Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

SharedStorageTest::testGetPermissions fails #4683

Closed
MorrisJobke opened this issue May 4, 2017 · 3 comments
Closed

SharedStorageTest::testGetPermissions fails #4683

MorrisJobke opened this issue May 4, 2017 · 3 comments
Labels
bug high regression
Milestone
Nextcloud 12.0

Comments

@MorrisJobke
Copy link
Member 

There was 1 failure:
1) OCA\Files_Sharing\Tests\SharedStorageTest::testGetPermissions
Failed asserting that 11 is identical to 9.
/drone/src/github.com/nextcloud/server/apps/files_sharing/tests/SharedStorageTest.php:213

@icewind1991 @schiessle @rullzer Do you have an idea why this test is failing? I try to bisect this.
@MorrisJobke MorrisJobke added this to the Nextcloud 12.0 milestone May 4, 2017
@MorrisJobke
Copy link
Member Author

Broken by #4623 - let me revert it.

@MorrisJobke
Copy link
Member Author

Fixed via #4684

@rullzer
Copy link
Member

rullzer commented May 4, 2017

🙈 that test is just wrong... we should have permissions to update it... since you can rename a share mountpoint... will take care of pr with fix...

@nextcloud-bot nextcloud-bot mentioned this issue Jul 11, 2018
Closed
@nextcloud-bot nextcloud-bot mentioned this issue Sep 12, 2018
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug high regression
Projects
None yet
Milestone
Nextcloud 12.0
Development

No branches or pull requests
2 participants
@rullzer @MorrisJobke