android
android copied to clipboard
Upload in background
- [ ] Tests written, or not not needed
/rebase
Codecov Report
Merging #10737 (73aebc0) into master (64d8b0e) will increase coverage by
27.94%. The diff coverage is70.08%.
:exclamation: Current head 73aebc0 differs from pull request most recent head e2ef6c4. Consider uploading reports for the commit e2ef6c4 to get more accurate results
Additional details and impacted files
@@ Coverage Diff @@
## master #10737 +/- ##
=============================================
+ Coverage 3.29% 31.23% +27.94%
- Complexity 420 3281 +2861
=============================================
Files 539 542 +3
Lines 40264 40433 +169
Branches 5555 5568 +13
=============================================
+ Hits 1325 12628 +11303
+ Misses 38858 25961 -12897
- Partials 81 1844 +1763
| Impacted Files | Coverage Δ | |
|---|---|---|
| ...java/com/nextcloud/client/di/ComponentsModule.java | 0.00% <ø> (ø) |
|
| .../com/nextcloud/client/jobs/BackgroundJobManager.kt | 0.00% <ø> (ø) |
|
| ...pp/src/main/java/com/owncloud/android/MainApp.java | 54.71% <ø> (+53.80%) |
:arrow_up: |
| ...d/android/operations/SynchronizeFileOperation.java | 37.23% <0.00%> (+37.23%) |
:arrow_up: |
| ...android/operations/SynchronizeFolderOperation.java | 43.41% <ø> (+43.41%) |
:arrow_up: |
| ...ncloud/android/operations/UploadFileOperation.java | 58.59% <ø> (+58.59%) |
:arrow_up: |
| ...ud/android/providers/DocumentsStorageProvider.java | 52.70% <ø> (+52.70%) |
:arrow_up: |
| .../android/ui/activity/ConflictsResolveActivity.java | 23.76% <0.00%> (+23.76%) |
:arrow_up: |
| ...roid/ui/activity/ReceiveExternalFilesActivity.java | 0.35% <0.00%> (+0.35%) |
:arrow_up: |
| ...ncloud/android/ui/activity/UploadListActivity.java | 5.21% <0.00%> (+5.21%) |
:arrow_up: |
| ... and 292 more |
blue-Light-IT test failed: https://www.kaminsky.me/nc-dev/android-integrationTests/10737-Screenshot-blue-Light-07-43
Let's see what CI says :)
/rebase
@AlvaroBrey spotlessKotlinCheck fails, but auto format in AS is fine with it.
Rebased and fixed for m3 ✨
@tobiasKaminsky please check my last 2 commits, for reference
stable-IT test failed: https://www.kaminsky.me/nc-dev/android-integrationTests/6299-IT-stable-15-53
master-IT test failed: https://www.kaminsky.me/nc-dev/android-integrationTests/6299-IT-master-15-54
APK file: https://www.kaminsky.me/nc-dev/android-artifacts/10737.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.
blue-Light-Screenshot test failed: https://www.kaminsky.me/nc-dev/android-integrationTests/10737-Screenshot-blue-Light-16-08