mirror of
https://github.com/status-im/status-react.git
synced 2025-01-10 02:56:07 +00:00
0d6c553f3f
Adds a new linter to verify all test names are consistent with one common convention we already follow, for the most part, which is: > Test vars (test names) should be suffixed with -test. There's no strong reason for following this convention, although it's quite common in Clojure, but in any case, these are the reasons I can think of and remember: - Naming consistency. Sometimes tests start with "test-", others end with "test" and others don't prefix/suffix at all. - The suffix removes potential conflicts with core Clojure functions. - The suffix mostly eliminates potential conflicts with other vars in the test namespace. Example: you can declare a function delete and have a test named delete-test. - For someone using Emacs imenu feature, it helps differentiate which vars are tests and which are just local functions supporting the tests.