Skip to content

Conversation

@adamzev
Copy link
Contributor

@adamzev adamzev commented Mar 14, 2025

Configures the repository to use pytest for unit testing and integrates it into the pr_checks_backend workflow.

This PR addresses issue #1010

Type of change: New dev workflow feature

Most of these changes are courtesy of @rmartinsen. This just provides some final changes on their work to ignore some tests which aren't traditional unit tests.

rmartinsen and others added 8 commits December 18, 2024 08:22
… mocks

Removed the duplicate file data new_etl/data_utils/diff_backup.py which is the same as
test/test_diff_backup.py

added an underscore to some required variables names for mocking which are never accessed to
supress a vulture warning (mock_getenv -> _mock_getenv)

added mocks to TestDiffBackup
@vercel
Copy link

vercel bot commented Mar 14, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
vacant-lots-proj ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 14, 2025 8:17pm

@adamzev adamzev changed the title 1010 setup pytest Task: 1010 setup pytest Mar 14, 2025
@nlebovits nlebovits merged commit 31e13e5 into CodeForPhilly:staging Mar 15, 2025
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants