node-oauth / node-oauth2-server

🚀 The successor to oauthjs/oauth2-server. 🔒 Complete, compliant, maintained and well tested OAuth2 Server for node.js. Includes native async await and PKCE.

https://www.npmjs.com/package/@node-oauth/oauth2-server

MIT License

288 stars 39 forks source link

tests: add deep integration tests #224

Closed jankapunkt closed 1 year ago

jankapunkt commented 1 year ago

Summary

This PR improves the integration tests to cover integrity of the model parameters.

Furthermore, compliance tests are added, where possible.

Finally, coverage is improved, if possible.

Linked issue(s)

219

Involved parts of the project

Tests, grant types, handlers

OAuth2 standard

Compliance tests to deeply conform RFC 6749

Reproduction

checkout branch, run npm run test or npm run test:coverage

jankapunkt commented 1 year ago

@jorenvandeweyer you are right, let's merge and open a "Part II" PR for this