Use actual GitHub workflow URL format in test mock workflow run data

Author: Kobzol

src/bors/handlers/info.rs

@@ -182,7 +182,7 @@ mod tests {
             - Priority: 10
             - Mergeable: yes
             - Try build is in progress
-            	- Workflow URL: https://github.com/workflows/Workflow1/1
+            	- Workflow URL: https://github.com/rust-lang/borstest/actions/runs/1
             "
             );
             Ok(())

src/bors/handlers/pr_events.rs

@@ -733,7 +733,7 @@ mod tests {
 
                 Auto build cancelled due to push. Cancelled workflows:
 
-                - https://github.com/workflows/Workflow1/1
+                - https://github.com/rust-lang/borstest/actions/runs/1
                 ");
                 Ok(())
             })

src/bors/handlers/review.rs

@@ -1343,7 +1343,7 @@ merge_queue_enabled = true
 
                 Auto build cancelled due to unapproval. Cancelled workflows:
 
-                - https://github.com/workflows/Workflow1/1
+                - https://github.com/rust-lang/borstest/actions/runs/1
                 ");
                 Ok(())
             })

src/bors/handlers/trybuild.rs

@@ -387,7 +387,7 @@ mod tests {
             insta::assert_snapshot!(
                 tester.get_comment().await?,
                 @r#"
-            :sunny: Try build successful ([Workflow1](https://github.com/workflows/Workflow1/1))
+            :sunny: Try build successful ([Workflow1](https://github.com/rust-lang/borstest/actions/runs/1))
             Build commit: merge-0-pr-1 (`merge-0-pr-1`, parent: `main-sha1`)
 
             <!-- homu: {"type":"TryBuildCompleted","merge_sha":"merge-0-pr-1"} -->
@@ -406,7 +406,7 @@ mod tests {
             tester.workflow_full_failure(tester.try_branch()).await?;
             insta::assert_snapshot!(
                 tester.get_comment().await?,
-                @":broken_heart: Test failed ([Workflow1](https://github.com/workflows/Workflow1/1))"
+                @":broken_heart: Test failed ([Workflow1](https://github.com/rust-lang/borstest/actions/runs/1))"
             );
             Ok(())
         })
@@ -438,7 +438,7 @@ mod tests {
             insta::assert_snapshot!(
                 tester.get_comment().await?,
                 @r"
-            :broken_heart: Test failed ([Workflow1](https://github.com/workflows/Workflow1/1)). Failed jobs:
+            :broken_heart: Test failed ([Workflow1](https://github.com/rust-lang/borstest/actions/runs/1)). Failed jobs:
 
             - `Job 42` ([web logs](https://github.com/job-logs/42), [extended logs](https://triage.rust-lang.org/gha-logs/rust-lang/borstest/42))
             - `Job 50` ([web logs](https://github.com/job-logs/50), [extended logs](https://triage.rust-lang.org/gha-logs/rust-lang/borstest/50))
@@ -811,7 +811,7 @@ try-job: Bar
                 )
                 .await?;
             insta::assert_snapshot!(tester.get_comment().await?, @r#"
-            :sunny: Try build successful ([Workflow1](https://github.com/workflows/Workflow1/2))
+            :sunny: Try build successful ([Workflow1](https://github.com/rust-lang/borstest/actions/runs/2))
             Build commit: merge-1-pr-1 (`merge-1-pr-1`, parent: `main-sha1`)
 
             <!-- homu: {"type":"TryBuildCompleted","merge_sha":"merge-1-pr-1"} -->
@@ -851,8 +851,8 @@ try-job: Bar
             tester.post_comment("@bors try cancel").await?;
             insta::assert_snapshot!(tester.get_comment().await?, @r"
             Try build cancelled. Cancelled workflows:
-            - https://github.com/workflows/Workflow1/123
-            - https://github.com/workflows/Workflow1/124
+            - https://github.com/rust-lang/borstest/actions/runs/123
+            - https://github.com/rust-lang/borstest/actions/runs/124
             ");
             Ok(())
         })
@@ -921,7 +921,7 @@ try-job: Bar
             tester.post_comment("@bors try cancel").await?;
             insta::assert_snapshot!(tester.get_comment().await?, @r"
             Try build cancelled. Cancelled workflows:
-            - https://github.com/workflows/Workflow1/3
+            - https://github.com/rust-lang/borstest/actions/runs/3
             ");
             Ok(())
         })

src/bors/handlers/workflow.rs

@@ -553,8 +553,8 @@ mod tests {
                 tester.get_comment().await?,
                 @r#"
             :sunny: Try build successful
-            - [Workflow1](https://github.com/workflows/Workflow1/1) :white_check_mark:
-            - [Workflow1](https://github.com/workflows/Workflow1/2) :white_check_mark:
+            - [Workflow1](https://github.com/rust-lang/borstest/actions/runs/1) :white_check_mark:
+            - [Workflow1](https://github.com/rust-lang/borstest/actions/runs/2) :white_check_mark:
             Build commit: merge-0-pr-1 (`merge-0-pr-1`, parent: `main-sha1`)
 
             <!-- homu: {"type":"TryBuildCompleted","merge_sha":"merge-0-pr-1"} -->
@@ -583,8 +583,8 @@ mod tests {
                 tester.get_comment().await?,
                 @r#"
             :sunny: Try build successful
-            - [Workflow1](https://github.com/workflows/Workflow1/1) :white_check_mark:
-            - [Workflow1](https://github.com/workflows/Workflow1/2) :white_check_mark:
+            - [Workflow1](https://github.com/rust-lang/borstest/actions/runs/1) :white_check_mark:
+            - [Workflow1](https://github.com/rust-lang/borstest/actions/runs/2) :white_check_mark:
             Build commit: merge-0-pr-1 (`merge-0-pr-1`, parent: `main-sha1`)
 
             <!-- homu: {"type":"TryBuildCompleted","merge_sha":"merge-0-pr-1"} -->
@@ -610,7 +610,7 @@ mod tests {
             tester.workflow_event(WorkflowEvent::failure(w2)).await?;
             insta::assert_snapshot!(
                 tester.get_comment().await?,
-                @":broken_heart: Test failed ([Workflow1](https://github.com/workflows/Workflow1/1), [Workflow1](https://github.com/workflows/Workflow1/2))"
+                @":broken_heart: Test failed ([Workflow1](https://github.com/rust-lang/borstest/actions/runs/1), [Workflow1](https://github.com/rust-lang/borstest/actions/runs/2))"
             );
             Ok(())
         })
@@ -750,7 +750,7 @@ auto_build_failed = ["+foo", "+bar", "-baz"]
                 }).await?;
                 insta::assert_snapshot!(
                     tester.get_comment().await?,
-                    @":broken_heart: Test failed ([Workflow1](https://github.com/workflows/Workflow1/1))"
+                    @":broken_heart: Test failed ([Workflow1](https://github.com/rust-lang/borstest/actions/runs/1))"
                 );
 
                 Ok(())

src/bors/merge_queue.rs

@@ -465,7 +465,7 @@ mod tests {
             insta::assert_snapshot!(
                 tester.get_comment().await?,
                 @r"
-            :sunny: Test successful - [Workflow1](https://github.com/workflows/Workflow1/1)
+            :sunny: Test successful - [Workflow1](https://github.com/rust-lang/borstest/actions/runs/1)
             Approved by: `default-user`
             Pushing merge-0-pr-1 to `main`...
             "

src/tests/mocks/workflow.rs

@@ -113,8 +113,8 @@ impl From<WorkflowEvent> for GitHubWorkflowEventPayload {
         let WorkflowEvent { event, workflow } = value;
 
         let url: Url = format!(
-            "https://github.com/workflows/{}/{}",
-            workflow.name, workflow.run_id
+            "https://github.com/{}/actions/runs/{}",
+            workflow.repository, workflow.run_id
         )
         .parse()
         .unwrap();