-
Notifications
You must be signed in to change notification settings - Fork 3k
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
[HOLD for payment 2023-10-24] [Reassure] Migrate some of the GooglePlacesUtilsTest to reassure test framework #28585
Comments
Triggered auto assignment to @JmillsExpensify ( |
Bug0 Triage Checklist (Main S/O)
|
Current assignee @JmillsExpensify is eligible for the Bug assigner, not assigning anyone new. |
Bug0 Triage Checklist (Main S/O)
|
@mountiny Commenting for @adhorodyski as Adam is ooo. 😃 (link) |
@mountiny could you please assign @adhorodyski instead of me? 🙂 |
I made the PR to skip the flaky tests, no we just need to wait for Reassure to support such tests |
@mountiny i will happily pick this up after we merge callstack/reassure#271 :) |
@adhorodyski is there anything we can do to help with the PR? If you need to allocate time for it or another engineer it makes sense to me to push this one ahead |
I now work 50/50 between the flashlist migration under #28902 & this PR. |
Thanks Adam! |
The new API from callstack/reassure#271 just got released under v0.10.0 🚀 - I will pick up this task shortly! |
Are any additional PRs due for this issue? It looks like no payments are due, so I think we might be ok to close it? |
|
The solution for this issue has been 🚀 deployed to production 🚀 in version 1.3.85-4 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 2023-10-24. 🎊 After the hold period is over and BZ checklist items are completed, please complete any of the applicable payments for this issue, and check them off once done.
For reference, here are some details about the assignees on this issue:
As a reminder, here are the bonuses/penalties that should be applied for any External issue:
|
BugZero Checklist: The PR fixing this issue has been merged! The following checklist (instructions) will need to be completed before the issue can be closed:
|
No payments are required on this issue, so the main question left is whether a regression is required. @koko57 / @adhorodyski thoughts? |
Nope this was an update to automated tests so all good here, thanks |
Problem
We have implemented one of the GooglePlacesUtilsTests so it measures time of some function execution in a test framework which is not ideal for that. This test is repeatadly flaky as a result, for example look here https://github.com/Expensify/App/actions/runs/6377422632/job/17306058621
Solution
First, lets skip these tests and then migrate them to Reassure framework, once we can do that. Currently this exact type of tests is not supported according to Callstack
The text was updated successfully, but these errors were encountered: