search

scottschafer / cypressautomocker

cypress-based library for capturing APIs and replaying them as mocks
131 stars 17 forks source link

Cypress ^3.3.0 Doesn't Recognize `/` at Beginning of Relative Path #17

Open jimberley opened 5 years ago

jimberley commented 5 years ago

https://github.com/scottschafer/cypressautomocker/blob/1eebe019e537c19a7a48c925a401755b68ec3ec2/include-in-tests/autoMockCommands.js#L53

https://github.com/cypress-io/cypress/issues/4352

This is causing Automock to fail when using Cypress @^3.3.0

scottschafer commented 5 years ago

I'm not seeing this issue in sample code, using Cypress 3.4.0.

jimberley commented 5 years ago

Hi. Per this -- https://docs.cypress.io/guides/references/changelog.html#3-4-0 -- 3.4.0 is the latest version of Cypress. Am I missing something?

On Wed, Jul 10, 2019 at 1:25 PM scottschafer notifications@github.com wrote:

I'm not seeing this issue in sample code, using Cypress 4.0.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/scottschafer/cypressautomocker/issues/17?email_source=notifications&email_token=ACURKLBL2J5WQF3WLHTFYOLP6YLPZA5CNFSM4H652KMKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODZUFIQY#issuecomment-510153795, or mute the thread https://github.com/notifications/unsubscribe-auth/ACURKLBX3W3BZF4RSIOIWM3P6YLPZANCNFSM4H652KMA .

-- Jim VanNest SDET

175 Varick, 7th Floor https://maps.google.com/?q=175+Varick,+7th+Floor+New+York,+NY+10014&entry=gmail&source=g

New York, NY 10014 https://maps.google.com/?q=175+Varick,+7th+Floor+New+York,+NY+10014&entry=gmail&source=g

We're Hiring! https://grnh.se/2wn5bg671

Facebook http://www.facebook.com/diaandco | Twitter http://twitter.com/diaandco | Instagram http://instagram.com/diaandco

Read our February 2018 NYTimes feature! https://www.nytimes.com/2018/02/08/business/smallbusiness/plus-size-fashion.html

scottschafer commented 5 years ago

Sorry, I miswrote. Yes, I meant 3.4.0. In any case, I pushed an update just now that uses Cypress 3.4.0 and I'm not seeing the issue that you're referring to, so I'm a bit confused.

jimberley commented 5 years ago

Cypress is a fickle application. What I'm seeing is that played-back automocks just stop working in Cypress 3.4.0. I get:

CypressError: Timed out retrying: cy.readFile("/cypress/integration/../automocks/admin_note.json") failed because the file does not exist at the following path: /cypress/automocks/admin_note.json

I then found that Cypress is no longer allowing for / in their relative paths for cy.readFile (linked above) and decided to talk to you. FWIW, the previous version of Cypress (3.2.x) works just fine with Automocker.

On Wed, Jul 10, 2019 at 2:19 PM scottschafer notifications@github.com wrote:

Sorry, I miswrote. Yes, I meant 3.4.0. In any case, I pushed an update just now that uses Cypress 3.4.0 and I'm not seeing the issue that you're referring to, so I'm a bit confused.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/scottschafer/cypressautomocker/issues/17?email_source=notifications&email_token=ACURKLHSG5D2LGP5AIJPTODP6YR2PA5CNFSM4H652KMKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODZUJ5WQ#issuecomment-510172890, or mute the thread https://github.com/notifications/unsubscribe-auth/ACURKLHA4QBC6KUF3362QFTP6YR2PANCNFSM4H652KMA .

-- Jim VanNest SDET

175 Varick, 7th Floor https://maps.google.com/?q=175+Varick,+7th+Floor+New+York,+NY+10014&entry=gmail&source=g

New York, NY 10014 https://maps.google.com/?q=175+Varick,+7th+Floor+New+York,+NY+10014&entry=gmail&source=g

We're Hiring! https://grnh.se/2wn5bg671

Facebook http://www.facebook.com/diaandco | Twitter http://twitter.com/diaandco | Instagram http://instagram.com/diaandco

Read our February 2018 NYTimes feature! https://www.nytimes.com/2018/02/08/business/smallbusiness/plus-size-fashion.html