Ref #796: Wait for specific text in UI instead of arbitrary wait (#801)

* Ref #796: Wait for specific text in UI instead of arbitrary wait

* make lint

Author: leplatrem

browser-tests/test_review.py

@@ -1,5 +1,4 @@
 import re
-import time
 
 from playwright.sync_api import expect
 
@@ -153,9 +152,9 @@ def test_review_existing_record(
     page.get_by_text("Request review...").first.click()
     page.get_by_placeholder("Comment...").last.fill("Modified record and attachment")
     page.get_by_text("Request review").last.click()
+    expect(page.get_by_text("Review requested.")).to_be_visible()
 
     # Check that preview record has the appropriate values
-    time.sleep(1) # wait 1 second as the preview collection can take a few ms to sync
     record = editor_client.get_record(id=record_id, bucket=preview_bucket)
     assert record["data"]["title"] == "new val"
     assert record["data"]["attachment"]["filename"] == "kinto-logo.jpg"