Improve the tests [#3464533]

The tests can be improved. For example:

Tests should verify the APCu extension has been loaded and its functions are available
Tests should verify the cache bins used for the tests are effectively handled by the DrupalApcCache class
Tests should use DrupalTestCase::randomName() and DrupalTestCase::randomString() for the test data, when possible
Tests should verify that other data written in the APCu storage from other modules are not changed from this module
Tests should verify that data written in the APCu storage with a 0 TTL are still stored after X seconds
Tests should not call an assertXYZ() method when a previous related call to another assertZYX() method failed

Issue fork apc-3464533

Show commands

Start within a Git clone of the project using the version control instructions.

Add & fetch this issue fork’s repository

Or, if you do not have SSH keys set up on git.drupalcode.org:

Add & fetch this issue fork’s repository

3464533-re-enable-phpunit-tests changes, plain diff MR !53
Check out this branch for the first time

Check out existing branch, if you already have it locally

About issue forks

Comments

Comment #1

29 July 2024 at 18:33

avpaderno created an issue. See original summary.

Comment #2

avpaderno

he/him

Italian

Brescia, 🇮🇹 🇪🇺

commented 3 August 2024 at 14:56

The lines to add to the .gitlab-ci.yml file should be the following ones.

.phpunit-base:
  before_script:
    - apt-get update
    - apt-get install -y --no-install-recommends $PHPIZE_DEPS
    - pecl install apcu && docker-php-ext-enable apcu