Refine access-rules output to show separate host, domain, and path columns

Changed table format from:
  route                    selector  scope  source
  backend.apps.identity    ...       app    frontend-app

To:
  host     domain          path     selector  scope  source
  backend  apps.identity            ...       app    frontend-app
  api      apps.identity   /metrics ...       space  monitoring

This provides better clarity by separating the route components into
individual columns, making it easier to scan and filter visually.

Author: rkoster

command/v7/access_rules_command.go

@@ -1,9 +1,6 @@
 package v7
 
 import (
-	"fmt"
-
-	"code.cloudfoundry.org/cli/v9/resources"
 	"code.cloudfoundry.org/cli/v9/util/ui"
 )
 
@@ -64,7 +61,9 @@ func (cmd AccessRulesCommand) Execute(args []string) error {
 	// Build table data
 	table := [][]string{
 		{
-			cmd.UI.TranslateText("route"),
+			cmd.UI.TranslateText("host"),
+			cmd.UI.TranslateText("domain"),
+			cmd.UI.TranslateText("path"),
 			cmd.UI.TranslateText("selector"),
 			cmd.UI.TranslateText("scope"),
 			cmd.UI.TranslateText("source"),
@@ -73,7 +72,9 @@ func (cmd AccessRulesCommand) Execute(args []string) error {
 
 	for _, ruleWithRoute := range rulesWithRoutes {
 		table = append(table, []string{
-			formatRoute(ruleWithRoute.Route, ruleWithRoute.DomainName),
+			ruleWithRoute.Route.Host,
+			ruleWithRoute.DomainName,
+			ruleWithRoute.Route.Path,
 			ruleWithRoute.Selector,
 			ruleWithRoute.ScopeType,
 			ruleWithRoute.SourceName,
@@ -86,16 +87,3 @@ func (cmd AccessRulesCommand) Execute(args []string) error {
 	return nil
 }
 
-// formatRoute formats a route as hostname.domain/path
-func formatRoute(route resources.Route, domainName string) string {
-	var formatted string
-	if route.Host != "" {
-		formatted = fmt.Sprintf("%s.%s", route.Host, domainName)
-	} else {
-		formatted = domainName
-	}
-	if route.Path != "" {
-		formatted += route.Path
-	}
-	return formatted
-}

command/v7/access_rules_command_test.go

@@ -146,9 +146,9 @@ var _ = Describe("access-rules Command", func() {
 			Expect(executeErr).ToNot(HaveOccurred())
 
 			Expect(testUI.Out).To(Say(`Getting access rules in org some-org / space some-space as steve\.\.\.`))
-			Expect(testUI.Out).To(Say(`route\s+selector\s+scope\s+source`))
-			Expect(testUI.Out).To(Say(`myapp\.example\.com/api\s+cf:app:app-guid-1\s+app\s+my-app`))
-			Expect(testUI.Out).To(Say(`webapp\.test\.com\s+cf:any\s+any`))
+			Expect(testUI.Out).To(Say(`host\s+domain\s+path\s+selector\s+scope\s+source`))
+			Expect(testUI.Out).To(Say(`myapp\s+example\.com\s+/api\s+cf:app:app-guid-1\s+app\s+my-app`))
+			Expect(testUI.Out).To(Say(`webapp\s+test\.com\s+cf:any\s+any`))
 
 			Expect(testUI.Err).To(Say("warning-1"))
 
@@ -298,8 +298,8 @@ var _ = Describe("access-rules Command", func() {
 			Expect(executeErr).ToNot(HaveOccurred())
 
 			Expect(testUI.Out).To(Say(`Getting access rules in org some-org / space some-space as steve\.\.\.`))
-			Expect(testUI.Out).To(Say(`route\s+selector\s+scope\s+source`))
-			Expect(testUI.Out).To(Say(`myapp\.example\.com/api\s+cf:app:app-guid-1\s+app\s+my-app`))
+			Expect(testUI.Out).To(Say(`host\s+domain\s+path\s+selector\s+scope\s+source`))
+			Expect(testUI.Out).To(Say(`myapp\s+example\.com\s+/api\s+cf:app:app-guid-1\s+app\s+my-app`))
 		})
 	})
 
@@ -340,11 +340,11 @@ var _ = Describe("access-rules Command", func() {
 		It("formats routes correctly", func() {
 			Expect(executeErr).ToNot(HaveOccurred())
 
-			// No host, with path: "example.com/api"
-			Expect(testUI.Out).To(Say(`example\.com/api`))
+			// No host, with path: empty host, example.com, /api
+			Expect(testUI.Out).To(Say(`\s+example\.com\s+/api`))
 
-			// With host, no path: "myapp.test.com"
-			Expect(testUI.Out).To(Say(`myapp\.test\.com`))
+			// With host, no path: myapp, test.com, empty path
+			Expect(testUI.Out).To(Say(`myapp\s+test\.com\s+`))
 		})
 	})
 })