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

[Discovery] Fix flaky test #8596

Merged
merged 1 commit into from
May 23, 2024
Merged

[Discovery] Fix flaky test #8596

merged 1 commit into from
May 23, 2024

Conversation

nicoback2
Copy link
Contributor

Description

Fix test that would flake if the current timestamp ended in 0 due to Postgres truncating trailing 0s when converting timestamp to string

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide repro instructions & any configuration.

Copy link

changeset-bot bot commented May 23, 2024

⚠️ No Changeset found

Latest commit: fa61d59

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

Copy link
Contributor

@dylanjeffers dylanjeffers left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

freeze!

Copy link
Contributor

@schottra schottra left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🧊

@nicoback2 nicoback2 merged commit c474195 into main May 23, 2024
11 of 12 checks passed
@nicoback2 nicoback2 deleted the nkang--fix-mm-test branch May 23, 2024 14:35
audius-infra pushed a commit that referenced this pull request May 23, 2024
[c474195] [Discovery] Fix flaky test (#8596) nicoback2
[4e3dc09] [QA-1313] Fix purchase/sale email random null (#8591) Reed
[276f714] [PAY-2912] Add purchase album support to SDK (#8593) Marcus Pasell
[f20d3dd] [PAY-3024] Skip onFavorite notifications for purchased content (#8580) Andrew Mendelsohn
[117067d] [PROTO-1834] Enabling disallow third party streaming from client (#8587) Isaac Solo
[d5ce34d] [PAY-2947] Implement spl_usdc_payout_wallet user entity manager actions (#8479) Raymond Jacobson
[9590688] Add cascade delete on album price history (#8588) Isaac Solo
[3721596] mediorum: ignore reachability in repair.go (#8586) Steve Perkins
[8a1f226] ddex: remove docker user (#8572) Steve Perkins
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants