You need to sign in or sign up before continuing.
Follow-up from "WIP:Ops/unit test js"
The following discussion from !248 (merged) should be addressed:
-
@cyntss started a discussion: Im fixing this, but @scurescu also check here that you dont need to mock the entire component, since all you are testing is
expect( $('.cookies-alert')).to.have.css('visibility', 'visible')
so, all you need is a div with the class
.cookies-alert
(btw... why is it that the class doesnt have the.js-
in front? - we must fix that.. JS classes always should have.js-
prefix ;) )
This issue is about using ALWAYS the .js-
prefix for classes that require to be handled by JS. In this case the class .cookies-alert
, but we should revisit all our components to make sure we are good everywhere else.
Edited by Cynthia Sanchez