chore: update test scripts for improved test coverage

Signed-off-by: Sam Poyigi <[email protected]>
tastyigniter · Jan 25, 2025 · 8ee2bcf · 8ee2bcf
1 parent 2949f26
commit 8ee2bcf
Show file tree

Hide file tree

Showing 218 changed files with 16,701 additions and 1,002 deletions.
diff --git a/phpstan.neon.dist b/phpstan.neon.dist
@@ -1,13 +1,14 @@
 includes:
+    - ./vendor/larastan/larastan/extension.neon
     - phpstan-baseline.neon
 
 parameters:
     level: 5
     paths:
-        - src/
-        - config/
-        - database/
-        - resources/
+        - src
+        - config
+        - database
+        - resources
     treatPhpDocTypesAsCertain: false
 #    ignoreErrors:
 #        - '#PHPDoc tag @var#'

diff --git a/phpunit.xml.dist b/phpunit.xml.dist
@@ -1,18 +1,21 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <phpunit xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
-         xsi:noNamespaceSchemaLocation="https://schema.phpunit.de/10.5/phpunit.xsd"
-         backupGlobals="false"
+         xsi:noNamespaceSchemaLocation="vendor/phpunit/phpunit/phpunit.xsd"
          bootstrap="vendor/autoload.php"
          colors="true"
-         processIsolation="false"
-         stopOnFailure="false"
          cacheDirectory=".phpunit.cache"
-         backupStaticProperties="false">
+>
     <testsuites>
         <testsuite name="Tests">
-            <directory suffix="Test.php">./tests</directory>
+            <directory>tests</directory>
         </testsuite>
     </testsuites>
+    <coverage/>
+    <source>
+        <include>
+            <directory suffix=".php">./src</directory>
+        </include>
+    </source>
     <php>
         <env name="APP_ENV" value="testing"/>
         <env name="APP_DEBUG" value="true"/>

diff --git a/rector.php b/rector.php
@@ -0,0 +1,15 @@
+<?php
+
+declare(strict_types=1);
+
+use Rector\Config\RectorConfig;
+use Rector\TypeDeclaration\Rector\ClassMethod\ReturnTypeFromStrictNewArrayRector;
+
+return RectorConfig::configure()
+    ->withPaths([__DIR__.'/src', __DIR__.'/tests'])
+    ->withSkip([
+        ReturnTypeFromStrictNewArrayRector::class,
+    ])
+    ->withTypeCoverageLevel(1)
+    ->withDeadCodeLevel(0)
+    ->withCodeQualityLevel(0);
diff --git a/tests/Pest.php b/tests/Pest.php
@@ -1,22 +1,20 @@
 <?php
 
-use Illuminate\Http\Request;
-use Illuminate\Routing\Route;
+use Igniter\User\Models\User;
 
 uses(Igniter\Tests\TestCase::class)->in(__DIR__.'/src');
 
+pest()->group('admin')->in('src/Admin');
+pest()->group('main')->in('src/Main');
+pest()->group('system')->in('src/System');
+pest()->group('flame')->in('src/Flame');
+
 function testThemePath()
 {
     return realpath(__DIR__.'/resources/themes/tests-theme');
 }
 
-function createRequest($uri, $routeName)
+function actingAsSuperUser()
 {
-    $request = new Request([], [], [], [], [], ['REQUEST_URI' => $uri]);
-
-    $request->setRouteResolver(function() use ($uri, $routeName, $request) {
-        return (new Route('GET', $uri, ['as' => $routeName]))->bind($request);
-    });
-
-    return $request;
+    return test()->actingAs(User::factory()->superUser()->create(), 'igniter-admin');
 }
