android-library icon indicating copy to clipboard operation
android-library copied to clipboard

Expose share by mail

Open tobiasKaminsky opened this issue 4 years ago • 8 comments
trafficstars

Signed-off-by: tobiasKaminsky [email protected]

tobiasKaminsky avatar Oct 26 '21 06:10 tobiasKaminsky

Change looks good, but tests are failing

AlvaroBrey avatar Oct 26 '21 06:10 AlvaroBrey

stable-IT test failed: https://www.kaminsky.me/nc-dev/library-integrationTests/581-IT-stable-06-59

@AlvaroBrey do you have an idea why suddenly so many tests on stable, but only on stable, fail?

tobiasKaminsky avatar Oct 27 '21 07:10 tobiasKaminsky

stable-IT test failed: https://www.kaminsky.me/nc-dev/library-integrationTests/581-IT-stable-06-59

@AlvaroBrey do you have an idea why suddenly so many tests on stable, but only on stable, fail?

It seems that most of the crashes are related to OCCapability. Any recent changes there?

AlvaroBrey avatar Oct 27 '21 09:10 AlvaroBrey

All solved except of lint, which complains as another dependency is outdated.

tobiasKaminsky avatar Nov 03 '21 07:11 tobiasKaminsky

/rebase

tobiasKaminsky avatar Nov 04 '21 08:11 tobiasKaminsky

Lint

TypemasterPR
Warnings11
Errors00

SpotBugs (new)

Warning Type Number
Bad practice Warnings 14
Correctness Warnings 38
Internationalization Warnings 6
Malicious code vulnerability Warnings 7
Multithreaded correctness Warnings 3
Performance Warnings 13
Security Warnings 1
Dodgy code Warnings 40
Total 122

SpotBugs (master)

Warning Type Number
Bad practice Warnings 14
Correctness Warnings 38
Internationalization Warnings 6
Malicious code vulnerability Warnings 7
Multithreaded correctness Warnings 3
Performance Warnings 13
Security Warnings 1
Dodgy code Warnings 40
Total 122

nextcloud-android-bot avatar Nov 04 '21 14:11 nextcloud-android-bot

Codecov Report

Merging #735 (20b435d) into master (012db19) will decrease coverage by 2.11%. The diff coverage is 71.42%.

:exclamation: Current head 20b435d differs from pull request most recent head c6c209c. Consider uploading reports for the commit c6c209c to get more accurate results

Additional details and impacted files
@@             Coverage Diff              @@
##             master     #735      +/-   ##
============================================
- Coverage     48.57%   46.47%   -2.11%     
============================================
  Files           200      164      -36     
  Lines          7697     6361    -1336     
  Branches        989      835     -154     
============================================
- Hits           3739     2956     -783     
+ Misses         3438     2965     -473     
+ Partials        520      440      -80     
Files Coverage Δ
...oud/android/lib/resources/status/OCCapability.java 59.63% <100.00%> (ø)
...sources/status/GetCapabilitiesRemoteOperation.java 48.83% <66.66%> (ø)

... and 362 files with indirect coverage changes

codecov[bot] avatar Feb 01 '24 06:02 codecov[bot]

SpotBugs

CategoryBaseNew
Bad practice3535
Correctness3434
Dodgy code2626
Internationalization66
Malicious code vulnerability4949
Multithreaded correctness33
Performance88
Total161161

github-actions[bot] avatar Feb 01 '24 06:02 github-actions[bot]

SpotBugs

CategoryBaseNew
Bad practice3535
Correctness3434
Dodgy code2626
Internationalization66
Malicious code vulnerability4949
Multithreaded correctness33
Performance88
Total161161

github-actions[bot] avatar Feb 12 '24 22:02 github-actions[bot]