chore: Add mutation testing workflow and configuration files.

.github/workflows/mutation.yml

@@ -0,0 +1,26 @@
+on:
+  pull_request:
+    paths-ignore:
+      - 'docs/**'
+      - 'README.md'
+      - 'CHANGELOG.md'
+      - '.gitignore'
+      - '.gitattributes'
+
+  push:
+    paths-ignore:
+      - 'docs/**'
+      - 'README.md'
+      - 'CHANGELOG.md'
+      - '.gitignore'
+      - '.gitattributes'
+
+name: mutation test
+
+jobs:
+  mutation:
+    uses: php-forge/actions/.github/workflows/infection.yml@main
+    with:
+      phpstan: true
+    secrets:
+      STRYKER_DASHBOARD_API_KEY: ${{ secrets.STRYKER_DASHBOARD_API_KEY }}

composer.json

@@ -15,6 +15,7 @@
     },
     "require-dev": {
         "ext-simplexml": "*",
+        "infection/infection": "^0.27|^0.29",
         "maglnet/composer-require-checker": "^4.1",
         "phpstan/extension-installer": "^1.4",
         "phpstan/phpstan-strict-rules": "^2.0.3",
@@ -41,13 +42,15 @@
     "config": {
         "sort-packages": true,
         "allow-plugins": {
+            "infection/extension-installer": true,
             "phpstan/extension-installer": true,
             "yiisoft/yii2-composer": true
         }
     },
     "scripts": {
         "check-dependencies": "./vendor/bin/composer-require-checker check",
         "ecs": "./vendor/bin/ecs --fix",
+        "mutation": "./vendor/bin/infection --threads=4 --ignore-msi-with-no-mutations --only-covered --min-msi=100 --min-covered-msi=100",
         "rector": "./vendor/bin/rector process src",
         "static": "./vendor/bin/phpstan --memory-limit=512M",
         "tests": "./vendor/bin/phpunit"

infection.json.dist → infection.json5

@@ -1,16 +1,17 @@
 {
-    "source": {
-        "directories": [
-            "src"
-        ]
-    },
+    "$schema": "vendor/infection/infection/resources/schema.json",
     "logs": {
         "text": "php:\/\/stderr",
         "stryker": {
             "report": "main"
         }
     },
-    "mutators": {
-        "@default": true
-    }
+    "source": {
+        "directories": [
+            "src"
+        ]
+    },
+    "phpStan": {
+        "configDir": "."
+    },
 }

phpunit.xml.dist

@@ -1,7 +1,7 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <phpunit
     xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
-    xsi:noNamespaceSchemaLocation="https://schema.phpunit.de/10.2/phpunit.xsd"
+    xsi:noNamespaceSchemaLocation="vendor/phpunit/phpunit/phpunit.xsd"
     bootstrap="tests/bootstrap.php"
     cacheDirectory="runtime/.phpunit.cache"
     colors="true"

src/NestedSetsBehavior.php

@@ -527,21 +527,21 @@
     {
         $this->operation = self::OPERATION_DELETE_WITH_CHILDREN;
 
         if ($this->getOwner()->isTransactional(ActiveRecord::OP_DELETE) !== false) {
             return $this->deleteWithChildrenInternal();
         }
 
         $transaction = $this->getOwner()::getDb()->beginTransaction();
 
         try {
             match ($result = $this->deleteWithChildrenInternal()) {
                 false => $transaction->rollBack(),
                 default => $transaction->commit(),
             };
 
             return $result;
         } catch (Exception $e) {
             $transaction->rollBack();
 
             throw $e;
         }
@@ -691,7 +691,7 @@
         $nodeLeft = $node->getAttribute($this->leftAttribute);
         $nodeRight = $node->getAttribute($this->rightAttribute);
 
         if ($currentLeft <= $nodeLeft || $currentRight >= $nodeRight) {
             return false;
         }
 
@@ -800,7 +800,7 @@
 
         $this->applyTreeAttributeCondition($condition);
 
         return $this->getOwner()::find()->andWhere($condition)->addOrderBy([$this->leftAttribute => SORT_ASC]);
     }
 
     /**
@@ -943,7 +943,7 @@
 
         $this->applyTreeAttributeCondition($condition);
 
         return $this->getOwner()::find()->andWhere($condition)->addOrderBy([$this->leftAttribute => SORT_ASC]);
     }
 
     /**
@@ -1152,7 +1152,7 @@
 
         $this->applyTreeAttributeCondition($condition);
         $result = $this->getOwner()::deleteAll($condition);
         $this->getOwner()->setOldAttributes(null);
         $this->getOwner()->afterDelete();
 
         return $result;
@@ -1199,7 +1199,7 @@
 
             $this->shiftLeftRightAttribute($value, $delta);
 
-            if ($leftValue >= $value) {
+            if ($leftValue > $value) {
                 $leftValue += $delta;
                 $rightValue += $delta;
             }