Clean up test suites

[gudmdharalds]

We need to clean up the unit and integration test suites a bit, ensuring more consistency and making them easier to work with.

The following elements need to be worked on:

• All tests should have an initial doc comment.
  • Unit tests [ Update unit and integration tests to comply with a common standard, PHP 8.2 fixes, template update #346 ]
  • Integration tests
• All tests should have a namespace, either namespace Vipgoci\Tests\Unit or namespace Vipgoci\Tests\Integration
  • Unit tests [ Update unit and integration tests to comply with a common standard, PHP 8.2 fixes, template update #346 ]
  • Integration tests
• All tests should declare strict types declare(strict_types=1)
  • Unit tests [ Update unit and integration tests to comply with a common standard, PHP 8.2 fixes, template update #346 ]
  • Integration tests
• All tests should comply with the WordPress coding standard.
  • Unit tests [ Update unit and integration tests to comply with a common standard, PHP 8.2 fixes, template update #346 ]
  • Integration tests
• As possible, tests should use @preserveGlobalState disabled and @runTestsInSeparateProcesses, explain in README.md
  • Unit tests [ Update unit and integration tests to comply with a common standard, PHP 8.2 fixes, template update #346 ]
  • Integration tests
• All testing functions should use :void as return type
  • Unit tests [ Update unit and integration tests to comply with a common standard, PHP 8.2 fixes, template update #346 ]
  • Integration tests
• All testing functions should define variable types
• All testing functions should call require_once() during setUp() stage to make the most of running tests in separate processes - this avoids problems with functions being declared multiple times.
  • Unit tests [ Update unit and integration tests to comply with a common standard, PHP 8.2 fixes, template update #346 ]
  • Integration tests
• Remove phpcs:disable PSR1.Files.SideEffects from tests
  • Unit tests [ Update unit and integration tests to comply with a common standard, PHP 8.2 fixes, template update #346 ]
  • Integration tests
• Ensure Skeleton.php files follow all the above.
• Rename vipgoci_unittests_output_suppress() and vipgoci_unittests_output_unsuppress() so that they include _maybe_.
• Some unit-tests are linked to a particular token holder, such as ResultsFilterCommentsToMaxTest.php - this should be avoided.
• Common folder for helper functions; all unit/helper files should go to common/helper, and all integrations/IncludesForTests*.php files to the same folder.
• Remove WordPress.Files.FileName and Squiz.Commenting.FileComment exceptions from .vipgoci_options and fix tests accordingly
• Consistent naming of tests - ensure each test has a prefix consistent with the file were the tested function lives. These tests need updating:
  • units/A00IrcApiAlertQueueTest.php [ Break vipgoci_run() into multiple functions #207 ]
  • units/A00IrcApiAlertQueueUniqueTest.php [ Break vipgoci_run() into multiple functions #207 ]
  • units/A00StatsCountersTest.php [ Update unit and integration tests to comply with a common standard, PHP 8.2 fixes, template update #346 ]
  • units/StatsRunTimeMeasureTest.php [ Update logging & memory management, retry certain HTTP requests, catch out of memory situations #258 ]
  • units/StatsStatsInitTest.php [ Add support for WPScan API scanning #273 ]
  • units/VipgociOptionsArrayHandleTest.php [ Improve reporting to users #246 ]
  • units/VipgociOptionsBoolHandleTest.php [ Rename test class to match file name #292 ]
  • units/VipgociOptionsIntegerHandleTest.php [ Update unit and integration tests to comply with a common standard, PHP 8.2 fixes, template update #346 ]
  • units/VipgociOptionsPhpcsRuntimeSetTest.php [ Update unit and integration tests to comply with a common standard, PHP 8.2 fixes, template update #346 ]
  • units/VipgociOptionsRecognizedTest.php [ Break vipgoci_run() into multiple functions #207 ]
  • units/VipgociLintScanTest.php [ PHP lint using multiple PHP versions #255 ]
  • units/VipgociSkipFileTest.php [ Add support for WPScan API scanning #273 ]
  • integrations/A01StatsStatsPerFileTest.php [ Apply WP CS and variable types to final files  #272 ]
  • integrations/A09GitHubLabelsTest.php [ Update unit and integration tests to comply with a common standard, PHP 8.2 fixes, template update #346 ]
  • integrations/A09LintLintScanCommitTest.php [ PHP lint using multiple PHP versions #255 ]
  • integrations/GitDiffsFetchTest.php [ Update unit and integration tests to comply with a common standard, PHP 8.2 fixes, template update #346 ]
  • integrations/HttpRespSunsetHeaderCheckTest.php [ Update unit and integration tests to comply with a common standard, PHP 8.2 fixes, template update #346 ]
  • integrations/StatsRunTimeMeasureShellExecTest.php [ Update two integration tests relating to git functionality #267 ]
  • integrations/ReviewCommentsSortBySeverityTest.php [ Apply WP CS for a few files, move functionality between files #247 ]
  • integrations/VipgociOptionsFileHandleTest.php [ Update unit and integration tests to comply with a common standard, PHP 8.2 fixes, template update #346 ]
  • integrations/VipgociOptionsReadEnvTest.php [ Update unit and integration tests to comply with a common standard, PHP 8.2 fixes, template update #346 ]
  • integrations/VipgociOptionsTeamsTest.php [ Update unit and integration tests to comply with a common standard, PHP 8.2 fixes, template update #346 ]
• Ensure only one function is tested per test-file.