Fix: test_delete_ws_groups_should_delete_renamed_and_reflected_groups_only and test_running_real_remove_backup_groups_job (#1476)

## Changes
The opposite of the `retried` logic should be applied: pass when
`NotFound` otherwise fail after retrying a couple times.

### Linked issues
Resolves #1473 
Resolves #1472

### Functionality 

- [ ] added relevant user documentation
- [ ] added new CLI command
- [ ] modified existing command: `databricks labs ucx ...`
- [ ] added a new workflow
- [ ] modified existing workflow: `...`
- [ ] added a new table
- [ ] modified existing table: `...`

### Tests
<!-- How is this tested? Please see the checklist below and also
describe any other relevant tests -->

- [ ] manually tested
- [ ] added unit tests
- [ ] added integration tests
- [ ] verified on staging environment (screenshot attached)

Author: JCZuurmond

tests/integration/test_installation.py

@@ -170,11 +170,14 @@ def test_running_real_remove_backup_groups_job(ws, installation_ctx):
 
     installation_ctx.deployed_workflows.run_workflow("remove-workspace-local-backup-groups")
 
-    @retried(on=[NotFound], timeout=timedelta(seconds=120))
-    def wait():
-        ws.groups.get(ws_group_a.id)
+    # The API needs a moment to delete a group, i.e. until the group is not found anymore
+    @retried(on=[KeyError], timeout=timedelta(minutes=2))
+    def get_group(group_id: str):
+        ws.groups.get(group_id)
+        raise KeyError(f"Group is not deleted: {group_id}")
 
-    wait()
+    with pytest.raises(NotFound):
+        get_group(ws_group_a.id)
 
 
 @retried(on=[NotFound, InvalidParameterValue], timeout=timedelta(minutes=3))

tests/integration/workspace_access/test_groups.py

@@ -99,11 +99,14 @@ def test_delete_ws_groups_should_delete_renamed_and_reflected_groups_only(
     group_manager.reflect_account_groups_on_workspace()
     group_manager.delete_original_workspace_groups()
 
-    @retried(on=[NotFound], timeout=timedelta(seconds=120))
-    def wait():
-        ws.groups.get(ws_group.id)
-
-    wait()
+    # The API needs a moment to delete a group, i.e. until the group is not found anymore
+    @retried(on=[KeyError], timeout=timedelta(minutes=2))
+    def get_group(group_id: str):
+        ws.groups.get(group_id)
+        raise KeyError(f"Group is not deleted: {group_id}")
+
+    with pytest.raises(NotFound):
+        get_group(ws_group.id)
 
 
 @retried(on=[NotFound], timeout=timedelta(minutes=2))