inveniosoftware / inveniosoftware/invenio-logging

Raven in tests.

Open
#76 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
2
Forks
36
PR merge metrics
No merged PRs in 30d

Description

Package version (if known): > 2.0

## Describe the bug
There are still raven imports in tests:
```
@pytest.yield_fixture()
def sentry_emit():
"""Mock Sentry emit."""
from raven.handlers.logging import SentryHandler

with patch.object(SentryHandler, "emit") as sentry_emit:
yield sentry_emit
```

Contributor guide

Open the contributing guide

Research direction

Search the test suite for the raven.handlers.logging.SentryHandler import shown in the issue and inspect the sentry_emit fixture. Determine the intended replacement or removal for this obsolete test dependency, then verify that the affected tests pass without Raven imports.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
testing-qa
Issue type
Bug
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.