search

onc-healthit / onc-certification-g10-test-kit

ONC Certification (g)(10) Standardized API Tests
Apache License 2.0
34 stars 12 forks source link

g(10) 6.2 Token Revocation Test improvement suggestion #134

Open vganshin opened 2 years ago

vganshin commented 2 years ago

I run all Inferno tests from ONC Certification (g)(10) Standardized API test suite. And run into issue that I have to revoke token concurrently before 6.2 Token Revocation Test has started. I would like Inferno shows "User Action Required" modal window asking to revoke specific access and refresh tokens, . What do you think of it?

Screenshot 2022-05-20 at 15 52 24

and we could remove checkbox with message "Prior to executing test, Health IT developer demonstrated revoking tokens provided during patient standalone launch" then.

Screenshot 2022-05-20 at 15 39 53
arscan commented 2 years ago

Hi @vganshin -- I think this is a great idea. This is something we had in mind but didn't want to change too much in the move to v2.0.0. It probably is a good time to see if we can move in this direction now though, especially since the current solution doesn't work for users that like to run all tests.

I think we'll just need to figure out if the 'Notes' field is important enough to keep. It would be simpler to not have that field.