search

solid-contrib / web-access-control-tests

Tests if a Solid server implements web access control correctly
MIT License
2 stars 5 forks source link

Test titles mismatching the purpose of the test #46

Open edwardsph opened 2 years ago

edwardsph commented 2 years ago

The ongoing conversation about test suites reminded me that I had very recently spotted problems with some tests and hadn't raised an issue. I should have done it when I found them but was busy trying to get a large change done and forgot. The timing of this is unfortunate as I do not wish to inflame the current debate but it serves the community best to fix things like this where we can.

The descriptions of these 2 tests are duplicates of other tests in this file and don't match the tests themselves. It looks like a copy/paste error. This probably explains why T153 and T155 are missing from the report at: https://github.com/solid/test-suite/blob/main/test-suite-report.md

It might be worth adding something to the report generation to detect missing/duplicate tests.

michielbdejong commented 2 years ago

@edwardsph hanks for reporting! @mrvahedi68 can you have a look?