Commit 118dfd1b authored by Luciano Pacheco's avatar Luciano Pacheco Committed by Commit Bot

Files app: Refactor the test disabled for FilesNg

Body tag can have other CSS classes thus the class equality check can
fail. Change to mark explicitly in the C++ browser test that the test
is disabled for FilesNg.

Bug: 1035691
Change-Id: I618a201fdecbb9785812c733196af47d160dac6d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2098163Reviewed-by: default avatarNoel Gordon <noel@chromium.org>
Commit-Queue: Luciano Pacheco <lucmult@chromium.org>
Cr-Commit-Position: refs/heads/master@{#749500}
parent e73c1f26
......@@ -958,7 +958,7 @@ WRAPPED_INSTANTIATE_TEST_SUITE_P(
FilesAppBrowserTest,
::testing::Values(TestCase("breadcrumbsNavigate").DisableFilesNg(),
TestCase("breadcrumbsLeafNoFocus").DisableFilesNg(),
TestCase("breadcrumbsTooltip"),
TestCase("breadcrumbsTooltip").DisableFilesNg(),
TestCase("breadcrumbsDownloadsTranslation")));
WRAPPED_INSTANTIATE_TEST_SUITE_P(
......
......@@ -121,12 +121,6 @@
const appId = await setupAndWaitUntilReady(
RootPath.DOWNLOADS, nestedFolderTestEntries, []);
// files-ng bail out: it does not have breadcrumb tooltips.
const body = await remoteCall.waitForElement(appId, 'body');
if (body.attributes['class'] === 'files-ng') {
return;
}
// Navigate to deepest folder.
const breadcrumb = '/My files/Downloads/' +
nestedFolderTestEntries.map(e => e.nameText).join('/');
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment