Added ACL test file

juanfont · Jun 8, 2022 · 39f03b8 · 39f03b8
1 parent 8287ba2
commit 39f03b8
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 1 deletion.
diff --git a/acls_test.go b/acls_test.go
@@ -321,6 +321,20 @@ func (s *Suite) TestPortRange(c *check.C) {
 	c.Assert(rules[0].DstPorts[0].Ports.Last, check.Equals, uint16(5500))
 }
 
+func (s *Suite) TestProtocolParsing(c *check.C) {
+	err := app.LoadACLPolicy("./tests/acls/acl_policy_basic_protocols.hujson")
+	c.Assert(err, check.IsNil)
+
+	rules, err := app.generateACLRules()
+	c.Assert(err, check.IsNil)
+	c.Assert(rules, check.NotNil)
+
+	c.Assert(rules, check.HasLen, 3)
+	c.Assert(rules[0].IPProto[0], check.Equals, 6)  // tcp
+	c.Assert(rules[1].IPProto[0], check.Equals, 17) // udp
+	c.Assert(rules[2].IPProto[1], check.Equals, 58) // icmp v4
+}
+
 func (s *Suite) TestPortWildcard(c *check.C) {
 	err := app.LoadACLPolicy("./tests/acls/acl_policy_basic_wildcards.hujson")
 	c.Assert(err, check.IsNil)

diff --git a/tests/acls/acl_policy_1.hujson b/tests/acls/acl_policy_1.hujson
@@ -35,7 +35,6 @@
         // ports on git-server or ci-server.
         {
             "action": "accept",
-            "protocol": "tcp",
             "src": [
                 "group:example2",
                 "192.168.1.0/24"