status-mobile/src/legacy/status_im/ethereum
Icaro Motta 0d6c553f3f
Add linter for inconsistent deftest name (#20224)
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.
2024-06-03 19:10:36 -03:00
..
transactions Partial handlers update (re-frame/reg-event-fx instead of rf/defn) (#19005) 2024-03-14 10:24:37 +01:00
decode.cljs move status-im under legacy (#18237) 2023-12-19 18:41:30 +01:00
encode.cljs move status-im under legacy (#18237) 2023-12-19 18:41:30 +01:00
ens.cljs FINAL BOSS! rename status-im2 to status-im (#18241) 2023-12-19 20:59:07 +01:00
macros.clj move status-im under legacy (#18237) 2023-12-19 18:41:30 +01:00
mnemonic.cljs move status-im under legacy (#18237) 2023-12-19 18:41:30 +01:00
mnemonic_test.cljs Add linter for inconsistent deftest name (#20224) 2024-06-03 19:10:36 -03:00
tokens.cljs move status-im under legacy (#18237) 2023-12-19 18:41:30 +01:00