Skip to content

Commit 04a1c82

Browse files
committed
Support customising test directories in PR check specs
1 parent 09e4bd3 commit 04a1c82

File tree

1 file changed

+24
-11
lines changed

1 file changed

+24
-11
lines changed

pr-checks/sync.py

Lines changed: 24 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -83,10 +83,18 @@ def writeHeader(checkStream):
8383
runnerImagesForOs = [image for image in runnerImages if image.startswith(operatingSystem)]
8484

8585
for runnerImage in runnerImagesForOs:
86-
matrix.append({
87-
'os': runnerImage,
88-
'version': version
89-
})
86+
if 'testDirectories' in checkSpecification:
87+
for testDirectory in checkSpecification.get('testDirectories'):
88+
matrix.append({
89+
'os': runnerImage,
90+
'version': version,
91+
'test-directory': testDirectory
92+
})
93+
else:
94+
matrix.append({
95+
'os': runnerImage,
96+
'version': version
97+
})
9098

9199
useAllPlatformBundle = "false" # Default to false
92100
if checkSpecification.get('useAllPlatformBundle'):
@@ -129,17 +137,22 @@ def writeHeader(checkStream):
129137
},
130138
])
131139

140+
prepareWith = {
141+
'version': '${{ matrix.version }}',
142+
'use-all-platform-bundle': useAllPlatformBundle,
143+
# If the action is being run from a container, then do not setup kotlin.
144+
# This is because the kotlin binaries cannot be downloaded from the container.
145+
'setup-kotlin': str(not 'container' in checkSpecification).lower(),
146+
}
147+
148+
if 'testDirectories' in checkSpecification:
149+
prepareWith['test-directory'] = '${{ matrix.test-directory }}'
150+
132151
steps.append({
133152
'name': 'Prepare test',
134153
'id': 'prepare-test',
135154
'uses': './.github/actions/prepare-test',
136-
'with': {
137-
'version': '${{ matrix.version }}',
138-
'use-all-platform-bundle': useAllPlatformBundle,
139-
# If the action is being run from a container, then do not setup kotlin.
140-
# This is because the kotlin binaries cannot be downloaded from the container.
141-
'setup-kotlin': str(not 'container' in checkSpecification).lower(),
142-
}
155+
'with': prepareWith,
143156
})
144157

145158
installGo = is_truthy(checkSpecification.get('installGo', ''))

0 commit comments

Comments
 (0)