Skip to content

Conversation

@soul-codes
Copy link
Collaborator

This PR adds a testing infrastructure for our analyzers. The test paradigm is snapshot testing -- tests are written by providing input data and expected output data.

Tests are added to our example analyzer as a means to extend our documentation, and the CI has been extended to run tests automatically.

Copy link
Collaborator

@andi-halim andi-halim left a comment

Choose a reason for hiding this comment

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

Checked out your forked repo and branch and test out the example analyzer. It worked end-to-end and I got the following outputs by running the reddit_vm dataset.

More importantly, I updated my env with the new requirements-dev.txt and ran pytest. The two test cases passed! Nice job Tar!!

Screenshot 2025-02-08 at 2 46 42 PM Screenshot 2025-02-08 at 2 45 54 PM

@soul-codes soul-codes merged commit 62f5427 into civictechdc:main Feb 18, 2025
2 checks passed
@soul-codes soul-codes mentioned this pull request Mar 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

2 participants