Closes #13219: Fixing long-tap delay for UI tests

[sv-ohorvath]

Changes the long-tap delay before starting the activity so it applies to all tests.
Fixed the Libraries menu toolbar resource ID because it has changed.

[codecov-commenter]

Codecov Report

Merging #13502 into master will increase coverage by 0.02%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master   #13502      +/-   ##
============================================
+ Coverage     29.81%   29.83%   +0.02%     
- Complexity     1115     1118       +3     
============================================
  Files           422      422              
  Lines         17172    17170       -2     
  Branches       2229     2227       -2     
============================================
+ Hits           5119     5123       +4     
+ Misses        11678    11673       -5     
+ Partials        375      374       -1     

Impacted Files	Coverage Δ	Complexity Δ
...pp/src/main/java/org/mozilla/fenix/FeatureFlags.kt	100.00% <ø> (ø)	3.00 <0.00> (ø)
...a/org/mozilla/fenix/browser/BaseBrowserFragment.kt	0.00% <0.00%> (ø)	0.00 <0.00> (ø)
.../java/org/mozilla/fenix/browser/BrowserFragment.kt	0.00% <0.00%> (ø)	0.00 <0.00> (ø)
...zilla/fenix/components/metrics/MetricController.kt	35.51% <0.00%> (-0.68%)	0.00 <0.00> (ø)
...mponents/searchengine/FenixSearchEngineProvider.kt	63.96% <0.00%> (+1.80%)	14.00% <0.00%> (+1.00%)
...in/java/org/mozilla/fenix/settings/SupportUtils.kt	77.55% <0.00%> (+6.12%)	13.00% <0.00%> (+2.00%)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 35f63f8...efba9aa. Read the comment docs.

[sv-ohorvath]

@AaronMT @isabelrios Do you think I should add this to all test classes to be safe?

[AaronMT]

We should move this into a spot where this setting can be inherited into tests like into the ActivityTestRule. E.g, can use the apply method and write an override in https://github.com/mozilla-mobile/fenix/blob/master/app/src/androidTest/java/org/mozilla/fenix/helpers/HomeActivityTestRule.kt e.g, https://github.com/timrijckaert/BetterActivityTestRule/blob/61ad0802859218d67cbb4907df35f78bd05b3220/app/src/androidTest/java/be/rijckaert/tim/disableanimations/BetterActivityTestRule.kt#L36 – doesn't have to be as convoluted but see where they use executeShellCommand and where the functions are called there.

[sv-ohorvath]

We should move this into a spot where this setting can be inherited into tests like into the ActivityTestRule. E.g, can use the apply method and write an override in https://github.com/mozilla-mobile/fenix/blob/master/app/src/androidTest/java/org/mozilla/fenix/helpers/HomeActivityTestRule.kt e.g, https://github.com/timrijckaert/BetterActivityTestRule/blob/61ad0802859218d67cbb4907df35f78bd05b3220/app/src/androidTest/java/be/rijckaert/tim/disableanimations/BetterActivityTestRule.kt#L36 – doesn't have to be as convoluted but see where they use executeShellCommand and where the functions are called there.

@AaronMT Done, please review: Adding this to the apply method would stop the automatic launch of the activity for reasons I couldn't explain. So, I used a different method from ActivityTestRule, beforeActivityLaunched() which according to the documentation does what we want: "Override this method to execute any code that should run before your {@link Activity} is created and launched. This method is called before each test method, including any method annotated with Before".

[AaronMT]

Awesome. Good to know about beforeActivityLaunched