Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[STORY] Add unittests to cover CLI #62

Open
6 tasks
tclose opened this issue Apr 14, 2022 · 1 comment
Open
6 tasks

[STORY] Add unittests to cover CLI #62

tclose opened this issue Apr 14, 2022 · 1 comment
Labels
deploy-framework Deployment framework epic mid-level Requires moderate framework/domain-specific knowledge pipelines story a unit of work
Milestone

Comments

@tclose
Copy link
Contributor

tclose commented Apr 14, 2022

Description

As a developer, I want to have high test coverage of the command line interface, so that I can refactor the code with confidence.

Acceptance Criteria

Unittest coverage of the following functions

  • arcana.cli.deploy.build_docs
  • arcana.cli.deploy.required_packages
  • arcana.cli.deploy.inspect_docker_exec
  • arcana.cli.deploy.run_pipeline -> branch when dataset name doesn't match existing dataset
  • arcana.cli.deploy.run_pipeline -> branch when pipeline exists in dataset already
  • arcana.cli.store.refresh
@tclose tclose added the testing label Apr 14, 2022
@tclose tclose added this to the 2.0.0a milestone Apr 14, 2022
@tclose tclose moved this to Backlog in AIS Master Project Apr 14, 2022
@tclose tclose moved this from Backlog to Todo in AIS Master Project Apr 14, 2022
@tclose tclose modified the milestones: 2.0.0a, 2.0.0b Apr 14, 2022
@ablachair ablachair moved this from Todo to In Progress in AIS Master Project Apr 26, 2022
@tclose
Copy link
Contributor Author

tclose commented Jun 19, 2022

@tclose tclose added incomplete-desc story a unit of work labels Jun 21, 2022
@tclose tclose changed the title Improve test coverage by adding unittests [STORY] Improve test coverage by adding unittests Jun 21, 2022
@tclose tclose added mid-level Requires moderate framework/domain-specific knowledge deploy-framework Deployment framework epic and removed testing labels Jun 21, 2022
@tclose tclose changed the title [STORY] Improve test coverage by adding unittests [STORY] Add unittests to cover CLI Jul 28, 2022
@tclose tclose added the ready label Jul 28, 2022
@tclose tclose removed the ready label Aug 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deploy-framework Deployment framework epic mid-level Requires moderate framework/domain-specific knowledge pipelines story a unit of work
Projects
None yet
Development

No branches or pull requests

2 participants