Reorder Assertions in test_set_permissions

hGriff0n · hGriff0n · commit c8e14eb0811e · 2026-02-21T14:40:10.000-05:00
diff --git a/tests/gkeep/test_set_permissions.py b/tests/gkeep/test_set_permissions.py
@@ -126,16 +126,16 @@ async def test_set_permissions_no_existing_writers():
         ]
     }
 
-    # Verify batchDelete was NOT called since there were no non-owner permissions
-    mock_service.notes().permissions().batchDelete.assert_not_called()
-
     result = await unwrap(set_permissions)(
         service=mock_service,
         user_google_email="test@example.com",
         note_id="notes/abc123",
         emails=["new@example.com"],
     )
 
+    # Verify batchDelete was NOT called since there were no non-owner permissions
+    mock_service.notes().permissions().batchDelete.assert_not_called()
+
     assert "Removed 0" in result
     assert "Added 1" in result
 

Original file line number	Diff line number	Diff line change
`@@ -126,16 +126,16 @@ async def test_set_permissions_no_existing_writers():`
`126`	`126`	`]`
`127`	`127`	`}`
`128`	`128`
`129`		`- # Verify batchDelete was NOT called since there were no non-owner permissions`
`130`		`- mock_service.notes().permissions().batchDelete.assert_not_called()`
`131`		`-`
`132`	`129`	`result = await unwrap(set_permissions)(`
`133`	`130`	`service=mock_service,`
`134`	`131`	`user_google_email="test@example.com",`
`135`	`132`	`note_id="notes/abc123",`
`136`	`133`	`emails=["new@example.com"],`
`137`	`134`	`)`
`138`	`135`
	`136`	`+ # Verify batchDelete was NOT called since there were no non-owner permissions`
	`137`	`+ mock_service.notes().permissions().batchDelete.assert_not_called()`
	`138`	`+`
`139`	`139`	`assert "Removed 0" in result`
`140`	`140`	`assert "Added 1" in result`
`141`	`141`