Run phpunit code quality rector

jbtronics · jbtronics · commit f3ad3c1ffe91 · 2025-07-14T00:38:54.000+02:00
diff --git a/rector.php b/rector.php
@@ -35,6 +35,9 @@
         PHPUnitSetList::ANNOTATIONS_TO_ATTRIBUTES,
         PHPUnitSetList::PHPUNIT_90,
         PHPUnitSetList::PHPUNIT_110,
+        PHPUnitSetList::PHPUNIT_CODE_QUALITY,
+
+        
     ])
 
     ->withRules([
diff --git a/tests/DataTables/Filters/Constraints/FilterTraitTest.php b/tests/DataTables/Filters/Constraints/FilterTraitTest.php
@@ -52,7 +52,7 @@ public function isAggregateFunctionStringDataProvider(): iterable
     #[DataProvider('isAggregateFunctionStringDataProvider')]
     public function testIsAggregateFunctionString(bool $expected, string $input): void
     {
-        $this->assertEquals($expected, $this->isAggregateFunctionString($input));
+        $this->assertSame($expected, $this->isAggregateFunctionString($input));
     }
 
 }
diff --git a/tests/Entity/Attachments/AttachmentTest.php b/tests/Entity/Attachments/AttachmentTest.php
@@ -231,7 +231,7 @@ public function testSetExternalPath(): void
         //Ensure that changing the external path does reset the internal one
         $attachment->setInternalPath('%MEDIA%/foo/bar.txt');
         $attachment->setExternalPath('https://example.de');
-        $this->assertSame(null, $attachment->getInternalPath());
+        $this->assertNull($attachment->getInternalPath());
 
         //Ensure that setting the same value to the external path again doesn't reset the internal one
         $attachment->setExternalPath('https://google.de');
diff --git a/tests/Entity/Parts/PartTest.php b/tests/Entity/Parts/PartTest.php
@@ -41,7 +41,7 @@ public function testAddRemovePartLot(): void
         $lot = new PartLot();
         $part->addPartLot($lot);
         $this->assertSame($part, $lot->getPart());
-        $this->assertSame(1, $part->getPartLots()->count());
+        $this->assertCount(1, $part->getPartLots());
 
         //Remove element
         $part->removePartLot($lot);
diff --git a/tests/Entity/PriceSystem/OrderdetailTest.php b/tests/Entity/PriceSystem/OrderdetailTest.php
@@ -38,7 +38,7 @@ public function testAddRemovePricdetails(): void
         $pricedetail = new Pricedetail();
         $orderdetail->addPricedetail($pricedetail);
         $this->assertSame($orderdetail, $pricedetail->getOrderdetail());
-        $this->assertSame(1, $orderdetail->getPricedetails()->count());
+        $this->assertCount(1, $orderdetail->getPricedetails());
 
         //After removal of the pricedetail, the orderdetail must be empty again
         $orderdetail->removePricedetail($pricedetail);
diff --git a/tests/Entity/UserSystem/ApiTokenTypeTest.php b/tests/Entity/UserSystem/ApiTokenTypeTest.php
@@ -35,7 +35,7 @@ public function testGetTokenPrefix(): void
 
     public function testGetTypeFromToken(): void
     {
-        $this->assertEquals(ApiTokenType::PERSONAL_ACCESS_TOKEN, ApiTokenType::getTypeFromToken('tcp_123'));
+        $this->assertSame(ApiTokenType::PERSONAL_ACCESS_TOKEN, ApiTokenType::getTypeFromToken('tcp_123'));
     }
 
     public function testGetTypeFromTokenInvalid(): void
diff --git a/tests/Helpers/Projects/ProjectBuildRequestTest.php b/tests/Helpers/Projects/ProjectBuildRequestTest.php
@@ -33,9 +33,6 @@
 class ProjectBuildRequestTest extends TestCase
 {
 
-    /** @var MeasurementUnit $float_unit */
-    private MeasurementUnit $float_unit;
-
     /** @var Project */
     private Project $project1;
     /** @var ProjectBOMEntry */
@@ -49,30 +46,25 @@ class ProjectBuildRequestTest extends TestCase
     private PartLot $lot1b;
     private PartLot $lot2;
 
-    /** @var Part */
-    private Part $part1;
-    /** @var Part */
-    private Part $part2;
-
 
     public function setUp(): void
     {
-        $this->float_unit = new MeasurementUnit();
-        $this->float_unit->setName('float');
-        $this->float_unit->setUnit('f');
-        $this->float_unit->setIsInteger(false);
-        $this->float_unit->setUseSIPrefix(true);
+        $float_unit = new MeasurementUnit();
+        $float_unit->setName('float');
+        $float_unit->setUnit('f');
+        $float_unit->setIsInteger(false);
+        $float_unit->setUseSIPrefix(true);
 
         //Setup some example parts and part lots
-        $this->part1 = new Part();
-        $this->part1->setName('Part 1');
+        $part1 = new Part();
+        $part1->setName('Part 1');
         $this->lot1a = new class extends PartLot {
             public function getID(): ?int
             {
                 return 1;
             }
         };
-        $this->part1->addPartLot($this->lot1a);
+        $part1->addPartLot($this->lot1a);
         $this->lot1a->setAmount(10);
         $this->lot1a->setDescription('Lot 1a');
 
@@ -82,25 +74,25 @@ public function getID(): ?int
                 return 2;
             }
         };
-        $this->part1->addPartLot($this->lot1b);
+        $part1->addPartLot($this->lot1b);
         $this->lot1b->setAmount(20);
         $this->lot1b->setDescription('Lot 1b');
 
-        $this->part2 = new Part();
+        $part2 = new Part();
 
-        $this->part2->setName('Part 2');
-        $this->part2->setPartUnit($this->float_unit);
+        $part2->setName('Part 2');
+        $part2->setPartUnit($float_unit);
         $this->lot2 = new PartLot();
-        $this->part2->addPartLot($this->lot2);
+        $part2->addPartLot($this->lot2);
         $this->lot2->setAmount(2.5);
         $this->lot2->setDescription('Lot 2');
 
         $this->bom_entry1a = new ProjectBOMEntry();
-        $this->bom_entry1a->setPart($this->part1);
+        $this->bom_entry1a->setPart($part1);
         $this->bom_entry1a->setQuantity(2);
 
         $this->bom_entry1b = new ProjectBOMEntry();
-        $this->bom_entry1b->setPart($this->part2);
+        $this->bom_entry1b->setPart($part2);
         $this->bom_entry1b->setQuantity(1.5);
 
         $this->bom_entry1c = new ProjectBOMEntry();
diff --git a/tests/Repository/NamedDBElementRepositoryTest.php b/tests/Repository/NamedDBElementRepositoryTest.php
@@ -32,7 +32,6 @@
  */
 class NamedDBElementRepositoryTest extends WebTestCase
 {
-    private $entityManager;
     /**
      * @var StructuralDBElementRepository
      */
@@ -42,11 +41,11 @@ protected function setUp(): void
     {
         $kernel = self::bootKernel();
 
-        $this->entityManager = $kernel->getContainer()
+        $entityManager = $kernel->getContainer()
             ->get('doctrine')
             ->getManager();
 
-        $this->repo = $this->entityManager->getRepository(User::class);
+        $this->repo = $entityManager->getRepository(User::class);
     }
 
     public function testGetGenericNodeTree(): void
diff --git a/tests/Repository/StructuralDBElementRepositoryTest.php b/tests/Repository/StructuralDBElementRepositoryTest.php
@@ -32,7 +32,6 @@
  */
 class StructuralDBElementRepositoryTest extends WebTestCase
 {
-    private $entityManager;
     /**
      * @var StructuralDBElementRepository
      */
@@ -42,11 +41,11 @@ protected function setUp(): void
     {
         $kernel = self::bootKernel();
 
-        $this->entityManager = $kernel->getContainer()
+        $entityManager = $kernel->getContainer()
             ->get('doctrine')
             ->getManager();
 
-        $this->repo = $this->entityManager->getRepository(AttachmentType::class);
+        $this->repo = $entityManager->getRepository(AttachmentType::class);
     }
 
     public function testFindRootNodes(): void
diff --git a/tests/Repository/UserRepositoryTest.php b/tests/Repository/UserRepositoryTest.php
@@ -30,7 +30,6 @@
 class UserRepositoryTest extends WebTestCase
 {
 
-    private $entityManager;
     /**
      * @var UserRepository
      */
@@ -40,11 +39,11 @@ protected function setUp(): void
     {
         $kernel = self::bootKernel();
 
-        $this->entityManager = $kernel->getContainer()
+        $entityManager = $kernel->getContainer()
             ->get('doctrine')
             ->getManager();
 
-        $this->repo = $this->entityManager->getRepository(User::class);
+        $this->repo = $entityManager->getRepository(User::class);
     }
 
 
diff --git a/tests/Services/EntityMergers/Mergers/PartMergerTest.php b/tests/Services/EntityMergers/Mergers/PartMergerTest.php
@@ -178,7 +178,9 @@ public function testMergeOfPartLots(): void
         //But the new lots, should be assigned to the target part and contain the same info
         $clone3 = $merged->getPartLots()->get(2);
         $clone4 = $merged->getPartLots()->get(3);
+        $this->assertInstanceOf(PartLot::class, $clone3);
         $this->assertSame($merged, $clone3->getPart());
+        $this->assertInstanceOf(PartLot::class, $clone4);
         $this->assertSame($merged, $clone4->getPart());
 
     }
diff --git a/tests/Services/InfoProviderSystem/DTOtoEntityConverterTest.php b/tests/Services/InfoProviderSystem/DTOtoEntityConverterTest.php
@@ -22,6 +22,7 @@
  */
 namespace App\Tests\Services\InfoProviderSystem;
 
+use App\Entity\PriceInformations\Currency;
 use App\Entity\Attachments\AttachmentType;
 use App\Entity\Parts\ManufacturingStatus;
 use App\Services\InfoProviderSystem\DTOs\FileDTO;
@@ -83,6 +84,7 @@ public function testConvertPriceOtherCurrency(): void
 
         //For non-base currencies, a new currency entity is created
         $currency = $entity->getCurrency();
+        $this->assertInstanceOf(Currency::class, $currency);
         $this->assertEquals($dto->currency_iso_code, $currency->getIsoCode());
     }
 
diff --git a/tests/Services/LabelSystem/BarcodeScanner/EIGP114BarcodeScanResultTest.php b/tests/Services/LabelSystem/BarcodeScanner/EIGP114BarcodeScanResultTest.php
@@ -53,7 +53,7 @@ public function testGuessBarcodeVendor(): void
             '4L' => 'US',
             '13Z' => 'Digi-Key',
         ]);
-        $this->assertEquals('digikey', $barcode->guessBarcodeVendor());
+        $this->assertSame('digikey', $barcode->guessBarcodeVendor());
 
         //Mouser barcode:
         $barcode = new EIGP114BarcodeScanResult([
@@ -66,7 +66,7 @@ public function testGuessBarcodeVendor(): void
             '1V' => 'Mouser',
         ]);
 
-        $this->assertEquals('mouser', $barcode->guessBarcodeVendor());
+        $this->assertSame('mouser', $barcode->guessBarcodeVendor());
 
         //Farnell barcode:
         $barcode = new EIGP114BarcodeScanResult([
@@ -79,7 +79,7 @@ public function testGuessBarcodeVendor(): void
             '3P' => 'Farnell',
         ]);
 
-        $this->assertEquals('element14', $barcode->guessBarcodeVendor());
+        $this->assertSame('element14', $barcode->guessBarcodeVendor());
     }
 
     public function testIsFormat06Code(): void
@@ -104,7 +104,7 @@ public function testParseFormat06CodeInvalid(): void
     public function testParseFormat06Code(): void
     {
         $barcode = EIGP114BarcodeScanResult::parseFormat06Code("[)>\x1E06\x1DP596-777A1-ND\x1D1PXAF4444\x1DQ3\x1D10D1452\x1D1TBF1103\x1D4LUS\x1E\x04");
-        $this->assertEquals([
+        $this->assertSame([
             'P' => '596-777A1-ND',
             '1P' => 'XAF4444',
             'Q' => '3',
@@ -125,32 +125,32 @@ public function testDataParsing(): void
             '4L' => 'US',
         ]);
 
-        $this->assertEquals('596-777A1-ND', $barcode->customerPartNumber);
-        $this->assertEquals('XAF4444', $barcode->supplierPartNumber);
-        $this->assertEquals(3, $barcode->quantity);
-        $this->assertEquals('1452', $barcode->alternativeDateCode);
-        $this->assertEquals('BF1103', $barcode->lotCode);
-        $this->assertEquals('US', $barcode->countryOfOrigin);
+        $this->assertSame('596-777A1-ND', $barcode->customerPartNumber);
+        $this->assertSame('XAF4444', $barcode->supplierPartNumber);
+        $this->assertSame(3, $barcode->quantity);
+        $this->assertSame('1452', $barcode->alternativeDateCode);
+        $this->assertSame('BF1103', $barcode->lotCode);
+        $this->assertSame('US', $barcode->countryOfOrigin);
     }
 
     public function testDigikeyParsing(): void
     {
         $barcode = EIGP114BarcodeScanResult::parseFormat06Code("[)>\x1e06\x1dPQ1045-ND\x1d1P364019-01\x1d30PQ1045-ND\x1dK12432 TRAVIS FOSS P\x1d1K85732873\x1d10K103332956\x1d9D231013\x1d1TQJ13P\x1d11K1\x1d4LTW\x1dQ3\x1d11ZPICK\x1d12Z7360988\x1d13Z999999\x1d20Z0000000000000000000000000000000000000000000000000000000000000000000000000000000000000");
 
-        $this->assertEquals('digikey', $barcode->guessBarcodeVendor());
-
-        $this->assertEquals('Q1045-ND', $barcode->customerPartNumber);
-        $this->assertEquals('364019-01', $barcode->supplierPartNumber);
-        $this->assertEquals(3, $barcode->quantity);
-        $this->assertEquals('231013', $barcode->dateCode);
-        $this->assertEquals('QJ13P', $barcode->lotCode);
-        $this->assertEquals('TW', $barcode->countryOfOrigin);
-        $this->assertEquals('Q1045-ND', $barcode->digikeyPartNumber);
-        $this->assertEquals('85732873', $barcode->digikeySalesOrderNumber);
-        $this->assertEquals('103332956', $barcode->digikeyInvoiceNumber);
-        $this->assertEquals('PICK', $barcode->digikeyLabelType);
-        $this->assertEquals('7360988', $barcode->digikeyPartID);
-        $this->assertEquals('999999', $barcode->digikeyNA);
-        $this->assertEquals('0000000000000000000000000000000000000000000000000000000000000000000000000000000000000', $barcode->digikeyPadding);
+        $this->assertSame('digikey', $barcode->guessBarcodeVendor());
+
+        $this->assertSame('Q1045-ND', $barcode->customerPartNumber);
+        $this->assertSame('364019-01', $barcode->supplierPartNumber);
+        $this->assertSame(3, $barcode->quantity);
+        $this->assertSame('231013', $barcode->dateCode);
+        $this->assertSame('QJ13P', $barcode->lotCode);
+        $this->assertSame('TW', $barcode->countryOfOrigin);
+        $this->assertSame('Q1045-ND', $barcode->digikeyPartNumber);
+        $this->assertSame('85732873', $barcode->digikeySalesOrderNumber);
+        $this->assertSame('103332956', $barcode->digikeyInvoiceNumber);
+        $this->assertSame('PICK', $barcode->digikeyLabelType);
+        $this->assertSame('7360988', $barcode->digikeyPartID);
+        $this->assertSame('999999', $barcode->digikeyNA);
+        $this->assertSame('0000000000000000000000000000000000000000000000000000000000000000000000000000000000000', $barcode->digikeyPadding);
     }
 }
diff --git a/tests/Services/LogSystem/TimeTravelTest.php b/tests/Services/LogSystem/TimeTravelTest.php
@@ -47,7 +47,7 @@ public function testUndeleteEntity(): void
         $undeletedCategory = $this->service->undeleteEntity(Category::class, 100);
 
         $this->assertInstanceOf(Category::class, $undeletedCategory);
-        $this->assertEquals(100, $undeletedCategory->getId());
+        $this->assertSame(100, $undeletedCategory->getId());
     }
 
     public function testApplyEntry(): void
