Improve enclave validation test coverage. (#682)
Signed-off-by: Michel Hidalgo <michel@ekumenlabs.com>
This commit is contained in:
parent
a4e6485313
commit
e2988b25ae
1 changed files with 8 additions and 1 deletions
|
@ -64,7 +64,10 @@ TEST(TestValidateEnclaveName, test_validation_string) {
|
||||||
{"/foo/$", RCL_ENCLAVE_NAME_INVALID_CONTAINS_UNALLOWED_CHARACTERS, 5},
|
{"/foo/$", RCL_ENCLAVE_NAME_INVALID_CONTAINS_UNALLOWED_CHARACTERS, 5},
|
||||||
{"/bar#", RCL_ENCLAVE_NAME_INVALID_CONTAINS_UNALLOWED_CHARACTERS, 4},
|
{"/bar#", RCL_ENCLAVE_NAME_INVALID_CONTAINS_UNALLOWED_CHARACTERS, 4},
|
||||||
{"/foo//bar", RCL_ENCLAVE_NAME_INVALID_CONTAINS_REPEATED_FORWARD_SLASH, 5},
|
{"/foo//bar", RCL_ENCLAVE_NAME_INVALID_CONTAINS_REPEATED_FORWARD_SLASH, 5},
|
||||||
{"/1bar", RCL_ENCLAVE_NAME_INVALID_NAME_TOKEN_STARTS_WITH_NUMBER, 1}
|
{"/1bar", RCL_ENCLAVE_NAME_INVALID_NAME_TOKEN_STARTS_WITH_NUMBER, 1},
|
||||||
|
{"/" + std::string(RCL_ENCLAVE_NAME_MAX_LENGTH, 'o'),
|
||||||
|
RCL_ENCLAVE_NAME_INVALID_TOO_LONG,
|
||||||
|
RCL_ENCLAVE_NAME_MAX_LENGTH - 1}
|
||||||
};
|
};
|
||||||
for (const auto & case_tuple : enclave_cases_that_should_fail) {
|
for (const auto & case_tuple : enclave_cases_that_should_fail) {
|
||||||
std::string enclave = case_tuple.enclave;
|
std::string enclave = case_tuple.enclave;
|
||||||
|
@ -82,4 +85,8 @@ TEST(TestValidateEnclaveName, test_validation_string) {
|
||||||
"Enclave '" << enclave << "' failed with '" << validation_result << "'.";
|
"Enclave '" << enclave << "' failed with '" << validation_result << "'.";
|
||||||
EXPECT_NE(nullptr, rcl_enclave_name_validation_result_string(validation_result)) << enclave;
|
EXPECT_NE(nullptr, rcl_enclave_name_validation_result_string(validation_result)) << enclave;
|
||||||
}
|
}
|
||||||
|
EXPECT_STREQ(
|
||||||
|
"unknown result code for rcl context name validation",
|
||||||
|
rcl_enclave_name_validation_result_string(-1)); // invalid result
|
||||||
|
EXPECT_EQ(nullptr, rcl_enclave_name_validation_result_string(RCL_ENCLAVE_NAME_VALID));
|
||||||
}
|
}
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue