commit 5d12127: [Test] Add test for the added features

Sat Dec 10 22:56:04 UTC 2022

Author: Vsevolod Stakhov
Date: 2022-12-10 22:41:52 +0000
URL: https://github.com/rspamd/rspamd/commit/5d12127d4a5843174f516559e39afe580fd1af6c (HEAD -> master)

[Test] Add test for the added features

---
 test/lua/unit/selectors.combined.lua | 45 ++++++++++++++++++++----------------
 1 file changed, 25 insertions(+), 20 deletions(-)

diff --git a/test/lua/unit/selectors.combined.lua b/test/lua/unit/selectors.combined.lua
index 06c5c0fca..31133dbb0 100644
--- a/test/lua/unit/selectors.combined.lua
+++ b/test/lua/unit/selectors.combined.lua
@@ -33,35 +33,40 @@ context("Selectors test", function()
 
   local cases = {
     ["rcpts + weekend"] = {
-                selector = "rcpts:addr.take_n(5).lower;time('message', '!%w').in(6, 7).id('weekends')",
-                expect = {
-                  "nobody at example.com:weekends",
-                  "no-one at example.com:weekends"}},
+      selector = "rcpts:addr.take_n(5).lower;time('message', '!%w').in(6, 7).id('weekends')",
+      expect = {
+        "nobody at example.com:weekends",
+        "no-one at example.com:weekends"}},
 
     ["weekend + rcpts"] = {
-                selector = "time('message', '!%w').in(6, 7).id('weekends');rcpts:addr.take_n(5).lower",
-                expect = {
-                  "weekends:nobody at example.com",
-                  "weekends:no-one at example.com"}},
+      selector = "time('message', '!%w').in(6, 7).id('weekends');rcpts:addr.take_n(5).lower",
+      expect = {
+        "weekends:nobody at example.com",
+        "weekends:no-one at example.com"}},
 
     ["id(rcpt) + rcpts + weekend"] = {
-                selector = "id('rcpt');rcpts:addr.take_n(5).lower;time('message', '!%w').in(6, 7).id('weekends')",
-                expect = {
-                  "rcpt:nobody at example.com:weekends",
-                  "rcpt:no-one at example.com:weekends"}},
+      selector = "id('rcpt');rcpts:addr.take_n(5).lower;time('message', '!%w').in(6, 7).id('weekends')",
+      expect = {
+        "rcpt:nobody at example.com:weekends",
+        "rcpt:no-one at example.com:weekends"}},
 
     ["id(rcpt) + id(2) rcpts + weekend"] = {
-                selector = "id('rcpt'); id(2); rcpts:addr.take_n(5).lower; time('message', '!%w').in(6, 7).id('weekends')",
-                expect = {
-                  "rcpt:2:nobody at example.com:weekends",
-                  "rcpt:2:no-one at example.com:weekends"}},
+      selector = "id('rcpt'); id(2); rcpts:addr.take_n(5).lower; time('message', '!%w').in(6, 7).id('weekends')",
+      expect = {
+        "rcpt:2:nobody at example.com:weekends",
+        "rcpt:2:no-one at example.com:weekends"}
+    },
 
     -- There are two rcpts but only one url in the message
     -- resulting table size is the size of the smallest table
     ["id(rcpt) + id(2) + rcpts and urls + weekend"] = {
-                selector = "id('rcpt'); id(2); rcpts:addr.take_n(5).lower; id('urls'); urls:get_host; time('message', '!%w').in(6, 7).id('weekends')",
-                expect = {
-                  "rcpt:2:nobody at example.com:urls:example.net:weekends"}},
+      selector = "id('rcpt'); id(2); rcpts:addr.take_n(5).lower; id('urls'); urls:get_host; time('message', '!%w').in(6, 7).id('weekends')",
+      expect = { "rcpt:2:nobody at example.com:urls:example.net:weekends"}
+    },
+    ["url + apply_methods"] = {
+      selector = "urls.apply_methods('get_host', 'get_path').join_tables('/')",
+      expect = {"example.net/path"}
+    },
   }
 
   for case_name, case in pairs(cases) do
@@ -99,7 +104,7 @@ Hello world
 Content-Type: text/html; charset="utf-8"
 
 <html><body>
-<a href="http://example.net">http://example.net</a>
+<a href="http://example.net/path?query">http://example.net</a>
 <a href="mailto:test at example.net">mail me</a>
 </html>
 
