Merge pull request #1879 from Expensify/marcaaron-redactTest

Add test for `SRedactSensitiveValues`
Expensify · Oct 8, 2024 · 745389d · 745389d
2 parents 599ae10 + 920970f
commit 745389d
Showing 1 changed file with 17 additions and 2 deletions.
diff --git a/test/tests/LibStuffTest.cpp b/test/tests/LibStuffTest.cpp
@@ -34,7 +34,8 @@ struct LibStuff : tpunit::TestFixture {
                                     TEST(LibStuff::testFirstOfMonth),
                                     TEST(LibStuff::SREMatchTest),
                                     TEST(LibStuff::SREReplaceTest),
-                                    TEST(LibStuff::SQResultTest)
+                                    TEST(LibStuff::SQResultTest),
+                                    TEST(LibStuff::SRedactSensitiveValuesTest)
                                     )
     { }
 
@@ -647,7 +648,7 @@ struct LibStuff : tpunit::TestFixture {
 
         // Case sensitive but case doesn't match.
         ASSERT_FALSE(SREMatch(".*CAT.*", "this contains cat"));
-        
+
         // Case-insensitive.
         ASSERT_TRUE(SREMatch(".*CAT.*", "this contains cat", false));
         ASSERT_FALSE(SREMatch(".*CAT.*", "this does not", false));
@@ -744,4 +745,18 @@ struct LibStuff : tpunit::TestFixture {
         db.rollback();
         ASSERT_EQUAL(result[0]["coco"], "name1");
     }
+
+    void SRedactSensitiveValuesTest() {
+        string logValue = R"({"edits":["test1", "test2", "test3"]})";
+        SRedactSensitiveValues(logValue);
+        ASSERT_EQUAL(R"({"edits":["REDACTED"]})", logValue);
+
+        logValue = R"({"authToken":"123IMANAUTHTOKEN321"})";
+        SRedactSensitiveValues(logValue);
+        ASSERT_EQUAL(R"({"authToken":<REDACTED>)", logValue);
+
+        logValue = R"({"html":"private conversation happens here"})";
+        SRedactSensitiveValues(logValue);
+        ASSERT_EQUAL(R"({"html":"<REDACTED>"})", logValue);
+    }
 } __LibStuff;