]> gitweb.michael.orlitzky.com - hath.git/blobdiff - src/Cidr.hs
Add more Haddock comments.
[hath.git] / src / Cidr.hs
index a07117d5382842b94d23f87fe469c6831c997aad..002ec0b849fc08d555a2e6ab221212143c51d160 100644 (file)
@@ -1,6 +1,9 @@
+-- | The CIDR modules contains most of the functions used for working
+--   with the CIDR type.
 module Cidr
 ( Cidr(..),
   cidr_from_string,
+  cidr_properties,
   cidr_tests,
   combine_all,
   contains,
@@ -18,13 +21,17 @@ module Cidr
 ) where
 
 import Data.List (nubBy)
+import Data.List.Split (splitOneOf)
 import Data.Maybe (catMaybes, fromJust)
-import Test.HUnit
-import Test.QuickCheck
+
+import Test.HUnit (assertEqual)
+import Test.Framework (Test, testGroup)
+import Test.Framework.Providers.HUnit (testCase)
+import Test.Framework.Providers.QuickCheck2 (testProperty)
+import Test.QuickCheck (Arbitrary(..), Gen, Property, (==>))
 
 import qualified Bit as B
 import IPv4Address
-import ListUtils
 import Maskable
 import Maskbits
 import Octet
@@ -50,27 +57,27 @@ instance Eq Cidr where
   cidr1 == cidr2 = (cidr1 `equivalent` cidr2)
 
 
--- Two CIDR ranges are equivalent if they have the same network bits
--- and the masks are the same.
+-- Two CIDR ranges are equivalent if they have the same network bits
+--   and the masks are the same.
 equivalent :: Cidr -> Cidr -> Bool
 equivalent (Cidr addr1 mbits1) (Cidr addr2 mbits2) =
     (mbits1 == mbits2) && ((apply_mask addr1 mbits1 B.Zero) == (apply_mask addr2 mbits2 B.Zero))
 
--- Returns the mask portion of a CIDR address. That is, everything
--- after the trailing slash.
+-- Returns the mask portion of a CIDR address. That is, everything
+--   after the trailing slash.
 maskbits_from_cidr_string :: String -> Maybe Maskbits
 maskbits_from_cidr_string s
   | length partlist == 2 = maskbits_from_string (partlist !! 1)
   | otherwise = Nothing
     where
-      partlist = (splitWith (`elem` "/") s)
+      partlist = splitOneOf "/" s
 
 
 -- | Takes an IP address String in CIDR notation, and returns a list
 --   of its octets (as Ints).
 octets_from_cidr_string :: String -> [Octet]
 octets_from_cidr_string s =
-  catMaybes $ map octet_from_string (take 4 (splitWith (`elem` "./") s))
+  catMaybes $ map octet_from_string (take 4 (splitOneOf "./" s))
 
 
 -- | Return Nothing if we can't parse both maskbits and octets from
@@ -87,35 +94,53 @@ cidr_from_string s =
 
 
 
+-- | Given a CIDR, return the minimum valid IPv4 address contained
+--   within it.
 min_host :: Cidr -> IPv4Address
 min_host (Cidr addr mask) = apply_mask addr mask B.Zero
 
-
+-- | Given a CIDR, return the maximum valid IPv4 address contained
+--   within it.
 max_host :: Cidr -> IPv4Address
 max_host (Cidr addr mask) = apply_mask addr mask B.One
 
-
+-- | Given a CIDR, return the first octet of the minimum valid IPv4
+--   address contained within it.
 min_octet1 :: Cidr -> Octet
 min_octet1 cidr = octet1 (min_host cidr)
 
+-- | Given a CIDR, return the second octet of the minimum valid IPv4
+--   address contained within it.
 min_octet2 :: Cidr -> Octet
 min_octet2 cidr = octet2 (min_host cidr)
 
+-- | Given a CIDR, return the third octet of the minimum valid IPv4
+--   address contained within it.
 min_octet3 :: Cidr -> Octet
 min_octet3 cidr = octet3 (min_host cidr)
 
+-- | Given a CIDR, return the fourth octet of the minimum valid IPv4
+--   address contained within it.
 min_octet4 :: Cidr -> Octet
 min_octet4 cidr = octet4 (min_host cidr)
 
+-- | Given a CIDR, return the first octet of the maximum valid IPv4
+--   address contained within it.
 max_octet1 :: Cidr -> Octet
 max_octet1 cidr = octet1 (max_host cidr)
 
+-- | Given a CIDR, return the second octet of the maximum valid IPv4
+--   address contained within it.
 max_octet2 :: Cidr -> Octet
 max_octet2 cidr = octet2 (max_host cidr)
 
+-- | Given a CIDR, return the third octet of the maximum valid IPv4
+--   address contained within it.
 max_octet3 :: Cidr -> Octet
 max_octet3 cidr = octet3 (max_host cidr)
 
+-- | Given a CIDR, return the fourth octet of the maximum valid IPv4
+--   address contained within it.
 max_octet4 :: Cidr -> Octet
 max_octet4 cidr = octet4 (max_host cidr)
 
@@ -162,6 +187,7 @@ contains (Cidr addr1 mbits1) (Cidr addr2 mbits2)
     addr2masked = apply_mask addr2 mbits1 B.Zero
 
 
+-- | Contains but is not equal to.
 contains_proper :: Cidr -> Cidr -> Bool
 contains_proper cidr1 cidr2 =
     (cidr1 `contains` cidr2) && (not (cidr2 `contains` cidr1))
@@ -235,137 +261,151 @@ adjacent cidr1 cidr2
 
 test_min_host1 :: Test
 test_min_host1 =
-  TestCase $ assertEqual "The minimum host in 10.0.0.0/24 is 10.0.0.0"
-             expected
-             actual
+  testCase desc $
+    assertEqual desc
+    expected
+    actual
   where
+    desc = "The minimum host in 10.0.0.0/24 is 10.0.0.0"
     actual = show $ min_host (fromJust $ cidr_from_string "10.0.0.0/24")
     expected = "10.0.0.0"
 
 
 test_max_host1 :: Test
 test_max_host1 =
-  TestCase $ assertEqual "The maximum host in 10.0.0.0/24 is 10.0.0.255"
-             expected
-             actual
+  testCase desc $
+    assertEqual desc
+    expected
+    actual
   where
+    desc = "The maximum host in 10.0.0.0/24 is 10.0.0.255"
     actual = show $ max_host (fromJust $ cidr_from_string "10.0.0.0/24")
     expected = "10.0.0.255"
 
 
 test_equality1 :: Test
 test_equality1 =
-  TestCase $
+  testCase desc $
     assertEqual
-    "10.1.1.0/23 equals itself"
+    desc
     True
     (cidr1 == cidr1)
   where
+    desc = "10.1.1.0/23 equals itself"
     cidr1 = fromJust $ cidr_from_string "10.1.1.0/23"
 
 
 test_contains1 :: Test
 test_contains1 =
-  TestCase $
+  testCase desc $
     assertEqual
-    "10.1.1.0/23 contains 10.1.1.0/24"
+    desc
     True
     (cidr1 `contains` cidr2)
   where
+    desc = "10.1.1.0/23 contains 10.1.1.0/24"
     cidr1 = fromJust $ cidr_from_string "10.1.1.0/23"
     cidr2 = fromJust $ cidr_from_string "10.1.1.0/24"
 
 
 test_contains2 :: Test
 test_contains2 =
-  TestCase $
+  testCase desc $
     assertEqual
-    "10.1.1.0/23 contains itself"
+    desc
     True
     (cidr1 `contains` cidr1)
   where
+    desc = "10.1.1.0/23 contains itself"
     cidr1 = fromJust $ cidr_from_string "10.1.1.0/23"
 
 
 test_contains_proper1 :: Test
 test_contains_proper1 =
-  TestCase $
+  testCase desc $
     assertEqual
-    "10.1.1.0/23 contains 10.1.1.0/24 properly"
+    desc
     True
     (cidr1 `contains_proper` cidr2)
   where
