[ignore] fix test for aci_management_access_policy to only be executed on supported versions

Author: akinross

gen/definitions/classes.yaml

@@ -1010,7 +1010,7 @@ commPol:
   sub_category: "Fabric Policies"
   ui_locations:
     - "Fabric -> Fabric Policies -> Policies -> Pod -> Management Access"
-  class_version: "6.0(2h)"
+  class_version: "6.0(2h)-6.0(8g),6.1(3f)-"
   contained_by:
       - "polUni"
   rn_prepend: "fabric"

gen/testvars/commPol.yaml

@@ -263,5 +263,5 @@ child_targets:
         name: "test_name"
 
 test_type: both
-class_version: 6.0(2h)
+class_version: 6.0(2h)-6.0(8g),6.1(3f)-
 version_mismatch: true

internal/provider/data_source_aci_management_access_policy_test.go

@@ -104,12 +104,14 @@ func testAccPreCheck(t *testing.T, testType string, testApplicableFromVersion st
 	if err != nil {
 		t.Fatalf("Error fetching APIC controller information: %v", err)
 	}
+
 	apicVersion := extractControllerVersion(infoController)
-	// TODO process the version when it has ranges associated with it
-	if apicVersion != strings.TrimSuffix(testApplicableFromVersion, "-") {
-		if IsVersionGreater(*ParseVersion(testApplicableFromVersion).Version, *ParseVersion(apicVersion).Version) {
-			t.Skip("[WARNING] Test skipped because it is not supported on APIC version:", apicVersion)
-		}
+
+	comparisonResult, err := CompareVersionsRange(apicVersion, testApplicableFromVersion, "inside")
+	if err != nil {
+		t.Skipf("[ERROR] Test skipped because the version '%s' couldn't be checked, error: %s", testApplicableFromVersion, err)
+	} else if !comparisonResult {
+		t.Skipf("[WARNING] Test skipped because the version '%s' is not supported on APIC version: %s", testApplicableFromVersion, apicVersion)
 	}
 
 }