OCHA-DAP / hdx-cli-toolkit

A commandline tool for interacting with HDX with a view to doing bulk updates
MIT License
1 stars 0 forks source link

Improve test coverage #18

Closed IanHopkinson closed 5 months ago

IanHopkinson commented 5 months ago

Currently test coverage is not great:

image

In practice new commands are tested on the CLI. Most commands are "read" actions so relatively low risk. Potentially the commands in DEMO.md could be run in a shell script to provide some confidence that all is working.

IanHopkinson commented 5 months ago

Added a shell script with all the ready only commands in #21

IanHopkinson commented 5 months ago

Coverage now at 65% overall

IanHopkinson commented 5 months ago

Improving in #24

IanHopkinson commented 5 months ago

image

Test coverage is rather improved, the missing pieces are tests for the add_quickchart and add_showcase because this is potentially destructive and requires a degree of setup. Perhaps best to merge this rather than waiting for perfection

IanHopkinson commented 5 months ago

Added tests for add_quickcharts and add_showcase against a test dataset. Happy with this coverage, the cli is a thin skin on the hdx_utilities: image

IanHopkinson commented 5 months ago

Fixed with #24