Skip to content

Commit 3e269ae

Browse files
committed
fix StdBase unit tests
fix WMTask unit tests
1 parent 06f5b59 commit 3e269ae

File tree

2 files changed

+16
-14
lines changed

2 files changed

+16
-14
lines changed

test/python/WMCore_t/WMSpec_t/StdSpecs_t/StepChain_t.py

Lines changed: 14 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -2489,9 +2489,10 @@ def testGPUStepChainsTasks(self):
24892489
testArguments[s]['ConfigCacheID'] = configDocs[s]
24902490
testArguments['Step2']['KeepOutput'] = False
24912491

2492-
gpuParams = {"GPUMemoryMB": 1234, "CUDARuntime": "11.2.3", "CUDACapabilities": ["7.5", "8.0"]}
2493-
testArguments['Step1'].update({"RequiresGPU": "optional", "GPUParams": json.dumps(gpuParams)})
2494-
testArguments['Step2'].update({"RequiresGPU": "required", "GPUParams": json.dumps(gpuParams)})
2492+
gpuParams1 = {"GPUMemoryMB": 1234, "CUDARuntime": "11.2.3", "CUDACapabilities": ["7.5", "8.0"]}
2493+
testArguments['Step1'].update({"RequiresGPU": "optional", "GPUParams": json.dumps(gpuParams1)})
2494+
gpuParams2 = {"GPUMemoryMB": 2345, "CUDARuntime": "9.6", "CUDACapabilities": ["7.4"]}
2495+
testArguments['Step2'].update({"RequiresGPU": "required", "GPUParams": json.dumps(gpuParams2)})
24952496
factory = StepChainWorkloadFactory()
24962497
testWorkload = factory.factoryWorkloadConstruction("TestWorkload", testArguments)
24972498

@@ -2503,8 +2504,8 @@ def testGPUStepChainsTasks(self):
25032504

25042505
# validate GPU parameters
25052506
self.assertEqual(testArguments['GPUParams'], json.dumps(None))
2506-
self.assertEqual(testArguments["Step1"]['GPUParams'], json.dumps(gpuParams))
2507-
self.assertEqual(testArguments["Step2"]['GPUParams'], json.dumps(gpuParams))
2507+
self.assertEqual(testArguments["Step1"]['GPUParams'], json.dumps(gpuParams1))
2508+
self.assertEqual(testArguments["Step2"]['GPUParams'], json.dumps(gpuParams2))
25082509
self.assertTrue("GPUParams" not in testArguments["Step3"])
25092510

25102511
for taskName in testWorkload.listAllTaskNames():
@@ -2520,10 +2521,10 @@ def testGPUStepChainsTasks(self):
25202521
elif stepHelper.stepType() == "CMSSW" and taskName == "GENSIM":
25212522
if stepHelper.name() == "cmsRun1":
25222523
self.assertEqual(stepHelper.data.application.gpu.gpuRequired, testArguments["Step1"]['RequiresGPU'])
2523-
self.assertItemsEqual(stepHelper.data.application.gpu.gpuRequirements, gpuParams)
2524+
self.assertItemsEqual(stepHelper.data.application.gpu.gpuRequirements, gpuParams1)
25242525
elif stepHelper.name() == "cmsRun2":
25252526
self.assertEqual(stepHelper.data.application.gpu.gpuRequired, testArguments["Step2"]['RequiresGPU'])
2526-
self.assertItemsEqual(stepHelper.data.application.gpu.gpuRequirements, gpuParams)
2527+
self.assertItemsEqual(stepHelper.data.application.gpu.gpuRequirements, gpuParams2)
25272528
elif stepHelper.name() == "cmsRun3":
25282529
self.assertEqual(stepHelper.data.application.gpu.gpuRequired, "forbidden")
25292530
self.assertIsNone(stepHelper.data.application.gpu.gpuRequirements)
@@ -2535,18 +2536,17 @@ def testGPUStepChainsTasks(self):
25352536
prodTask = testWorkload.getTask('GENSIM')
25362537
gpuRequired, gpuRequirements = prodTask.getStepHelper('cmsRun1').getGPUSettings()
25372538
self.assertEqual(gpuRequired, testArguments["Step1"]['RequiresGPU'])
2538-
self.assertItemsEqual(gpuRequirements, gpuParams)
2539+
self.assertItemsEqual(gpuRequirements, gpuParams1)
25392540

