Check and fix mock names in pre-commit hook

[cameron-craig-etas]

#422

This PR adds a Python script to validate test doubles (mocks, stubs, fakes etc.), to ensure they use the test double name as a prefix (e.g mock_component.cpp). We recently updated all our mocks to use a consistent format (/pull/457). This pre-commit will keep things nice and tidy 🧹

As far as testing goes I have ran through these scenarios manually:

• Attempting to commit a mock with a backwards name:

  Check and fix test double file names.....................................Failed
  - hook id: test-doubles-naming-fix
  - exit code: 1
  - files were modified by this hook
  
  WARNING: Non-conformant files have been found, fixing now.
  WARNING: Renaming: score/launch_manager/src/daemon/src/process_group_manager/details/component_mock.hpp -> score/launch_manager/src/daemon/src/process_group_manager/details/mock_component.hpp.
  

• Attempting to commit a stub with a backwards name:

  Check and fix test double file names.....................................Failed
  - hook id: test-doubles-naming-fix
  - exit code: 1
  - files were modified by this hook
  
  WARNING: Non-conformant files have been found, fixing now.
  WARNING: Renaming: bad_stub.hpp -> stub_bad.hpp.
  

• Attempting to commit a mock stub mutant hybrid:

  Check and fix test double file names.....................................Failed
  - hook id: test-doubles-naming-fix
  - exit code: 1
  
  Traceback (most recent call last):
    File "/home/crc1yok/git/lifecycle/./scripts/fix_test_doubles_naming_precommit.py", line 236, in <module>
      raise SystemExit(main())
                     ^^^^^^
    File "/home/crc1yok/git/lifecycle/./scripts/fix_test_doubles_naming_precommit.py", line 201, in main
      operations = define_operations(args.filenames)
                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    File "/home/crc1yok/git/lifecycle/./scripts/fix_test_doubles_naming_precommit.py", line 136, in define_operations
      raise ValueError(
  ValueError: Invalid file name mock_fake.hpp contains multiple double names.
  

• Adding a mock with multiple file extensions:

  Check and fix test double file names.....................................Failed
  - hook id: test-doubles-naming-fix
  - exit code: 1
  
  Traceback (most recent call last):
    File "/home/crc1yok/git/lifecycle/./scripts/fix_test_doubles_naming_precommit.py", line 239, in <module>
      raise SystemExit(main())
                       ^^^^^^
    File "/home/crc1yok/git/lifecycle/./scripts/fix_test_doubles_naming_precommit.py", line 204, in main
      operations = define_operations(args.filenames)
                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    File "/home/crc1yok/git/lifecycle/./scripts/fix_test_doubles_naming_precommit.py", line 141, in define_operations
      raise ValueError(
  ValueError: More than one file extension found for test double test_mock.old.cpp. Fix this!
  

• A valid commit with no changes to test doubles

  Check and fix copyright headers with cr_checker..........................Passed
  

[github-actions]

License Check Results

🚀 The license check job ran with the Bazel command:

bazel run --lockfile_mode=error //:license-check

Status: ⚠️ Needs Review

Click to expand output

[License Check Output]
Extracting Bazel installation...
Starting local Bazel server (8.6.0) and connecting to it...
INFO: Invocation ID: 9db264b1-e800-434e-b44f-edac16ef640e
Computing main repo mapping: 
Computing main repo mapping: 
Loading: 
Loading: 0 packages loaded
Loading: 0 packages loaded
Loading: 0 packages loaded
    currently loading: 
Loading: 0 packages loaded
    currently loading: 
Loading: 0 packages loaded
    currently loading: 
Loading: 0 packages loaded
    currently loading: 
Loading: 0 packages loaded
    currently loading: 
Analyzing: target //:license-check (1 packages loaded)
Analyzing: target //:license-check (1 packages loaded, 0 targets configured)
Analyzing: target //:license-check (1 packages loaded, 0 targets configured)

Analyzing: target //:license-check (41 packages loaded, 10 targets configured)

Analyzing: target //:license-check (90 packages loaded, 10 targets configured)

Analyzing: target //:license-check (145 packages loaded, 1493 targets configured)

Analyzing: target //:license-check (158 packages loaded, 3170 targets configured)

Analyzing: target //:license-check (160 packages loaded, 9334 targets configured)

Analyzing: target //:license-check (170 packages loaded, 9397 targets configured)

Analyzing: target //:license-check (170 packages loaded, 9397 targets configured)

INFO: Analyzed target //:license-check (175 packages loaded, 11535 targets configured).
[1 / 14] Expanding template license.check.license_check; 0s local ... (2 actions running)
[12 / 16] JavaToolchainCompileClasses external/rules_java+/toolchains/platformclasspath_classes; 0s disk-cache, processwrapper-sandbox ... (2 actions running)
[14 / 16] JavaToolchainCompileBootClasspath external/rules_java+/toolchains/platformclasspath.jar; 0s disk-cache, processwrapper-sandbox
[16 / 16] no actions running
INFO: Found 1 target...
Target //:license.check.license_check up-to-date:
  bazel-bin/license.check.license_check
  bazel-bin/license.check.license_check.jar
INFO: Elapsed time: 25.524s, Critical Path: 2.59s
INFO: 16 processes: 12 internal, 3 processwrapper-sandbox, 1 worker.
INFO: Build completed successfully, 16 total actions
INFO: Running command line: bazel-bin/license.check.license_check ./formatted.txt <args omitted>
usage: org.eclipse.dash.licenses.cli.Main [-batch <int>] [-cd <url>]
       [-confidence <int>] [-ef <url>] [-excludeSources <sources>] [-help] [-lic
       <url>] [-project <shortname>] [-repo <url>] [-review] [-summary <file>]
       [-timeout <seconds>] [-token <token>]

[github-actions]

The created documentation from the pull request is available at: docu-html

[MaciejKaszynski]

Mostly nit picks

[danth]

This will replace dashes with underscores, seems unintended since the rest of the code preserves dashes.