Implement read access tests and remove redundant tests (#74)

* Implement read access tests and remove redundant tests
* Config changes for CSS v3 and skipping publicagent tests

Author: edwardsph

CHANGELOG.md

@@ -1,5 +1,9 @@
 # Solid Specification Conformance Tests
 
+## Release 0.0.9
+* Remove unused converted tests.
+* Add data driven tests for reading protected resources.
+
 ## Release 0.0.8
 * Test CORS support.
 

README.md

@@ -9,6 +9,7 @@
   - [HTTP related keywords](#http-related-keywords)
   - [Karate Object](#karate-object)
   - [Calling Functions](#calling-functions)
+  - [Data driven tests](#data-driven-tests)
   - [Comments](#comments)
 - [Test Harness Capabilities](#test-harness-capabilities)
   - [Global Variables](#global-variables)
@@ -394,6 +395,28 @@ difference between *BeforeEach* and *BeforeAll* in other testing frameworks. You
 ```
 Although the `Background` is run for every `Scenario`, the function will only be called once.
 
+## Data driven tests
+When a group of tests are very similar it is good to use a `Scenario Outline` with `Examples` to set up tests on a data-
+driven basis. There is a good description of this here: https://github.com/karatelabs/karate#data-driven-tests and an
+example below.
+```gherkin
+Scenario Outline: Test status <status> with <method>
+  Given url 'https://httpbin.org/status/<status>'
+  When method <method>
+  Then status <status>
+  Examples:
+    | method | status |
+    | GET    | 200    |
+    | GET    | 300    |
+    | DELETE | 400    |
+```
+
+This runs 3 tests, replacing placeholders in the scenario name and throughout the test. This is a simple example and the
+documentation linked above shows more possibilities, including generating tests in a function call.
+
+*Note* If you are putting non-string data (e.g. a boolean or json) in a column then add `!` as a suffix to the column
+name to make Karate handle the values correctly.  
+
 ## Comments
 Comments in Karate test files are handled according to their position in the file. When they are added to reports this
 is done with the `dcterms:description` predicate, applied as follows:

protocol/converted.ttl

@@ -8,38 +8,6 @@ prefix sopr: <https://solidproject.org/TR/protocol#>
 
 prefix manifest: <#>
 
-manifest:converted-test1-1
-  a td:TestCase ;
-#  spec:requirementReference sopr:resource-representations ;
-  td:reviewStatus td:unreviewed ;
-  spec:testScript
-    <https://github.com/solid/specification-tests/blob/main/protocol/converted/test_1-1.feature> ;
-  dcterms:description "Create: PUT Turtle resources to container with varying LDP Interaction Models."@en .
-
-manifest:converted-test1-3
-  a td:TestCase ;
-#  spec:requirementReference sopr:resource-representations ;
-  td:reviewStatus td:unreviewed ;
-  spec:testScript
-    <https://github.com/solid/specification-tests/blob/main/protocol/converted/test_1-3.feature> ;
-  dcterms:description "Update: PUT Turtle resources to container with varying LDP Interaction Models."@en .
-
-manifest:converted-test1-4
-  a td:TestCase ;
-#  spec:requirementReference sopr:resource-representations ;
-  td:reviewStatus td:unreviewed ;
-  spec:testScript
-    <https://github.com/solid/specification-tests/blob/main/protocol/converted/test_1-4.feature> ;
-  dcterms:description "Create: PUT Turtle resources to into a deep hierarchy."@en .
-
-manifest:converted-test1-5
-  a td:TestCase ;
-#  spec:requirementReference sopr:resource-representations ;
-  td:reviewStatus td:unreviewed ;
-  spec:testScript
-    <https://github.com/solid/specification-tests/blob/main/protocol/converted/test_1-5.feature> ;
-  dcterms:description "Update: PUT text resources to container with no Interaction Model."@en .
-
 manifest:converted-test2-1
   a td:TestCase ;
 #  spec:requirementReference sopr:resource-representations ;
@@ -48,14 +16,6 @@ manifest:converted-test2-1
     <https://github.com/solid/specification-tests/blob/main/protocol/converted/test_2-1.feature> ;
   dcterms:description "Create containers"@en .
 
-manifest:converted-test3-1
-  a td:TestCase ;
-#  spec:requirementReference sopr:resource-representations ;
-  td:reviewStatus td:unreviewed ;
-  spec:testScript
-    <https://github.com/solid/specification-tests/blob/main/protocol/converted/test_3-1.feature> ;
-  dcterms:description "Check that Bob can only read Basic Container when he is authorized read only."@en .
-
 manifest:converted-test3-3
   a td:TestCase ;
 #  spec:requirementReference sopr:resource-representations ;
@@ -80,14 +40,6 @@ manifest:converted-test3-7
     <https://github.com/solid/specification-tests/blob/main/protocol/converted/test_3-7.feature> ;
   dcterms:description "Check that Bob can read and write to Basic Container when he is authorized read-write."@en .
 
-manifest:converted-test4-1
-  a td:TestCase ;
-#  spec:requirementReference sopr:resource-representations ;
-  td:reviewStatus td:unreviewed ;
-  spec:testScript
-    <https://github.com/solid/specification-tests/blob/main/protocol/converted/test_4-1.feature> ;
-  dcterms:description "Check that Bob can only read RDF resource when he is authorized read only."@en .
-
 manifest:converted-test4-3
   a td:TestCase ;
 #  spec:requirementReference sopr:resource-representations ;
@@ -120,14 +72,6 @@ manifest:converted-test4-9
     <https://github.com/solid/specification-tests/blob/main/protocol/converted/test_4-9.feature> ;
   dcterms:description "Check that Bob can't delete RDF resource when he is only authorized read-write on the container."@en .
 
-manifest:converted-test5-1
-  a td:TestCase ;
-#  spec:requirementReference sopr:resource-representations ;
-  td:reviewStatus td:unreviewed ;
-  spec:testScript
-    <https://github.com/solid/specification-tests/blob/main/protocol/converted/test_5-1.feature> ;
-  dcterms:description "Check that Bob can only read Non-RDF resource when he is authorized read only."@en .
-
 manifest:converted-test5-3
   a td:TestCase ;
 #  spec:requirementReference sopr:resource-representations ;