Browser-laptop: Add a prefix/suffix for classes which are used for automated tests only

Created on 12 Dec 2016  路  3Comments  路  Source: brave/browser-laptop

Expected behavior:
Add a prefix/suffix for classes which are used for automated tests only, eg. test__. It would help to avoid unexpected regressions.

  • Any related issues:
automated-tests
Source
picture luixxiul

Most helpful comment

I believe this isn't valid anymore according to the style guide, since at the end all the classes will be used just for this.

picture michalbe on 11 Feb 2017
👍2

All 3 comments

I believe this isn't valid anymore according to the style guide, since at the end all the classes will be used just for this.

michalbe picture michalbe on 11 Feb 2017
👍2

Exactly, thanks for pinging!

luixxiul picture luixxiul on 11 Feb 2017
👍1

correct 馃槃 the replacement (for testing only) would be (per the style guide) to create a new attribute, data-test-id

bsclifton picture bsclifton on 12 Feb 2017
Was this page helpful?
0 / 5 - 0 ratings

Related issues

octohedron picture octohedron  路  3Comments
lukemulks picture lukemulks  路  3Comments
jkup picture jkup  路  3Comments
antiroyalty picture antiroyalty  路  3Comments
luixxiul picture luixxiul  路  3Comments