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

Add actual report tests with data to CI/CD #28

Open
krowvin opened this issue May 16, 2024 · 0 comments
Open

Add actual report tests with data to CI/CD #28

krowvin opened this issue May 16, 2024 · 0 comments
Labels
feature Potentially a missing feature or a new feature

Comments

@krowvin
Copy link
Collaborator

krowvin commented May 16, 2024

Running the pytest -v and checking the forms for #26 I realized it does not actually appear to be generating output with data.

It does check conversions, size compares, etc. Which is important.

Consider another CI/CD pipeline that tests various reports from districts so that as we make changes we can confirm it will run on district production reports.

Thoughts:

  • Picks specific dates in time in the past so that the expected output can match what is requested
  • Add a #FORMURL method that does a fetch for a plaintext form elsewhere (another github repo possibly) and runs from that form
    On second thought, decoupling the DEF from the FORM this much is probably a bad idea
@krowvin krowvin added the feature Potentially a missing feature or a new feature label May 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature Potentially a missing feature or a new feature
Projects
None yet
Development

No branches or pull requests

1 participant