Moxios not working in 0.19.1

[medeirosle]

Moxios last version (0.4.0) just stopped working with Axios 0.19.1.

I just changed back Axios version in my project (0.19.0) and it's working fine.

Any idea why?

[chinesedfan]

@neomedeiros Can you provide detailed code examples or error messages, as the issue template required?

[DanielRuf]

axios/moxios#71

v0.19.0...v0.19.1

Maybe you can find the culprit with git bisect by cloning axios into your node_modules or using axios/axios.

[medeirosle]

I've got no error message in my real-world project, the stubRequest() function simply did not stub a given request. I've just tried to create a minimal project in order to reproduce the error, but without success. Everything is working fine:

https://github.com/neomedeiros/axios-2650

I will try to get more details about the issue in the real-world project.

[brandon-leapyear]

✨ This is an old work account. Please reference @brandonchinn178 for all future communication ✨

---

@neomedeiros I think the issue is if you change todos.js in your repro project to use

axios.get('/todos', { baseURL: 'https://...' })

I'm finding that if I do stubRequest with just the path instead of the full path, it works

[brandon-leapyear]

✨ This is an old work account. Please reference @brandonchinn178 for all future communication ✨

---

It broke here: https://github.com/brandon-leapyear/axios/commit/6fe506fda290ba935c2641f68f1fcba7f4a16cd3

Will try to put up a PR soon

[github-actions]

Hello! 👋

This issue is being automatically marked as stale because it has not been updated in a while. Please confirm that the issue is still present and reproducible. If no updates or new comments are received the issue will be closed in a few days.

Thanks.

[brandon-leapyear]

✨ This is an old work account. Please reference @brandonchinn178 for all future communication ✨

---

this issue still has open PRs against it

[chinesedfan]

@brandon-leapyear See #2751 (comment). Can you check for that?

[github-actions]

Hello! 👋

This issue is being automatically marked as stale because it has not been updated in a while. Please confirm that the issue is still present and reproducible. If no updates or new comments are received the issue will be closed in a few days.

Thanks.

[fraywing]

@brandon-leapyear @chinesedfan have you all not achieved consensus on a fix for this?