Skip to content

test: Resolve flakey tests #1098

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.

Sign up for GitHub

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

Jump to bottom
Merged
jaissica12 merged 20 commits into from
Oct 31, 2025
Merged

test: Resolve flakey tests #1098

jaissica12 merged 20 commits into from
Oct 31, 2025

Conversation

@jaissica12
Copy link
Contributor

@jaissica12 jaissica12 commented Oct 7, 2025

Instructions

  1. PR target branch should be against development
  2. PR title name should follow this format: https://github.com/mParticle/mparticle-workflows/blob/main/.github/workflows/pr-title-check.yml
  3. PR branch prefix should follow this format: https://github.com/mParticle/mparticle-workflows/blob/main/.github/workflows/pr-branch-check-name.yml

Summary

  • {provide a thorough description of the changes}

Testing Plan

  • Was this tested locally? If not, explain why.
  • {explain how this has been tested, and what, if any, additional testing should be done}

Reference Issue (For mParticle employees only. Ignore if you are an outside contributor)

@jaissica12
test: converted .then to async/await in tests-forwarders (#1086)
166e2c0
@jaissica12
test:convert then to async/await in requirejs (#1097)
d4bf71a
@jaissica12
test: convert .then to async/await in self hosting specific test (#1092)
6c715c8
@jaissica12
test:convert .then to async/await in serverModel test (#1093)
247f3e9
@jaissica12
test:convert .then to async/await in session-manager test (#1094)
d8ecd50
@jaissica12
test:convert .then to async/await in user test (#1095)
dc8cc39
@jaissica12
test: converted .then to async/await for identity tests (#1091)
e3b424c
@jaissica12
test: convert .then to async/await for integration capture test (#1090)
be59b37
@jaissica12
test: convert .then to async/await in mParticleUser tests (#1087)
24364b1
@jaissica12
test: converted .then to async/await in feature flags test (#1085)
3f9fc1a
@jaissica12
test: converted .then to async/await in core sdk tests (#1083)
1963719
@jaissica12
test: convert .then to async/await in kit-blocking test (#1089)
26e60b4
@jaissica12
test: convert .then to async/await in legacy-alias-request test (#1088)
edb0ab7
@jaissica12
test: convert .then to async/await for beaconUpload test (#1081)
a515d8e
@jaissica12
test: convert .then to async/await for batchUploader tests (#1080)
32ad7c1
@jaissica12
test: converted .then to async/await in tests-consent (#1082)
285d56f
@jaissica12
test: converted .then to async/await in event logging test (#1084)
3895436
@jaissica12
test: convert .then to async/await in eCommerce test (#1096)
8e3410d
@jaissica12
test: resolve failing tests after converting .then to async/await (#1099
a438f0d 
)
@jaissica12
test: evaluate resetForTests function (#1100)
302dd47
@jaissica12 jaissica12 marked this pull request as ready for review October 31, 2025 18:37
@sonarqubecloud
Copy link

sonarqubecloud bot commented Oct 31, 2025

Quality Gate Failed Quality Gate failed

Failed conditions
13.1% Duplication on New Code (required ≤ 3%)

See analysis details on SonarQube Cloud

@jaissica12 jaissica12 requested a review from rmi22186 October 31, 2025 18:43
@rmi22186
Copy link
Member

rmi22186 commented Oct 31, 2025

change the commit message to test: Resolve flakey tests

@jaissica12 jaissica12 changed the title test: SDKE 300 convert all then to async await in tests test: Resolve flakey tests Oct 31, 2025
@jaissica12 jaissica12 merged commit 06a2743 into development Oct 31, 2025
27 of 31 checks passed
@mparticle-automation
Copy link
Collaborator

mparticle-automation commented Nov 10, 2025

🎉 This PR is included in version 2.49.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@rmi22186 rmi22186 rmi22186 approved these changes

Assignees

No one assigned

Labels

released

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@jaissica12 @rmi22186 @mparticle-automation