Join GitHub today
GitHub is home to over 28 million developers working together to host and review code, manage projects, and build software together.
Sign upClean up List tests #154
Conversation
pushed a commit
that referenced
this pull request
Feb 2, 2015
evancz
merged commit b9604c0
into
elm:master
Feb 2, 2015
1 check passed
continuous-integration/travis-ci
The Travis CI build passed
Details
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
jonathanhefner commentedFeb 2, 2015
I cleaned up many List tests to follow the discussion in #150. For the most part, the only non-primitive functions the tests rely on are
foldl,(::), andmap. If we really wanted to be extreme we could construct a "trusted" version ofmap(andfoldr) in terms offoldland(::), but that would be considerably less efficient, and I didn't want to slow the tests down. Incidentally, these tests should run faster due to the reduction of temporary list constructions.