aboutsummaryrefslogtreecommitdiffstats
path: root/python
diff options
context:
space:
mode:
authorMartin Robinson <mrobinson@igalia.com>2024-08-28 04:29:51 -0700
committerGitHub <noreply@github.com>2024-08-28 11:29:51 +0000
commitb29b614775b562cc57d420d29ea190d5bafef014 (patch)
tree5d6a6faf7e1cb912edeba427c4a65e68b24d4cb6 /python
parentbb5547a5d05b1f002d9cce3197cfb9cdcb71d33c (diff)
downloadservo-b29b614775b562cc57d420d29ea190d5bafef014.tar.gz
servo-b29b614775b562cc57d420d29ea190d5bafef014.zip
ci: Use Servo's `wpt` repository for WPT export (#33224)
This switches the WPT export script to use Servo's fork the `wpt` upstream repository. The reasoning is that Servo developers's can fix issues with branches there. This isn't possible because no one has permissions to the servo-wpt-sync fork of the repository. I have given `servo-wpt-sync` write persmissions to `servo/wpt`. This also updates the tests a bit to reflect the new default repository. Signed-off-by: Martin Robinson <mrobinson@igalia.com>
Diffstat (limited to 'python')
-rwxr-xr-xpython/wpt/export.py2
-rw-r--r--python/wpt/test.py60
2 files changed, 31 insertions, 31 deletions
diff --git a/python/wpt/export.py b/python/wpt/export.py
index 6860a7a8ed9..e1c58228016 100755
--- a/python/wpt/export.py
+++ b/python/wpt/export.py
@@ -27,7 +27,7 @@ def main() -> int:
success = WPTSync(
servo_repo='servo/servo',
wpt_repo='web-platform-tests/wpt',
- downstream_wpt_repo='servo-wpt-sync/web-platform-tests',
+ downstream_wpt_repo='servo/wpt',
servo_path='./servo',
wpt_path='./wpt',
github_api_token=os.environ['WPT_SYNC_TOKEN'],
diff --git a/python/wpt/test.py b/python/wpt/test.py
index 742796035c6..e6231f8d0a0 100644
--- a/python/wpt/test.py
+++ b/python/wpt/test.py
@@ -363,8 +363,8 @@ class TestFullSyncRun(unittest.TestCase):
self.assertListEqual(
self.run_test("opened.json", ["18746.diff"]),
[
- "CreateOrUpdateBranchForPRStep:1:servo-wpt-sync/wpt/servo_export_18746",
- "OpenPRStep:servo-wpt-sync/wpt/servo_export_18746→wpt/wpt#1",
+ "CreateOrUpdateBranchForPRStep:1:servo/wpt/servo_export_18746",
+ "OpenPRStep:servo/wpt/servo_export_18746→wpt/wpt#1",
"CommentStep:servo/servo#18746:🤖 Opened new upstream WPT pull request "
"(wpt/wpt#1) with upstreamable changes.",
],
@@ -374,8 +374,8 @@ class TestFullSyncRun(unittest.TestCase):
self.assertListEqual(
self.run_test("opened.json", ["move-into-wpt.diff"]),
[
- "CreateOrUpdateBranchForPRStep:1:servo-wpt-sync/wpt/servo_export_18746",
- "OpenPRStep:servo-wpt-sync/wpt/servo_export_18746→wpt/wpt#1",
+ "CreateOrUpdateBranchForPRStep:1:servo/wpt/servo_export_18746",
+ "OpenPRStep:servo/wpt/servo_export_18746→wpt/wpt#1",
"CommentStep:servo/servo#18746:🤖 Opened new upstream WPT pull request "
"(wpt/wpt#1) with upstreamable changes.",
],
@@ -395,8 +395,8 @@ class TestFullSyncRun(unittest.TestCase):
],
),
[
- "CreateOrUpdateBranchForPRStep:1:servo-wpt-sync/wpt/servo_export_18746",
- "OpenPRStep:servo-wpt-sync/wpt/servo_export_18746→wpt/wpt#1",
+ "CreateOrUpdateBranchForPRStep:1:servo/wpt/servo_export_18746",
+ "OpenPRStep:servo/wpt/servo_export_18746→wpt/wpt#1",
"CommentStep:servo/servo#18746:🤖 Opened new upstream WPT pull request "
"(wpt/wpt#1) with upstreamable changes.",
],
@@ -406,8 +406,8 @@ class TestFullSyncRun(unittest.TestCase):
self.assertListEqual(
self.run_test("opened.json", ["move-out-of-wpt.diff"]),
[
- "CreateOrUpdateBranchForPRStep:1:servo-wpt-sync/wpt/servo_export_18746",
- "OpenPRStep:servo-wpt-sync/wpt/servo_export_18746→wpt/wpt#1",
+ "CreateOrUpdateBranchForPRStep:1:servo/wpt/servo_export_18746",
+ "OpenPRStep:servo/wpt/servo_export_18746→wpt/wpt#1",
"CommentStep:servo/servo#18746:🤖 Opened new upstream WPT pull request "
"(wpt/wpt#1) with upstreamable changes.",
],
@@ -436,11 +436,11 @@ class TestFullSyncRun(unittest.TestCase):
self.run_test(
"opened.json",
["18746.diff"],
- [MockPullRequest("servo-wpt-sync:servo_export_18746", 1)],
+ [MockPullRequest("servo:servo_export_18746", 1)],
),
[
"ChangePRStep:wpt/wpt#1:opened:This is a test:<!-- Please...[95]",
- "CreateOrUpdateBranchForPRStep:1:servo-wpt-sync/wpt/servo_export_18746",
+ "CreateOrUpdateBranchForPRStep:1:servo/wpt/servo_export_18746",
"CommentStep:servo/servo#18746:📝 Transplanted new upstreamable changes to "
"existing upstream WPT pull request (wpt/wpt#1).",
],
@@ -451,13 +451,13 @@ class TestFullSyncRun(unittest.TestCase):
self.run_test(
"opened.json",
["non-wpt.diff"],
- [MockPullRequest("servo-wpt-sync:servo_export_18746", 1)],
+ [MockPullRequest("servo:servo_export_18746", 1)],
),
[
"CommentStep:wpt/wpt#1:👋 Downstream pull request (servo/servo#18746) no longer "
"contains any upstreamable changes. Closing pull request without merging.",
"ChangePRStep:wpt/wpt#1:closed",
- "RemoveBranchForPRStep:servo-wpt-sync/wpt/servo_export_18746",
+ "RemoveBranchForPRStep:servo/wpt/servo_export_18746",
"CommentStep:servo/servo#18746:🤖 This change no longer contains upstreamable changes "
"to WPT; closed existing upstream pull request (wpt/wpt#1).",
]
@@ -467,8 +467,8 @@ class TestFullSyncRun(unittest.TestCase):
self.assertListEqual(
self.run_test("opened.json", ["add-non-utf8-file.diff"]),
[
- "CreateOrUpdateBranchForPRStep:1:servo-wpt-sync/wpt/servo_export_18746",
- "OpenPRStep:servo-wpt-sync/wpt/servo_export_18746→wpt/wpt#1",
+ "CreateOrUpdateBranchForPRStep:1:servo/wpt/servo_export_18746",
+ "OpenPRStep:servo/wpt/servo_export_18746→wpt/wpt#1",
"CommentStep:servo/servo#18746:🤖 Opened new upstream WPT pull request "
"(wpt/wpt#1) with upstreamable changes.",
],
@@ -481,7 +481,7 @@ class TestFullSyncRun(unittest.TestCase):
self.run_test(
"opened.json",
["does-not-apply-cleanly.diff"],
- [MockPullRequest("servo-wpt-sync:servo_export_18746", 1)],
+ [MockPullRequest("servo:servo_export_18746", 1)],
),
[
"ChangePRStep:wpt/wpt#1:opened:This is a test:<!-- Please...[95]",
@@ -501,11 +501,11 @@ class TestFullSyncRun(unittest.TestCase):
self.run_test(
"closed.json",
["18746.diff"],
- [MockPullRequest("servo-wpt-sync:servo_export_18746", 10)],
+ [MockPullRequest("servo:servo_export_18746", 10)],
),
[
"ChangePRStep:wpt/wpt#10:closed",
- "RemoveBranchForPRStep:servo-wpt-sync/wpt/servo_export_18746"
+ "RemoveBranchForPRStep:servo/wpt/servo_export_18746"
]
)
@@ -514,11 +514,11 @@ class TestFullSyncRun(unittest.TestCase):
self.run_test(
"synchronize.json",
["18746.diff"],
- [MockPullRequest("servo-wpt-sync:servo_export_19612", 10)],
+ [MockPullRequest("servo:servo_export_19612", 10)],
),
[
"ChangePRStep:wpt/wpt#10:opened:deny warnings:<!-- Please...[142]",
- "CreateOrUpdateBranchForPRStep:1:servo-wpt-sync/wpt/servo_export_19612",
+ "CreateOrUpdateBranchForPRStep:1:servo/wpt/servo_export_19612",
"CommentStep:servo/servo#19612:📝 Transplanted new upstreamable changes to existing "
"upstream WPT pull request (wpt/wpt#10).",
]
@@ -529,13 +529,13 @@ class TestFullSyncRun(unittest.TestCase):
self.run_test(
"synchronize.json",
["non-wpt.diff"],
- [MockPullRequest("servo-wpt-sync:servo_export_19612", 11)],
+ [MockPullRequest("servo:servo_export_19612", 11)],
),
[
"CommentStep:wpt/wpt#11:👋 Downstream pull request (servo/servo#19612) no longer contains any "
"upstreamable changes. Closing pull request without merging.",
"ChangePRStep:wpt/wpt#11:closed",
- "RemoveBranchForPRStep:servo-wpt-sync/wpt/servo_export_19612",
+ "RemoveBranchForPRStep:servo/wpt/servo_export_19612",
"CommentStep:servo/servo#19612:🤖 This change no longer contains upstreamable changes to WPT; "
"closed existing upstream pull request (wpt/wpt#11).",
]
@@ -545,8 +545,8 @@ class TestFullSyncRun(unittest.TestCase):
self.assertListEqual(
self.run_test("synchronize.json", ["18746.diff"]),
[
- "CreateOrUpdateBranchForPRStep:1:servo-wpt-sync/wpt/servo_export_19612",
- "OpenPRStep:servo-wpt-sync/wpt/servo_export_19612→wpt/wpt#1",
+ "CreateOrUpdateBranchForPRStep:1:servo/wpt/servo_export_19612",
+ "OpenPRStep:servo/wpt/servo_export_19612→wpt/wpt#1",
"CommentStep:servo/servo#19612:🤖 Opened new upstream WPT pull request "
"(wpt/wpt#1) with upstreamable changes.",
]
@@ -559,7 +559,7 @@ class TestFullSyncRun(unittest.TestCase):
self.run_test(
"synchronize.json",
["does-not-apply-cleanly.diff"],
- [MockPullRequest("servo-wpt-sync:servo_export_19612", 11)],
+ [MockPullRequest("servo:servo_export_19612", 11)],
),
[
"ChangePRStep:wpt/wpt#11:opened:deny warnings:<!-- Please...[142]",
@@ -575,7 +575,7 @@ class TestFullSyncRun(unittest.TestCase):
self.assertListEqual(
self.run_test(
"edited.json", ["wpt.diff"],
- [MockPullRequest("servo-wpt-sync:servo_export_19620", 10)]
+ [MockPullRequest("servo:servo_export_19620", 10)]
),
[
"ChangePRStep:wpt/wpt#10:open:A cool new title:Reference #<!--...[136]",
@@ -595,8 +595,8 @@ class TestFullSyncRun(unittest.TestCase):
"synchronize-multiple.json", ["18746.diff", "non-wpt.diff", "wpt.diff"]
),
[
- "CreateOrUpdateBranchForPRStep:2:servo-wpt-sync/wpt/servo_export_19612",
- "OpenPRStep:servo-wpt-sync/wpt/servo_export_19612→wpt/wpt#1",
+ "CreateOrUpdateBranchForPRStep:2:servo/wpt/servo_export_19612",
+ "OpenPRStep:servo/wpt/servo_export_19612→wpt/wpt#1",
"CommentStep:servo/servo#19612:"
"🤖 Opened new upstream WPT pull request (wpt/wpt#1) with upstreamable changes.",
]
@@ -610,11 +610,11 @@ class TestFullSyncRun(unittest.TestCase):
self.run_test(
"merged.json",
["18746.diff"],
- [MockPullRequest("servo-wpt-sync:servo_export_19620", 100)]
+ [MockPullRequest("servo:servo_export_19620", 100)]
),
[
"MergePRStep:wpt/wpt#100",
- "RemoveBranchForPRStep:servo-wpt-sync/wpt/servo_export_19620"
+ "RemoveBranchForPRStep:servo/wpt/servo_export_19620"
]
)
@@ -633,7 +633,7 @@ def setUpModule():
SYNC = WPTSync(
servo_repo="servo/servo",
wpt_repo="wpt/wpt",
- downstream_wpt_repo="servo-wpt-sync/wpt",
+ downstream_wpt_repo="servo/wpt",
servo_path=os.path.join(TMP_DIR, "servo-mock"),
wpt_path=os.path.join(TMP_DIR, "wpt-mock"),
github_api_token="",