Open jensscheerlinck opened 8 years ago
First part duplicates #2. Agree the test case should be renamed. A.3.1 (requirement 4) is about whether the service implements the "additional search attributes". A.2.3 (requirement 5) is about whether they are advertised in the GetCapabilities response. I don't think that either is a pre-requisite of the other, although it could be argued (from CSW) that any client (including the test suite) shouldn't test something which isn't advertised - so A.2.3 would be the pre-requisite! The second clause of Requirement 22 specifies the XPath, i.e. how to advertise them. This indicates that A.2.1 includes this test case. Proposed resolutions: Either, split A.2.1 so that it only covers the first clause of Requirement 22 (the ISO queryables), and keep this A.2.3 for the additional ones. Or, delete this test case and mark A.2.1 as covering Requirement 5 as well.
The XPath (see Requirement 22) is ows:OperationsMetadata/ows:Operation[name="GetRecords"]/ows:Constraint[name="AdditionalQueryables"]/ows:Value - this should return a set of nodes whose contents match the Name column of TG Table 5 & 6.
If read to mean that the record returned is a valid ISO 19139 record (allowing for ISO 19119 extensions), then A.3.1 is a pre-requisite of several of the other tests. But not particularly this one, as this is about how one can query, whereas A.3.1 is about how the response is structured.
PLEASE IGNORE THIS COMMENT - I mixed up A.1.3 and A.3.1 last night!
On reflection, preference is to amend test case A.2.1 so that it only covers what its title says it does, the ISO parameters, leaving this test case to cover the additional INSPIRE ones - the first option in my fiirst comment yesterday. See #34
Proper XPath (attribute syntax) at #12
This issue has been extracted from the issue list on:https://ies-svn.jrc.ec.europa.eu/issues/2685
Comment
Typo in the URL XPATH expression is missing
A.3.1 as prerequisite? Additional search attributes are supported by the Discovery Service.
Proposed Change
Addiotional -> additional Add XPATH reference