@@ -65,8 +65,8 @@ public function testApplyEntry(): void
 
         $this->service->applyEntry($category, $logEntry);
 
-        $this->assertEquals('Old Category', $category->getName());
-        $this->assertEquals('Old Comment', $category->getComment());
+        $this->assertSame('Old Category', $category->getName());
+        $this->assertSame('Old Comment', $category->getComment());
     }
 
     public function testRevertEntityToTimestamp(): void
diff --git a/tests/Services/UserSystem/PermissionSchemaUpdaterTest.php b/tests/Services/UserSystem/PermissionSchemaUpdaterTest.php
@@ -90,9 +90,9 @@ public function testUpgradeSchemaToVersion1(): void
 
         //Do an upgrade and afterward the move, add, and withdraw permissions should be set to ALLOW
         self::assertTrue($this->service->upgradeSchema($user, 1));
-        self::assertEquals(PermissionData::ALLOW, $user->getPermissions()->getPermissionValue('parts_stock', 'move'));
-        self::assertEquals(PermissionData::ALLOW, $user->getPermissions()->getPermissionValue('parts_stock', 'add'));
-        self::assertEquals(PermissionData::ALLOW, $user->getPermissions()->getPermissionValue('parts_stock', 'withdraw'));
+        self::assertSame(PermissionData::ALLOW, $user->getPermissions()->getPermissionValue('parts_stock', 'move'));
+        self::assertSame(PermissionData::ALLOW, $user->getPermissions()->getPermissionValue('parts_stock', 'add'));
+        self::assertSame(PermissionData::ALLOW, $user->getPermissions()->getPermissionValue('parts_stock', 'withdraw'));
     }
 
     public function testUpgradeSchemaToVersion2(): void
