Skip to content

fix: Properly mock OIDC responses#49

Merged
kosciCZ merged 1 commit intomainfrom
fix-test-warnings
Jun 9, 2025
Merged

fix: Properly mock OIDC responses#49
kosciCZ merged 1 commit intomainfrom
fix-test-warnings

Conversation

@kosciCZ
Copy link
Copy Markdown
Contributor

@kosciCZ kosciCZ commented Jun 9, 2025

Gets rid of the following warnings introduced in #35

tests/cmd/upload/test_oidc.py::test_put
  /home/mj/redhat/mobster/src/mobster/cmd/upload/oidc.py:232: RuntimeWarning: coroutine 'AsyncMockMixin._execute_mock_call' was never awaited
    response.raise_for_status()
  Enable tracemalloc to get traceback where the object was allocated.
  See https://docs.pytest.org/en/stable/how-to/capture-warnings.html#resource-warnings for more info.

tests/cmd/upload/test_oidc.py::test_post
  /home/mj/redhat/mobster/src/mobster/cmd/upload/oidc.py:253: RuntimeWarning: coroutine 'AsyncMockMixin._execute_mock_call' was never awaited
    response.raise_for_status()
  Enable tracemalloc to get traceback where the object was allocated.
  See https://docs.pytest.org/en/stable/how-to/capture-warnings.html#resource-warnings for more info.

-- Docs: https://docs.pytest.org/en/stable/how-to/capture-warnings.html

@kosciCZ kosciCZ requested a review from jedinym June 9, 2025 14:38
@kosciCZ kosciCZ self-assigned this Jun 9, 2025
@kosciCZ kosciCZ force-pushed the fix-test-warnings branch from b8d936c to 6874935 Compare June 9, 2025 14:49
@codecov-commenter
Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 97.61%. Comparing base (f297aa8) to head (6874935).

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #49   +/-   ##
=======================================
  Coverage   97.61%   97.61%           
=======================================
  Files          26       26           
  Lines        1384     1384           
=======================================
  Hits         1351     1351           
  Misses         33       33           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@kosciCZ kosciCZ added this pull request to the merge queue Jun 9, 2025
Merged via the queue into main with commit 8e38a55 Jun 9, 2025
4 checks 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.

3 participants