Merge pull request #86 from ChadSikorra/v1-migrate-tests-to-phpunit

[1.0.0] Migate all unit tests from PHPSpec to PHPUnit

Author: ChadSikorra

.github/workflows/analysis.yml

@@ -24,7 +24,7 @@ jobs:
         run: echo "dir=$(composer config cache-files-dir)" >> $GITHUB_OUTPUT
 
       - name: Install Composer Dependencies
-        run: composer install --no-progress --no-suggest --prefer-dist --optimize-autoloader
+        run: composer install --no-progress --prefer-dist --optimize-autoloader
 
       - name: Cache Dependencies
         uses: actions/cache@v3
@@ -43,11 +43,11 @@ jobs:
           COMPOSER_ALLOW_SUPERUSER: 1
         run: sudo composer run-script --timeout=0 test-coverage
 
-      - name: Upload Spec Coverage to Codecov
+      - name: Upload Unit Coverage to Codecov
         uses: codecov/codecov-action@v3
         with:
           token: ${{ secrets.CODECOV_TOKEN }}
-          file: ./coverage.xml
+          file: ./coverage-unit.xml
           name: FreeDSx-LDAP-Unit
 
       - name: Upload Integration Coverage to Codecov

.github/workflows/build.yml

@@ -7,7 +7,7 @@ jobs:
       fail-fast: false
       matrix:
         operating-system: [ubuntu-latest, windows-latest]
-        php-versions: ['8.1', '8.2']
+        php-versions: ['8.1', '8.2', '8.3', '8.4', '8.5']
     name: PHP ${{ matrix.php-versions }} Test on ${{ matrix.operating-system }}
     steps:
       - name: Checkout
@@ -30,7 +30,7 @@ jobs:
         run: echo "dir=$(composer config cache-files-dir)" >> $GITHUB_OUTPUT
 
       - name: Install Composer dependencies
-        run: composer install --no-progress --no-suggest --prefer-dist --optimize-autoloader
+        run: composer install --no-progress --prefer-dist --optimize-autoloader
 
       - name: Cache dependencies
         uses: actions/cache@v3
@@ -39,8 +39,8 @@ jobs:
           key: ${{ runner.os }}-composer-${{ hashFiles('**/composer.lock') }}
           restore-keys: ${{ runner.os }}-composer-
 
-      - name: Run Specs
-        run: composer run-script test-spec
+      - name: Run Unit Tests
+        run: composer run-script test-unit
 
       # Not ideal, but we run as root here because the LDAP server needs access to certs / keys system directories.
       # For tests this seems fine, but in a real scenario this would be bad...

CHANGELOG.md

@@ -1,7 +1,7 @@
 CHANGELOG
 =========
 
-1.0.0 (2023-xx-xx)
+1.0.0 (202x-xx-xx)
 ------------------
 * Updated the minimum version of PHP to version 8.1.
 * All classes now use "strict_types=1". This internal change should not impact external usage.
@@ -10,6 +10,9 @@ CHANGELOG
 * Added entry handler processing of searches. This allows immediate processing of entries via anonymous functions.
 * Added the ability to cancel search or sync requests when using entry handler processing for searches.
 * Added support for the SensitiveParameter attribute to mask bind related credentials from stack traces.
+* Removed various deprecations for PHP >= 8.1 (implicit nullability, return type declarations, etc.).
+* Added PHP 8.3, 8.4, and 8.5 to normal CI runs.
+* Migrated all unit tests from PHPSpec to PHPUnit. 
 
 0.8.0 (2022-05-21)
 ------------------

composer.json

