Skip to content

General refactor in tests directory

Tristan Van Berkom requested to merge tristan/organizing-tests into master

Test cases have become a bit messy, such that it is difficult to know where to look for a given test or where is the right place to add one. As a result, some tests have become redundant and we've also grown tests on similar material in different subdirectories.

This branch does the following things:

  • Created a new tests/internals/ subdirectory to group together subsystem tests
  • Created a new tests/elements/ subdirectory to mirror the tests/sources/ directory (for the filter test)
  • Removed all of the following directories:
    • tests/context/ (-> tests/internals/)
    • tests/loader/
    • tests/pipeline/
    • tests/plugins/
    • tests/storage/ (-> tests/internals/)
    • tests/utils/ (-> tests/internals/)
    • tests/yaml/ (-> tests/internals/)
  • Tests in the removed directories were either:
    • Moved to tests/internals/, tests/frontend/ or tests/format/
    • Merged into another related test in one of the above three directories
    • Removed with a comment explaining where the test is already covered
    • Ported to use the cli test fixture and moved into either tests/frontend/ or tests/format/

Merge request reports