25402541
gpuRequired, gpuRequirements = prodTask.getStepHelper('cmsRun2').getGPUSettings()
25412542
self.assertEqual(gpuRequired, testArguments["Step2"]['RequiresGPU'])
2542-
self.assertItemsEqual(gpuRequirements, gpuParams)
2543+
self.assertItemsEqual(gpuRequirements, gpuParams2)
25432544

25442545
gpuRequired, gpuRequirements = prodTask.getStepHelper('cmsRun3').getGPUSettings()
25452546
self.assertEqual(gpuRequired, testArguments["Step3"].get('RequiresGPU', "forbidden"))
25462547
self.assertIsNone(gpuRequirements)
25472548

25482549

2549-
25502550
# test assignment with wrong Trust flags
25512551
assignDict = {"SiteWhitelist": ["T2_US_Nebraska"], "Team": "The-A-Team",
25522552
"RequestStatus": "assigned"}
@@ -2560,18 +2560,18 @@ def testGPUStepChainsTasks(self):
25602560

25612561
# validate GPU parameters
25622562
self.assertEqual(testArguments['GPUParams'], json.dumps(None))
2563-
self.assertEqual(testArguments["Step1"]['GPUParams'], json.dumps(gpuParams))
2564-
self.assertEqual(testArguments["Step2"]['GPUParams'], json.dumps(gpuParams))
2563+
self.assertEqual(testArguments["Step1"]['GPUParams'], json.dumps(gpuParams1))
2564+
self.assertEqual(testArguments["Step2"]['GPUParams'], json.dumps(gpuParams2))
25652565
self.assertTrue("GPUParams" not in testArguments["Step3"])
25662566

25672567
prodTask = testWorkload.getTask('GENSIM')
25682568
gpuRequired, gpuRequirements = prodTask.getStepHelper('cmsRun1').getGPUSettings()
25692569
self.assertEqual(gpuRequired, testArguments["Step1"]['RequiresGPU'])
2570-
self.assertItemsEqual(gpuRequirements, gpuParams)
2570+
self.assertItemsEqual(gpuRequirements, gpuParams1)
25712571

25722572
gpuRequired, gpuRequirements = prodTask.getStepHelper('cmsRun2').getGPUSettings()
25732573
self.assertEqual(gpuRequired, testArguments["Step2"]['RequiresGPU'])
2574-
self.assertItemsEqual(gpuRequirements, gpuParams)
2574+
self.assertItemsEqual(gpuRequirements, gpuParams2)
25752575

25762576
gpuRequired, gpuRequirements = prodTask.getStepHelper('cmsRun3').getGPUSettings()
25772577
self.assertEqual(gpuRequired, testArguments["Step3"].get('RequiresGPU', "forbidden"))

test/python/WMCore_t/WMSpec_t/WMTask_t.py

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -858,6 +858,8 @@ def testGPUTaskSettings(self):
858858
### Now set a single value for both tasks
859859
gpuParams = {"GPUMemoryMB": 1234, "CUDARuntime": "11.2.3", "CUDACapabilities": ["7.5", "8.0"]}
860860
task1.setTaskGPUSettings("required", json.dumps(gpuParams))
861+
# CUDARuntime returns as a list
862+
gpuParams["CUDARuntime"] = [gpuParams["CUDARuntime"]]
861863
for taskObj in task1.taskIterator():
862864
# task level check
863865
self.assertEqual(taskObj.getRequiresGPU(), "required")

0 commit comments

Comments
 (0)