Merge pull request #35 from seomoz/wildcard-tests

More wildcard tests
seomoz · Feb 12, 2018 · a2f304e · a2f304e
2 parents 713577a + 208dbf4
commit a2f304e
Show file tree

Hide file tree

Showing 2 changed files with 38 additions and 0 deletions.
diff --git a/test/test-agent.cpp b/test/test-agent.cpp
@@ -60,6 +60,17 @@ TEST(AgentTest, DisallowNone)
     EXPECT_TRUE(agent.allowed("/anything"));
 }
 
+TEST(AgentTest, MiddleWildcard)
+{
+    Rep::Agent agent = Rep::Agent("a.com").disallow("/test*foo");
+    EXPECT_FALSE(agent.allowed("/testfoo"));
+    EXPECT_FALSE(agent.allowed("/testafoo"));
+    EXPECT_FALSE(agent.allowed("/testaasdffoo"));
+    EXPECT_FALSE(agent.allowed("/test/foo"));
+    EXPECT_TRUE(agent.allowed("/testfo"));
+    EXPECT_TRUE(agent.allowed("/estfoo"));
+}
+
 TEST(AgentTest, EscapedRule)
 {
     Rep::Agent agent = Rep::Agent("a.com").disallow("/a%3cd.html");

diff --git a/test/test-directive.cpp b/test/test-directive.cpp
@@ -70,6 +70,33 @@ TEST(DirectiveTest, WildcardTest)
     }
 }
 
+TEST(DirectiveTest, LeadingWildcard)
+{
+    std::vector<std::string> examples = {
+        "/test",
+        "/a/test",
+        "/ab/test",
+        "/abc/test",
+    };
+    std::string directive("*/test");
+    Rep::Directive parsed(directive, true);
+    for (auto example : examples)
+    {
+        EXPECT_TRUE(parsed.match(example)) <<
+            example << " didn't match " << directive;
+    }
+
+    std::vector<std::string> antiexamples = {
+        "/tes",
+        "/est",
+    };
+    for (auto example : antiexamples)
+    {
+        EXPECT_FALSE(parsed.match(example)) <<
+            example << " matched " << directive;
+    }
+}
+
 TEST(DirectiveTest, MultipleWildcardTest)
 {
     std::vector<std::string> examples = {