@@ -106,9 +106,9 @@ public function testUpgradeSchemaToVersion2(): void
 
         //After the upgrade all operations should be available under the name "projects" with the same values
         self::assertTrue($this->service->upgradeSchema($user, 2));
-        self::assertEquals(PermissionData::ALLOW, $user->getPermissions()->getPermissionValue('projects', 'read'));
-        self::assertEquals(PermissionData::INHERIT, $user->getPermissions()->getPermissionValue('projects', 'edit'));
-        self::assertEquals(PermissionData::DISALLOW, $user->getPermissions()->getPermissionValue('projects', 'delete'));
+        self::assertSame(PermissionData::ALLOW, $user->getPermissions()->getPermissionValue('projects', 'read'));
+        self::assertSame(PermissionData::INHERIT, $user->getPermissions()->getPermissionValue('projects', 'edit'));
+        self::assertSame(PermissionData::DISALLOW, $user->getPermissions()->getPermissionValue('projects', 'delete'));
     }
 
     public function testUpgradeSchemaToVersion3(): void

Original file line number	Diff line number	Diff line change
`@@ -52,7 +52,7 @@ public function isAggregateFunctionStringDataProvider(): iterable`
`52`	`52`	`#[DataProvider('isAggregateFunctionStringDataProvider')]`
`53`	`53`	`public function testIsAggregateFunctionString(bool $expected, string $input): void`
`54`	`54`	`{`
`55`		`- $this->assertEquals($expected, $this->isAggregateFunctionString($input));`
	`55`	`+ $this->assertSame($expected, $this->isAggregateFunctionString($input));`
