Skip to content

Integration test improvements#267

Merged
BryceStevenWilley merged 8 commits into
mainfrom
integration_test_improvements
Jul 9, 2025
Merged

Integration test improvements#267
BryceStevenWilley merged 8 commits into
mainfrom
integration_test_improvements

Conversation

@BryceStevenWilley
Copy link
Copy Markdown
Contributor

  • Randomized the emails we use to make accounts
  • Factored out variables needed for different jurisdictions (MA vs IL, etc.) (not completely done, need a json to make a filing into MA still)
  • switched to unittest: easier to understand the half-baked self-rolled assertions that stop immediately
  • changed the account env vars to be reasonable (no assumptions of the account, better env var name)

So there's never a conflict if a previous test fails.
Set just in the proxy connection object, not used anywhere else.
Instead of hardcoding them in the functions themselves
Lol, I have no idea why I thought that was a good idea in the first place.
@BryceStevenWilley BryceStevenWilley force-pushed the integration_test_improvements branch from 0b0f462 to d6b2ea4 Compare July 9, 2025 15:52
@BryceStevenWilley BryceStevenWilley merged commit e6e2dd9 into main Jul 9, 2025
5 of 6 checks passed
@BryceStevenWilley BryceStevenWilley deleted the integration_test_improvements branch July 9, 2025 17:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant