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.

Sign up for GitHub

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

Jump to bottom

Unit tests for dic2owl CLI and setup testing framework #62

Merged
merged 8 commits into from
Dec 3, 2021
Merged

Unit tests for dic2owl CLI and setup testing framework #62

merged 8 commits into from
Dec 3, 2021

Conversation

CasperWA
Copy link
Contributor

@CasperWA CasperWA commented Aug 30, 2021 •
edited
Loading

Add unit tests using PyTest for the dic2owl Python package:

  • Set up PyTest framework.
  • Test cli.py.
  • Test dic2owl.py.

@CasperWA CasperWA added the dic2owl Issue or PR related specifically to the dic2owl Python package label Aug 30, 2021
@CasperWA CasperWA force-pushed the close_57_unit-tests-dic2owl branch 2 times, most recently from c81764a to 7d0d77e Compare August 30, 2021 16:37
@CasperWA CasperWA force-pushed the close_57_unit-tests-dic2owl branch from 7d0d77e to ac3240e Compare September 23, 2021 10:05
@CasperWA
Uncomment unusable CLI options
8d470b6 
Uncommenting `--log-level` and `--debug`.
They should be uncommented and used when support for them are available,
or eventually removed if it's determined they should not be implemented.
@CasperWA
Add unit tests for dic2owl CLI
616f0bc
@CasperWA
Run unit tests in CI
2b6210f 
Add generated coverage report to `.gitignore` when running `pytest` with
coverage locally.
@CasperWA
Enhance failure messages
3e09cc7 
This is specifically for the CLI runner pytest fixture: Explicitly print
out stdout and stderr.
@CasperWA
First tests for dic2owl.Generator class
0e5ac7d 
Implement better typing all-round.
@CasperWA CasperWA force-pushed the close_57_unit-tests-dic2owl branch from 7751097 to 0e5ac7d Compare December 1, 2021 15:53
@CasperWA
Copy link
Contributor Author

CasperWA commented Dec 1, 2021 •
edited
Loading

I'll get this in, creating a new issue for the remaining dic2owl unit tests.

@CasperWA CasperWA marked this pull request as ready for review December 1, 2021 15:58
@CasperWA CasperWA changed the title Unit tests for dic2owl Unit tests for dic2owl CLI and setup testing framework Dec 1, 2021
@CasperWA CasperWA force-pushed the close_57_unit-tests-dic2owl branch from 2cb6986 to 88babb9 Compare December 3, 2021 08:04
@CasperWA CasperWA merged commit bfeaf81 into main Dec 3, 2021
@CasperWA CasperWA deleted the close_57_unit-tests-dic2owl branch December 3, 2021 09:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
dic2owl Issue or PR related specifically to the dic2owl Python package
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@CasperWA