Skip to content

Conversation

@tobiasKaminsky
Copy link
Member

@tobiasKaminsky tobiasKaminsky commented Apr 4, 2018

Depends on #137

This one re-activates all integration tests that were already in the project.
While doing this, I found nextcloud/server#9028.
Also if during upload ([Chunked]UploadRemoteFileOperation) a file is not found, it returns ResultCode.LOCAL_FILE_NOT_FOUND.

TODO

  • fix codacy
  • see if drone is working with it, also with local server docker image
  • rebase once Integration Tests #137 is merged

Signed-off-by: tobiasKaminsky tobias@kaminsky.me

@nextcloud nextcloud deleted a comment Apr 4, 2018
@nextcloud nextcloud deleted a comment Apr 4, 2018
@nextcloud nextcloud deleted a comment Apr 4, 2018
@nextcloud nextcloud deleted a comment Apr 5, 2018
@nextcloud nextcloud deleted a comment Apr 5, 2018
@nextcloud nextcloud deleted a comment Apr 5, 2018
@nextcloud nextcloud deleted a comment Aug 29, 2018
@AndyScherzinger
Copy link
Member

some tests keep failing while the log doesn't show which assertion failed

@AndyScherzinger
Copy link
Member

Tests aren't failing anymore while the test build still fails:

Tests on android-27(AVD) - 8.1.0 failed: Test run failed to complete. Expected 54 tests, received 3

@nextcloud nextcloud deleted a comment Aug 29, 2018
@AndyScherzinger AndyScherzinger force-pushed the reactivateTests branch 4 times, most recently from fd7de28 to 99a1daa Compare August 29, 2018 11:28
@nextcloud nextcloud deleted a comment Sep 12, 2018
@nextcloud nextcloud deleted a comment Sep 21, 2018
tobiasKaminsky and others added 9 commits September 25, 2018 17:34
- add integration test for ExceptionParser
- include integration tests
Signed-off-by: tobiasKaminsky <tobias@kaminsky.me>
Signed-off-by: tobiasKaminsky <tobias@kaminsky.me>
fire up two servers

Signed-off-by: tobiasKaminsky <tobias@kaminsky.me>
Signed-off-by: tobiasKaminsky <tobias@kaminsky.me>
Signed-off-by: tobiasKaminsky <tobias@kaminsky.me>
@tobiasKaminsky
Copy link
Member Author

Closing for now, but keeping branch.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants