Address peer review comments.

Signed-off-by: Michel Hidalgo <michel@ekumenlabs.com>
ros2 · Jun 15, 2020 · 38103ef · 38103ef
1 parent 536e844
commit 38103ef
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/rcl/test/rcl/test_validate_enclave_name.cpp b/rcl/test/rcl/test_validate_enclave_name.cpp
@@ -64,7 +64,7 @@ TEST(TestValidateEnclaveName, test_validation_string) {
     {"/foo/$", RCL_ENCLAVE_NAME_INVALID_CONTAINS_UNALLOWED_CHARACTERS, 5},
     {"/bar#", RCL_ENCLAVE_NAME_INVALID_CONTAINS_UNALLOWED_CHARACTERS, 4},
     {"/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}
@@ -85,6 +85,8 @@ TEST(TestValidateEnclaveName, test_validation_string) {
       "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(-1));  // invalid result
+  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));
 }