Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Testcase 12379 no_support has wrong expectation #265

Closed
Thopap opened this issue Mar 29, 2017 · 1 comment
Closed

Testcase 12379 no_support has wrong expectation #265

Thopap opened this issue Mar 29, 2017 · 1 comment
Assignees
Milestone

Comments

@Thopap
Copy link
Contributor

Thopap commented Mar 29, 2017

XDS.b conformance testcase 12379 for registry that has "no_support" has a wrong expectation. The current assertion error is:

Status: Fail
Section: 12379/no_support Step: no_support
Endpoint: https://localhost:8443/pxs/webservices/xdsb/documentRegistry/iti42RegisterDocuments
Goals:
XDSExtraMetadataNotSaved : Request contain non-IHE metadata, which were ignored.
Status is urn:oasis:names:tc:ebxml-regrep:ResponseStatusType:Success, expected status is urn:oasis:names:tc:ebxml-regrep:ResponseStatusType:Warning (stepId=no_support)
------------------------------------------------------------------------------------------------
Finished

Registry response was:
<ns3:RegistryResponse status="urn:oasis:names:tc:ebxml-regrep:ResponseStatusType:Success" xmlns:ns3="urn:oasis:names:tc:ebxml-regrep:xsd:rs:3.0"> <ns3:RegistryErrorList> <ns3:RegistryError codeContext="Request contain non-IHE metadata, which were ignored." errorCode="XDSExtraMetadataNotSaved" severity="urn:oasis:names:tc:ebxml-regrep:ErrorSeverityType:Warning"/> </ns3:RegistryErrorList> </ns3:RegistryResponse>

According to ITI TF3 "Table 4.2.4.2-1: [ITI-41] Provide and Register Document Set-b Responses" there is no ResponseStatusType:Warning defined. Instead "ResponseStatusType:Success" with the embedded warning "XDSExtraMetadataNotSaved" is correct.

@Thopap
Copy link
Contributor Author

Thopap commented Apr 21, 2022

From my understanding, all testcases defining <ExpectedStatus>Warning</ExpectedStatus> (e.g. "https://github.com/usnistgov/iheos-toolkit2/blob/master/services/src/test/resources/war/toolkitx/testkit/tests/12379/no_support/testplan.xml#L8") must be replaced with <ExpectedStatus>Success</ExpectedStatus>

@skbhaskarla skbhaskarla self-assigned this Apr 21, 2022
@skbhaskarla skbhaskarla added this to the v7.8.0 milestone Apr 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants