oidctest Issue #7.

[sozkan]

I added a try-except to prevent abrupt failures when response code is unexpected. Also added http response code checks to OP-OAuth-2nd and OP-OAuth-2nd-30s.

How to test it:

• Run docker exec -it docker_op_1 /bin/bash
• Edit ./lib/helpers/errors.js: and change status code to 403, e.g:
  super(403, 'invalid_grant');
• Stop and restart (do not rebuild) docker_op_1.
• Run https://op-test:60003/OP-OAuth-2nd and https://op-test:60003/OP-OAuth-2nd-30s

@rohe OP-OAuth-2nd-30s returns an error status while OP-OAuth-2nd returns a warning in case of an invalid_grant response. Should they be consistent? I can add another commit if we would like to make them consistent. Let me know if any additional changes are needed.

[rohe]

@sozkan There is a reason behind them not being consistent. Why the standard says SHOULD and not MUST in section 10.5 of RFC6749 was because the speed of light :-) If we follow the same reasoning as RFC6749 OP-OAuth-2nd may fail due to the speed of light but OP-OAuth-2nd-30s can not. That's why one is a warning and the other an error.