Skip to content

[Eng-7752] tests deps upgrade #379

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

Conversation

mkovalua
Copy link

@mkovalua mkovalua commented Jun 3, 2025

Ticket

https://openscience.atlassian.net/browse/ENG-7752

Purpose

Update dependencies in dep-group “test”.

The dep-group tickets are about updating a batch of dependencies at a time. In the MFR requirements DB, column C is "group". I've tried to group similar dependencies together. When you take on a dep-group ticket, you're trying to upgrade all the tickets in that group to their latest version.

It may turn out that some of the modules in one group break or require changes for a module in another group. That's okay; we'll deal with those on a case-by-case basis. If we have to leave something out or drag something in it, that's fine. We can also break things out into smaller dep-groups with new tickets if that makes sense.

Changes

Update code to make it possible to run the tests for python3.13

make the PR from the following one to keep its changings
#378

The tests pytest triggers test execution but see not all of them are passing

mfr test execution logs(
https://docs.google.com/document/d/15x_ar0pI8i7EmiddErK7g84o4oViWNed0s29vlSm6r0/edit?usp=sharing
)

image

Side effects

QA Notes

Deployment Notes

mkovalua added 4 commits June 13, 2025 21:39
 * make it possible to run unittest tests

 * add '__init__.py' into tests/core to avoid import errors
 * Update `async with await aiohttp.request` ->
   `async with aiohttp.request`
 * ContentEncodingError has been replaced with except Exception:
   (maybe some additional logic is needed for pass and more strict
   Exception class )
@felliott felliott force-pushed the ENG-7752/tests-deps-upgrade branch from e660d25 to 9ab77ae Compare June 14, 2025 01:51
@felliott felliott merged commit ec8a0d2 into CenterForOpenScience:feature/buff-worms Jun 14, 2025
1 check passed
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