Refactor DataCollectionViewModel

[shobhitagarwal1612]

This PR is a pure refactor which is split from #2986 so that main logic change can be separated from refactors.

Things done:

• Instead of using currentTaskId.value as default task id in all method, pass the id from the callers
• Prevent re-creating task sequence for each value when storing values to local db.
• Cleanup unused code.

@anandwana001 PTAL?

[codecov]

Codecov Report

Attention: Patch coverage is 59.37500% with 13 lines in your changes missing coverage. Please review.

Project coverage is 63.41%. Comparing base (5d2e5ec) to head (dc09650).
Report is 1 commits behind head on master.

Files with missing lines	Patch %	Lines
...round/ui/datacollection/DataCollectionViewModel.kt	59.37%	9 Missing and 4 partials ⚠️

Additional details and impacted files

@@            Coverage Diff            @@
##             master    #2996   +/-   ##
=========================================
  Coverage     63.41%   63.41%           
+ Complexity     1244     1241    -3     
=========================================
  Files           267      267           
  Lines          6513     6505    -8     
  Branches        913      912    -1     
=========================================
- Hits           4130     4125    -5     
+ Misses         1796     1792    -4     
- Partials        587      588    +1     

Files with missing lines	Coverage Δ
...round/ui/datacollection/DataCollectionViewModel.kt	80.52% <59.37%> (+1.23%)	⬆️

... and 1 file with indirect coverage changes

[shobhitagarwal1612]

@anandwana001 PTAL?