-
Notifications
You must be signed in to change notification settings - Fork 52
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
DXCDT-600: Show active tenant in auth0 tenants list
#910
Conversation
@@ -86,6 +86,7 @@ func buildRootCmd(cli *cli) *cobra.Command { | |||
PersistentPreRunE: func(cmd *cobra.Command, args []string) error { | |||
ansi.Initialize(cli.noColor) | |||
prepareInteractivity(cmd) | |||
cli.configureRenderer() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This method mostly serves to attach the active tenant to the renderer struct so that we can make the comparison below. Otherwise, the tenant commands' renderers do not have knowledge of the currently active tenant.
Given that there can only be a single active tenant at a time (so only a single |
…h0/auth0-cli into DXCDT-600-show-active-tenant
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #910 +/- ##
==========================================
- Coverage 72.59% 72.58% -0.02%
==========================================
Files 93 93
Lines 12393 12399 +6
==========================================
+ Hits 8997 9000 +3
- Misses 2874 2876 +2
- Partials 522 523 +1
☔ View full report in Codecov by Sentry. |
…h0/auth0-cli into DXCDT-600-show-active-tenant
🔧 Changes
Adds a tabular structure to the
auth0 tenants list
command including an indicator for the currently active tenant.🔬 Testing
Added slightly stronger assertions to the existing test case.
📝 Checklist