`test:` or `tests:` in spago.yaml?

[ad-si]

The documentation says test: and spago init also creates a YAML file with test:.
However, spago test only seems to work with tests:.

[f-f]

We don't use the tests key - are you able to put together a small repro to demonstrate the problem with the latest Spago?

The code that parses the configuration only looks for test:

spago/core/src/Config.purs

Line 91 in b450892

$ CA.recordPropOptional (Proxy :: _ "test") testConfigCodec

[ad-si]

Haha, I know what's going on:
When I have a tests: field, it uses the default settings (since it ignores the tests: field).
When I rename it to test: I get an error because my test.main field was incorrect (Test.Main.purs instead of Test.Main).

Maybe there should be a warning if you specify undefined fields?

[JordanMartinez]

Maybe there should be a warning if you specify undefined fields?

That'd probably be useful for clarifying what schema we expect, especially as the config file changes during this alpha work.

[f-f]

Good points - I opened #1165 to track this