Split Maestro CI flow in 2 jobs.

[jmartinesp]

Type of change

• Feature
• Bugfix
• Technical
• Other :

Content

Split the Maestro CI flow into 2 jobs.

Motivation and context

This way, if the Maestro tests fail for some runtime reason we don't have to build the APK (~10min) again to re-test it.

Tests

• The Maestro job should be restartable, independent from the 'Build APK' one.

A 2nd run of this job ran here: https://github.com/element-hq/element-x-android/actions/runs/9013826868/job/24765945553.

Tested devices

• Physical
• Emulator
• OS version(s):

Checklist

• Changes have been tested on an Android device or Android emulator with API 23
• 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 includes a new file under ./changelog.d. See https://github.com/element-hq/element-x-android/blob/develop/CONTRIBUTING.md#changelog
• Pull request includes screenshots or videos if containing UI changes
• Pull request includes a sign off
• You've made a self review of your PR

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

[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/2MZppG

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 73.93%. Comparing base (4c1bd2d) to head (6dd19ac).
Report is 1 commits behind head on develop.

Additional details and impacted files

@@           Coverage Diff            @@
##           develop    #2823   +/-   ##
========================================
  Coverage    73.93%   73.93%           
========================================
  Files         1528     1528           
  Lines        36467    36467           
  Branches      7053     7053           
========================================
  Hits         26961    26961           
  Misses        5811     5811           
  Partials      3695     3695           

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

[bmarty]

Thanks, this is definitely a very good improvement for devx

[bmarty]

Note urgent, but I am wondering if we could reuse the APK from the build.yml action. Not sure how to properly have the same ability to trigger a partial workflow when we want to trigger Maestro using a label or a worflow dispatch though.

[jmartinesp]

That's also my end goal here, but I think the complexity would skyrocket since we'd need few extra checks, something that triggers the flow if the artefact no longer exists, etc.