fix: crash on code screen while requesting focus (WPB-6876)

[yamilmedina]

WPB-6876 [Android] Crash in CreateAccountCodeScreen.CodeContent

---

PR Submission Checklist for internal contributors

• The PR Title

  • conforms to the style of semantic commits messages¹ supported in Wire's Github Workflow²
  • contains a reference JIRA issue number like SQPIT-764
  • answers the question: If merged, this PR will: ... ³

• The PR Description

  • is free of optional paragraphs and you have filled the relevant parts to the best of your ability

---

What's new in this PR?

Issues

Sometimes when requesting focus, we are having an unhandled Exception:

Exception java.lang.IllegalStateException: FocusRequester is not initialized

Causes (Optional)

Crash when requesting focus during composition.

Solutions

The solutions suggested, were, applying delay, using onGloballyPositioned and the selected one:
Applying requestFocus call inside a coroutineContext.invokeOnCompletion so the effect will be canceled when leaving composition.

Testing

How to Test

Manually tested.

---

PR Post Submission Checklist for internal contributors (Optional)

• Wire's Github Workflow has automatically linked the PR to a JIRA issue

---

PR Post Merge Checklist for internal contributors

• If any soft of configuration variable was introduced by this PR, it has been added to the relevant documents and the CI jobs have been updated.

---

References

1. https://sparkbox.com/foundry/semantic_commit_messages
2. https://github.com/wireapp/.github#usage
3. E.g. feat(conversation-list): Sort conversations by most emojis in the title #SQPIT-764.

[AndroidBob]

Build 5025 failed.

[github-actions]

Test Results

931 tests  ±0   931 ✅ ±0   7m 43s ⏱️ - 1m 59s
124 suites ±0     0 💤 ±0 
124 files   ±0     0 ❌ ±0 

Results for commit d75760f. ± Comparison against base commit 40ebea4.

♻️ This comment has been updated with latest results.

[github-actions]

APKs built during tests are available here. Scroll down to Artifacts!

[AndroidBob]

Build 5031 succeeded.

The build produced the following APK's:

• com.wire.android-v4.8.0(9791923)(9791923)-staging-compat.apk

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 44.32%. Comparing base (40ebea4) to head (d75760f).

Additional details and impacted files

@@             Coverage Diff             @@
##           develop    #3043      +/-   ##
===========================================
+ Coverage    44.15%   44.32%   +0.16%     
===========================================
  Files          450      450              
  Lines        14520    14521       +1     
  Branches      2496     2497       +1     
===========================================
+ Hits          6411     6436      +25     
+ Misses        7407     7375      -32     
- Partials       702      710       +8     

see 4 files with indirect coverage changes

---

Continue to review full report in Codecov by Sentry.

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

[sonarcloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

[github-actions]

APKs built during tests are available here. Scroll down to Artifacts!

[AndroidBob]

Build 5037 succeeded.

The build produced the following APK's:

• com.wire.android-v4.8.0(9792485)(9792485)-staging-compat.apk