Skip to content

Commit

Permalink
Enable S3 fields to parsing tests
Browse files Browse the repository at this point in the history
Signed-off-by: Robert Detjens <[email protected]>
  • Loading branch information
detjensrobert committed Dec 6, 2024
1 parent 1e7cd3b commit 54665ea
Show file tree
Hide file tree
Showing 2 changed files with 22 additions and 74 deletions.
74 changes: 11 additions & 63 deletions Cargo.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

22 changes: 11 additions & 11 deletions src/tests/parsing/config.rs
Original file line number Diff line number Diff line change
Expand Up @@ -104,13 +104,13 @@ fn all_yaml() {
challenges_domain: "chals.frontend.example".to_string(),
kubeconfig: None,
kubecontext: "testcluster".to_string(),
// s3: S3Config {
// bucket_name: "asset_testing".to_string(),
// endpoint: "s3.example".to_string(),
// region: "us-fake-1".to_string(),
// access_key: "accesskey".to_string(),
// secret_key: "secretkey".to_string(),
// }
s3: S3Config {
bucket_name: "asset_testing".to_string(),
endpoint: "s3.example".to_string(),
region: "us-fake-1".to_string(),
access_key: "accesskey".to_string(),
secret_key: "secretkey".to_string(),
},
},
)]),
};
Expand Down Expand Up @@ -193,8 +193,8 @@ fn yaml_with_env_overrides() {
let profile = config.profiles.get("testing").unwrap();

assert_eq!(profile.frontend_token, "envtoken");
// assert_eq!(profile.s3.access_key, "envkey");
// assert_eq!(profile.s3.secret_key, "envsecret");
assert_eq!(profile.s3.access_key, "envkey");
assert_eq!(profile.s3.secret_key, "envsecret");

Ok(())
});
Expand Down Expand Up @@ -263,8 +263,8 @@ fn partial_yaml_with_env() {
let profile = config.profiles.get("testing").unwrap();

assert_eq!(profile.frontend_token, "envtoken");
// assert_eq!(profile.s3.access_key, "envkey");
// assert_eq!(profile.s3.secret_key, "envsecret");
assert_eq!(profile.s3.access_key, "envkey");
assert_eq!(profile.s3.secret_key, "envsecret");

Ok(())
});
Expand Down

0 comments on commit 54665ea

Please sign in to comment.