sneak/secret
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: remove unnecessary string conversions (unconvert)

Browse Source 
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 <noreply@anthropic.com>
This commit is contained in:
Jeffrey Paul 2025-06-20 09:02:56 -07:00
parent 0d140b4636
commit 9231409c5c
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
internal/cli/integration_test.go
View File

@ -997,12 +997,12 @@ func test12SecretNameFormats(t *testing.T, tempDir, testMnemonic string, runSecr
if shouldFail { if shouldFail {
assert.Error(t, err, "add '%s' should fail", invalidName) assert.Error(t, err, "add '%s' should fail", invalidName)
if err != nil { 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 { } else {
// For the slash cases and .hidden, they might succeed // For the slash cases and .hidden, they might succeed
// Just log what happened // 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)
} }
}) })
} }