separate test workflow for datasets #117
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR outsources tests to new
test_datasets.yaml
GitHub CI workflow. This is the first approach to handle #112.The following items in the workflow configs are adjusted:
on.(push|pull_request).paths(-ignore)
: only execute the workflow if the respective files did change--ignore=tests/dataset_builders
/tests/dataset_builders
topytest
arguments: only execute the respective tests--cov=src
/--cov
: only calculate the coverage for the respective filescoverage.xml
/coverage_datasets.xml
to save and upload the coverage reportNotes:
BratBuilder
. Until now, this was covered by the Brat dataset tests, but since these are now separated, we also need explicit tests for the builder.BratBuilder._generate_example()
to make it "API-complete"