diff --git a/tests/resources/css/style.css b/tests/resources/css/style.css
diff --git a/tests/resources/js/script.js b/tests/resources/js/script.js
diff --git a/tests/resources/models/test_settings.php b/tests/resources/models/test_settings.php
@@ -0,0 +1,14 @@
+<?php
+
+return [
+    'form' => [
+        'toolbar' => [],
+        'fields' => [
+            'name' => [
+                'label' => 'Name',
+                'type' => 'text',
+                'span' => 'left',
+            ],
+        ],
+    ],
+];
diff --git a/tests/resources/scss/style.scss b/tests/resources/scss/style.scss
@@ -0,0 +1,4 @@
+body {
+  background-color: #f00;
+  color: #fff;
+}
diff --git a/tests/resources/themes/tests-theme/_content/test-content.blade.php b/tests/resources/themes/tests-theme/_content/test-content.blade.php
@@ -0,0 +1 @@
+This is a test content
diff --git a/tests/resources/themes/tests-theme/_layouts/default-with-lifecycle.blade.php b/tests/resources/themes/tests-theme/_layouts/default-with-lifecycle.blade.php
@@ -0,0 +1,12 @@
+---
+name: Default Layout
+
+'[testComponentWithLifecycle]':
+---
+<html>
+<body>
+
+@themePage
+
+</body>
+</html>
diff --git a/tests/resources/themes/tests-theme/_layouts/default.blade.php b/tests/resources/themes/tests-theme/_layouts/default.blade.php
@@ -0,0 +1,12 @@
+---
+name: Default Layout
+
+'[testComponent]':
+---
+<html>
+<body>
+
+@themePage
+
+</body>
+</html>
diff --git a/tests/resources/themes/tests-theme/_meta/blueprint.json b/tests/resources/themes/tests-theme/_meta/blueprint.json
diff --git a/tests/resources/themes/tests-theme/_meta/fields.php b/tests/resources/themes/tests-theme/_meta/fields.php
@@ -0,0 +1,17 @@
+<?php
+
+return [
+    'form' => [
+        'general' => [
+            'title' => 'lang:igniter.tests::default.theme.label_title',
+            'fields' => [
+                'field1' => [
+                    'label' => 'lang:igniter.tests::default.theme.label_field1',
+                    'type' => 'text',
+                    'span' => 'left',
+                    'cssClass' => 'flex-width',
+                ],
+            ],
+        ],
+    ],
+];
diff --git a/tests/resources/themes/tests-theme/_pages/components.blade.php b/tests/resources/themes/tests-theme/_pages/components.blade.php
@@ -1,3 +1,10 @@
+---
+title: Components
+description: ''
+permalink: /components
+layout: default
+'[testComponent]': []
+---
 <div class="container">
     <div class="row">
         @push('scripts')
@@ -16,8 +23,8 @@
         <p>This is a logged customer</p>
         @endmainauth
 
-        @partialIf('scriprts')
+        @partialIf('scripts')
 
-        @component('account')
+        @component('testComponent')
     </div>
 </div>