+    desc = "10.1.1.0/23 contains 10.1.1.0/24 properly"
     cidr1 = fromJust $ cidr_from_string "10.1.1.0/23"
     cidr2 = fromJust $ cidr_from_string "10.1.1.0/24"
 
 
 test_contains_proper2 :: Test
 test_contains_proper2 =
-  TestCase $
+  testCase desc $
     assertEqual
-    "10.1.1.0/23 does not contain itself properly"
+    desc
     False
     (cidr1 `contains_proper` cidr1)
   where
+    desc = "10.1.1.0/23 does not contain itself properly"
     cidr1 = fromJust $ cidr_from_string "10.1.1.0/23"
 
 
 test_adjacent1 :: Test
 test_adjacent1 =
-  TestCase $
+  testCase desc $
     assertEqual
-    "10.1.0.0/24 is adjacent to 10.1.1.0/24"
+    desc
     True
     (cidr1 `adjacent` cidr2)
   where
+    desc = "10.1.0.0/24 is adjacent to 10.1.1.0/24"
     cidr1 = fromJust $ cidr_from_string "10.1.0.0/24"
     cidr2 = fromJust $ cidr_from_string "10.1.1.0/24"
 
 
 test_adjacent2 :: Test
 test_adjacent2 =
-  TestCase $
+  testCase desc $
     assertEqual
-    "10.1.0.0/23 is not adjacent to 10.1.0.0/24"
+    desc
     False
     (cidr1 `adjacent` cidr2)
   where
+    desc = "10.1.0.0/23 is not adjacent to 10.1.0.0/24"
     cidr1 = fromJust $ cidr_from_string "10.1.0.0/23"
     cidr2 = fromJust $ cidr_from_string "10.1.0.0/24"
 
 
 test_adjacent3 :: Test
 test_adjacent3 =
-  TestCase $
+  testCase desc $
     assertEqual
-    "10.1.0.0/24 is not adjacent to 10.2.5.0/24"
+    desc
     False
     (cidr1 `adjacent` cidr2)
   where
+    desc = "10.1.0.0/24 is not adjacent to 10.2.5.0/24"
     cidr1 = fromJust $ cidr_from_string "10.1.0.0/24"
     cidr2 = fromJust $ cidr_from_string "10.2.5.0/24"
 
 
 test_adjacent4 :: Test
 test_adjacent4 =
-  TestCase $
+  testCase desc $
     assertEqual
-    "10.1.1.0/24 is not adjacent to 10.1.2.0/24"
+    desc
     False
     (cidr1 `adjacent` cidr2)
   where
+    desc = "10.1.1.0/24 is not adjacent to 10.1.2.0/24"
     cidr1 = fromJust $ cidr_from_string "10.1.1.0/24"
     cidr2 = fromJust $ cidr_from_string "10.1.2.0/24"
 
 
 test_combine_contained1 :: Test
 test_combine_contained1 =
-  TestCase $
+  testCase desc $
     assertEqual
-    "10.0.0.0/8, 10.1.0.0/16, and 10.1.1.0/24 combine to 10.0.0.0/8"
+    desc
     expected_cidrs
     (combine_contained test_cidrs)
   where
+    desc = "10.0.0.0/8, 10.1.0.0/16, and 10.1.1.0/24 combine to 10.0.0.0/8"
     cidr1 = fromJust $ cidr_from_string "10.0.0.0/8"
     cidr2 = fromJust $ cidr_from_string "10.1.0.0/16"
     cidr3 = fromJust $ cidr_from_string "10.1.1.0/24"
@@ -375,24 +415,27 @@ test_combine_contained1 =
 
 test_combine_contained2 :: Test
 test_combine_contained2 =
-  TestCase $
+  testCase desc $
     assertEqual
-    "192.168.3.0/23 does not contain 192.168.1.0/24"
+    desc
     [cidr1, cidr2]
     (combine_contained [cidr1, cidr2])
   where
+    desc = "192.168.3.0/23 does not contain 192.168.1.0/24"
     cidr1 = fromJust $ cidr_from_string "192.168.3.0/23"
     cidr2 = fromJust $ cidr_from_string "192.168.1.0/24"
 
 
 test_combine_all1 :: Test
 test_combine_all1 =
