onBeforeLoad does not work when overriding visit

• Operating System: osx
• Cypress Version: 0.20.0
• Browser Version: chrome 60

Is this a Feature or Bug?

Bug

Current behavior:

Commands.overwrite seems to break onBeforeLoad for visit command

Test code:

 Cypress.Commands.add('visitRoute', (route) => {
     cy.visit(route, {
         onBeforeLoad(win) {
             console.log('this will happen');
         }
     });
 });
// However if I replace visitRoute with visit and the above with
Cypress.Commands.overwrite('visit', (visit, route) => {
     console.log('this will happen');
     visit(route, {
         onBeforeLoad(win) {
             console.log('this will not happen');
         }
     });
});

[canercandan]

@brian-mann I am running into the same issue though I am using the last release of cypress v2.1.0:

Cypress.Commands.overwrite('visit', (visit, url, options) => {
  console.log('overwritten visit')
  return visit(url, {
    onBeforeLoad: (win) => {
      console.log('onBeforeLoad')
    },
  })
})

onBeforeLoad does not get triggered, thanks.

[RobbinHabermehl]

Unfortunately, this still isn’t working as of v3.0.3 @brian-mann.