wip timing

Author: jzohrab

tests/acceptance/lute_test_client.py

@@ -17,7 +17,7 @@
 import time
 import json
 import requests
-from playwright.sync_api import Keyboard, Mouse
+from playwright.sync_api import Keyboard, Mouse, expect
 
 
 class LuteTestClient:  # pylint: disable=too-many-public-methods
@@ -159,9 +159,11 @@ def get_book_table_content(self):
             # - "last opened" date is a hassle to check
             # - "actions" is just "..."
             ret = "; ".join(rowtext[:-2]).strip()
+            # Hacky cleanup ok for tests.
+            ret = ret.replace("\u200B", "").replace("\n", "").replace("\\n", "")
             content.append(ret)
 
-        return "\n".join(content)
+        return "\n".join([c for c in content if c.strip() != ""]).strip()
 
     def get_book_page_start_dates(self):
         "get content from sql check"
@@ -219,9 +221,17 @@ def _fill_tagify_field(self, frame, fieldid, text):
         if span is None:
             raise RuntimeError(f"unable to find {fieldid}")
 
-        # Focus the span and type text
+        # Focus the span and type text.  Various timing hacks added to
+        # ensure that tagify wires things up correctly in response to
+        # clicks ...  hacky, but fixed errors during ci runs.
+        expect(span).to_be_visible()
+        expect(span).to_be_enabled()
         span.click()
-        span.type(text)
+        time.sleep(0.1)
+        expect(span).to_be_focused()
+        time.sleep(0.1)
+        span.fill(text)
+        time.sleep(0.1)
         span.press("Enter")
         time.sleep(0.3)
 
@@ -320,6 +330,7 @@ def get_term_table_content(self):
 
         def _delimited_tags(td):
             """Get the cleaned tags."""
+            # print(f"GOT RAW TAGS = {td.inner_text()}", flush=True)
             tags = [t.strip() for t in td.inner_text().split("\n")]
             tags = [t for t in tags if t not in ["", "\u200B"]]
             return ", ".join(tags)
@@ -353,11 +364,17 @@ def _delimited_tags(td):
             )
             rowtext.append(selected_text)
 
+            # Hacky cleanup, ok for tests.
+            # print(f"RAW ROWTEXT = {rowtext}", flush=True)
+            rowtext = [
+                r.replace("\u200B", "").replace("\n", "").replace("\\n", "")
+                for r in rowtext
+            ]
             rowval = "; ".join(rowtext).strip()
-            cleaned_rowval = rowval.replace("\u200B", "")
-            rowstring.append(cleaned_rowval)
+            rowstring.append(rowval)
 
-        return "\n".join(rowstring)
+        # print(f"RAW ROWSTRINGs = {rowstring}", flush=True)
+        return "\n".join([r for r in rowstring if r.strip() != ""]).strip()
 
     ################################3
     # Reading/rendering