getsentry / getsentry/sentry

Flaky test: tests/sentry/integrations/opsgenie/test_integration.py::OpsgenieMigrationIntegrationTest::test_no_duplicate_keys

Open
#108,915 1 comment 0 reactions 0 assignees View on GitHub

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/integrations/opsgenie/test_integration.py::OpsgenieMigrationIntegrationTest::test_no_duplicate_keys
Sentry sha: 4a8dadc9c184ad93a761561c5f57437a29cd3300
Run: https://github.com/getsentry/sentry/actions/runs/22283662003

tests/sentry/integrations/opsgenie/test_integration.py:381: in test_no_duplicate_keys
    assert org_integration.config == {
E   AssertionError: assert {'team_table'...[MIGRATED]'}]} == {'team_table'...[MIGRATED]'}]}
E     
E     Differing items:
E     {'team_table': [{'id': '222-thinkies', 'integration_key': '123-key', 'team': 'thinkies [MIGRATED]'}]} != {'team_table': [{'id': '222-thonk', 'integration_key': '123-key', 'team': 'thonk [MIGRATED]'}]}
E     
E     Full diff:
E       {
E           'team_table': [
E               {
E     -             'id': '222-thonk',
E     ?                          ^
E     +             'id': '222-thinkies',
E     ?                          ^  +++
E                   'integration_key': '123-key',
E     -             'team': 'thonk [MIGRATED]',
E     ?                        ^
E     +             'team': 'thinkies [MIGRATED]',
E     ?                        ^  +++
E               },
E           ],
E       }

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start with tests/sentry/integrations/opsgenie/test_integration.py and run OpsgenieMigrationIntegrationTest::test_no_duplicate_keys, using the linked Actions run and reported Sentry SHA as context. Investigate why the migration produces either the thonk or thinkies team entry; done means the test passes consistently with the expected configuration.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
backend
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.