fix: autologin with empty fetch (#3947)

codeceptjs · Oct 28, 2023 · 7a0f7ff · 7a0f7ff
1 parent f78c027
commit 7a0f7ff
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/docs/webdriver.md b/docs/webdriver.md
@@ -447,7 +447,7 @@ To share the same user session across different tests CodeceptJS provides [autoL
 This plugin requires some configuration but is very simple in use:
 
 ```js
-Scenario('do something with logged in user', ({ I, login) }) => {
+Scenario('do something with logged in user', ({ I, login }) => {
   login('user');
   I.see('Dashboard','h1');
 });

diff --git a/lib/plugin/autoLogin.js b/lib/plugin/autoLogin.js
@@ -9,6 +9,7 @@ const isAsyncFunction = require('../utils').isAsyncFunction;
 
 const defaultUser = {
   fetch: I => I.grabCookie(),
+  check: () => {},
   restore: (I, cookies) => {
     I.amOnPage('/'); // open a page
     I.setCookie(cookies);
@@ -283,7 +284,12 @@ module.exports = function (config) {
       } else {
         userSession.login(I);
       }
+
       const cookies = await userSession.fetch(I);
+      if (!cookies) {
+        debug('Cannot save user session with empty cookies from auto login\'s fetch method');
+        return;
+      }
       if (config.saveToFile) {
         debug(`Saved user session into file for ${name}`);
         fs.writeFileSync(path.join(global.output_dir, `${name}_session.json`), JSON.stringify(cookies));