hooks: fix failing tests with mismatched renderer

[ljharb]

There's a few tests failing, currently marked as allowed failures in 3db6013, that shouldn't be failing.

Ideally, the functionality can be fixed; if not, then the tests should be skipped when the breakages are expected.

[AlokTakshak]

@ljharb I would like to work on it can you explain what all needs to be done?

[ljharb]

@AlokTakshak sure! I'm not exactly sure what needs to be done to fix it; but basically, remove these two test jobs from allowed_failures in travis.yml, and then fix the react 16 adapter so the tests pass :-)

[AlokTakshak]

@ljharb is it closed now?

[ljharb]

@AlokTakshak no, those jobs are still allowed failures.

[AlokTakshak]

@ljharb i am running test cases in my local it is passing, i think someones changes fixed the issue

[ljharb]

In that case, a PR that solely removes the allowed failures would close this issue :-)

[AlokTakshak]

@ljharb yeah Sure i will raise it ASAP

[AlokTakshak]

#2122

[AlokTakshak]

@ljharb
npm run test:all
for me it is passing in local but not in travis
I have tried for a specific version that also works fine
npm run react 16.8
npm run build
npm test

[ljharb]

You have to use the RENDERER env var with npm run react if you want to replicate what happens in Travis.