test suite should test json files

jllodra / ncdump-json

Modified ncdump to output data in json format

85 stars 28 forks source link

test suite should test json files #9

Open ocehugo opened 8 years ago

ocehugo commented 8 years ago

Currently the run_tests.sh script is not testing the output of ncdump-json, just if the command should run.

I also believe that more cdls can be add to the testing suite (unidata has a lot of them). I think the best way to handle this is a list of the file locations (or http locations) so you could load the file, run ncdump-json in all entries, and test a json reader on the output.

Cheers

jllodra commented 8 years ago

Hi @ocehugo,

tests were done, but I switched to python json.tool, added a new unidata example file, and improved the tests/README.md to make things clear.

Thank you