`56`	`56`	`}`
`57`	`57`
`58`	`58`	`}`
Original file line number	Diff line number	Diff line change
`@@ -35,7 +35,7 @@ public function testGetTokenPrefix(): void`
`35`	`35`
`36`	`36`	`public function testGetTypeFromToken(): void`
`37`	`37`	`{`
`38`		`- $this->assertEquals(ApiTokenType::PERSONAL_ACCESS_TOKEN, ApiTokenType::getTypeFromToken('tcp_123'));`
	`38`	`+ $this->assertSame(ApiTokenType::PERSONAL_ACCESS_TOKEN, ApiTokenType::getTypeFromToken('tcp_123'));`
`39`	`39`	`}`
`40`	`40`
`41`	`41`	`public function testGetTypeFromTokenInvalid(): void`
Original file line number	Diff line number	Diff line change
`@@ -32,7 +32,6 @@`
`32`	`32`	`*/`
`33`	`33`	`class NamedDBElementRepositoryTest extends WebTestCase`
`34`	`34`	`{`
`35`		`- private $entityManager;`
`36`	`35`	`/**`
`37`	`36`	`* @var StructuralDBElementRepository`
`38`	`37`	`*/`
`@@ -42,11 +41,11 @@ protected function setUp(): void`
`42`	`41`	`{`
`43`	`42`	`$kernel = self::bootKernel();`
`44`	`43`
`45`		`- $this->entityManager = $kernel->getContainer()`
	`44`	`+ $entityManager = $kernel->getContainer()`
`46`	`45`	`->get('doctrine')`
`47`	`46`	`->getManager();`
`48`	`47`
`49`		`- $this->repo = $this->entityManager->getRepository(User::class);`
	`48`	`+ $this->repo = $entityManager->getRepository(User::class);`
`50`	`49`	`}`
`51`	`50`
`52`	`51`	`public function testGetGenericNodeTree(): void`
Original file line number	Diff line number	Diff line change
`@@ -30,7 +30,6 @@`
`30`	`30`	`class UserRepositoryTest extends WebTestCase`
`31`	`31`	`{`
`32`	`32`
`33`		`- private $entityManager;`
`34`	`33`	`/**`
`35`	`34`	`* @var UserRepository`
`36`	`35`	`*/`
`@@ -40,11 +39,11 @@ protected function setUp(): void`
`40`	`39`	`{`
`41`	`40`	`$kernel = self::bootKernel();`
`42`	`41`
`43`		`- $this->entityManager = $kernel->getContainer()`
	`42`	`+ $entityManager = $kernel->getContainer()`
`44`	`43`	`->get('doctrine')`
`45`	`44`	`->getManager();`
`46`	`45`
`47`		`- $this->repo = $this->entityManager->getRepository(User::class);`
	`46`	`+ $this->repo = $entityManager->getRepository(User::class);`
`48`	`47`	`}`
`49`	`48`
`50`	`49`