Skip to content

Conversation

@PVince81
Copy link
Contributor

Description

Port sharing related tests to use the Share2.0 API.
Also moved old ShareTest tests to be integration tests instead, as they
cannot be applied directly as unit tests.

Related Issue

Ref #26608

Motivation and Context

We want to move away from the old sharing code for files and folders, so the first step is to adjust the tests to not rely on that one. Then later on adjust the code itself, this is what I tried here #26608

How Has This Been Tested?

Just run the tests.

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Tech debt

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@DeepDiver1975 @jvillafanez @butonic

@PVince81 PVince81 added this to the 9.2 milestone Nov 11, 2016
@mention-bot
Copy link

@PVince81, thanks for your PR! By analyzing the history of the files in this pull request, we identified @nickvergessen, @rullzer and @butonic to be potential reviewers.

@PVince81
Copy link
Contributor Author

Note: there is likely more but this is a first step forward.

@PVince81
Copy link
Contributor Author

17:43:27 1) Test\Files\ViewTest::testMoveMountPointIntoSharedFolder
17:43:27 InvalidArgumentException: Share does not have a full id
17:43:27 
17:43:27 /var/lib/jenkins/workspace/owncloud-core_core_PR-26610-7CGLOWUWW7YCOI3Q3BQJZ42IU2XWJIECIS423LISWQMF5CKP646Q/lib/private/Share20/Manager.php:775
17:43:27 /var/lib/jenkins/workspace/owncloud-core_core_PR-26610-7CGLOWUWW7YCOI3Q3BQJZ42IU2XWJIECIS423LISWQMF5CKP646Q/tests/lib/Files/ViewTest.php:1630
17:43:27 
17:43:27 2) OCA\Files_Sharing\Tests\UpdaterTest::testDeleteParentFolder
17:43:27 OCP\Share\Exceptions\ShareNotFound: Unspecified share exception
17:43:27 
17:43:27 /var/lib/jenkins/workspace/owncloud-core_core_PR-26610-7CGLOWUWW7YCOI3Q3BQJZ42IU2XWJIECIS423LISWQMF5CKP646Q/lib/private/Share20/DefaultShareProvider.php:534
17:43:27 /var/lib/jenkins/workspace/owncloud-core_core_PR-26610-7CGLOWUWW7YCOI3Q3BQJZ42IU2XWJIECIS423LISWQMF5CKP646Q/lib/private/Share20/Manager.php:963
17:43:27 /var/lib/jenkins/workspace/owncloud-core_core_PR-26610-7CGLOWUWW7YCOI3Q3BQJZ42IU2XWJIECIS423LISWQMF5CKP646Q/apps/files_sharing/tests/UpdaterTest.php:113

Hmm strange, but on the other PR these passed...

Also moved old ShareTest tests to be integration tests instead, as they
cannot be applied directly as unit tests.
@PVince81
Copy link
Contributor Author

Here, fixed.

@DeepDiver1975
Copy link
Member

👍

@DeepDiver1975 DeepDiver1975 merged commit 2d77366 into master Nov 14, 2016
@DeepDiver1975 DeepDiver1975 deleted the port-tests-share20 branch November 14, 2016 08:43
PVince81 pushed a commit that referenced this pull request Nov 24, 2016
Partial backport of the integration tests from #26610
PVince81 pushed a commit that referenced this pull request Nov 24, 2016
Partial backport of the integration tests from #26610
PVince81 pushed a commit that referenced this pull request Nov 24, 2016
Partial backport of the integration tests from #26610
@lock
Copy link

lock bot commented Aug 4, 2019

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked as resolved and limited conversation to collaborators Aug 4, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants