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

feat(history): adds tests for history plugin #752

Merged
merged 1 commit into from
Aug 21, 2024

Conversation

sennierer
Copy link
Collaborator

currently tests for the history plugin lack an ontology for running. This is a draft PR until we added an ontology to the testing env.

@sennierer sennierer marked this pull request as ready for review May 13, 2024 09:48
@sennierer sennierer requested a review from b1rger May 13, 2024 09:48
Copy link
Contributor

@b1rger b1rger left a comment

Choose a reason for hiding this comment

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

We will have to move the import

@sennierer
Copy link
Collaborator Author

We will have to move the import

is moved

@b1rger
Copy link
Contributor

b1rger commented May 27, 2024

This should now be updated for the sample_project ontology

@sennierer sennierer self-assigned this Jun 19, 2024
@b1rger
Copy link
Contributor

b1rger commented Aug 13, 2024

ping

@sennierer
Copy link
Collaborator Author

adapted the existing tests to the sample project. presumably the check in line 11 (for the CI env variable) can be removed as the tests should now run in CI as well. However, i suggest to merge these simple tests first and lift the check for CI in another PR

Copy link
Contributor

@b1rger b1rger left a comment

Choose a reason for hiding this comment

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

please change the commit message to test (history): ... - if we cut a release, feat leads to a version bump

@sennierer
Copy link
Collaborator Author

good point, changed that

@sennierer sennierer requested a review from b1rger August 21, 2024 06:04
@sennierer sennierer merged commit df1438d into main Aug 21, 2024
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants