search

opengeospatial / ets-wmts10

WMTS Test suite
Other
1 stars 3 forks source link

Bug fixes #92

Closed cmorriscmps closed 1 month ago

cmorriscmps commented 1 month ago

Fixes for issue #90 and issue #91.

Closes #90 Closes #91

cmorriscmps commented 1 month ago

The reason for using a variety of different cases for parameter names is that servers are required to ignore case. If all the parameters are lowercase, a server that only recognizes lowercase would pass when it should fail. But there is nothing different about the acceptversions parameter. That applies to all parameters.

dstenger commented 1 month ago

Sure, I am aware of that. Still, the ignoring of cases should be tested in an individual test and not mixed into other tests. Otherwise, debugging might be quite hard as the description of the test does not mention that upper/lower case of parameters is also tested.

cmorriscmps commented 1 month ago

I have no objection if you want to change them all to lowercase.