Skip to content

[stable20] Replace some usages of OC_DB in OC\Share\* with query builder - #23955

Merged
rullzer merged 1 commit into
stable20from
backport/23938/stable20
Nov 10, 2020
Merged

[stable20] Replace some usages of OC_DB in OC\Share\* with query builder#23955
rullzer merged 1 commit into
stable20from
backport/23938/stable20

Conversation

@backportbot-nextcloud

Copy link
Copy Markdown

backport of #23938

Signed-off-by: Joas Schilling <coding@schilljs.com>

@MorrisJobkeMorrisJobke left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🐘

@MorrisJobkeMorrisJobke added the 3. to review Waiting for reviews label Nov 6, 2020
@faily-bot

faily-botBot commented Nov 7, 2020

Copy link
Copy Markdown

🤖 beep boop beep 🤖

Here are the logs for the failed build:

Status of 35158: failure

mysql8.0-php7.2

Show full log
There were 2 warnings:
1) Test\Files\ViewTest::testRenameFailDeleteTargetKeepSource
Trying to configure method "writeStream" which cannot be configured because it does not exist, has not been specified, is final, or is static
2) Test\Files\ViewTest::testCopyFailDeleteTargetKeepSource
Trying to configure method "writeStream" which cannot be configured because it does not exist, has not been specified, is final, or is static
--
There was 1 failure:
1) OCA\OAuth2\Tests\Db\ClientMapperTest::testGetByIdentifier
Failed asserting that two objects are equal.
--- Expected
+++ Actual
@@ @@
'redirectUri' => 'https://example.com/'
'clientIdentifier' => 'MyAwesomeClientIdentifier'
'secret' => 'TotallyNotSecret'
- 'id' => 0
+ 'id' => 1
'_updatedFields' => Array ()
'_fieldTypes' => Array (...)
)
/drone/src/apps/oauth2/tests/Db/ClientMapperTest.php:51

acceptance-app-files-sharing-link

  • tests/acceptance/features/app-files-sharing-link.feature:101
Show full log
 Scenario: owner sees folder created in the public page of an editable shared folder # /drone/src/tests/acceptance/features/app-files-sharing-link.feature:101
Given I act as John # ActorContext::iActAs()
And I am logged in # LoginPageContext::iAmLoggedIn()
And I create a new folder named "Editable shared folder" # FileListContext::iCreateANewFolderNamed()
And I close the details view # FilesAppContext::iCloseTheDetailsView()
And I see that the details view is closed # FilesAppContext::iSeeThatTheDetailsViewIsClosed()
And I share the link for "Editable shared folder" # FilesAppSharingContext::iShareTheLinkFor()
And I set the shared link as editable # FilesAppSharingContext::iSetTheSharedLinkAsEditable()
And I write down the shared link # FilesAppSharingContext::iWriteDownTheSharedLink()
Element is no longer attached to the DOM
For documentation on this error, please visit: http://seleniumhq.org/exceptions/stale_element_reference.html
Build info: version: '2.53.1', revision: 'a36b8b1', time: '2016-06-30 17:37:03'
System info: host: '50d1343058b1', ip: '172.26.0.2', os.name: 'Linux', os.arch: 'amd64', os.version: '4.15.0-76-generic', java.version: '1.8.0_91'
Driver info: driver.version: unknown (WebDriver\Exception\StaleElementReference)
And I act as Jane # ActorContext::iActAs()
And I visit the shared link I wrote down # PublicShareContext::iVisitTheSharedLinkIWroteDown()
And I see that the current page is the shared link I wrote down # PublicShareContext::iSeeThatTheCurrentPageIsTheSharedLinkIWroteDown()
And I create a new folder named "Subfolder" # FileListContext::iCreateANewFolderNamed()
And I see that the file list contains a file named "Subfolder" # FileListContext::iSeeThatTheFileListContainsAFileNamed()
When I act as John # ActorContext::iActAs()
And I enter in the folder named "Editable shared folder" # FileListContext::iEnterInTheFolderNamed()
Then I see that the file list contains a file named "Subfolder" # FileListContext::iSeeThatTheFileListContainsAFileNamed()

@rullzerrullzer mentioned this pull request Nov 9, 2020

@rullzerrullzer left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Applied clean lets do it

@rullzer
rullzer merged commit 8edc29b into stable20Nov 10, 2020
@rullzer
rullzer deleted the backport/23938/stable20 branch November 10, 2020 07:59
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewWaiting for reviewsbugfeature: sharingtechnical debt🧱 🤔🚀

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@rullzer@MorrisJobke@nickvergessen