[rfontanarosa]

closes #3974

[gemini-code-assist]

Code Review

This pull request introduces a mechanism to prevent duplicate submissions in DataCollectionViewModel by tracking the submission state with an AtomicBoolean (isSubmitting). It refactors the submission logic into a submit function and adds a corresponding unit test to verify that multiple taps on the last task only trigger a single submission. The review feedback highlights a potential data loss issue where clearDraft() is called before saveChanges(); if saving fails, the draft is already deleted. It is recommended to move clearDraft() to execute only after saveChanges() completes successfully.

[codecov]

Codecov Report

❌ Patch coverage is 82.35294% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 70.03%. Comparing base (44237d4) to head (b7f0db1).

Files with missing lines	Patch %	Lines
...droid/ui/datacollection/DataCollectionViewModel.kt	82.35%	3 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff              @@
##             master    #3989      +/-   ##
============================================
- Coverage     70.03%   70.03%   -0.01%     
- Complexity     2106     2108       +2     
============================================
  Files           441      441              
  Lines         11869    11877       +8     
  Branches       1545     1545              
============================================
+ Hits           8313     8318       +5     
- Misses         2780     2783       +3     
  Partials        776      776              

Files with missing lines	Coverage Δ
...droid/ui/datacollection/DataCollectionViewModel.kt	80.32% <82.35%> (-0.61%)	⬇️

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.