From ace44499246124663d3e602e6462e5cc4b7d0c55 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Daniel=20Calvi=C3=B1o=20S=C3=A1nchez?= Date: Wed, 3 Apr 2019 16:29:52 +0200 Subject: [PATCH] Adjust acceptance tests to changes of file names in file picker MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit 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 --- tests/acceptance/features/bootstrap/FilePickerContext.php | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/tests/acceptance/features/bootstrap/FilePickerContext.php b/tests/acceptance/features/bootstrap/FilePickerContext.php index c62a505f70..d7659dbac8 100644 --- a/tests/acceptance/features/bootstrap/FilePickerContext.php +++ b/tests/acceptance/features/bootstrap/FilePickerContext.php @@ -48,7 +48,9 @@ class FilePickerContext implements Context, ActorAwareInterface { * @return Locator */ public static function rowForFile($fileName) { - return Locator::forThe()->xpath("//*[@id = 'picker-filestable']//*[contains(concat(' ', normalize-space(@class), ' '), ' filename ') and normalize-space() = '$fileName']/ancestor::tr")-> + // File names in the file picker are split in two span elements, so + // their texts need to be concatenated to get the full file name. + return Locator::forThe()->xpath("//*[@id = 'picker-filestable']//*[contains(concat(' ', normalize-space(@class), ' '), ' filename-parts ') and concat(span[1], span[2]) = '$fileName']/ancestor::tr")-> descendantOf(self::fileListContainer())-> describedAs("Row for file $fileName in the file picker dialog"); }