Merge pull request #18 from webxdc/simon/fix17

add test for Duplicated Updates Race
webxdc · Jul 21, 2023 · 72adcdc · 72adcdc
2 parents 4add6f7 + fdacf5c
commit 72adcdc
Show file tree

Hide file tree

Showing 2 changed files with 70 additions and 0 deletions.
diff --git a/duplicated_updates_race.html b/duplicated_updates_race.html
@@ -0,0 +1,61 @@
+<!DOCTYPE html>
+<html lang="en">
+  <head>
+    <meta charset="UTF-8" />
+    <meta http-equiv="X-UA-Compatible" content="IE=edge" />
+    <meta name="viewport" content="width=device-width, initial-scale=1.0" />
+    <title>test webxdc - duplicated updates race test</title>
+    <script src="webxdc.js"></script>
+  </head>
+  <body>
+    <a href="./index.html">&lt; Back</a>
+
+    <h1>Duplicated Updates Race Test</h1>
+    <p>
+      At the time of writing this test desktop has a bug that each update causes
+      sending all new update from the update's serial onwards, which results in
+      updates getting processed multiple times, which can confuse your webxdc
+      app if it does not account for it.
+      <br /><br />Read More deltachat-desktop#3296
+    </p>
+
+    <div id="updates" style="max-height: 50vh; overflow-y: scroll"></div>
+
+    <script>
+      const updatesDiv = document.getElementById("updates");
+      let known_ids = [];
+      let batch = [];
+      let active = false;
+      window.webxdc.setUpdateListener((update) => {
+        batch.push(update.serial);
+
+        if (!known_ids.includes(update.serial)) {
+          known_ids.push(update.serial);
+        } else {
+          const note = document.createElement("div");
+          note.innerText = `(Processing duplicated update ${update.serial})`;
+          updatesDiv.appendChild(note);
+        }
+
+        if (update.serial === update.max_serial) {
+          if (active) {
+            const updateBatch = document.createElement("div");
+            updateBatch.innerText = `[${batch.join(",")}]`;
+            updatesDiv.appendChild(updateBatch);
+          }
+          batch = [];
+          updatesDiv.scrollTop = updatesDiv.scrollHeight;
+        }
+      }, 0);
+      function try_to_trigger_update_race() {
+        active = true;
+        window.webxdc.sendUpdate({ payload: "race test" }, "race test");
+        window.webxdc.sendUpdate({ payload: "race test" }, "race test");
+        window.webxdc.sendUpdate({ payload: "race test" }, "race test");
+      }
+    </script>
+    <button onclick="try_to_trigger_update_race()">
+      Try to trigger update race
+    </button>
+  </body>
+</html>
diff --git a/index.html b/index.html
@@ -46,6 +46,15 @@
         <script src="js/webrtc.js"></script>
         <iframe src="./iframe-webrtc-test.html" sandbox="allow-scripts" width="100%" height="200"></iframe>
 
+        <div class="card">
+            <header class="container"><h2>Webxdc Status Update Tests</h2></header>
+            <div class="container">
+                <a href="./duplicated_updates_race.html">
+                    Duplicated Status Updates Race Test
+                </a>
+            </div>
+        </div>
+
         <!-- debugging -->
         <script src="js/eruda.min.js"></script>
         <script>window.addEventListener("load", () => eruda.init());</script>