Mentions légales du service

Skip to content

Names of tests should be cleaned

Created by: stephanemagnenat

While PR #584 cleaned up the directory structure of unit tests, the tests themselves are not named very consistently. In particular they lack of prefix denoting their category. The current naming is informative, separates words with dashes, but lacks a proper prefixing of the type of test. The goal of this issue is to discuss such a namespacing.

A proposal with minimal changes could be to use an underscore to provide namespacing, and keep the dash to separate words in the test names within one scope. That would lead to:

  • simulator_basic-thymio2
  • vm_natives-count
  • compiler_basic-arithmetic
  • compiler_simulateinput_basic-arithmetic
  • compiler_fuzzyinput_basic-arithemtic
  • compiler_valgrind

But that does not look really good. Maybe the reverse is better:

  • simulator-basic_thymio2
  • vm-natives_count
  • compiler-basic_arithmetic
  • compiler-simulateinput-basic_arithmetic
  • compiler-fuzzyinput-basic_arithemtic
  • compiler-valgrind

Does anyone knows best practices or has an informed opinion?