Skip to content

Conversation

@danxuliu
Copy link
Member

@danxuliu danxuliu commented Apr 3, 2019

Follow up to #14912

The acceptance tests in server did not fail because the file picker is not exercised in them, although it is in Talk tests.

File names are no longer shown directly in the ".filename" element, but
split in two "span" elements inside a ".filename-parts" element, so now
the texts in those span elements need to be concatenated to get the file
name.

Signed-off-by: Daniel Calviño Sánchez <[email protected]>
@danxuliu danxuliu added the 3. to review Waiting for reviews label Apr 3, 2019
@danxuliu danxuliu added this to the Nextcloud 16 milestone Apr 3, 2019
Copy link
Member

@skjnldsv skjnldsv left a comment

Choose a reason for hiding this comment

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

make sense

@faily-bot
Copy link

faily-bot bot commented Apr 3, 2019

🤖 beep boop beep 🤖

Here are the logs for the failed build:

Status of 17477: failure

TESTS=ui-regression

@MorrisJobke MorrisJobke merged commit f73f7bb into master Apr 3, 2019
@MorrisJobke MorrisJobke deleted the adjust-acceptance-tests-to-changes-of-file-names-in-file-picker branch April 3, 2019 15:55
@MorrisJobke MorrisJobke mentioned this pull request Apr 3, 2019
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants