Skip to content

[HOLD for payment 2024-07-24] Investigate workflow job failing on main: test / test (job 3) #45340

Closed

Description

@github-actions

🚨 Failure Summary 🚨:

⚠️ Action Required ⚠️:

🛠️ A recent merge appears to have caused a failure in the job named test / test (job 3).
This issue has been automatically created and labeled with Workflow Failure for investigation.

👀 Please look into the following:

  1. Why the PR caused the job to fail?
  2. Address any underlying issues.

🐛 We appreciate your help in squashing this bug!

Activity

  1. dangrous commented on Jul 12, 2024

    @dangrous
    Contributor

    @JKobrynski do you mind taking a look at this one? Looks like it's the new tests you added in #44838 #44072 - I'm wondering if the Onyx.multiSet() calls are overlapping and causing the tests to be flakey.

  2. dangrous commented on Jul 12, 2024

    @dangrous
    Contributor

    sorry not that PR - this one! #44072

  3. dangrous commented on Jul 12, 2024

    @dangrous
    Contributor

    I think it's because the last test group doesn't have an afterEach? Maybe should add:

    afterEach(async () => {
        await Onyx.clear();
        await Onyx.multiSet({
     
     
     
     [ONYXKEYS.NVP_PRIVATE_OWNER_BILLING_GRACE_PERIOD_END]: null,
        [ONYXKEYS.NVP_PRIVATE_AMOUNT_OWED]: null,
        [ONYXKEYS.NVP_PRIVATE_BILLING_DISPUTE_PENDING]: null,
        [ONYXKEYS.NVP_PRIVATE_STRIPE_CUSTOMER_ID]: null,
        [ONYXKEYS.NVP_PRIVATE_BILLING_STATUS]: null,
        [ONYXKEYS.FUND_LIST]: null,
     
     
     
     [ONYXKEYS.SUBSCRIPTION_RETRY_BILLING_STATUS_SUCCESSFUL]: null,
        [ONYXKEYS.SUBSCRIPTION_RETRY_BILLING_STATUS_FAILED]: null,
        });
    });
    

    (but correct spacing)

  4. dangrous commented on Jul 12, 2024

    @dangrous
    Contributor

    Fix PR up!

  5. added
    Awaiting PaymentAuto-added when associated PR is deployed to production
    and removed on Jul 17, 2024
  6. changed the title [-]Investigate workflow job failing on main: test / test (job 3)[/-] [+][HOLD for payment 2024-07-24] Investigate workflow job failing on main: test / test (job 3)[/+] on Jul 17, 2024
  7. melvin-bot commented on Jul 17, 2024

    @melvin-bot

    Reviewing label has been removed, please complete the "BugZero Checklist".

  8. melvin-bot commented on Jul 17, 2024

    @melvin-bot

    The solution for this issue has been 🚀 deployed to production 🚀 in version 9.0.7-8 and is now subject to a 7-day regression period 📆. Here is the list of pull requests that resolve this issue:

    If no regressions arise, payment will be issued on 2024-07-24. 🎊

  9. dangrous commented on Jul 24, 2024

    @dangrous
    Contributor

    Closing this out, no payment required.

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

Metadata

Metadata

Assignees

Labels

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions