Make Maestro run for each PR push

[jmartinesp]

Content

Follow up of #4092, since it wasn't working as expected.

• Remove the workflow_run configuration for the Maestro job.
• Make the 'build apk' job always run again too.
• Make sure the test results are uploaded when the Maestro job fails too (this broke somehow).

Motivation and context

Ideally we wanted to run Maestro once the 'Build APK' flow was finished and reuse its output so we don't have to build the APK twice, but this has proven difficult to achieve, since the 2 options GH actions gives us for that are:

• workflow_run, which can set up dependencies in a 'when X flow finishes, run flow Y' way, and it's technically what we want, but has the downside that it'll always use the .yml contents of the develop branch, which makes it difficult to maintain or change (as proven by Test using Maestro CLI + emulator instead of Cloud #4092 not really working after being merged).
• workflow_call, which allows a flow to explicitly start another: it worked as seen in this commit, but ended up in either the Maestro flow being run and displayed as part of the Build APK one or viceversa, and in both cases it was quite awkward since you couldn't see the results directly in the 'checks' UI.

So in the end we're just re-building the APK until a better option appears.

Tests

There should be a working 'Maestro (local)' flow in the list of actions run.

Checklist

• Changes have been tested on an Android device or Android emulator with API 24
• UI change has been tested on both light and dark themes
• Accessibility has been taken into account. See https://github.com/element-hq/element-x-android/blob/develop/CONTRIBUTING.md#accessibility
• Pull request is based on the develop branch
• Pull request title will be used in the release note, it clearly define what will change for the user
• Pull request includes screenshots or videos if containing UI changes
• You've made a self review of your PR

[github-actions]

📱 Scan the QR code below to install the build (arm64 only) for this PR.

If you can't scan the QR code you can install the build via this link: https://i.diawi.com/xJ5t3f

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 83.28%. Comparing base (6d1dac6) to head (42a276b).
Report is 21 commits behind head on develop.

Additional details and impacted files

@@           Coverage Diff            @@
##           develop    #4121   +/-   ##
========================================
  Coverage    83.28%   83.28%           
========================================
  Files         1885     1885           
  Lines        49096    49096           
  Branches      5764     5764           
========================================
  Hits         40888    40888           
  Misses        6139     6139           
  Partials      2069     2069           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[bmarty]

Thanks for the follow-up!