Deleting an imported key deletes the old key

[jefft0]

Description

Describe your issue in as much detail as possible here

Your environment

• OS and version: macOS Ventura
• version of gno: Latest master branch
• branch that causes this issue (with the commit hash): master 28c3b1a

Steps to reproduce

• Edit keybase_test.go. At the end of TestExportImport add these lines:

	// Delete the imported key
	err = cstore.Delete("john2", "secretcpw", false)
	require.NoError(t, err)
	// The original key should still exist
	has, err := cstore.HasByName("john")
	require.NoError(t, err)
	require.True(t, has)

• In a terminal cd to gno/tm2/pkg/crypto/keys
• Enter go test .

Expected behaviour

The test should pass. (The original exported key should still exist.)

Actual behaviour

The test fails because deleting the imported key by the new name deletes the key by the original name.

Logs

Output of go test .:

--- FAIL: TestExportImport (0.56s)
    keybase_test.go:246:
        	Error Trace:	/Users/jefft0/work/gno/gno/tm2/pkg/crypto/keys/keybase_test.go:246
        	Error:      	Should be true
        	Test:       	TestExportImport

Proposed solution

The Export function exports the Info struct which has the key name, in this case "john". Then Import imports the key with a new name, but does not change the Info struct. (Is this by design?) The Delete function fetches the Info struct from the database using the new name "john2", but it deletes based on the original name in the struct. This deletes the key under the original name.

Maybe Import needs to modify the Info struct with the new name before storing in the database.

[jefft0]

Closing in favor of a PR with code that demonstrates the bug. #2472