cli client [DO NOT MERGE]

[pcgeek86]

• Separate dealership REST API into dealership.js
• Moved dbutils.js to its own, shared directory, and updated references
• Started CLI client to call REST APIs, using Python Click library

@wallnerryan could you help me figure out what's causing this error when POST'ing to /dealerships?

This is the PowerShell call that I'm using, sending a simple JSON payload in the HTTP body.

Invoke-RestMethod -Uri http://localhost:8000/dealerships -Method POST `
  -Body ([PSCustomObject]@{ 
    name = 'asdf'; 
    phone = '555-555-5556'; 
    addr = '123 street' } | ConvertTo-Json) `
  -Headers @{ 'Content-Type' = 'application/json' }

[wallnerryan]

This seems like leftovers? If the insert was a success the endpoint should return a success message or json object that has "success" or "failure" etc.

[pcgeek86]

Agreed -- this is just there until I can figure out what the bug is.

[pcgeek86]

Eventually, it will return HTTP 201 (Created).

[wallnerryan]

What is this file @pcgeek86 ?

[pcgeek86]

Hmmmm, good question. I presume this is coming from Visual Studio Code. Here's an article that talks about it:

https://code.visualstudio.com/docs/languages/javascript

I can remove it from the repo, although it might make sense to leave it. We could both benefit from it, if we use VSCode, although it obviously isn't a requirement. Looks like it mainly provides Intellisense hints to VSCode.

[wallnerryan]

Lets remove it, since its really not source code the particular the the app, but the specific developer.

[wallnerryan]

@pcgeek86 should work now.

$:-> curl -H "Content-Type: application/json" -X POST -d '{"name": "asdfg", "phone": "555-555-5555", "addr": "1234 Main St." }' localhost:8000/dealerships
{"message":"hello"}$:->

Logs from front end container

Creating a new dealership
{ name: 'asdf', phone: '555-555-5555', addr: '123 Main St.' }
Dealership name is asdf
Connecting to RethinkDB
{ deleted: 0,
  errors: 0,
  generated_keys: [ 'a7430c19-6576-4e51-85b1-b20ab51af630' ],
  inserted: 1,
  replaced: 0,
  skipped: 0,
  unchanged: 0 }

[wallnerryan]

@pcgeek86 you can merge in master to this branch, master is updated against new FH instance and slaves use new cli now. So should build correctly without FH/cli specific issues

[wallnerryan]

I will be working on this to update the APIs, then merge in. Once done, I will merge the APIs back into #75 so we can write more tests and merge those in.

[wallnerryan]

Renaming this now that #75 is merged. This PR can now get rid of any API related things because #75 included it.