Add tests for gamepad support about jsnes-web HOT 10 OPEN

Ok! I will work on that. I will start by writing tests for GamepadController class (mocking gamepad API) . We need tests for react components that were modified too ?

from jsnes-web.

Awesome thank you! Testing the integration of the modal with the controllers is probably a good idea (I don’t think there is any testing for the custom keyboard commands) but a test for the gamepadcontroller would also be useful. :)

from jsnes-web.

I can't see tests on current version of jsnes-web

So, I will follow this guide https://facebook.github.io/create-react-app/docs/running-tests to start adding the tests. It is ok ? (sorry, I am newbie on react)

from jsnes-web.

No worries! There are a couple really simple ones - the files ending in .test.js in src/.

from jsnes-web.

Great, I will follow the actual scheme then

from jsnes-web.

@tario Seems like the gamepad support is breaking non-webkit browsers too. This error is coming from IE: https://sentry.io/share/issue/b0c3a6b734294b30b6414c548d57a410/

from jsnes-web.

I am sorry to hear that, maybe I should fix that in a separated pull-request. What platforms we should cover ? IE? edge too?

from jsnes-web.

Just tested jsnes.org on edge and IE11 and the gamepad seems to be broken at all. Also tested on https://html5gamepad.com/ and again the gamepad doesn't seems to work at all. Gamepad API should be working on these browsers ?

from jsnes-web.

Anyway. I will fix the case where both getGamepads and webkitGetGamepads are not present. I think I have to code more defensively, sorry

from jsnes-web.

Here it is #167

from jsnes-web.