Crash on Assertion: 6.5.20

[samerhaj]

The tool run up till 6.5.20 then crashed. It looks like it found an entry in the $metadata for an OEM Schema (OemAuthorization_v1.xml) and decided to go look for it on dmtf.org: http://redfish.dmtf.org/schemas/v1/OemAuthorization_v1.xml, and crashed with HTTP Error 404

---> Assertion: 6.5.20
.....
....
....
The uri is /redfish/v1/metadata/OemAuthorization_v1.xml
OemAuthorization_v1.xml
http://redfish.dmtf.org/schemas/v1/OemAuthorization_v1.xml
Traceback (most recent call last):
File "rf_client.py", line 779, in
main()
File "rf_client.py", line 773, in main
rfs_test.run(sut)
File "C:\DevTools\Redfish-Service-Conformance-Check\rfs_test_init_.py", line 34, in run
TEST_protocol_details.run(sut, log)
File "C:\DevTools\Redfish-Service-Conformance-Check\rfs_test\TEST_protocol_details.py", line 5019, in
run
assertion_status = Assertion_6_5_20(self, log)
File "C:\DevTools\Redfish-Service-Conformance-Check\rfs_test\TEST_protocol_details.py", line 3751, in
Assertion_6_5_20
f = urllib.request.urlopen(uris)
File "C:\Users\smahmoud\AppData\Local\Programs\Python\Python35\lib\urllib\request.py", line 163, in urlopen
return opener.open(url, data, timeout)
File "C:\Users\smahmoud\AppData\Local\Programs\Python\Python35\lib\urllib\request.py", line 472, in open
response = meth(req, response)
File "C:\Users\smahmoud\AppData\Local\Programs\Python\Python35\lib\urllib\request.py", line 582, in http_response
'http', request, response, code, msg, hdrs)
File "C:\Users\smahmoud\AppData\Local\Programs\Python\Python35\lib\urllib\request.py", line 510, in error
return self._call_chain(*args)
File "C:\Users\smahmoud\AppData\Local\Programs\Python\Python35\lib\urllib\request.py", line 444, in _call_chain
result = func(*args)
File "C:\Users\smahmoud\AppData\Local\Programs\Python\Python35\lib\urllib\request.py", line 590, in http_error_default

raise HTTPError(req.full_url, code, msg, hdrs, fp)

urllib.error.HTTPError: HTTP Error 404: Not Found

[billdodd]

Assertion_6_5_20 is changed by 3 other pending Pull Requests (#22, #27, #28). And the issue reported here may well be fixed by the change in PR #28.

I'd still like to add proper try/catch code for the urlopen() call. But I'll wait until after the current batch of PRs get merged on Thursday in order to avoid the merge conflicts.

[billdodd]

This should be fixed by PR #37 (along with previous PRs #22, #27 and #28).