Fix piping input from json

[DrSensor]

Description

This PR will enable the CLI to accept json from process.stdin

echo '{"states":[]}' | ./bin/smcat

Motivation and Context

Currently, this is what happens:

$ echo '{"states":[]}' | smcat

syntax error on line 1, column 1:
Expected comment, end of input, line end, or whitespace but "{" found.

How Has This Been Tested?

(Testing CLI apps is always tricky, especially this one which I don't know how to do it. Please let me know if you have some tips 🙂)

Screenshots (if appropriate):

Before

After

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

• The code I add will be subject to The MIT license, and I'm OK with that.
• The code I've added is my own original work.
• My code follows the code style of this project.
• My change requires a change to the documentation.
• I have updated the documentation accordingly.
• I have read the CONTRIBUTING document.
• I have added tests to cover my changes.
• All new and existing tests passed.

[sverweij]

hi @DrSensor - thanks for this PR. Before I dive into the details - have you tried using --input-type on the command line? e.g.

echo '{"states":[]}' | ./bin/smcat --input-type json

[sverweij]

Dôh - I now see; smcat ignores the --input-type option altogether, reading from stdin or otherwise.

[sverweij]

... that means this PR is not going to work, until that is fixed. I've created an issue (#57) and a fix for it (#58 )

[DrSensor]

Thanks, that will do. I will close this now