Dredd skips transaction if there's no value for URI template's parameter

If “Ambigous URI parameter” happens (there’s URI template with parameter, but the parameter isn’t described anywhere), Dredd warns about it and just seems to skip the transaction in v1.0.x version.

Note: In v1.1.0-pre.x versions this behavior is broken (https://github.com/apiaryio/dredd/issues/469, https://github.com/apiaryio/dredd/issues/478), but this issue is about the fact that I think even the original approach isn’t completely correct.

Should Dredd skip the transaction or fail? I think it would be better if:

• For URI template parameters such as /people/{id}/address the “Ambigous URI parameter” annotation should be an error and Dredd should end with that error.
• For URI template parameters such as /people{?sort} the “Ambigous URI parameter” annotation should be a warning and Dredd should test with URI created without the parameter.

@netmilk @smizell

[jackbentley]

This has already caught me out. I had an endpoint that was returning an incorrect status code, and dredd had silently skipped the test, as the route had URI parameters specified for another method.

[honzajavorek]

@lteacher Hello! Thanks for getting here. This is not resolved and I agree with you this can lead into flawed trust into how Dredd is able to test your contract, which is a fundamentally the reason why Dredd exists in the first place. This needs a bit more analysis though on how we want to approach and execute this. I’ll get some opinion from @netmilk and get back here.