Added support for CSS modules in hasClass

[nblackburn]

Checks the className against $style where available.

• Allows for checking classes that were converted with css modules without needing to know what it was converted to.
• Only works when $style exists on the vm and a mapping is available for the target class.

DISCLAIMER: Not a fan of defining a function argument but it's the cleanest solution.

[nblackburn]

Made some tweaks so that the function variables are not getting redefined at the cost of consiceness, sorry 😢.

[eddyerburgh]

Can you add a unit test please

[nblackburn]

Sure, i was going to add one but didn't have time yesterday.

[nblackburn]

Added a unit test

[eddyerburgh]

The compatibility tests are failing, it looks like $style is implemented differently in Vue 2.0.x

To debug you can run the npm run test:compatibility

[nblackburn]

Should be good to go now, sorry about that.

FYI: You can supress all of the stuff being spewed out on console in the karma runner with the client.captureConsole property.

[eddyerburgh]

The test was correct, but the code needs to be changed.

Can you check how scoped classes are dealt with in Vue 2.0.x and add a fix?

You can see an example where we have different behavior depending on Vue version here - https://github.com/vuejs/vue-test-utils/blob/dev/src/wrappers/wrapper.js#L297.

[eddyerburgh]

And can you revert the test to the previous test, where you had a class but had a scoped style block

[nblackburn]

@eddyerburgh I am not convinced the test was correct as to be able to use css modules, the class needs to be that of the mapped (found in $style) one rather than the original which to my understanding is not done automatically.

https://vue-loader.vuejs.org/en/features/css-modules.html

[eddyerburgh]

The test was correct, you can see the example in the vue loader docs —https://vue-loader.vuejs.org/en/features/scoped-css.html

[nblackburn]

@eddyerburgh this feature is for CSS modules, not scoped CSS. Hence the reference to css-modules in my last comment, the fact i am using $style and the test using the module directive instead of scoped.

[eddyerburgh]

Ah sorry 👍

You're right then, the test you have now looks correct

[nblackburn]

Cool, thanks 👍