Add/amend logging to use TestTopologyBasic.Name instead of TestTopologyMetadata.Name (#116)

Co-authored-by: Johan Jatko <Johan Jatko>

Author: ArmedGuy

pkg/scheduler/actions/allocate/allocateElastic_test.go

@@ -25,6 +25,8 @@ func TestHandleElasticAllocation(t *testing.T) {
 	defer controller.Finish()
 
 	for testNumber, testMetadata := range getElasticTestsMetadata() {
+		t.Logf("Running test %d: %s", testNumber, testMetadata.TestTopologyBasic.Name)
+
 		ssn := test_utils.BuildSession(testMetadata.TestTopologyBasic, controller)
 		allocateAction := allocate.New()
 		allocateAction.Execute(ssn)

pkg/scheduler/actions/allocate/allocateFractionalGpu_test.go

@@ -28,6 +28,8 @@ func TestHandleFractionalGPUAllocation(t *testing.T) {
 
 	testsMetadata := getFractionalGPUTestsMetadata()
 	for testNumber, testMetadata := range testsMetadata {
+		t.Logf("Running test %d: %s", testNumber, testMetadata.TestTopologyBasic.Name)
+
 		ssn := test_utils.BuildSession(testMetadata.TestTopologyBasic, controller)
 		allocateAction := allocate.New()
 		allocateAction.Execute(ssn)

pkg/scheduler/actions/allocate/allocateGang_test.go

@@ -28,6 +28,8 @@ func TestHandleGangAllocation(t *testing.T) {
 
 func runTests(t *testing.T, testsMetadata []integration_tests_utils.TestTopologyMetadata, controller *Controller) {
 	for testNumber, testMetadata := range testsMetadata {
+		t.Logf("Running test %d: %s", testNumber, testMetadata.TestTopologyBasic.Name)
+
 		ssn := test_utils.BuildSession(testMetadata.TestTopologyBasic, controller)
 		allocateAction := allocate.New()
 		allocateAction.Execute(ssn)

pkg/scheduler/actions/allocate/allocateGpuMemory_test.go

@@ -27,6 +27,8 @@ func TestHandleMemoryGPUAllocation(t *testing.T) {
 
 	testsMetadata := getMemoryGPUTestsMetadata()
 	for testNumber, testMetadata := range testsMetadata {
+		t.Logf("Running test %d: %s", testNumber, testMetadata.TestTopologyBasic.Name)
+
 		ssn := test_utils.BuildSession(testMetadata.TestTopologyBasic, controller)
 		allocateAction := allocate.New()
 		allocateAction.Execute(ssn)

pkg/scheduler/actions/allocate/allocateMIG_test.go

@@ -30,6 +30,8 @@ func TestHandleMIGAllocation(t *testing.T) {
 
 	testsMetadata := getMIGTestsMetadata()
 	for testNumber, testMetadata := range testsMetadata {
+		t.Logf("Running test %d: %s", testNumber, testMetadata.TestTopologyBasic.Name)
+
 		ssn := test_utils.BuildSession(testMetadata.TestTopologyBasic, controller)
 		allocateAction := allocate.New()
 		allocateAction.Execute(ssn)

pkg/scheduler/actions/allocate/allocate_test.go

@@ -31,6 +31,8 @@ func TestHandleAllocation(t *testing.T) {
 	defer controller.Finish()
 	testsMetadata := getTestsMetadata()
 	for testNumber, testMetadata := range testsMetadata {
+		t.Logf("Running test %d: %s", testNumber, testMetadata.TestTopologyBasic.Name)
+
 		ssn := test_utils.BuildSession(testMetadata.TestTopologyBasic, controller)
 		allocateAction := allocate.New()
 		allocateAction.Execute(ssn)

pkg/scheduler/actions/consolidation/consolidation_test.go

@@ -27,7 +27,7 @@ func TestConsolidation(t *testing.T) {
 	testsMetadata := getTestsMetadata()
 
 	for testNumber, testMetadata := range testsMetadata {
-		fmt.Printf("Running test %d/%d: %s\n", testNumber, len(testsMetadata), testMetadata.Name)
+		fmt.Printf("Running test %d/%d: %s\n", testNumber, len(testsMetadata), testMetadata.TestTopologyBasic.Name)
 		ssn := test_utils.BuildSession(testMetadata.TestTopologyBasic, controller)
 		consolidationAction := consolidation.New()
 		consolidationAction.Execute(ssn)

pkg/scheduler/actions/preempt/preemptGang_test.go

@@ -26,6 +26,7 @@ func TestHandleGangPreempt(t *testing.T) {
 	defer controller.Finish()
 	testsMetadata := getTestsGangMetadata()
 	for testNumber, testMetadata := range testsMetadata {
+		t.Logf("Running Test %d: %s", testNumber, testMetadata.TestTopologyBasic.Name)
 
 		ssn := test_utils.BuildSession(testMetadata.TestTopologyBasic, controller)
 		preemptAction := preempt.New()

pkg/scheduler/actions/preempt/preemptMIG_test.go

@@ -26,7 +26,7 @@ func TestMIGPreempt(t *testing.T) {
 	defer controller.Finish()
 	testsMetadata := getMIGTestsMetadata()
 	for testNumber, testMetadata := range testsMetadata {
-		t.Logf("Running Test %d: %s", testNumber, testMetadata.Name)
+		t.Logf("Running Test %d: %s", testNumber, testMetadata.TestTopologyBasic.Name)
 
 		ssn := test_utils.BuildSession(testMetadata.TestTopologyBasic, controller)
 		preemptAction := preempt.New()

pkg/scheduler/actions/preempt/preempt_elastic_test.go

@@ -26,7 +26,7 @@ func TestHandlePreemptElastic(t *testing.T) {
 	defer controller.Finish()
 	testsMetadata := getElasticTestsMetadata()
 	for testNumber, testMetadata := range testsMetadata {
-		t.Logf("Running Test %d: %s", testNumber, testMetadata.Name)
+		t.Logf("Running Test %d: %s", testNumber, testMetadata.TestTopologyBasic.Name)
 
 		ssn := test_utils.BuildSession(testMetadata.TestTopologyBasic, controller)
 		preemptAction := preempt.New()