Skip to content

Commit 4aeabe0

Browse files
committed
Address review comments
1 parent 9bf7953 commit 4aeabe0

File tree

3 files changed

+6
-10
lines changed

3 files changed

+6
-10
lines changed

AzureDevOps/DistributeTests.ps1

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,6 @@ Write-Host "Agent number: $agentNumber"
2727
Write-Host "Total tests: $testCount"
2828

2929
$testsToRun= @()
30-
3130
# Slice test files so each agent gets a unique set of files to execute
3231
For ($i=$agentNumber; $i -le $testCount;) {
3332
$file = $tests[$i-1]

AzureDevOps/DistributeTests.sh

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,6 @@ while IFS= read -r file; do
1919
tests+=("$file")
2020
done < <(find ./tests -type f -name "*.m" | sort)
2121

22-
# Use Azure DevOps variables
2322
totalAgents=${SYSTEM_TOTALJOBSINPHASE} # Standard VSTS variable containing the number of parallel jobs
2423
agentNumber=${SYSTEM_JOBPOSITIONINPHASE} # Current job positioN
2524
testCount=${#tests[@]}
@@ -37,7 +36,6 @@ echo "Agent number: $agentNumber"
3736
echo "Total tests: $testCount"
3837

3938
testsToRun=()
40-
4139
# Slice test files so each agent gets a unique set of files
4240
for (( i=agentNumber; i<=testCount; i+=totalAgents )); do
4341
file="${tests[i-1]}"

AzureDevOps/ParallelStrategy.yml

Lines changed: 6 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
jobs:
22
- job: ParallelWindows
3-
# Parallel strategy to run tests in parallel
3+
# Specify 'parallel' strategy to run tests in parallel
44
strategy:
55
parallel: 2
66
pool:
@@ -18,7 +18,7 @@ jobs:
1818
MLM_LICENSE_TOKEN: $(MLM_LICENSE_TOKEN)
1919

2020
- powershell: .\AzureDevOps\DistributeTests.ps1
21-
displayName: 'PowerShell Script to distribute tests'
21+
displayName: 'PowerShell script to distribute tests'
2222

2323
- task: RunMATLABTests@1
2424
inputs:
@@ -28,7 +28,7 @@ jobs:
2828
MLM_LICENSE_TOKEN: $(MLM_LICENSE_TOKEN)
2929

3030
- job: ParallelLinux
31-
# Parallel strategy to run tests in parallel
31+
# Specify 'parallel' strategy to run tests in parallel
3232
strategy:
3333
parallel: 2
3434
pool:
@@ -39,15 +39,14 @@ jobs:
3939
inputs:
4040
products: MATLAB_Compiler_SDK MATLAB_Test
4141

42-
# Builds Python package from MATLAB function and verifies functionality through equivalence tests
4342
- task: RunMATLABBuild@1
4443
inputs:
4544
tasks: equivalenceTest
4645
env:
4746
MLM_LICENSE_TOKEN: $(MLM_LICENSE_TOKEN)
4847

4948
- bash: chmod +x ./AzureDevOps/DistributeTests.sh && ./AzureDevOps/DistributeTests.sh
50-
displayName: 'Bash Script to distribute tests'
49+
displayName: 'Bash script to distribute tests'
5150

5251
- task: RunMATLABTests@1
5352
inputs:
@@ -57,7 +56,7 @@ jobs:
5756
MLM_LICENSE_TOKEN: $(MLM_LICENSE_TOKEN)
5857

5958
- job: ParallelMac
60-
# Parallel strategy to run tests in parallel
59+
# Specify 'parallel' strategy to run tests in parallel
6160
strategy:
6261
parallel: 2
6362
pool:
@@ -75,7 +74,7 @@ jobs:
7574
MLM_LICENSE_TOKEN: $(MLM_LICENSE_TOKEN)
7675

7776
- bash: chmod +x ./AzureDevOps/DistributeTests.sh && ./AzureDevOps/DistributeTests.sh
78-
displayName: 'Bash Script to distribute tests'
77+
displayName: 'Bash script to distribute tests'
7978

8079
- task: RunMATLABTests@1
8180
inputs:

0 commit comments

Comments
 (0)