Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

sets mariadb as db, changes version retrieval #10

Merged
merged 5 commits into from
Sep 26, 2023
Merged

Conversation

jmbhughes
Copy link
Member

#9 noted that there were issues running the database tests. This fixed them on my computer. We should probably determine the appropriate mariaDB version to use too though.

@jmbhughes
Copy link
Member Author

This appears to be failing because of how we're dynamically getting __version__. Interestingly it only fails for Level 1 and not Level 2. I'm not sure why though.

@codecov
Copy link

codecov bot commented Sep 26, 2023

Codecov Report

All modified lines are covered by tests ✅

Files Coverage Δ
punchpipe/__init__.py 100.00% <100.00%> (+9.09%) ⬆️
punchpipe/controlsegment/tests/conftest.py 100.00% <100.00%> (ø)
punchpipe/flows/tests/conftest.py 100.00% <100.00%> (ø)

📢 Thoughts on this report? Let us know!.

@jmbhughes jmbhughes changed the title sets mariadb as db sets mariadb as db, changes version retrieval Sep 26, 2023
@jmbhughes jmbhughes requested a review from danseaton September 26, 2023 20:12
Copy link

@danseaton danseaton left a comment

Choose a reason for hiding this comment

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

Minor changes only

@jmbhughes jmbhughes merged commit 389c842 into develop Sep 26, 2023
@jmbhughes jmbhughes deleted the fix-test-db branch September 26, 2023 20:26
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants