Skip to content

Commit 1f35d1d

Browse files
committed
test(sdk): use neutral domain examples in the v1 suites
1 parent bca3311 commit 1f35d1d

3 files changed

Lines changed: 27 additions & 27 deletions

File tree

sdk/go/dstack/client_v1_test.go

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,7 @@ func TestV1GetKey(t *testing.T) {
4343
{"secp256k1", 33},
4444
{"ed25519", 32},
4545
} {
46-
resp, err := client.GetKey(context.Background(), "wallet", tc.algorithm)
46+
resp, err := client.GetKey(context.Background(), "storage-encryption", tc.algorithm)
4747
if err != nil {
4848
t.Fatalf("%s: %v", tc.algorithm, err)
4949
}
@@ -74,11 +74,11 @@ func TestV1GetKeyIsDomainSeparated(t *testing.T) {
7474
client := dstack.NewDstackClientV1()
7575
ctx := context.Background()
7676

77-
k256, err := client.GetKey(ctx, "wallet", "secp256k1")
77+
k256, err := client.GetKey(ctx, "storage-encryption", "secp256k1")
7878
if err != nil {
7979
t.Fatal(err)
8080
}
81-
ed, err := client.GetKey(ctx, "wallet", "ed25519")
81+
ed, err := client.GetKey(ctx, "storage-encryption", "ed25519")
8282
if err != nil {
8383
t.Fatal(err)
8484
}
@@ -95,7 +95,7 @@ func TestV1GetKeyIsDomainSeparated(t *testing.T) {
9595
}
9696

9797
// Derivation is stable for a given (domain, algorithm).
98-
again, err := client.GetKey(ctx, "wallet", "secp256k1")
98+
again, err := client.GetKey(ctx, "storage-encryption", "secp256k1")
9999
if err != nil {
100100
t.Fatal(err)
101101
}
@@ -109,11 +109,11 @@ func TestV1GetKeyIsDomainSeparated(t *testing.T) {
109109
func TestV1GetKeyDiffersFromV0(t *testing.T) {
110110
ctx := context.Background()
111111

112-
v1Resp, err := dstack.NewDstackClientV1().GetKey(ctx, "wallet", "secp256k1")
112+
v1Resp, err := dstack.NewDstackClientV1().GetKey(ctx, "storage-encryption", "secp256k1")
113113
if err != nil {
114114
t.Fatal(err)
115115
}
116-
v0Resp, err := dstack.NewDstackClientV0().GetKey(ctx, "wallet", "", "secp256k1")
116+
v0Resp, err := dstack.NewDstackClientV0().GetKey(ctx, "storage-encryption", "", "secp256k1")
117117
if err != nil {
118118
t.Fatal(err)
119119
}
@@ -133,7 +133,7 @@ func TestV1GetKeyRejectsMissingOrUnknownAlgorithms(t *testing.T) {
133133
client := dstack.NewDstackClientV1()
134134
ctx := context.Background()
135135

136-
_, err := client.GetKey(ctx, "wallet", "")
136+
_, err := client.GetKey(ctx, "storage-encryption", "")
137137
if err == nil {
138138
t.Fatal("expected an empty algorithm to be rejected")
139139
}
@@ -147,7 +147,7 @@ func TestV1GetKeyRejectsMissingOrUnknownAlgorithms(t *testing.T) {
147147
}
148148

149149
for _, algorithm := range []string{"k256", "secp256k1_prehashed", "rsa"} {
150-
if _, err := client.GetKey(ctx, "wallet", algorithm); err == nil {
150+
if _, err := client.GetKey(ctx, "storage-encryption", algorithm); err == nil {
151151
t.Errorf("expected algorithm %q to be rejected", algorithm)
152152
}
153153
}

sdk/js/src/__tests__/index-v1.test.ts

Lines changed: 11 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -68,7 +68,7 @@ describe('DstackClientV1', () => {
6868
describe('getKey', () => {
6969
it('should derive a secp256k1 key with a public key and a two-link chain', async () => {
7070
const client = new DstackClientV1()
71-
const result = await client.getKey('wallet', 'secp256k1')
71+
const result = await client.getKey('storage-encryption', 'secp256k1')
7272
expect(result.key).toBeInstanceOf(Uint8Array)
7373
expect(result.key.length).toBe(32)
7474
// SEC1 compressed, so the chain's first link commits to these exact bytes.
@@ -83,29 +83,29 @@ describe('DstackClientV1', () => {
8383

8484
it('should derive an ed25519 key with a 32-byte public key', async () => {
8585
const client = new DstackClientV1()
86-
const result = await client.getKey('wallet', 'ed25519')
86+
const result = await client.getKey('storage-encryption', 'ed25519')
8787
expect(result.key.length).toBe(32)
8888
expect(result.public_key.length).toBe(32)
8989
expect(result.signature_chain.length).toBe(2)
9090
})
9191

9292
it('should be deterministic for the same domain and algorithm', async () => {
9393
const client = new DstackClientV1()
94-
const first = await client.getKey('wallet', 'secp256k1')
95-
const second = await client.getKey('wallet', 'secp256k1')
94+
const first = await client.getKey('storage-encryption', 'secp256k1')
95+
const second = await client.getKey('storage-encryption', 'secp256k1')
9696
expect(first.key).toEqual(second.key)
9797
})
9898

9999
it('should separate the two curves, which v0 did not', async () => {
100100
const client = new DstackClientV1()
101-
const secp = await client.getKey('wallet', 'secp256k1')
102-
const ed = await client.getKey('wallet', 'ed25519')
101+
const secp = await client.getKey('storage-encryption', 'secp256k1')
102+
const ed = await client.getKey('storage-encryption', 'ed25519')
103103
expect(secp.key).not.toEqual(ed.key)
104104
})
105105

106106
it('should derive different material than v0 for the same name', async () => {
107-
const v0 = await new DstackClientV0().getKey('wallet', '', 'secp256k1')
108-
const v1 = await new DstackClientV1().getKey('wallet', 'secp256k1')
107+
const v0 = await new DstackClientV0().getKey('storage-encryption', '', 'secp256k1')
108+
const v1 = await new DstackClientV1().getKey('storage-encryption', 'secp256k1')
109109
expect(v1.key).not.toEqual(v0.key)
110110
})
111111

@@ -124,17 +124,17 @@ describe('DstackClientV1', () => {
124124

125125
it('should reject an empty algorithm without a round trip', async () => {
126126
const client = new DstackClientV1()
127-
await expect(() => client.getKey('wallet', '')).rejects.toThrow('algorithm is required')
127+
await expect(() => client.getKey('storage-encryption', '')).rejects.toThrow('algorithm is required')
128128
})
129129

130130
it('should reject the v0 k256 alias', async () => {
131131
const client = new DstackClientV1()
132-
await expect(() => client.getKey('wallet', 'k256')).rejects.toThrow()
132+
await expect(() => client.getKey('storage-encryption', 'k256')).rejects.toThrow()
133133
})
134134

135135
it('should reject secp256k1_prehashed, which named a signing mode', async () => {
136136
const client = new DstackClientV1()
137-
await expect(() => client.getKey('wallet', 'secp256k1_prehashed')).rejects.toThrow()
137+
await expect(() => client.getKey('storage-encryption', 'secp256k1_prehashed')).rejects.toThrow()
138138
})
139139
})
140140

sdk/python/tests/test_client_v1.py

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -43,9 +43,9 @@ def test_v0_call_shapes_fail_loudly_on_the_default_client():
4343
"""
4444
client = DstackClient()
4545
with pytest.raises(TypeError):
46-
client.get_key(path="wallet/eth", purpose="mainnet")
46+
client.get_key(path="storage-encryption", purpose="mainnet")
4747
with pytest.raises(TypeError):
48-
client.get_key("wallet/eth")
48+
client.get_key("storage-encryption")
4949
for name in ["sign", "verify", "emit_event", "get_quote", "get_tls_key"]:
5050
assert not hasattr(client, name)
5151

@@ -115,15 +115,15 @@ def check_info_response(result: InfoResponseV1):
115115

116116
def test_sync_v1_get_key():
117117
client = DstackClientV1()
118-
result = client.get_key("wallet/eth", "secp256k1")
118+
result = client.get_key("storage-encryption", "secp256k1")
119119
assert isinstance(result, GetKeyResponseV1)
120120
assert len(result.decode_key()) == 32
121121
# secp256k1 public keys are SEC1 compressed, and the chain's first link
122122
# commits to exactly these bytes.
123123
assert len(result.decode_public_key()) == 33
124124
assert len(result.decode_signature_chain()) == 2
125125

126-
ed = client.get_key("wallet/eth", "ed25519")
126+
ed = client.get_key("storage-encryption", "ed25519")
127127
assert len(ed.decode_key()) == 32
128128
assert len(ed.decode_public_key()) == 32
129129
# The v1 KDF binds the algorithm, so one name no longer serves two curves.
@@ -144,22 +144,22 @@ async def test_async_v1_get_key_is_deterministic_per_domain():
144144
def test_v1_get_key_requires_an_algorithm():
145145
client = DstackClientV1()
146146
with pytest.raises(ValueError, match="algorithm is required"):
147-
client.get_key("wallet/eth", "")
147+
client.get_key("storage-encryption", "")
148148

149149

150150
@pytest.mark.asyncio
151151
async def test_async_v1_get_key_rejects_the_v0_k256_alias():
152152
"""v0 accepted `k256`; v1 refuses rather than guess what the caller meant."""
153153
client = AsyncDstackClientV1()
154154
with pytest.raises(Exception) as excinfo:
155-
await client.get_key("wallet/eth", "k256")
155+
await client.get_key("storage-encryption", "k256")
156156
assert "k256" in str(excinfo.value)
157157

158158

159159
def test_v1_keys_differ_from_v0_keys():
160160
"""No compatibility mode: the same name yields different key material."""
161-
v0 = DstackClientV0().get_key("wallet/eth", "")
162-
v1 = DstackClientV1().get_key("wallet/eth", "secp256k1")
161+
v0 = DstackClientV0().get_key("storage-encryption", "")
162+
v1 = DstackClientV1().get_key("storage-encryption", "secp256k1")
163163
assert v1.decode_key() != v0.decode_key()
164164

165165

0 commit comments

Comments
 (0)