Merge pull request #20 from conduktor/cons-862-consistent-message-bet…

…ween-live-dry-run Consistent message between live and dry run
conduktor · Mar 20, 2024 · a4fd8ca · a4fd8ca
2 parents 6819870 + 2c75ec2
commit a4fd8ca
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 10 deletions.
diff --git a/client/client.go b/client/client.go
@@ -74,15 +74,6 @@ func (client *Client) Apply(resource *resource.Resource, dryMode bool) (string,
 	if err != nil {
 		return resp.String(), nil
 	}
-	if dryMode && upsertResponse.UpsertResult == "Created" {
-		return "To be created", nil
-	}
-	if dryMode && upsertResponse.UpsertResult == "Updated" {
-		return "To be updated", nil
-	}
-	if dryMode && upsertResponse.UpsertResult == "NotChanged" {
-		return "Nothing to do", nil
-	}
 	return upsertResponse.UpsertResult, nil
 }
 

diff --git a/client/client_test.go b/client/client_test.go
@@ -72,7 +72,7 @@ func TestApplyWithDryModeShouldWork(t *testing.T) {
 	if err != nil {
 		t.Error(err)
 	}
-	if body != "Nothing to do" {
+	if body != "NotChanged" {
 		t.Errorf("Bad result expected NotChanged got: %s", body)
 	}
 }