search

p2-inc / keycloak-orgs

Single realm, multi-tenancy for SaaS apps
https://phasetwo.io
Other
417 stars 72 forks source link

[Refactor] Active Organization Test #225

Open MGLL opened 7 months ago

MGLL commented 7 months ago

Opening this issue for traceability.

I would like to refactor my testOrganizationSwitch() in OrganizationResourceTest to clean it (make it more readable and maintainable) and maybe move it into a separate Test class / file.

Let me know for any recommendation or best practices you would like to enforce. Else, I will try and see what is best and what could be done with the context of the Testcontainer (as it keep state of other / previous tests).

Best,

xgp commented 7 months ago

Thanks for taking the lead on this. I will look to you for a suggested best practices within the context of Testcontainers.