Fix - AbstractIT

[alperozturk96]

Resolved issues encountered during local environment testing with AbstractIT.

1. The test was forcing the use http instead of https, which caused the waitForServer() function to fail.

2. The removeOnClient() method was unable to delete files as expected, leading to test failures. File can be locked, thus remove will fail.

[github-actions]

SpotBugs

Category	Base	New
Bad practice	35	35
Correctness	34	34
Dodgy code	26	26
Internationalization	6	6
Malicious code vulnerability	49	49
Multithreaded correctness	3	3
Performance	8	8
Total	161	161

[github-actions]

SpotBugs

Category	Base	New
Bad practice	35	35
Correctness	34	34
Dodgy code	26	26
Internationalization	6	6
Malicious code vulnerability	49	49
Multithreaded correctness	3	3
Performance	8	8
Total	161	161

[nextcloud-android-bot]

stable-IT test failed: https://www.kaminsky.me/nc-dev/android-library-integrationTests/4074-IT-stable-08-54/debug/

[nextcloud-android-bot]

master-IT test failed: https://www.kaminsky.me/nc-dev/android-library-integrationTests/4074-IT-master-08-55/debug/

[alperozturk96]

ToggleEncryptionRemoteOperation in removeOnClient() fails locally, even with these changes, but works on our server.

I'm getting a 403 Forbidden error.

The username for the local instance is "user".

[github-actions]

SpotBugs

Category	Base	New
Bad practice	35	35
Correctness	34	34
Dodgy code	26	26
Internationalization	6	6
Malicious code vulnerability	49	49
Multithreaded correctness	3	3
Performance	8	8
Total	161	161

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 43.07%. Comparing base (2d47aa4) to head (58bd46a).
Report is 132 commits behind head on master.

Additional details and impacted files

@@             Coverage Diff              @@
##             master    #1721      +/-   ##
============================================
- Coverage     43.20%   43.07%   -0.13%     
- Complexity      981      988       +7     
============================================
  Files           229      235       +6     
  Lines          8490     8562      +72     
  Branches       1109     1119      +10     
============================================
+ Hits           3668     3688      +20     
- Misses         4317     4359      +42     
- Partials        505      515      +10     

see 14 files with indirect coverage changes

🚀 New features to boost your workflow:

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

[github-actions]

SpotBugs

Category	Base	New
Bad practice	35	35
Correctness	34	34
Dodgy code	26	26
Internationalization	6	6
Malicious code vulnerability	49	49
Multithreaded correctness	3	3
Performance	8	8
Total	161	161

[ZetaTom]

ToggleEncryptionRemoteOperation in removeOnClient() fails locally, even with these changes, but works on our server.

I'm getting a 403 Forbidden error.

The username for the local instance is "user".

During testing we've noticed that this does indeed happen while running test cases explicitly. I've also been able to replicate this on my machine. However, when placing a breakpoint in ToggleEncryptionRemoteOperation and manually unlocking a folder in the app, a 200 status code is returned instead.

[github-actions]

SpotBugs

Category	Base	New
Bad practice	34	35
Correctness	34	34
Dodgy code	26	26
Internationalization	6	6
Malicious code vulnerability	49	49
Multithreaded correctness	11	3
Performance	8	8
Total	168	161