Add first Unit tests

[dxL1nus]

Add Unit Tests ass asked by @nuest here

classes/RetrieveReserveIdentifiers/CertificateIdentifierList.php

9    // get the certificate ID from the issue description
10  function getRawIdentifier(string $title): string

The task

This is a function that we can very nicely check in a unit test, and feed it different (and incorrect) inputs, like "Author | abc | 2025-999" or "Author - 2025-001 | 2026-001 - 2026-002".

Please add a first unit test (and tell me how to add unit tests to plugins).

See https://github.com/pkp/pkp-lib/blob/main/tests/classes/plugins/PluginTest.php and maybe(!!) this also helps: https://github.com/reviewqualitycollector/ojs-rqcplugin/tree/main/tests

You can give us the first unit test, and @Subhanaliweb will create the first integration/UI test in #35

[dxL1nus]

To run unit-tests use the following command from the OJS root directory:

lib/pkp/lib/vendor/phpunit/phpunit/phpunit -c lib/pkp/tests/phpunit.xml plugins/generic/codecheck/tests/

[dxL1nus]

So far we have 10 passing Tests for the getRawidentifier() function.

[dxL1nus]

To see how tests should look like, please view this test suite file.

[nuest]

That looks like a good start!

Currently you're only testing specfific function calls.

Could you also test more complex fromApi function? You could try to mock the fetchIssues() function and return some test data there to make sure you have the right number of identifiers in the list and the expected identifiers, etc.

[dxL1nus]

That looks like a good start!

Currently you're only testing specfific function calls.

Could you also test more complex fromApi function? You could try to mock the fetchIssues() function and return some test data there to make sure you have the right number of identifiers in the list and the expected identifiers, etc.

Yes of course I will write more unit tests. This was just your requested starting point from the PR

[dxL1nus]

I also managed to install a code coverage tool for the tests, so we can see which methods are covered to what extend!

[dxL1nus]

The CertificateIdentifierList Class has now 100% coverage and we can move on to the next classes. If you want to (@nuest ) then I can write uni tests for all the Classes for the Retrieving and Reserving of Identifiers as well as the Api Controller