-  TestCase $
+  testCase desc $
     assertEqual
-    "10.0.0.0/24 is adjacent to 10.0.1.0/24 and 10.0.3.0/23 contains 10.0.2.0/24"
+    desc
     expected_cidrs
     (combine_all test_cidrs)
   where
+    desc = "10.0.0.0/24 is adjacent to 10.0.1.0/24 "
+           ++ "and 10.0.3.0/23 contains 10.0.2.0/24"
     cidr1 = fromJust $ cidr_from_string "10.0.0.0/24"
     cidr2 = fromJust $ cidr_from_string "10.0.1.0/24"
     cidr3 = fromJust $ cidr_from_string "10.0.2.0/24"
@@ -404,12 +447,13 @@ test_combine_all1 =
 
 test_combine_all2 :: Test
 test_combine_all2 =
-    TestCase $
+    testCase desc $
       assertEqual
-      "127.0.0.1/32 combines with itself recursively"
+      desc
       expected_cidrs
       (combine_all test_cidrs)
     where
+      desc = "127.0.0.1/32 combines with itself recursively"
       cidr1 = fromJust $ cidr_from_string "127.0.0.1/32"
       expected_cidrs = [cidr1]
       test_cidrs = [cidr1, cidr1, cidr1, cidr1, cidr1]
@@ -417,12 +461,14 @@ test_combine_all2 =
 
 test_combine_all3 :: Test
 test_combine_all3 =
-  TestCase $
+  testCase desc $
     assertEqual
-    "10.0.0.16, 10.0.0.17, 10.0.0.18, and 10.0.0.19 get combined into 10.0.0.16/30"
+    desc
     expected_cidrs
     (combine_all test_cidrs)
   where
+    desc = "10.0.0.16, 10.0.0.17, 10.0.0.18, and "
+           ++ "10.0.0.19 get combined into 10.0.0.16/30"
     cidr1 = fromJust $ cidr_from_string "10.0.0.16/32"
     cidr2 = fromJust $ cidr_from_string "10.0.0.17/32"
     cidr3 = fromJust $ cidr_from_string "10.0.0.18/32"
@@ -431,24 +477,25 @@ test_combine_all3 =
     test_cidrs = [cidr1, cidr2, cidr3, cidr4]
 
 
-cidr_tests :: [Test]
-cidr_tests = [ test_min_host1,
-               test_max_host1,
-               test_equality1,
-               test_contains1,
-               test_contains2,
-               test_contains_proper1,
-               test_contains_proper2,
-               test_adjacent1,
-               test_adjacent2,
-               test_adjacent3,
-               test_adjacent4,
-               test_combine_contained1,
-               test_combine_contained2,
-               test_combine_all1,
-               test_combine_all2,
-               test_combine_all3
-             ]
+cidr_tests :: Test
+cidr_tests =
+  testGroup "CIDR Tests" [
+    test_min_host1,
+    test_max_host1,
+    test_equality1,
+    test_contains1,
+    test_contains2,
+    test_contains_proper1,
+    test_contains_proper2,
+    test_adjacent1,
+    test_adjacent2,
+    test_adjacent3,
+    test_adjacent4,
+    test_combine_contained1,
+    test_combine_contained2,
+    test_combine_all1,
+    test_combine_all2,
+    test_combine_all3 ]
 
 
 -- QuickCheck Tests
@@ -462,3 +509,15 @@ prop_contains_proper_intransitive :: Cidr -> Cidr -> Property
 prop_contains_proper_intransitive cidr1 cidr2 =
     (cidr1 `contains_proper` cidr2) ==>
       (not (cidr2 `contains_proper` cidr1))
+
+cidr_properties :: Test
+cidr_properties =
+  testGroup "CIDR Properties" [
+    testProperty
+      "All CIDRs contain themselves"
+      prop_all_cidrs_contain_themselves,
+
+    testProperty
+      "contains_proper is intransitive"
+      prop_contains_proper_intransitive
+  ]