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

Build Unit test boilerplate #1052

Merged
merged 14 commits into from
May 21, 2024
Merged

Build Unit test boilerplate #1052

merged 14 commits into from
May 21, 2024

Conversation

gcglinton
Copy link
Contributor

Added a ton of files, most of which are empty, containing no tests.

They're just there to setup the ground-work for future tests.

I also centralized the "debug" stuff into the conftest.py file in the root of the tests directory. Now when looking to debug anything during test writing, you can just call debug.pretty(YourThingToDebug), and it should work.
It's also possible to centralize some fixtures there if needed. Things like building a base config, or messages could be put there, and used by all tests.
You can also create a conftest.py file in any given directory, and have those contents be available in that context, either building on those defined above it, or over-riding them.

Some of that is outlined here

Had to rename flowcb/accept/delete_test.py becase there's on here too.
authenticate, destfn, download, filter, gather
…nstead

There's also some test renames as well for cases where there's duplicate module names in different folders. They're now "<moduleName>__<ParentFolder>_test.py"
Also removed un-neccessary imports from the boilerplate test files
Includes: accept, housekeeping, post, scheduled, send
Rename conflicting modules from other flowcb folders
Copy link

github-actions bot commented May 16, 2024

Test Results

220 tests   212 ✅  17s ⏱️
  1 suites    8 💤
  1 files      0 ❌

Results for commit b0728af.

♻️ This comment has been updated with latest results.

@petersilva
Copy link
Contributor

You didn't ask for any reviewers... (top-right?)
We normally don't merge without at least one peer review.
Is this a WIP or do you want it merged? fwiw... I have a bunch of unit test engineering (ute) branches... they are not yet merged...
they all just use plain print() what does the pretty print stuff give us?

@petersilva petersilva merged commit 4fb2cdf into development May 21, 2024
3 checks passed
@petersilva petersilva deleted the unitTestBoilerplate branch June 6, 2024 21:06
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