@@ -22,14 +22,14 @@
     "psr/log": "^3"
   },
   "require-dev": {
-    "phpspec/phpspec": "^7.2",
-    "phpunit/phpunit": "^9.6",
+    "phpunit/phpunit": "^10.5",
     "symplify/easy-coding-standard": "^9.4",
-    "friends-of-phpspec/phpspec-code-coverage": "^6.3",
-    "phpstan/phpstan": "^1.11",
+    "phpstan/phpstan": "^2.1",
     "symfony/process": "^5.4",
     "squizlabs/php_codesniffer": "^3.7",
-    "slevomat/coding-standard": "^7.2"
+    "slevomat/coding-standard": "^7.2",
+    "phpstan/phpstan-phpunit": "^2.0",
+    "phpstan/extension-installer": "^1.4"
   },
   "suggest": {
     "ext-openssl": "For SSL/TLS support and some SASL mechanisms.",
@@ -40,31 +40,32 @@
   },
   "autoload-dev": {
     "psr-4": {
-      "integration\\FreeDSx\\Ldap\\": "tests/integration/FreeDSx/Ldap",
-      "spec\\FreeDSx\\Ldap\\": "tests/spec/FreeDSx/Ldap"
+      "Tests\\Integration\\FreeDSx\\Ldap\\": "tests/integration",
+      "Tests\\Unit\\FreeDSx\\Ldap\\": "tests/unit"
     }
   },
   "config": {
     "allow-plugins": {
-      "dealerdirect/phpcodesniffer-composer-installer": true
+      "dealerdirect/phpcodesniffer-composer-installer": true,
+      "phpstan/extension-installer": true
     }
   },
   "scripts": {
     "test-coverage": [
-      "phpspec run --no-interaction -c phpspec.cov.yml",
-      "phpunit --coverage-clover=coverage-integration.xml"
+      "phpunit --testsuite unit --coverage-clover=coverage-unit.xml",
+      "phpunit --testsuite integration --coverage-clover=coverage-integration.xml"
     ],
-    "test-spec": [
-      "phpspec run --no-interaction"
+    "test-unit": [
+      "phpunit --testsuite unit"
     ],
     "test-integration": [
-      "phpunit"
+      "phpunit --testsuite integration"
     ],
     "analyse": [
       "phpstan --memory-limit=-1 analyse"
     ],
     "analyse-tests": [
-      "phpstan analyse -c phpstan.tests.neon"
+      "phpstan --memory-limit=-1 analyse -c phpstan.tests.neon"
     ],
     "cs-fix": [
       "phpcbf --standard=ruleset.xml --extensions=php --tab-width=4 -sp src",

phpspec.cov.yml

@@ -1,5 +1,5 @@
 parameters:
-    level: max
+    level: 9
     paths:
         - %currentWorkingDirectory%/src
     treatPhpDocTypesAsCertain: false

phpspec.yml

@@ -1,7 +1,4 @@
 parameters:
-    level: 1
+    level: 3
     paths:
         - %currentWorkingDirectory%/tests
-    ignoreErrors:
-        # Does not seem to be a great extension for phpspec :(
-        - '#Call to an undefined (static )?method spec\\FreeDSx\\Ldap\\.*Spec::#'

phpstan.neon

@@ -1,14 +1,34 @@
 <?xml version="1.0" encoding="UTF-8"?>
-<phpunit xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" backupGlobals="false" backupStaticAttributes="false" colors="true" convertErrorsToExceptions="true" convertNoticesToExceptions="true" convertWarningsToExceptions="true" processIsolation="false" stopOnFailure="false" bootstrap="vendor/autoload.php" xsi:noNamespaceSchemaLocation="https://schema.phpunit.de/9.3/phpunit.xsd">
-  <coverage processUncoveredFiles="true">
-    <include>
-      <directory suffix=".php">src</directory>
-    </include>
-  </coverage>
+<phpunit xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+         backupGlobals="false"
+         colors="true"
+         displayDetailsOnTestsThatTriggerDeprecations="true"
+         displayDetailsOnTestsThatTriggerErrors="true"
+         displayDetailsOnTestsThatTriggerNotices="true"
+         displayDetailsOnTestsThatTriggerWarnings="true"
+         displayDetailsOnPhpunitDeprecations="true"
+         processIsolation="false"
+         stopOnFailure="false"
+         bootstrap="vendor/autoload.php"
+         xsi:noNamespaceSchemaLocation="https://schema.phpunit.de/10.5/phpunit.xsd"
+>
+    <source>
+        <include>
+            <directory suffix=".php">./src</directory>
+        </include>
+    </source>
+    <coverage includeUncoveredFiles="true"
+              pathCoverage="false"
+              ignoreDeprecatedCodeUnits="true"
+              disableCodeCoverageIgnore="true">
+    </coverage>
   <testsuites>
-    <testsuite name="FreeDSx LDAP Unit Tests">
+    <testsuite name="integration">
       <directory>./tests/integration</directory>
     </testsuite>
+    <testsuite name="unit">
+      <directory>./tests/unit</directory>
+    </testsuite>
   </testsuites>
   <php>
     <env name="LDAP_SERVER" value="example.com"/>

phpstan.tests.neon

@@ -67,7 +67,7 @@ public function toAsn1(): AbstractType
             ));
         }
         $this->controlValue->addChild(Asn1::boolean(
-            $this->refreshDeletes ?? false
+            $this->refreshDeletes
         ));
 
         return parent::toAsn1();