Skip to content

Commit

Permalink
Merge pull request #295 from rbasso/hspec-sieve
Browse files Browse the repository at this point in the history
sieve: Rewrite tests to use hspec with fail-fast.
  • Loading branch information
rbasso authored Sep 15, 2016
2 parents 3e96908 + 057bd7e commit 998ea02
Show file tree
Hide file tree
Showing 2 changed files with 65 additions and 37 deletions.
2 changes: 1 addition & 1 deletion exercises/sieve/package.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -17,4 +17,4 @@ tests:
source-dirs: test
dependencies:
- sieve
- HUnit
- hspec
100 changes: 64 additions & 36 deletions exercises/sieve/test/Tests.hs
Original file line number Diff line number Diff line change
@@ -1,42 +1,70 @@
{-# OPTIONS_GHC -fno-warn-type-defaults #-}
{-# LANGUAGE RecordWildCards #-}

import Data.Foldable (for_)
import Test.Hspec (Spec, describe, it, shouldBe)
import Test.Hspec.Runner (configFastFail, defaultConfig, hspecWith)

import Test.HUnit (Assertion, (@=?), runTestTT, Test(..), Counts(..))
import System.Exit (ExitCode(..), exitWith)
import Sieve (primesUpTo)

exitProperly :: IO Counts -> IO ()
exitProperly m = do
counts <- m
exitWith $ if failures counts /= 0 || errors counts /= 0 then ExitFailure 1 else ExitSuccess
main :: IO ()
main = hspecWith defaultConfig {configFastFail = True} specs

testCase :: String -> Assertion -> Test
testCase label assertion = TestLabel label (TestCase assertion)
specs :: Spec
specs = describe "sieve" $
describe "primesUpTo" $ for_ cases test
where

main :: IO ()
main = exitProperly $ runTestTT $ TestList
[ TestList sieveTests ]

sieveTests :: [Test]
sieveTests =
[ testCase "small primes" $ do
[2, 3, 5, 7] @=? primesUpTo 10
[2, 3, 5, 7, 11] @=? primesUpTo 11
, testCase "primes up to 1000" $
[ 2, 3, 5, 7, 11, 13, 17, 19, 23, 29, 31, 37, 41, 43, 47, 53, 59, 61, 67, 71
, 73, 79, 83, 89, 97, 101, 103, 107, 109, 113, 127, 131, 137, 139, 149, 151
, 157, 163, 167, 173, 179, 181, 191, 193, 197, 199, 211, 223, 227, 229, 233
, 239, 241, 251, 257, 263, 269, 271, 277, 281, 283, 293, 307, 311, 313, 317
, 331, 337, 347, 349, 353, 359, 367, 373, 379, 383, 389, 397, 401, 409, 419
, 421, 431, 433, 439, 443, 449, 457, 461, 463, 467, 479, 487, 491, 499, 503
, 509, 521, 523, 541, 547, 557, 563, 569, 571, 577, 587, 593, 599, 601, 607
, 613, 617, 619, 631, 641, 643, 647, 653, 659, 661, 673, 677, 683, 691, 701
, 709, 719, 727, 733, 739, 743, 751, 757, 761, 769, 773, 787, 797, 809, 811
, 821, 823, 827, 829, 839, 853, 857, 859, 863, 877, 881, 883, 887, 907, 911
, 919, 929, 937, 941, 947, 953, 967, 971, 977, 983, 991, 997
] @=? primesUpTo 1000
, testCase "first thousand primes" $
1000 @=? length (primesUpTo 7919)
, testCase "edge cases" $ do
[] @=? primesUpTo 1
[2] @=? primesUpTo 2
]
test Case{..} = it description assertion
where
assertion = primesUpTo (fromIntegral limit)
`shouldBe`
fromIntegral <$> expected

-- Test cases adapted from `exercism/x-common` on 2016-09-12.

data Case = Case { description :: String
, limit :: Integer
, expected :: [Integer]
}

cases :: [Case]
cases = [ Case { description = "no primes under two"
, limit = 1
, expected = []
}
, Case { description = "find first prime"
, limit = 2
, expected = [2]
}
, Case { description = "find primes up to 10"
, limit = 10
, expected = [2, 3, 5, 7]
}
, Case { description = "limit is prime"
, limit = 13
, expected = [2, 3, 5, 7, 11, 13]
}
, Case { description = "find primes up to 1000"
, limit = 1000
, expected = [ 2, 3, 5, 7, 11, 13, 17, 19, 23, 29, 31, 37
, 41, 43, 47, 53, 59, 61, 67, 71, 73, 79, 83
, 89, 97, 101, 103, 107, 109, 113, 127, 131
, 137, 139, 149, 151, 157, 163, 167, 173, 179
, 181, 191, 193, 197, 199, 211, 223, 227, 229
, 233, 239, 241, 251, 257, 263, 269, 271, 277
, 281, 283, 293, 307, 311, 313, 317, 331, 337
, 347, 349, 353, 359, 367, 373, 379, 383, 389
, 397, 401, 409, 419, 421, 431, 433, 439, 443
, 449, 457, 461, 463, 467, 479, 487, 491, 499
, 503, 509, 521, 523, 541, 547, 557, 563, 569
, 571, 577, 587, 593, 599, 601, 607, 613, 617
, 619, 631, 641, 643, 647, 653, 659, 661, 673
, 677, 683, 691, 701, 709, 719, 727, 733, 739
, 743, 751, 757, 761, 769, 773, 787, 797, 809
, 811, 821, 823, 827, 829, 839, 853, 857, 859
, 863, 877, 881, 883, 887, 907, 911, 919, 929
, 937, 941, 947, 953, 967, 971, 977, 983, 991
, 997 ]
}
]

0 comments on commit 998ea02

Please sign in to comment.