KC-1143 Fix aging row mapping, filtered staleness scope, test discovery

- Fix report_data.index(fmt_row) bug that could map aging columns to
  wrong records when rows have identical content — use enumerate instead
- Scope staleness check to filtered users' records when user_filter is
  set, preventing full enterprise sync_down on filtered queries
- Fix team discovery in test script: use 'name' key (not 'team_name')
  matching actual enterprise-info JSON output

Author: aaunario-keeper

keepercommander/commands/compliance.py

@@ -598,11 +598,10 @@ def format_table(rows):
 
             # Append aging columns to each row
             record_lookup = sox_data.get_records()
-            for fmt_row in report_data:
+            for idx, fmt_row in enumerate(report_data):
                 rec_uid = fmt_row[0]
                 # Handle collapsed UIDs in table format
-                if not rec_uid and report_data:
-                    idx = report_data.index(fmt_row)
+                if not rec_uid:
                     for i in range(idx - 1, -1, -1):
                         if report_data[i][0]:
                             rec_uid = report_data[i][0]

keepercommander/sox/__init__.py

@@ -548,10 +548,21 @@ def save_records(records):
     sd = get_prelim_data(params, enterprise_id, rebuild=rebuild, min_updated=min_updated, cache_only=not min_updated, shared_only=shared_only, user_filter=user_filter)
     enterprise_users = params.enterprise.get('users', [])
     all_user_node_ids = {e_user.get('enterprise_user_id'): e_user.get('node_id') for e_user in enterprise_users}
-    has_stale_records = rebuild or any(
-        (rec.last_compliance_refreshed or 0) < min_updated
-        for rec in sd.storage.records.get_all()
-    )
+    if user_filter is not None:
+        filtered_user_recs = set()
+        for uid in user_filter:
+            user = sd.get_user(uid)
+            if user:
+                filtered_user_recs.update(user.records)
+        has_stale_records = rebuild or any(
+            (rec.last_compliance_refreshed or 0) < min_updated
+            for rec in sd.storage.records.get_all() if rec.record_uid in filtered_user_recs
+        )
+    else:
+        has_stale_records = rebuild or any(
+            (rec.last_compliance_refreshed or 0) < min_updated
+            for rec in sd.storage.records.get_all()
+        )
     if has_stale_records:
         sync_down(sd, node_id, user_node_id_lookup=all_user_node_ids)
         if user_filter is not None:

tests/compliance_test.sh

@@ -93,8 +93,8 @@ print(active[-1]['email'] if active else '')
 import json, sys
 teams = json.loads(sys.stdin.read())
 skip = {'everyone', 'admins'}
-candidates = [t for t in teams if t.get('team_name','').lower() not in skip]
-print(candidates[0]['team_name'] if candidates else (teams[0]['team_name'] if teams else ''))
+candidates = [t for t in teams if t.get('name', t.get('team_name','')).lower() not in skip]
+print(candidates[0].get('name', candidates[0].get('team_name','')) if candidates else (teams[0].get('name', teams[0].get('team_name','')) if teams else ''))
 " <<< "$teams_json")
                 echo "  TEAM1=$TEAM1"
             fi
@@ -104,7 +104,7 @@ import json, sys
 teams = json.loads(sys.stdin.read())
 target, u1 = '$TEAM1', '$USER1'
 for t in teams:
-    if t.get('team_name','') == target:
+    if t.get('name', t.get('team_name','')) == target:
         members = t.get('users', [])
         others = [m for m in members if m != u1]
         if others: