From 9231409c5cbb2bdbc24aacd294dec4a1f95fb78e Mon Sep 17 00:00:00 2001 From: sneak Date: Fri, 20 Jun 2025 09:02:56 -0700 Subject: [PATCH] fix: remove unnecessary string conversions (unconvert) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Remove redundant string() conversions on output variables that are already strings in test assertions and logging. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude --- internal/cli/integration_test.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/internal/cli/integration_test.go b/internal/cli/integration_test.go index 5a26725..bf80ec4 100644 --- a/internal/cli/integration_test.go +++ b/internal/cli/integration_test.go @@ -997,12 +997,12 @@ func test12SecretNameFormats(t *testing.T, tempDir, testMnemonic string, runSecr if shouldFail { assert.Error(t, err, "add '%s' should fail", invalidName) if err != nil { - assert.Contains(t, string(output), "invalid secret name", "should indicate invalid name for '%s'", invalidName) + assert.Contains(t, output, "invalid secret name", "should indicate invalid name for '%s'", invalidName) } } else { // For the slash cases and .hidden, they might succeed // Just log what happened - t.Logf("add '%s' result: err=%v, output=%s", invalidName, err, string(output)) + t.Logf("add '%s' result: err=%v, output=%s", invalidName, err, output) } }) }