Fixed: Test Dependency on elm/random

[ahankinson]

Running elm-test-rs on the test suite flagged an issue with the tests missing the elm/random dependency.

Random is imported in tests/ArrayTests.elm, but not listed in the test-dependenceies section of the elm.json.

I'm not sure how the regular tests are passing without this import, as it doesn't even seem to be a transient dependency. It might be that because elm/random is a dependency of elm-test itself, it doesn't flag it?

Anyway, it seems that both testing tools work with this change.

[lydell]

I'm not sure how the regular tests are passing without this import, as it doesn't even seem to be a transient dependency. It might be that because elm/random is a dependency of elm-test itself, it doesn't flag it?

Almost. To run tests, test runners generate en Elm app behind the scenes and run it. elm-test (node-test-runner) adds elm/random to the direct dependencies of that application (because it needs it for internal stuff), which means that in tests you (unfortunately) secretly have access to elm/random even if you don’t depend on it. https://github.com/mpizenberg/elm-test-rs#differences-with-elm-test

So this PR is doing the right thing!