Skip to content

Comments

fix: use pytest native method name#305

Merged
frostming merged 1 commit intopsf:masterfrom
frostming:frostming/issue298
May 31, 2023
Merged

fix: use pytest native method name#305
frostming merged 1 commit intopsf:masterfrom
frostming:frostming/issue298

Conversation

@frostming
Copy link
Contributor

@frostming frostming commented May 30, 2023

Fix #298

This eliminates the mass warning when running pytest:

    tests/test_storage_redis.py::TestRedisCache::test_set_expiration_int is using nose-specific method: `setup(self)`
  To remove this warning, rename it to `setup_method(self)`
  See docs: https://docs.pytest.org/en/stable/deprecations.html#support-for-tests-written-for-nose
    fixture_result = next(generator)

Signed-off-by: Frost Ming [email protected]

@frostming
Copy link
Contributor Author

The black formatting is triggered by bulk replace.

@frostming frostming merged commit 2b696f6 into psf:master May 31, 2023
@frostming frostming deleted the frostming/issue298 branch May 31, 2023 02:53
woodruffw pushed a commit to woodruffw-forks/cachecontrol that referenced this pull request Jul 9, 2024
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.

0.12.12: pytest is failing in tests/test_etag.py::TestReleaseConnection::test_not_modified_releases_connection unit

1 participant