Cypress runs commands twice when no baseUrl defined

[rafael-anachoreta]

Current behavior:

Right now, whenever you start Cypress it will kick off some of the commands twice.

This isn't a problem most of the time as tests should be self contained anyway, but if you are using before hooks (for instance, to login and store cookies, as suggested by the docs) it becomes quite detrimental.

Desired behavior:

Cypress should only run once.

Steps to reproduce:

//__bug.spec.js
describe('Bug?', ()=>{
    before(() =>{
        cy.task('message', 'I\'m running!');
    });

    it('Should only log once', () => {
        cy.visit('https://www.google.com');
    });
});

//index.js
module.exports = (on) => {
    on('task', {
        message (args) {
            console.log(args);

            return null;
        }
    });
};

• Run the spec
• Results:

====================================================================================================

  (Run Starting)

  ┌────────────────────────────────────────────────────────────────────────────────────────────────┐
  │ Cypress:    3.1.1                                                                              │
  │ Browser:    Electron 59 (headless)                                                             │
  │ Specs:      1 found (__bug.spec.js)                                                            │
  │ Searched:   cypress/integration/__bug.spec.js                                                  │
  └────────────────────────────────────────────────────────────────────────────────────────────────┘


────────────────────────────────────────────────────────────────────────────────────────────────────

  Running: __bug.spec.js...                                                                (1 of 1)


  Bug
I'm running!
I'm running!
    ✓ Should only log once (1159ms)

Note: this isn't exclusive to hooks! The below spec will also print unwanted results:

describe('Bug', ()=>{

    it('Should only log once', () => {
        cy.task('message', 'I\'m running!');
        cy.visit('https://www.google.com');
    });
    it('Should only log once', () => {
        cy.task('message', 'I\'m running!');
        cy.visit('https://www.google.com');
    });
    it('Should only log once', () => {
        cy.task('message', 'I\'m running!');
        cy.visit('https://www.google.com');
    });
    it('Should only log once', () => {
        cy.task('message', 'I\'m running!');
        cy.visit('https://www.google.com');
    });

});

Results:

Running: __bug.spec.js...                                                                (1 of 1)


  Bug
I'm running!
I'm running!
    ✓ Should only log once (1091ms)
I'm running!
    ✓ Should only log once (3876ms)
I'm running!
    ✓ Should only log once (992ms)
I'm running!
    ✓ Should only log once (1449ms)

If the commands are inverted (i.e., if you visit() before logging) then the results are fine, so maybe Cypress waits for visit then reboots?

Versions

Electron 59 headless
Cypress 3.1.1
macOS High Sierra 10.13.6

[dawidkostyszak]

I found that this happened when you visit url which is different to baseUrl in config.

[dirtyhenry]

I met the same bug.

In my case, it's worth noting that before code runs twice only when all specs are run at once (ie click on Run all specs). When run one by one, each spec behaves as expected.

[jennifer-shehane]

@dirtyhenry Any code provided within the support file will run before each spec file as documented here: https://on.cypress.io/writing-and-organizing-tests#Support-file

[jennifer-shehane]

Hey @rafael-anachoreta, thank you for providing this great detail on this issue.

I have confirmed that indeed, these cy.task() commands are running twice, which is unintended behavior. I've also confirmed that this bug is only present when baseUrl is not defined.

I imagine this has to do with how we handle visiting sites when no baseUrl is defined. See an explanation of this behavior here.

Workaround Today Provide a baseUrl within your Cypress configuration

[dirtyhenry]

Hi @jennifer-shehane: I don't really understand your message since none of my code is in the support file.

[dawidkostyszak]

@jennifer-shehane same behaviour is when baseUrl is defined but I want to visit other page. Ie. I set baseUrl='https://www.google.com/' and in my test will do cy.visit('https://duckduckgo.com/') then all before commands will run twice.

[jennifer-shehane]

@dirtyhenry Sorry for not being clear. If you want code to run only once before each spec file as opposed to before each test, make sure to place the before hook within your support file.

[rblmdst]

Hello everyone,
Any update ? In my case I am using cy.task() to load/unload data in database using the before/after hooks, so it is causing unexpected behavior.