diff --git a/tests/resources/themes/tests-theme/_pages/layout-with-lifecycle.blade.php b/tests/resources/themes/tests-theme/_pages/layout-with-lifecycle.blade.php
@@ -0,0 +1,13 @@
+---
+title: Layout with Lifecycle
+description: ''
+permalink: /layout-with-lifecycle
+layout: default-with-lifecycle
+
+'[testComponent]':
+---
+<div class="container">
+    <div class="row">
+        <p>This is a logged customer</p>
+    </div>
+</div>
diff --git a/tests/resources/themes/tests-theme/_pages/page-with-lifecycle.blade.php b/tests/resources/themes/tests-theme/_pages/page-with-lifecycle.blade.php
@@ -0,0 +1,13 @@
+---
+title: Page with Lifecycle
+description: ''
+permalink: /page-with-lifecycle
+layout: default
+
+'[testComponentWithLifecycle]':
+---
+<div class="container">
+    <div class="row">
+        <p>This is a logged customer</p>
+    </div>
+</div>
diff --git a/tests/resources/themes/tests-theme/_partials/test-partial.blade.php b/tests/resources/themes/tests-theme/_partials/test-partial.blade.php
@@ -0,0 +1 @@
+This is a test partial content
diff --git a/tests/resources/themes/tests-theme/public/style.css b/tests/resources/themes/tests-theme/public/style.css
diff --git a/tests/resources/views/_components/testcomponent/default.blade.php b/tests/resources/views/_components/testcomponent/default.blade.php
@@ -0,0 +1 @@
+{{ 'This is a test component partial content' }}
diff --git a/tests/resources/views/_mail/test_layout.blade.php b/tests/resources/views/_mail/test_layout.blade.php
@@ -0,0 +1,3 @@
+name = 'Test Layout'
+===
+This is a test layout!
diff --git a/tests/resources/views/_mail/test_partial.blade.php b/tests/resources/views/_mail/test_partial.blade.php
@@ -0,0 +1,3 @@
+name = 'Test Partial'
+===
+This is a test partial!
diff --git a/tests/resources/views/_partials/test-partial.blade.php b/tests/resources/views/_partials/test-partial.blade.php
@@ -1 +1 @@
-This is a test partial view
+This is a test partial content
diff --git a/tests/resources/views/layout.blade.php b/tests/resources/views/layout.blade.php
@@ -0,0 +1,7 @@
+<html>
+<body>
+
+{{Template::getBlock('body')}}
+
+</body>
+</html>
diff --git a/tests/resources/views/test.blade.php b/tests/resources/views/test.blade.php
@@ -1,3 +1 @@
-<html>
-<body>Hello World!</body>
-</html>
+This is a test view content
diff --git a/tests/resources/views/testcontroller/index.blade.php b/tests/resources/views/testcontroller/index.blade.php
diff --git a/tests/resources/views/testcontroller/view-with-exception.blade.php b/tests/resources/views/testcontroller/view-with-exception.blade.php
@@ -0,0 +1,3 @@
+<?php
+    throw new Exception('This is a test exception');
+?>
diff --git a/tests/resources/views/testcontroller/view-with-throwable.blade.php b/tests/resources/views/testcontroller/view-with-throwable.blade.php
@@ -0,0 +1,3 @@
+<?php
+    throw new Error('This is a test error');;
+?>
diff --git a/tests/src/Admin/BulkActionWidgets/DeleteTest.php b/tests/src/Admin/BulkActionWidgets/DeleteTest.php
@@ -23,3 +23,18 @@
 
     expect(StatusHistory::count())->toBe(0);
 });
+
+it('does noting when records is empty', function() {
+    $actionButton = new ToolbarButton('delete');
+    $actionButton->displayAs('link', []);
+
+    $controller = resolve(TestController::class);
+    $widget = new Delete($controller, $actionButton, []);
+    $widget->code = $actionButton->name;
+
+    expect(StatusHistory::count())->toBe(0);
+
+    $widget->handleAction([], StatusHistory::get());
+
+    expect(StatusHistory::count())->toBe(0);
+});
diff --git a/tests/src/Admin/BulkActionWidgets/StatusTest.php b/tests/src/Admin/BulkActionWidgets/StatusTest.php
@@ -27,3 +27,20 @@
 
     expect(StatusHistory::where($statusColumn, true)->count())->toBe(10);
 });
+
+it('does noting when records is empty', function() {
+    $statusColumn = 'notify';
+
+    $actionButton = new ToolbarButton('status');
+    $actionButton->displayAs('link', []);
+
+    $controller = resolve(TestController::class);
+    $widget = new Status($controller, $actionButton, ['statusColumn' => $statusColumn]);
+    $widget->code = $actionButton->name;
+
+    expect(StatusHistory::count())->toBe(0);
+
+    $widget->handleAction(['code' => 'action.disable'], StatusHistory::get());
+
+    expect(StatusHistory::count())->toBe(0);
+});
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1 @@
		{{ 'This is a test component partial content' }}
Original file line number	Diff line number	Diff line change
		@@ -1 +1 @@
		This is a test partial view
		This is a test partial content