Skip to content

Commit 5a37fe9

Browse files
FrankYang0529FrankYang0529
committed
test: add bindle keyring & secret keys
Signed-off-by: Frank Yang <[email protected]>
1 parent 565ad43 commit 5a37fe9

File tree

7 files changed

+72
-6
lines changed

7 files changed

+72
-6
lines changed

‎.github/workflows/build.yml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ jobs:
2525
extension: "",
2626
# We have this enabled for releases, so we should test it.
2727
extraArgs: "--features openssl/vendored",
28-
bindleUrl: "https://bindle.blob.core.windows.net/releases/bindle-v0.8.0-linux-amd64.tar.gz",
28+
bindleUrl: "https://bindle.blob.core.windows.net/releases/bindle-v0.9.0-rc.1-linux-amd64.tar.gz",
2929
bindleBinary: "bindle-server",
3030
pathInBindleArchive: "bindle-server",
3131
nomadUrl: "https://releases.hashicorp.com/nomad/1.3.1/nomad_1.3.1_linux_amd64.zip",
@@ -41,7 +41,7 @@ jobs:
4141
os: "macos-latest",
4242
extension: "",
4343
extraArgs: "",
44-
bindleUrl: "https://bindle.blob.core.windows.net/releases/bindle-v0.8.0-macos-amd64.tar.gz",
44+
bindleUrl: "https://bindle.blob.core.windows.net/releases/bindle-v0.9.0-rc.1-macos-amd64.tar.gz",
4545
bindleBinary: "bindle-server",
4646
pathInBindleArchive: "bindle-server",
4747
nomadUrl: "https://releases.hashicorp.com/nomad/1.3.1/nomad_1.3.1_darwin_amd64.zip",
@@ -53,7 +53,7 @@ jobs:
5353
os: "windows-latest",
5454
extension: ".exe",
5555
extraArgs: "",
56-
bindleUrl: "https://bindle.blob.core.windows.net/releases/bindle-v0.8.0-windows-amd64.tar.gz",
56+
bindleUrl: "https://bindle.blob.core.windows.net/releases/bindle-v0.9.0-rc.1-windows-amd64.tar.gz",
5757
bindleBinary: "bindle-server.exe",
5858
pathInBindleArchive: "bindle-server.exe",
5959
nomadUrl: "https://releases.hashicorp.com/nomad/1.3.1/nomad_1.3.1_windows_amd64.zip",

‎Makefile

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -30,8 +30,8 @@ test-integration:
3030

3131
.PHONY: test-e2e
3232
test-e2e:
33-
RUST_LOG=$(LOG_LEVEL) cargo test --test integration --features e2e-tests --no-fail-fast -- integration_tests::test_dependencies --nocapture
34-
RUST_LOG=$(LOG_LEVEL) cargo test --test integration --features e2e-tests --no-fail-fast -- --skip integration_tests::test_dependencies --nocapture
33+
RUST_LOG=$(LOG_LEVEL) cargo test --test integration --features e2e-tests --no-fail-fast -- integration_tests::e2e_tests::test_dependencies --nocapture
34+
RUST_LOG=$(LOG_LEVEL) cargo test --test integration --features e2e-tests --no-fail-fast -- --skip integration_tests::e2e_tests::test_dependencies --nocapture
3535

3636
.PHONY: test-sdk-go
3737
test-sdk-go:

