add workaround for applying MOs without a Name attribute

MOs without a Name attribute will always execute a POST rather than checking and updating

cmd/apply.go

@@ -131,7 +131,8 @@ func applyMOs(client *util.IsctlClient, rawMOs []rawMO) error {
 		var op *gen.Operation
 
 		getOperation := gen.GetGetOperationForClassID(classID)
-		if getOperation == nil {
+		_, moHasName := mo["Name"]
+		if getOperation == nil || !moHasName {
 			op = gen.GetCreateOperationForClassID(classID)
 			if op == nil {
 				return fmt.Errorf("no create operation for ClassId %s", classID)

tests/apply.bats

@@ -68,6 +68,18 @@ TEST_SECTION="Apply"
     run ./build/isctl ${ISCTL_OPTIONS} delete server profiletemplate name "${TEST_NAME}" 
 }
 
+@test "${TEST_SECTION}: apply for MO with no Name attribute" {
+    run ./build/isctl ${ISCTL_OPTIONS} apply -f tests/data/test-appliance-device-claim.yaml
+    # this should fail because we're are testing against SaaS, not an appliance but as long as the error code matches we know the request was sent properly
+    assert_failure
+    assert_line --partial "Performing create operation on new MO (ClassId: appliance.DeviceClaim)"
+    assert_line --partial "Error while applying MOs: error executing operation: request failed: 403 Forbidden: Operation not supported."
+}
+
+setup() {
+    load 'test_helper/bats-support/load' # this is required by bats-assert!
+    load 'test_helper/bats-assert/load'
+}
 
 setup_file() {
     # delete the test objects if they already exist. Don't check the exit code. 

tests/data/test-appliance-device-claim.yaml

@@ -0,0 +1,5 @@
+ClassId: appliance.DeviceClaim
+PlatformType: IMC
+Hostname: 192.168.9.10
+Username: admin
+Password: "<redacted>"