Flaky test: tests/sentry/incidents/serializers/test_workflow_engine_detector.py::TestDetectorSerializer::test_simple
Open
Nobody has claimed this yet.
flaky-test
Tests
- Dominant language
- Python
- Stars
- 44.8k
- Forks
- 4.9k
- Avg merge
- 21h 23m
- Merged PRs (30d)
- 607
Description
Failing test: tests/sentry/incidents/serializers/test_workflow_engine_detector.py::TestDetectorSerializer::test_simple
Sentry sha: ab471d2d1ede60c8a8c1e3911b78a71a83261eae
Run: https://github.com/getsentry/sentry/actions/runs/22271635213
[1m[31mtests/sentry/incidents/serializers/test_workflow_engine_detector.py[0m:36: in test_simple
assert serialized_detector == self.expected
[1m[31mE AssertionError: assert {'aggregate':...one.utc), ...} == {'aggregate':...one.utc), ...}[0m
[1m[31mE [0m
[1m[31mE Omitting 16 identical items, use -vv to show[0m
[1m[31mE Differing items:[0m
[1m[31mE {'triggers': [{'actions': [{'alertRuleTriggerId': '30', 'dateCreated': datetime.datetime(2024, 12, 11, 3, 21, 34, tzin... 'alertThreshold': 100, 'dateCreated': datetime.datetime(2024, 12, 11, 3, 21, 34, tzinfo=datetime.timezone.utc), ...}]} != {'triggers': [{'actions': [{'alertRuleTriggerId': '29', 'dateCreated': datetime.datetime(2024, 12, 11, 3, 21, 34, tzin..., 'alertThreshold': 50, 'dateCreated': datetime.datetime(2024, 12, 11, 3, 21, 34, tzinfo=datetime.timezone.utc), ...}]}[0m
[1m[31mE [0m
[1m[31mE Full diff:[0m
[1m[31mE {[0m
[1m[31mE 'aggregate': 'count()',[0m
[1m[31mE 'createdBy': None,[0m
[1m[31mE 'dateCreated': datetime.datetime(2024, 12, 11, 3, 21, 34, tzinfo=datetime.timezone.utc),[0m
[1m[31mE 'dateModified': datetime.datetime(2024, 12, 11, 3, 21, 34, tzinfo=datetime.timezone.utc),[0m
[1m[31mE 'description': '',[0m
[1m[31mE 'detectionType': 'static',[0m
[1m[31mE 'id': '141',[0m
[1m[31mE 'name': 'Mighty Seal',[0m
[1m[31mE 'organizationId': '4555195392327680',[0m
[1m[31mE 'owner': None,[0m
[1m[31mE 'projects': [[0m
[1m[31mE 'bar',[0m
[1m[31mE ],[0m
[1m[31mE 'query': 'level:error',[0m
[1m[31mE 'resolution': 1.0,[0m
[1m[31mE 'status': 0,[0m
[1m[31mE 'thresholdPeriod': 1,[0m
[1m[31mE 'timeWindow': 10.0,[0m
[1m[31mE 'triggers': [[0m
[1m[31mE + {[0m
[1m[31mE + 'actions': [[0m
[1m[31mE + {[0m
[1m[31mE + 'alertRuleTriggerId': '30',[0m
[1m[31mE + 'dateCreated': datetime.datetime(2024, 12, 11, 3, 21, 34, tzinfo=datetime.timezone.utc),[0m
[1m[31mE + 'desc': 'Send a notification to admin@localhost',[0m
[1m[31mE + 'id': '27',[0m
[1m[31mE + 'inputChannelId': None,[0m
[1m[31mE + 'integrationId': None,[0m
[1m[31mE + 'priority': None,[0m
[1m[31mE + 'sentryAppId': None,[0m
[1m[31mE + 'targetIdentifier': '2598',[0m
[1m[31mE + 'targetType': 'user',[0m
[1m[31mE + 'type': 'email',[0m
[1m[31mE + },[0m
[1m[31mE + ],[0m
[1m[31mE + 'alertRuleId': '141',[0m
[1m[31mE + 'alertThreshold': 50,[0m
[1m[31mE + 'dateCreated': datetime.datetime(2024, 12, 11, 3, 21, 34, tzinfo=datetime.timezone.utc),[0m
[1m[31mE + 'id': '30',[0m
[1m[31mE + 'label': 'warning',[0m
[1m[31mE + 'resolveThreshold': 50,[0m
[1m[31mE + 'thresholdType': 0,[0m
[1m[31mE + },[0m
[1m[31mE {[0m
[1m[31mE 'actions': [[0m
[1m[31mE {[0m
[1m[31mE 'alertRuleTriggerId': '29',[0m
[1m[31mE 'dateCreated': datetime.datetime(2024, 12, 11, 3, 21, 34, tzinfo=datetime.timezone.utc),[0m
[1m[31mE 'desc': 'Send a notification to admin@localhost',[0m
[1m[31mE 'id': '26',[0m
[1m[31mE 'inputChannelId': None,[0m
[1m[31mE 'integrationId': None,[0m
[1m[31mE 'priority': None,[0m
[1m[31mE 'sentryAppId': None,[0m
[1m[31mE 'targetIdentifier': '2598',[0m
[1m[31mE 'targetType': 'user',[0m
[1m[31mE 'type': 'email',[0m
[1m[31mE },[0m
[1m[31mE ],[0m
[1m[31mE 'alertRuleId': '141',[0m
[1m[31mE 'alertThreshold': 100,[0m
[1m[31mE 'dateCreated': datetime.datetime(2024, 12, 11, 3, 21, 34, tzinfo=datetime.timezone.utc),[0m
[1m[31mE 'id': '29',[0m
[1m[31mE 'label': 'critical',[0m
[1m[31mE 'resolveThreshold': 50,[0m
[1m[31mE 'thresholdType': 0,[0m
[1m[31mE },[0m
[1m[31mE - {[0m
[1m[31mE - 'actions': [[0m
[1m[31mE - {[0m
[1m[31mE - 'alertRuleTriggerId': '30',[0m
[1m[31mE - 'dateCreated': datetime.datetime(2024, 12, 11, 3, 21, 34, tzinfo=datetime.timezone.utc),[0m
[1m[31mE - 'desc': 'Send a notification to admin@localhost',[0m
[1m[31mE - 'id': '27',[0m
[1m[31mE - 'inputChannelId': None,[0m
[1m[31mE - 'integrationId': None,[0m
[1m[31mE - 'priority': None,[0m
[1m[31mE - 'sentryAppId': None,[0m
[1m[31mE - 'targetIdentifier': '2598',[0m
[1m[31mE - 'targetType': 'user',[0m
[1m[31mE - 'type': 'email',[0m
[1m[31mE - },[0m
[1m[31mE - ],[0m
[1m[31mE - 'alertRuleId': '141',[0m
[1m[31mE - 'alertThreshold': 50,[0m
[1m[31mE - 'dateCreated': datetime.datetime(2024, 12, 11, 3, 21, 34, tzinfo=datetime.timezone.utc),[0m
[1m[31mE - 'id': '30',[0m
[1m[31mE - 'label': 'warning',[0m
[1m[31mE - 'resolveThreshold': 50,[0m
[1m[31mE - 'thresholdType': 0,[0m
[1m[31mE - },[0m
[1m[31mE ],[0m
[1m[31mE }[0m
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by running tests/sentry/incidents/serializers/test_workflow_engine_detector.py::TestDetectorSerializer::test_simple and compare serialized_detector with self.expected. Investigate the differing trigger entries and their ordering or values shown in the failure, then rerun the test repeatedly; done means the test passes consistently.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- testing
- Issue type
- Bug
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100