‎crates/publish/src/expander.rs

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -494,7 +494,9 @@ async fn get_bindle_first_matching_key(
494494
role: &SignatureRole,
495495
label_match: Option<&LabelMatch>,
496496
) -> Result<SecretKeyEntry> {
497-
let keys = SecretKeyFile::load_file(&fpath).await?;
497+
let keys = SecretKeyFile::load_file(&fpath)
498+
.await
499+
.with_context(|| format!("can't read file {:?}", &fpath))?;
498500

499501
keys.key
500502
.iter()

‎tests/http/client_keyring.toml

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,7 @@
1+
version = "1.0"
2+
3+
[[key]]
4+
label = "Test<[email protected]>"
5+
roles = ["creator"]
6+
key = "HfXxdRvgUWEE0f36EIa8+njPJVoMvmF6/gO/T19aV90="
7+
labelSignature = "En2GtxiLbzAIoFu2zQ5HWXwck5lONq/ALYGt8RQYUDNpStk23JcaN/Gx9zh9oEJGnVeDwHLNj9/Cql6j23O1Cw=="

‎tests/http/client_secret_keys.toml

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,6 @@
1+
version = "1.0"
2+
3+
[[key]]
4+
label = "Test<[email protected]>"
5+
keypair = "O6QdMC7mvxepvqDIXtj9b66ooMf4Mlz4Mmz65Em2Q8Ed9fF1G+BRYQTR/foQhrz6eM8lWgy+YXr+A79PX1pX3Q=="
6+
roles = ["creator"]

‎tests/http/server_keyring.toml

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,7 @@
1+
version = "1.0"
2+
3+
[[key]]
4+
label = "Test<[email protected]>"
5+
roles = ["creator"]
6+
key = "HfXxdRvgUWEE0f36EIa8+njPJVoMvmF6/gO/T19aV90="
7+
labelSignature = "En2GtxiLbzAIoFu2zQ5HWXwck5lONq/ALYGt8RQYUDNpStk23JcaN/Gx9zh9oEJGnVeDwHLNj9/Cql6j23O1Cw=="

‎tests/integration.rs

Lines changed: 44 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -49,6 +49,10 @@ mod integration_tests {
4949
const BINDLE_SERVER_BASIC_AUTH_HTPASSWD_FILE: &str = "tests/http/htpasswd";
5050
const BINDLE_SERVER_BASIC_AUTH_USER: &str = "bindle-user";
5151
const BINDLE_SERVER_BASIC_AUTH_PASSWORD: &str = "topsecret";
52+
const BINDLE_CLIENT_SECRET_FILE: &str = "tests/http/client_secret_keys.toml";
53+
pub const BINDLE_CLIENT_KEYRING_FILE: &str = "tests/http/server_keyring.toml";
54+
const BINDLE_SERVER_KEYRING_FILE: &str = "tests/http/client_keyring.toml";
55+
const BINDLE_LABEL: &str = "\"Test<[email protected]>\"";
5256

5357
const HIPPO_BASIC_AUTH_USER: &str = "hippo-user";
5458
const HIPPO_BASIC_AUTH_PASSWORD: &str = "topsecret";
@@ -88,6 +92,12 @@ mod integration_tests {
8892
),
8993
"--bindle-server",
9094
&b.url,
95+
"--bindle-secret-file",
96+
BINDLE_CLIENT_SECRET_FILE,
97+
"--bindle-keyring-file",
98+
BINDLE_CLIENT_KEYRING_FILE,
99+
"--bindle-label",
100+
BINDLE_LABEL,
91101
],
92102
None,
93103
None,
@@ -129,6 +139,12 @@ mod integration_tests {
129139
BINDLE_SERVER_BASIC_AUTH_USER,
130140
"--bindle-password",
131141
BINDLE_SERVER_BASIC_AUTH_PASSWORD,
142+
"--bindle-secret-file",
143+
BINDLE_CLIENT_SECRET_FILE,
144+
"--bindle-keyring-file",
145+
BINDLE_CLIENT_KEYRING_FILE,
146+
"--bindle-label",
147+
BINDLE_LABEL,
132148
],
133149
None,
134150
None,
@@ -166,6 +182,12 @@ mod integration_tests {
166182
),
167183
"--bindle-server",
168184
&b.url,
185+
"--bindle-secret-file",
186+
BINDLE_CLIENT_SECRET_FILE,
187+
"--bindle-keyring-file",
188+
BINDLE_CLIENT_KEYRING_FILE,
189+
"--bindle-label",
190+
BINDLE_LABEL,
169191
],
170192
None,
171193
None,
@@ -211,6 +233,12 @@ mod integration_tests {
211233
),
212234
"--bindle-server",
213235
&b.url,
236+
"--bindle-secret-file",
237+
BINDLE_CLIENT_SECRET_FILE,
238+
"--bindle-keyring-file",
239+
BINDLE_CLIENT_KEYRING_FILE,
240+
"--bindle-label",
241+
BINDLE_LABEL,
214242
],
215243
None,
216244
None,
@@ -266,6 +294,12 @@ mod integration_tests {
266294
),
267295
"--bindle-server",
268296
&b.url,
297+
"--bindle-secret-file",
298+
BINDLE_CLIENT_SECRET_FILE,
299+
"--bindle-keyring-file",
300+
BINDLE_CLIENT_KEYRING_FILE,
301+
"--bindle-label",
302+
BINDLE_LABEL,
269303
],
270304
None,
271305
None,
@@ -318,6 +352,12 @@ mod integration_tests {
318352
HIPPO_BASIC_AUTH_USER,
319353
"--hippo-password",
320354
HIPPO_BASIC_AUTH_PASSWORD,
355+
"--bindle-secret-file",
356+
BINDLE_CLIENT_SECRET_FILE,
357+
"--bindle-keyring-file",
358+
BINDLE_CLIENT_KEYRING_FILE,
359+
"--bindle-label",
360+
BINDLE_LABEL,
321361
],
322362
None,
323363
None,
@@ -397,6 +437,8 @@ mod integration_tests {
397437
server_cache.path().to_string_lossy().to_string().as_str(),
398438
"-i",
399439
address.as_str(),
440+
"--keyring",
441+
BINDLE_SERVER_KEYRING_FILE,
400442
],
401443
auth_args.as_slice(),
402444
]
@@ -719,6 +761,8 @@ mod integration_tests {
719761
bindle_url,
720762
"--listen",
721763
&url,
764+
"--bindle-keyring-file",
765+
e2e_tests::BINDLE_CLIENT_KEYRING_FILE,
722766
];
723767
for v in env {
724768
args.push("--env");

0 commit comments

Comments
 (0)