Skip to content

Adjust acceptance tests to incoming shares being accepted by default - #20057

Merged
rullzer merged 3 commits into
masterfrom
adjust-acceptance-tests-to-incoming-shares-being-accepted-by-default
Mar 20, 2020
Merged

Adjust acceptance tests to incoming shares being accepted by default#20057
rullzer merged 3 commits into
masterfrom
adjust-acceptance-tests-to-incoming-shares-being-accepted-by-default

Conversation

@danxuliu

@danxuliudanxuliu commented Mar 20, 2020

Copy link
Copy Markdown
Member

Follow up to #18940

Now the acceptance tests assume that the share is accepted by default (so they no longer wait for the notification), and a specific test was added to verify the behaviour when the user has disabled accepting shares by default.

This reverts commit f402e23.
Since eb4c42d now shares are accepted
by default, so no notification is shown to accept them.
Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
@danxuliudanxuliu added bug 3. to review Waiting for reviews labels Mar 20, 2020
@danxuliudanxuliu added this to the Nextcloud 19 milestone Mar 20, 2020
@danxuliu

Copy link
Copy Markdown
MemberAuthor

/backport to stable18

@rullzer
rullzer merged commit fcf1810 into masterMar 20, 2020
@rullzer
rullzer deleted the adjust-acceptance-tests-to-incoming-shares-being-accepted-by-default branch March 20, 2020 20:23
@backportbot-nextcloud

Copy link
Copy Markdown

backport to stable18 in #20060

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

Labels

3. to reviewWaiting for reviewsbug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@danxuliu@rullzer@skjnldsv