Skip to content

Commit 216fcbb

Browse files
authored
Add uses format validation for step and job-level workflows (#220)
1 parent 03ffd0c commit 216fcbb

File tree

9 files changed

+1220
-29
lines changed

9 files changed

+1220
-29
lines changed

languageservice/src/complete.expressions.test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1268,7 +1268,7 @@ jobs:
12681268
on: push
12691269
jobs:
12701270
a:
1271-
uses: ./reusable-workflow-with-outputs.yaml
1271+
uses: ./.github/workflows/reusable-workflow-with-outputs.yaml
12721272
b:
12731273
needs: [a]
12741274
runs-on: ubuntu-latest

languageservice/src/complete.reusable-workflows.test.ts

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ on: push
2121
2222
jobs:
2323
build:
24-
uses: ./reusable-workflow-with-inputs.yaml
24+
uses: ./.github/workflows/reusable-workflow-with-inputs.yaml
2525
with:
2626
|
2727
`;
@@ -49,7 +49,7 @@ on: push
4949
5050
jobs:
5151
build:
52-
uses: ./reusable-workflow-with-inputs.yaml
52+
uses: ./.github/workflows/reusable-workflow-with-inputs.yaml
5353
with:
5454
username: monalisa
5555
|
@@ -74,7 +74,7 @@ on: push
7474
7575
jobs:
7676
build:
77-
uses: ./reusable-workflow-with-inputs.yaml
77+
uses: ./.github/workflows/reusable-workflow-with-inputs.yaml
7878
secrets:
7979
|
8080
`;
@@ -102,7 +102,7 @@ on: push
102102
103103
jobs:
104104
build:
105-
uses: ./reusable-workflow-with-inputs.yaml
105+
uses: ./.github/workflows/reusable-workflow-with-inputs.yaml
106106
secrets: |
107107
`;
108108
const result = await complete(...getPositionFromCursor(input), {fileProvider: testFileProvider});
@@ -117,7 +117,7 @@ on: push
117117
118118
jobs:
119119
build:
120-
uses: ./reusable-workflow-with-inputs.yaml
120+
uses: ./.github/workflows/reusable-workflow-with-inputs.yaml
121121
secrets:
122122
envPAT: "myPAT"
123123
|

languageservice/src/context-providers/needs.test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -111,7 +111,7 @@ jobs:
111111
on: push
112112
jobs:
113113
a:
114-
uses: ./reusable-workflow-with-outputs.yaml
114+
uses: ./.github/workflows/reusable-workflow-with-outputs.yaml
115115
116116
b:
117117
needs: [a]

languageservice/src/hover.reusable-workflow.test.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ on: push
1414
1515
jobs:
1616
build:
17-
uses: ./reusable-workflow-with-inputs.yaml
17+
uses: ./.github/workflows/reusable-workflow-with-inputs.yaml
1818
with:
1919
us|ername:
2020
`;
@@ -31,7 +31,7 @@ on: push
3131
3232
jobs:
3333
build:
34-
uses: ./reusable-workflow-with-inputs-no-description.yaml
34+
uses: ./.github/workflows/reusable-workflow-with-inputs-no-description.yaml
3535
with:
3636
us|ername:
3737
`;
@@ -48,7 +48,7 @@ on: push
4848
4949
jobs:
5050
build:
51-
uses: ./reusable-workflow-with-outputs.yaml
51+
uses: ./.github/workflows/reusable-workflow-with-outputs.yaml
5252
echo_outputs:
5353
runs-on: ubuntu-latest
5454
needs: build

languageservice/src/test-utils/test-file-provider.ts

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -5,9 +5,9 @@ export const testFileProvider: FileProvider = {
55
// eslint-disable-next-line @typescript-eslint/require-await
66
getFileContent: async ref => {
77
switch (fileIdentifier(ref)) {
8-
case "monalisa/octocat/workflow.yaml@main":
8+
case "monalisa/octocat/.github/workflows/workflow.yaml@main":
99
return {
10-
name: "monalisa/octocat/workflow.yaml",
10+
name: "monalisa/octocat/.github/workflows/workflow.yaml",
1111
content: `
1212
on: workflow_call
1313
jobs:
@@ -31,9 +31,9 @@ jobs:
3131
`
3232
};
3333

34-
case "./reusable-workflow.yaml":
34+
case "./.github/workflows/reusable-workflow.yaml":
3535
return {
36-
name: "reusable-workflow.yaml",
36+
name: ".github/workflows/reusable-workflow.yaml",
3737
content: `
3838
on: workflow_call
3939
jobs:
@@ -44,9 +44,9 @@ jobs:
4444
`
4545
};
4646

47-
case "./reusable-workflow-with-inputs.yaml":
47+
case "./.github/workflows/reusable-workflow-with-inputs.yaml":
4848
return {
49-
name: "reusable-workflow-with-inputs.yaml",
49+
name: ".github/workflows/reusable-workflow-with-inputs.yaml",
5050
content: `
5151
on:
5252
workflow_call:
@@ -76,9 +76,9 @@ jobs:
7676
`
7777
};
7878

79-
case "./reusable-workflow-with-inputs-no-description.yaml":
79+
case "./.github/workflows/reusable-workflow-with-inputs-no-description.yaml":
8080
return {
81-
name: "reusable-workflow-with-inputs.yaml",
81+
name: ".github/workflows/reusable-workflow-with-inputs.yaml",
8282
content: `
8383
on:
8484
workflow_call:
@@ -95,9 +95,9 @@ jobs:
9595
`
9696
};
9797

98-
case "./reusable-workflow-with-outputs.yaml":
98+
case "./.github/workflows/reusable-workflow-with-outputs.yaml":
9999
return {
100-
name: "reusable-workflow-with-outputs.yaml",
100+
name: ".github/workflows/reusable-workflow-with-outputs.yaml",
101101
content: `
102102
on:
103103
workflow_call:

languageservice/src/validate.expressions.test.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -635,7 +635,7 @@ jobs:
635635
fail-fast: true
636636
matrix:
637637
node: [14, 16]
638-
uses: ./reusable-workflow-with-inputs.yaml
638+
uses: ./.github/workflows/reusable-workflow-with-inputs.yaml
639639
with:
640640
username: User-\${{ strategy.fail-fast }}
641641
`;
@@ -654,7 +654,7 @@ jobs:
654654
strategy:
655655
matrix:
656656
node: [14, 16]
657-
uses: ./reusable-workflow-with-inputs.yaml
657+
uses: ./.github/workflows/reusable-workflow-with-inputs.yaml
658658
with:
659659
username: \${{ matrix.node }}
660660
`;

0 commit comments

Comments
 (0)