Skip to content
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

chore: disable e2e test which started failing on GitHub for no obvious reason #18812

Merged
merged 2 commits into from
Oct 14, 2024

Conversation

david-mackessy
Copy link
Contributor

@david-mackessy david-mackessy commented Oct 14, 2024

  • disable DB constraint test during data element merge for min max elements
  • test started failing in GitHub only (setup stage, posting metadata - 409 response)
  • not failing locally when run in isolation nor when whole suite run
  • possible that some other test state is not being cleaned up correctly, impacting this test
  • tried removing some metadata UIDs in setup where possible and changing others, but still failed in GitHub only

@david-mackessy david-mackessy changed the title chore: Alter e2e test, remove id where possible and make others more less likely to clash where possible chore: Alter e2e test, remove uid where possible and make others less likely to clash where possible Oct 14, 2024
Copy link

sonarcloud bot commented Oct 14, 2024

@david-mackessy david-mackessy changed the title chore: Alter e2e test, remove uid where possible and make others less likely to clash where possible chore: disable e2e test which started failing on GitHub for no obvious reason Oct 14, 2024
@david-mackessy david-mackessy enabled auto-merge (squash) October 14, 2024 09:05
@david-mackessy david-mackessy merged commit 112bd6f into master Oct 14, 2024
15 checks passed
@david-mackessy david-mackessy deleted the update-e2e-test-setup branch October 14, 2024 09:50
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