DEV: Increase wait duration between smoke test further during edit.

discourse
,
#1

DEV: Increase wait duration between smoke test further during edit.

Otherwise, the app detects that we’re replying too quickly.

diff --git a/test/smoke_test.js b/test/smoke_test.js
index 5eddf69..68ebfe9 100644
--- a/test/smoke_test.js
+++ b/test/smoke_test.js
@@ -286,7 +286,7 @@ const path = require("path");
         visible: true
       });
 
-      promise = promise.then(() => page.waitFor(2000));
+      promise = promise.then(() => page.waitFor(5000));
 
       promise = promise.then(() => {
         const post = `I edited this post`;

GitHub sha: 7bb94e87

#2

:frowning: now our smoke test is 3 seconds slower.

Maybe take it in as a param, then we can have a build 4 seconds faster by setting the rate limit to 1 (like it is today) and only waiting 1 second here.

#3

There are a bunch of waits through out the smoke test already but I’ll fix it properly once I find the time to do so. It is quite interesting though because the app is built to prevent bots from spamming us and it is doing a good job because getting the smoke test to run consistently is tricky due to those restrictions.

2 Likes
#4
#6