Modified: trunk/LayoutTests/ChangeLog (293110 => 293111)
--- trunk/LayoutTests/ChangeLog 2022-04-20 19:38:00 UTC (rev 293110)
+++ trunk/LayoutTests/ChangeLog 2022-04-20 20:18:21 UTC (rev 293111)
@@ -1,3 +1,18 @@
+2022-04-20 Commit Queue <[email protected]>
+
+ Unreviewed, reverting r292508.
+ https://bugs.webkit.org/show_bug.cgi?id=239561
+
+ Links to test results are partly broken in layout-tests-
+ results/results.html
+
+ Reverted changeset:
+
+ "Fix expected, actual links for variant-based imported wpt
+ tests"
+ https://bugs.webkit.org/show_bug.cgi?id=238832
+ https://commits.webkit.org/r292508
+
2022-04-20 Karl Rackler <[email protected]>
[ macOS wk2 ] fast/scrolling/overflow-scroll-past-max.html is a flaky failure
Modified: trunk/LayoutTests/fast/harness/results.html (293110 => 293111)
--- trunk/LayoutTests/fast/harness/results.html 2022-04-20 19:38:00 UTC (rev 293110)
+++ trunk/LayoutTests/fast/harness/results.html 2022-04-20 20:18:21 UTC (rev 293111)
@@ -280,19 +280,13 @@
return null;
}
- static testPrefix(testName)
+ static stripExtension(testName)
{
// Temporary fix, also in Tools/Scripts/webkitpy/layout_tests/constrollers/test_result_writer.py, line 95.
// FIXME: Refactor to avoid confusing reference to both test and process names.
- let parts = Utils.splitExtension(testName);
- let prefix = parts[0];
- if (parts[1].includes('?'))
- prefix += '_' + parts[1].split('?')[1]
- else if (parts[1].includes('#'))
- prefix += '_' + parts[1].split('#')[1]
- else if (Utils.splitExtension(parts[0])[1].length > 5)
+ if (Utils.splitExtension(testName)[1].length > 5)
return testName;
- return prefix;
+ return Utils.splitExtension(testName)[0];
}
static splitExtension(testName)
@@ -1174,7 +1168,7 @@
TestResultsController._getResultContainer(node).remove();
else if (url.match('-actual.png$')) {
let name = Utils.parentOfType(node, 'tbody').querySelector('.test-link').textContent;
- TestResultsController._getResultContainer(node).outerHTML = togglingImageFunction(Utils.testPrefix(name));
+ TestResultsController._getResultContainer(node).outerHTML = togglingImageFunction(Utils.stripExtension(name));
}
}
}
@@ -1350,7 +1344,7 @@
let actualTokens = testResult.info.actual.split(/\s+/);
- let testPrefix = Utils.testPrefix(testResult.name);
+ let testPrefix = Utils.stripExtension(testResult.name);
let imageResults = this.imageResultLinks(testResult, testPrefix, actualTokens[0]);
if (!imageResults && actualTokens.length > 1)
imageResults = this.imageResultLinks(testResult, 'retries/' + testPrefix, actualTokens[1]);
@@ -1380,12 +1374,12 @@
appendTextFailureLinks(testResult, cell)
{
- cell.innerHTML += this._resultsController.textResultLinks(Utils.testPrefix(testResult.name));
+ cell.innerHTML += this._resultsController.textResultLinks(Utils.stripExtension(testResult.name));
}
appendAudioFailureLinks(testResult, cell)
{
- let prefix = Utils.testPrefix(testResult.name);
+ let prefix = Utils.stripExtension(testResult.name);
cell.innerHTML += TestResultsController.resultLink(prefix, '-expected.wav', 'expected audio')
+ TestResultsController.resultLink(prefix, '-actual.wav', 'actual audio')
+ TestResultsController.resultLink(prefix, '-diff.txt', 'textual diff');
@@ -1393,7 +1387,7 @@
appendActualOnlyLinks(testResult, cell)
{
- let prefix = Utils.testPrefix(testResult.name);
+ let prefix = Utils.stripExtension(testResult.name);
if (testResult.isMissingAudio())
cell.innerHTML += TestResultsController.resultLink(prefix, '-actual.wav', 'audio result');
@@ -1477,7 +1471,7 @@
fillTestResultCell(testResult, cell)
{
- cell.innerHTML = TestResultsController.resultLink(Utils.testPrefix(testResult.name), '-stderr.txt', 'stderr');
+ cell.innerHTML = TestResultsController.resultLink(Utils.stripExtension(testResult.name), '-stderr.txt', 'stderr');
}
};
@@ -1487,7 +1481,7 @@
fillTestResultCell(testResult, cell)
{
// FIXME: only include timeout actual/diff results here if we actually spit out results for timeout tests.
- cell.innerHTML = this._resultsController.textResultLinks(Utils.testPrefix(testResult.name));
+ cell.innerHTML = this._resultsController.textResultLinks(Utils.stripExtension(testResult.name));
}
};
@@ -1496,8 +1490,8 @@
fillTestResultCell(testResult, cell)
{
- cell.innerHTML = TestResultsController.resultLink(Utils.testPrefix(testResult.name), '-crash-log.txt', 'crash log')
- + TestResultsController.resultLink(Utils.testPrefix(testResult.name), '-sample.txt', 'sample');
+ cell.innerHTML = TestResultsController.resultLink(Utils.stripExtension(testResult.name), '-crash-log.txt', 'crash log')
+ + TestResultsController.resultLink(Utils.stripExtension(testResult.name), '-sample.txt', 'sample');
}
};
@@ -1505,7 +1499,7 @@
sectionTitle() { return 'Other crashes'; }
fillTestResultCell(testResult, cell)
{
- cell.innerHTML = TestResultsController.resultLink(Utils.testPrefix(testResult.name), '-crash-log.txt', 'crash log');
+ cell.innerHTML = TestResultsController.resultLink(Utils.stripExtension(testResult.name), '-crash-log.txt', 'crash log');
}
createHistoryCell(testResult)
Modified: trunk/Tools/ChangeLog (293110 => 293111)
--- trunk/Tools/ChangeLog 2022-04-20 19:38:00 UTC (rev 293110)
+++ trunk/Tools/ChangeLog 2022-04-20 20:18:21 UTC (rev 293111)
@@ -1,3 +1,18 @@
+2022-04-20 Commit Queue <[email protected]>
+
+ Unreviewed, reverting r292508.
+ https://bugs.webkit.org/show_bug.cgi?id=239561
+
+ Links to test results are partly broken in layout-tests-
+ results/results.html
+
+ Reverted changeset:
+
+ "Fix expected, actual links for variant-based imported wpt
+ tests"
+ https://bugs.webkit.org/show_bug.cgi?id=238832
+ https://commits.webkit.org/r292508
+
2022-04-18 Justin Michaud <[email protected]>
[PGO] run-benchmark should allow pgo profiles to be collected during a benchmark run
Modified: trunk/Tools/Scripts/webkitpy/layout_tests/controllers/test_result_writer.py (293110 => 293111)
--- trunk/Tools/Scripts/webkitpy/layout_tests/controllers/test_result_writer.py 2022-04-20 19:38:00 UTC (rev 293110)
+++ trunk/Tools/Scripts/webkitpy/layout_tests/controllers/test_result_writer.py 2022-04-20 20:18:21 UTC (rev 293111)
@@ -102,14 +102,17 @@
output_basename = ext_parts[0]
if len(ext_parts) > 1 and '?' in ext_parts[1]:
output_basename += '_' + ext_parts[1].split('?')[1]
- elif len(ext_parts) > 1 and '#' in ext_parts[1]:
+ if len(ext_parts) > 1 and '#' in ext_parts[1]:
output_basename += '_' + ext_parts[1].split('#')[1]
- elif len(fs.splitext(output_basename)[1]) - 1 > 5:
- # Temporary fix, also in LayoutTests/fast/harness/results.html, line 275.
- # FIXME: Refactor to avoid confusing reference to both test and process names.
- return fs.join(self._root_output_dir, self._test_name) + modifier
- return fs.join(self._root_output_dir, output_basename) + modifier
+ output_filename = fs.join(self._root_output_dir, output_basename + ext_parts[1])
+
+ # Temporary fix, also in LayoutTests/fast/harness/results.html, line 275.
+ # FIXME: Refactor to avoid confusing reference to both test and process names.
+ if len(fs.splitext(output_filename)[1]) - 1 > 5:
+ return output_filename + modifier
+ return fs.splitext(output_filename)[0] + modifier
+
def _write_binary_file(self, path, contents):
if contents is not None:
self._make_output_directory()
Modified: trunk/Tools/Scripts/webkitpy/layout_tests/controllers/test_result_writer_unittest.py (293110 => 293111)
--- trunk/Tools/Scripts/webkitpy/layout_tests/controllers/test_result_writer_unittest.py 2022-04-20 19:38:00 UTC (rev 293110)
+++ trunk/Tools/Scripts/webkitpy/layout_tests/controllers/test_result_writer_unittest.py 2022-04-20 20:18:21 UTC (rev 293111)
@@ -61,17 +61,3 @@
fs = host.filesystem
writer = test_result_writer.TestResultWriter(fs, port, port.results_directory(), 'require-corp-revalidated-images.https.html')
self.assertEqual(writer.output_filename('-diff.txt'), fs.join(port.results_directory(), 'require-corp-revalidated-images.https-diff.txt'))
-
- def test_output_filename_variant(self):
- host = MockHost()
- port = TestPort(host)
- fs = host.filesystem
- writer = test_result_writer.TestResultWriter(fs, port, port.results_directory(), 'template_test/pbkdf2.https.any.worker.html?1-1000')
- self.assertEqual(fs.join(port.results_directory(), 'template_test/pbkdf2.https.any.worker_1-1000-diff.txt'), writer.output_filename('-diff.txt'))
-
- def test_output_filename_variant(self):
- host = MockHost()
- port = TestPort(host)
- fs = host.filesystem
- writer = test_result_writer.TestResultWriter(fs, port, port.results_directory(), 'template_test2/pbkdf2.https.any.html?1-1000')
- self.assertEqual(fs.join(port.results_directory(), 'template_test2/pbkdf2.https.any_1-1000-diff.txt'), writer.output_filename('-diff.txt'))
