-
Notifications
You must be signed in to change notification settings - Fork 5
Description
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.
- https://github.com/solid/web-access-control-tests/blob/97eafaef6ada78c9e918ca89ab9383a90c14c421/test/surface/read-public.test.ts#L177
- https://github.com/solid/web-access-control-tests/blob/97eafaef6ada78c9e918ca89ab9383a90c14c421/test/surface/read-public.test.ts#L226
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.