Skip to content

Commit 62a231b

Browse files
committed
fix(test): update error messages
1 parent 9db444b commit 62a231b

File tree

1 file changed

+10
-10
lines changed

1 file changed

+10
-10
lines changed

rust/crd/src/authentication.rs

+10-10
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,7 @@ pub enum Error {
7373
},
7474

7575
#[snafu(display(
76-
"tLS verification cannot be disabled in Superset (AuthenticationClass {auth_class_name:?})"
76+
"TLS verification cannot be disabled in Superset (AuthenticationClass {auth_class_name:?})"
7777
))]
7878
TlsVerificationCannotBeDisabled { auth_class_name: String },
7979

@@ -527,7 +527,7 @@ mod tests {
527527
.await;
528528

529529
assert_eq!(
530-
r#"The AuthenticationClass "oidc" is referenced several times which is not allowed."#,
530+
r#"the AuthenticationClass "oidc" is referenced several times which is not allowed"#,
531531
error_message
532532
);
533533
}
@@ -567,7 +567,7 @@ mod tests {
567567
.await;
568568

569569
assert_eq!(
570-
"Only one authentication type at a time is supported by Superset, see https://github.com/dpgaspar/Flask-AppBuilder/issues/1924.",
570+
"only one authentication type at a time is supported by Superset, see https://github.com/dpgaspar/Flask-AppBuilder/issues/1924",
571571
error_message
572572
);
573573
}
@@ -603,7 +603,7 @@ mod tests {
603603
.await;
604604

605605
assert_eq!(
606-
"Only one LDAP provider at a time is supported by Superset.",
606+
"only one LDAP provider at a time is supported by Superset",
607607
error_message
608608
);
609609
}
@@ -648,7 +648,7 @@ mod tests {
648648
.await;
649649

650650
assert_eq!(
651-
"The userRegistration settings must not differ between the authentication entries.",
651+
"the userRegistration settings must not differ between the authentication entries",
652652
error_message
653653
);
654654
}
@@ -693,7 +693,7 @@ mod tests {
693693
.await;
694694

695695
assert_eq!(
696-
"The userRegistrationRole settings must not differ between the authentication entries.",
696+
"the userRegistrationRole settings must not differ between the authentication entries",
697697
error_message
698698
);
699699
}
@@ -738,7 +738,7 @@ mod tests {
738738
.await;
739739

740740
assert_eq!(
741-
"The syncRolesAt settings must not differ between the authentication entries.",
741+
"the syncRolesAt settings must not differ between the authentication entries",
742742
error_message
743743
);
744744
}
@@ -767,7 +767,7 @@ mod tests {
767767

768768
assert_eq!(
769769
indoc! { r#"
770-
Invalid OIDC configuration
770+
invalid OIDC configuration
771771
772772
Caused by this error:
773773
1: OIDC authentication details not specified. The AuthenticationClass "oidc" uses an OIDC provider, you need to specify OIDC authentication details (such as client credentials) as well"#
@@ -801,7 +801,7 @@ mod tests {
801801
.await;
802802

803803
assert_eq!(
804-
r#""sub" is not a supported principalClaim in Superset for the Keycloak OIDC provider. Please use "preferred_username" in the AuthenticationClass "oidc""#,
804+
r#"invalid principalClaim "sub" in the "oidc" AuthenticationClass. Superset hard-codes the claim name to "preferred_username" for the Keycloak OIDC provider"#,
805805
error_message
806806
);
807807
}
@@ -834,7 +834,7 @@ mod tests {
834834
.await;
835835

836836
assert_eq!(
837-
r#"TLS verification cannot be disabled in Superset (AuthenticationClass "oidc")."#,
837+
r#"TLS verification cannot be disabled in Superset (AuthenticationClass "oidc")"#,
838838
error_message
839839
);
840840
}

0 commit comments

Comments
 (0)