search

elm-explorations / test

Write unit and fuzz tests for Elm code.
https://package.elm-lang.org/packages/elm-explorations/test/latest
BSD 3-Clause "New" or "Revised" License
236 stars 40 forks source link

Prefix internal Test type variants so node-test-runner can find them #153

Closed lydell closed 3 years ago

lydell commented 3 years ago

Explanation for why node-test-runner needs this: https://github.com/rtfeldman/node-test-runner/pull/442#issuecomment-713526271

(Original discussion leading to this PR: https://github.com/rtfeldman/node-test-runner/pull/442#discussion_r495579878)

harrysarson commented 3 years ago

I really want this change, it will future proof a hack we need to make in the test runner to detect tests without parsing the .elmi files. I think the ugliness is worth it.

harrysarson commented 3 years ago

Any objections to merging?

drathier commented 3 years ago

Yes, I want more context and documentation first! https://github.com/rtfeldman/node-test-runner/pull/442#issuecomment-713526271

drathier commented 3 years ago

Just want that link, then I'd be happy to merge this.

drathier commented 3 years ago

Cool, let's give it a go :)