Pokmon API is not available anymore: https://graphql-pokemon.now.sh about react-hooks

[kentcdodds]

Thanks for the help in the workshop repo @marcosvega91. I'm working on fixing this here now :)

[kentcdodds]

Actually... That reminds me!

@all-contributors please add @gugol2 for bugs

[kentcdodds]

Ha! Dang it. Well, I know what I'm working on now 😅

I've been wanting to replace this with MSW anyway.

Thanks for letting me know.

[gugol2]

Well, apparently it works randomly...

I think replacing it with something else in the long run is a good idea.

Thanks again for this material Kent!

[marcosvega91]

Hey I can submit a PR with MSW implementation if you haven't done yet.
I have only a doubt about react-workshop-app because in the setup-tests.js window.fetch should be mocked.
Maybe an env variable like process.env.MSW_ENABLED can be passed from this repo to react-workshop-app 🤔

[kentcdodds]

I wouldn't implement this as part of react-workshop-app, but rather in this repo directly. If you'd like to work on this that would be welcome. We could have a similar setup here that we have in the bookshelf app.

[marcosvega91]

MSW will be imported in this repo but on react-workshop-app side we have this https://github.com/kentcdodds/react-workshop-app/blob/master/src/setup-tests.js that requires that fetch should be mocked
we can do in this way

test('displays the pokemon', async () => {
  window.fetch.mockRestore()
  .......
  jest.spyOn(window, 'fetch')
});

but is not really good

or we can not use the setup-test file from react-workshop-app and do the following in this repo

import '@testing-library/jest-dom/extend-expect'
import '@kentcdodds/react-workshop-app/dist/jest-expect-message'
import {server} from './test/server'

beforeAll(() => {
  jest.spyOn(console, 'info')
  console.info.mockImplementation(() => {})
  server.listen()
})
afterAll(() => {
  console.info.mockRestore()
  server.close()
})

beforeEach(() => console.info.mockClear())

afterEach(() => server.resetHandlers())

[kentcdodds]

Ah, good point. Hmmmm... I set that up before I discovered MSW. I think the right approach here is to remove that from the react-workshop-app and add it to the individual apps and then later set up MSW instead.

[marcosvega91]

It should be good. I'll make a PR there

[marcosvega91]

Goood 🚀

[gugol2]

Sorry I didn't realize you wanted collaboration for this issue @kentcdodds

[kentcdodds]

No worries @gugol2 :) Glad that you reported the issue!

[allcontributors]

@kentcdodds

I've put up a pull request to add @gugol2! 🎉