Please address the comments from this code review:

## Overall Comments
- Switching from a slice to a map for `testCases` makes subtest execution order non-deterministic; consider keeping a slice (or storing the keys in a separate ordered slice) if stable ordering is useful for debugging and CI output.
- The map key and `desc` field in the test cases now diverge in at least one case (the Giedi=Prime test), which may cause confusion when reading failures; consider either dropping `desc` and relying on the map key or keeping them identical.
- In `conversion_test.go`, the test name string "empty ruleset result in an empty selector" has a subject-verb agreement typo; consider changing it back to "results" for clarity.

## Individual Comments

### Comment 1
<location path="pkg/sac/effectiveaccessscope/effective_access_scope_test.go" line_range="93-97" />
<code_context>

-	testCases := []testCase{
-		{
+	testCaseMap := map[string]testCase{
+		"no access scope includes nothing": {
 			desc:      "no access scope includes nothing",
</code_context>
<issue_to_address>
**suggestion (testing):** Using a map for table-driven tests makes subtest execution order nondeterministic

Using a map here makes subtest execution order arbitrary, which can introduce flaky tests or harder-to-reproduce failures if anything relies on a stable order (fixtures, logs, benchmarks). Prefer keeping a slice for deterministic iteration, or derive and sort the map keys before ranging so subtest order is stable.

Suggested implementation:

```golang
	testCases := []testCase{
		{
			desc:      "no access scope includes nothing",
			scopeDesc: `nil => { }`,
			scopeStr:  "",
			detail:   v1.ComputeEffectiveAccessScopeRequest_HIGH,
			hasError: false,
		},
		{
			desc:      "empty access scope includes nothing",
			scopeDesc: `∅ => { }`,
			scopeStr:  "",

```

1. Update any loop that iterates over `testCaseMap` to instead iterate over the ordered slice `testCases`, for example:
   - Replace `for name, tc := range testCaseMap { t.Run(name, func(t *testing.T) { ... }) }`
   - With `for _, tc := range testCases { t.Run(tc.desc, func(t *testing.T) { ... }) }`
2. Remove all remaining map literal keys (e.g., `"some case name": {`) in the test case composite literal and convert them to plain elements `{ ... }` in the `[]testCase` slice.
3. Ensure there is no remaining reference to `testCaseMap` in the file; all should now refer to `testCases`.
</issue_to_address>