discovering module for tests is broken

[stoeffel]

rtfeldman/node-test-runner#258

I think this is because we don't find a module in any of the source directories. We need to check tests (not defined in source-modules) as well.

[avh4]

The problem in rtfeldman/node-test-runner#258 is that there is an elmi file for an elm source file that no longer exists (the user deleted it). I know there used to be code somewhere that would check that the elm file exists, and if it doesn't, ignore the elmi file. I don't remember if that code was in elmi-to-json, or in node-test-runner? But maybe something's wrong there? I'm pretty sure it is correctly finding the elm file when the elm file does exist. (the tests folder is in the source-directories of the elm.json in elm-stuff/generated/elm-explorations/test, which is where elmi-to-json is not run from).

[stoeffel]

oh got it. I don't remember that there was code for that in elmi-to-json before.

[avh4]

I guess I was thinking of this code, which does that check, but throws an error if the source files doesn't exist. https://github.com/stoeffel/elmi-to-json/blob/master/src/Elmi.hs#L29-L33

Could we make it so that elmi-to-json with no arguments will ignore those extra elmi files?

[stoeffel]

Yes that sounds good to me.