Attachment 342921 Details for Bug 186755 – Patch

[patch] Patch

bug-186755-20180618125216.patch (text/plain), 2.03 KB, created by Zan Dobersek on 2018-06-18 03:52:17 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Zan Dobersek

Created: 2018-06-18 03:52:17 PDT

Size: 2.03 KB

patch

obsolete

>Subversion Revision: 232923
>diff --git a/Tools/ChangeLog b/Tools/ChangeLog
>index 2f6a513bb01d03f9329186c2e7c716f414daa060..ab6bd327b6751c97118ac7ef8075dfa2dd795b5d 100644
>--- a/Tools/ChangeLog
>+++ b/Tools/ChangeLog
>@@ -1,3 +1,24 @@
>+2018-06-18  Zan Dobersek  <zdobersek@igalia.com>
>+
>+        [webkitpy] WPTRunner should remove any metadata content before (re)generating it
>+        https://bugs.webkit.org/show_bug.cgi?id=186755
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        WPTRunner generates the WPT-compliant metadata content before each test
>+        run. Before it does so, it should remove the complete target directory
>+        for this metadata, allowing any change in the expectations JSON to be
>+        immediately reflected.
>+
>+        For example, if a test failure expectation is removed, the related .ini
>+        metadata file is not generated anymore, but the stale .ini file isn't
>+        removed from the current metadata directory. To avoid this, purging
>+        the metadata directory and regenerating it from scratch should be done
>+        for each test run.
>+
>+        * Scripts/webkitpy/w3c/wpt_runner.py:
>+        (WPTRunner._generate_metadata_directory):
>+
> 2018-06-12  Darin Adler  <darin@apple.com>
> 
>         [Cocoa] Make some RetainPtr refinements to get more ready for ARC
>diff --git a/Tools/Scripts/webkitpy/w3c/wpt_runner.py b/Tools/Scripts/webkitpy/w3c/wpt_runner.py
>index 22a528a4aaa63c725cfdada20e4b7216d779d3a5..94c52a3038b5d5b2315d6b5ab6550811bea3d285 100644
>--- a/Tools/Scripts/webkitpy/w3c/wpt_runner.py
>+++ b/Tools/Scripts/webkitpy/w3c/wpt_runner.py
>@@ -146,6 +146,8 @@ class WPTRunner(object):
>         with self._host.filesystem.open_text_file_for_reading(expectations_file) as fd:
>             expectations = json.load(fd)
> 
>+        self._host.filesystem.rmtree(metadata_path)
>+
>         for test_name, test_data in expectations.iteritems():
>             ini_file = self._host.filesystem.join(metadata_path, test_name + ".ini")
>             self._host.filesystem.maybe_make_directory(self._host.filesystem.dirname(ini_file))

Actions: View | Formatted Diff | Diff

Attachments on bug 186755: 342921 | 342926 | 342929