elm-explorations/test

Incorporate elm-html-test

rtfeldman opened this issue · 6 comments

rtfeldman commented

For several reasons...

  • Many people seem unaware http://package.elm-lang.org/packages/eeue56/elm-html-test/latest is available (which is a shame - it rocks!)
  • There's no more native whitelist, so it can't be re-published in 0.19 as-is
  • This project is going to have more dependencies on other packages when it gets into testing Cmd values anyway

...the plan is to add elm-html-test here.

This will entail the usual 0.19 changes, plus Native -> Kernel, and updating the README and other docs.

rtfeldman commented

Note: @eeue56 has already started working on this.

  • 👍5
gyzerok commented

There is also longstanding issue/pr from me there eeue56/elm-html-test#65. Probably there was not enough time to look into it due to all the work around 0.19. Would appreciate if it could be batched with the work mentioned here.

  • 👍1
avh4 commented

Taking a look at this now (though don't expect to finish today)

  • 👍2
  • ❤️2
avh4 commented

WIP: #41

A notable problem is the removal of Debug.crash, which is going is looking like it will needs a bit of non-trivial refactoring to get around.

andys8 commented

#41 is merged, 1.2.0 released. I guess this issue can be closed 👍

Maybe documentation has to be adapted. README.md is still mentioning and linking to the elm-html-test package.

Screenshot

image

avh4 commented

Thanks! I updated the README in 13bc9ba, and also realized that we haven't merged in elm-html-test's old README, so I created an issue for that: #51