Skip to content
This repository was archived by the owner on Jul 5, 2023. It is now read-only.

Support test case generation only and upload #110

Merged
merged 1 commit into from
Aug 6, 2020

Conversation

adracus
Copy link
Contributor

@adracus adracus commented Aug 5, 2020

Fixes #90

Introduce new generate command that generates the test cases,
outputting them to a target file.

Add new -t / --test-cases flag to the run command that allows
specifying custom test cases to upload instead of generating.

Add format handling read and write operations. Currently, only
YAML and JSON is supported.

Create and verify tests for obtaining the correct format depending
on the filename and / or an explicit format name.

@codecov
Copy link

codecov bot commented Aug 5, 2020

Codecov Report

Merging #110 into master will increase coverage by 1.03%.
The diff coverage is 72.50%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #110      +/-   ##
==========================================
+ Coverage   48.22%   49.26%   +1.03%     
==========================================
  Files           8        8              
  Lines         846      885      +39     
  Branches      191      196       +5     
==========================================
+ Hits          408      436      +28     
- Misses        389      399      +10     
- Partials       49       50       +1     
Impacted Files Coverage Δ
src/illuminatio/util.py 76.19% <70.27%> (-8.43%) ⬇️
src/illuminatio/test_case.py 78.46% <100.00%> (+0.68%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b5d4383...d14a3a6. Read the comment docs.

@johscheuer johscheuer self-requested a review August 5, 2020 13:09
@adracus adracus force-pushed the feature.test-cases-file branch from fcb096b to 4d05684 Compare August 5, 2020 15:27
Copy link
Contributor

@johscheuer johscheuer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for your PR, just some minor nitpicks.

Introduce new `generate` command that generates the test cases,
outputting them to a target file.

Add new `-t` / `--test-cases` flag to the `run` command that allows
specifying custom test cases to upload instead of generating.

Add format handling read and write operations. Currently, only
YAML and JSON is supported.

Create and verify tests for obtaining the correct format depending
on the filename and / or an explicit format name.
@adracus adracus force-pushed the feature.test-cases-file branch from 4d05684 to d14a3a6 Compare August 6, 2020 07:43
Copy link
Contributor

@johscheuer johscheuer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@johscheuer johscheuer merged commit cd93f86 into inovex:master Aug 6, 2020
@adracus adracus deleted the feature.test-cases-file branch August 6, 2020 07:51
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Implement running tests from test config
3 participants