Improve File Download Tracking

[alperozturk96]

• Tests written, or not not needed

Resolves: #14850

Tested with XLSX and PDF files

Downloading Small PDF

Screen.Recording.2025-05-02.at.15.55.14.mp4

Downloading Large PDF

ddd.mp4

Optional: Showing FileDetailFragment Based on Download Speed:

While 1MB may be considered small under typical internet speeds, it can take several minutes to download for users with slower connections. In such cases, displaying the FileDetailFragment instead of skipping it might be more meaningful. We could potentially detect the user's download speed and, if the estimated download time exceeds 5 seconds, show the FileDetailFragment. However, this may be a feature to consider in the future.

[alperozturk96]

This entire preview logic belongs in FileDisplayActivity and needs to be accessible from there as well,
because after the file download is completed, the fragment is no longer an instance of OCFileListFragment,
so this function cannot be used from FileDisplayActivity.

[github-actions]

Codacy

Lint

Type	master	PR
Warnings	51	50
Errors	11	11

SpotBugs

Category	Base	New
Bad practice	65	65
Correctness	62	62
Dodgy code	300	300
Experimental	1	1
Internationalization	7	7
Malicious code vulnerability	2	2
Multithreaded correctness	35	35
Performance	48	48
Security	18	18
Total	538	538

[github-actions]

APK file: https://www.kaminsky.me/nc-dev/android-artifacts/14857.apk



To test this change/fix you can simply download above APK file and install and test it in parallel to your existing Nextcloud app.

[codecov]

Codecov Report

Attention: Patch coverage is 1.02041% with 97 lines in your changes missing coverage. Please review.

Project coverage is 23.11%. Comparing base (b0b163f) to head (bc92642).
Report is 7 commits behind head on master.

Files with missing lines	Patch %	Lines
...cloud/android/ui/activity/FileDisplayActivity.java	2.08%	46 Missing and 1 partial ⚠️
...ncloud/android/ui/fragment/OCFileListFragment.java	0.00%	23 Missing and 1 partial ⚠️
...ncloud/android/ui/fragment/FileDetailFragment.java	0.00%	20 Missing ⚠️
...xtcloud/client/jobs/download/FileDownloadWorker.kt	0.00%	3 Missing ⚠️
...p/src/main/java/com/nextcloud/model/WorkerState.kt	0.00%	2 Missing ⚠️
...in/java/com/owncloud/android/datamodel/OCFile.java	0.00%	1 Missing ⚠️

Additional details and impacted files

@@              Coverage Diff              @@
##             master   #14857       +/-   ##
=============================================
+ Coverage          0   23.11%   +23.11%     
- Complexity        0     3223     +3223     
=============================================
  Files             0      712      +712     
  Lines             0    51713    +51713     
  Branches          0     6969     +6969     
=============================================
+ Hits              0    11955    +11955     
- Misses            0    37879    +37879     
- Partials          0     1879     +1879     

Files with missing lines	Coverage Δ
...loud/android/operations/DownloadFileOperation.java	32.16% <ø> (ø)
...in/java/com/owncloud/android/datamodel/OCFile.java	81.32% <0.00%> (ø)
...p/src/main/java/com/nextcloud/model/WorkerState.kt	50.00% <0.00%> (ø)
...xtcloud/client/jobs/download/FileDownloadWorker.kt	1.80% <0.00%> (ø)
...ncloud/android/ui/fragment/FileDetailFragment.java	0.00% <0.00%> (ø)
...ncloud/android/ui/fragment/OCFileListFragment.java	34.48% <0.00%> (ø)
...cloud/android/ui/activity/FileDisplayActivity.java	28.01% <2.08%> (ø)

... and 705 files with indirect coverage changes

🚀 New features to boost your workflow:

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

[alperozturk96]

/backport to stable-3.32

[alperozturk96]

/backport to stable-3.32