Android: Fix or remove failing tests

Created on 5 Nov 2018  Â·  6Comments  Â·  Source: nextcloud/android

atm 2 tests for uploading are failing in drone CI. This breaks every build and thus makes QA for PRs rather complicated since you always have to check the build logs to see if you hit the test issue or if something else broke.

So to get CI back we should either remove or fix the tests.

@tobiasKaminsky

approved bug

All 6 comments

I took a look on Friday, but I'll have deeper look now.

So…This seems to be a server bug.
Uploading a 0b file leads to a complaining dav…

So we need to create a non-zero-bytes file within the tests, right? Do you have the time to implement this? (Just asking)

Yes, and no…
No, as we/client/user can upload 0b files, it should be fixed on server.

And yes, we should also test upload with non-zero files to make sure that this works.
And also with bigger files, so we test chunked upload.

PR with new test cases is #3233
For server, @rullzer knows, so we can close this.

Was this page helpful?
0 / 5 - 0 ratings

Related issues

tobiasKaminsky picture tobiasKaminsky  Â·  3Comments

JSoko picture JSoko  Â·  3Comments

Tie-fighter picture Tie-fighter  Â·  3Comments

AndyScherzinger picture AndyScherzinger  Â·  3Comments

JSoko picture JSoko  Â·  3Comments