Error in Quality Assurance Course - suiteSetup Hook

[darkpsinight]

Describe the Issue

There seems to be a small error in the FreeCodeCamp Quality Assurance course regarding the usage of the suiteSetup hook. In the lesson about simulating actions using a headless browser (Course), there's a code snippet provided:

suiteSetup(function(done) {
  return browser.visit('/', done);
});

However, it appears that there's a mistake in this code. The done function should be called with parentheses, like this:

suiteSetup(function(done) {
  return browser.visit('/', done());
});

By adding parentheses after done , we ensure that done is actually invoked as a function, signaling the completion of the setup. Without the parentheses, done is passed as a callback function, which could lead to unexpected behavior.

Update: The solution to this issue has been found in the forum discussion at this link: (FCC Forum).

Looking forward to hearing your insights!

Affected Page

https://www.freecodecamp.org/learn/quality-assurance/quality-assurance-and-testing-with-chai/simulate-actions-using-a-headless-browser

Your code

suiteSetup(function(done) {
  return browser.visit('/', done);
});

Expected behavior

suiteSetup(function(done) {
  return browser.visit('/', done());
});

Screenshots

No response

System

• Device: [PC]
• OS: [Windows11]
• Browser: [Chrome]
• Version: [124.0.6367.78]

Additional context

No response

[gikf]

From my understanding the browser.visit uses the callback function once visit is completed. In here indicating completion of the setup at that point.

Otherwise, calling done() earlier would not mean that setup is completed.

[ShaunSHamilton]

Hello there,

I cannot reproduce any issue with the lesson. Also, as mentioned, done is a callback function that is supposed to be called by browser.visit - it should not be called beforehand.

You can see this in Zombie's docs: https://zombie.js.org/

As such, I am going to close this issue for now.