Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Drop support for Laravel 8 and 9 #629

Merged
merged 1 commit into from
May 27, 2024
Merged

Drop support for Laravel 8 and 9 #629

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
36 changes: 2 additions & 34 deletions .github/workflows/tests.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,44 +12,12 @@ jobs:
strategy:
fail-fast: true
matrix:
php: [7.3, 7.4, "8.0", 8.1, 8.2, 8.3]
laravel: [8, 9, 10, 11]
php: [8.1, 8.2, 8.3]
laravel: [10, 11]
stability: ["prefer-lowest", "prefer-stable"]
exclude:
- php: 7.3
laravel: 9
- php: 7.3
laravel: 10
- php: 7.3
laravel: 11
- php: 7.4
laravel: 9
- php: 7.4
laravel: 10
- php: 7.4
laravel: 11
- php: '8.0'
laravel: 10
- php: '8.0'
laravel: 11
- php: 8.1
laravel: 6
- php: 8.1
laravel: 7
- php: 8.1
laravel: 11
- php: 8.2
laravel: 6
- php: 8.2
laravel: 7
- php: 8.2
laravel: 8
- php: 8.3
laravel: 6
- php: 8.3
laravel: 7
- php: 8.3
laravel: 8

name: PHP ${{ matrix.php }} - Laravel ${{ matrix.laravel }} (w/ ${{ matrix.stability }})
steps:
Expand Down
13 changes: 8 additions & 5 deletions composer.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,10 @@
"name": "inertiajs/inertia-laravel",
"type": "library",
"description": "The Laravel adapter for Inertia.js.",
"keywords": ["laravel", "inertia"],
"keywords": [
"laravel",
"inertia"
],
"license": "MIT",
"authors": [
{
Expand All @@ -25,16 +28,16 @@
}
},
"require": {
"php": "^7.3|~8.0.0|~8.1.0|~8.2.0|~8.3.0",
"php": "^8.1.0",
"ext-json": "*",
"laravel/framework": "^8.74|^9.0|^10.0|^11.0",
"laravel/framework": "^10.0|^11.0",
"symfony/console": "^5.3|^6.0|^7.0"
},
"require-dev": {
"roave/security-advisories": "dev-master",
"orchestra/testbench": "^6.4|^7.0|^8.0|^9.0",
"orchestra/testbench": "^8.0|^9.0",
"mockery/mockery": "^1.3.3",
"phpunit/phpunit": "^8.0|^9.5.8|^10.4",
"phpunit/phpunit": "^10.4|^11.0",
"laravel/pint": "^1.16"
},
"suggest": {
Expand Down
42 changes: 14 additions & 28 deletions tests/Testing/AssertableInertiaTest.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -8,8 +8,7 @@

class AssertableInertiaTest extends TestCase
{
/** @test */
public function the_view_is_served_by_inertia(): void
public function test_the_view_is_served_by_inertia(): void
{
$response = $this->makeMockRequest(
Inertia::render('foo')
Expand All @@ -18,8 +17,7 @@ public function the_view_is_served_by_inertia(): void
$response->assertInertia();
}

/** @test */
public function the_view_is_not_served_by_inertia(): void
public function test_the_view_is_not_served_by_inertia(): void
{
$response = $this->makeMockRequest(view('welcome'));
$response->assertOk(); // Make sure we can render the built-in Orchestra 'welcome' view..
Expand All @@ -30,8 +28,7 @@ public function the_view_is_not_served_by_inertia(): void
$response->assertInertia();
}

/** @test */
public function the_component_matches(): void
public function test_the_component_matches(): void
{
$response = $this->makeMockRequest(
Inertia::render('foo')
Expand All @@ -42,8 +39,7 @@ public function the_component_matches(): void
});
}

/** @test */
public function the_component_does_not_match(): void
public function test_the_component_does_not_match(): void
{
$response = $this->makeMockRequest(
Inertia::render('foo')
Expand All @@ -57,8 +53,7 @@ public function the_component_does_not_match(): void
});
}

/** @test */
public function the_component_exists_on_the_filesystem(): void
public function test_the_component_exists_on_the_filesystem(): void
{
$response = $this->makeMockRequest(
Inertia::render('Stubs/ExamplePage')
Expand All @@ -70,8 +65,7 @@ public function the_component_exists_on_the_filesystem(): void
});
}

/** @test */
public function the_component_does_not_exist_on_the_filesystem(): void
public function test_the_component_does_not_exist_on_the_filesystem(): void
{
$response = $this->makeMockRequest(
Inertia::render('foo')
Expand All @@ -86,8 +80,7 @@ public function the_component_does_not_exist_on_the_filesystem(): void
});
}

/** @test */
public function it_can_force_enable_the_component_file_existence(): void
public function test_it_can_force_enable_the_component_file_existence(): void
{
$response = $this->makeMockRequest(
Inertia::render('foo')
Expand All @@ -102,8 +95,7 @@ public function it_can_force_enable_the_component_file_existence(): void
});
}

/** @test */
public function it_can_force_disable_the_component_file_existence_check(): void
public function test_it_can_force_disable_the_component_file_existence_check(): void
{
$response = $this->makeMockRequest(
Inertia::render('foo')
Expand All @@ -116,8 +108,7 @@ public function it_can_force_disable_the_component_file_existence_check(): void
});
}

/** @test */
public function the_component_does_not_exist_on_the_filesystem_when_it_does_not_exist_relative_to_any_of_the_given_paths(): void
public function test_the_component_does_not_exist_on_the_filesystem_when_it_does_not_exist_relative_to_any_of_the_given_paths(): void
{
$response = $this->makeMockRequest(
Inertia::render('fixtures/ExamplePage')
Expand All @@ -133,8 +124,7 @@ public function the_component_does_not_exist_on_the_filesystem_when_it_does_not_
});
}

/** @test */
public function the_component_does_not_exist_on_the_filesystem_when_it_does_not_have_one_of_the_configured_extensions(): void
public function test_the_component_does_not_exist_on_the_filesystem_when_it_does_not_have_one_of_the_configured_extensions(): void
{
$response = $this->makeMockRequest(
Inertia::render('fixtures/ExamplePage')
Expand All @@ -150,8 +140,7 @@ public function the_component_does_not_exist_on_the_filesystem_when_it_does_not_
});
}

/** @test */
public function the_page_url_matches(): void
public function test_the_page_url_matches(): void
{
$response = $this->makeMockRequest(
Inertia::render('foo')
Expand All @@ -162,8 +151,7 @@ public function the_page_url_matches(): void
});
}

/** @test */
public function the_page_url_does_not_match(): void
public function test_the_page_url_does_not_match(): void
{
$response = $this->makeMockRequest(
Inertia::render('foo')
Expand All @@ -177,8 +165,7 @@ public function the_page_url_does_not_match(): void
});
}

/** @test */
public function the_asset_version_matches(): void
public function test_the_asset_version_matches(): void
{
Inertia::version('example-version');

Expand All @@ -191,8 +178,7 @@ public function the_asset_version_matches(): void
});
}

/** @test */
public function the_asset_version_does_not_match(): void
public function test_the_asset_version_does_not_match(): void
{
Inertia::version('example-version');

Expand Down
9 changes: 3 additions & 6 deletions tests/Testing/TestResponseMacrosTest.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -8,8 +8,7 @@

class TestResponseMacrosTest extends TestCase
{
/** @test */
public function it_can_make_inertia_assertions(): void
public function test_it_can_make_inertia_assertions(): void
{
$response = $this->makeMockRequest(
Inertia::render('foo')
Expand All @@ -24,8 +23,7 @@ public function it_can_make_inertia_assertions(): void
$this->assertTrue($success);
}

/** @test */
public function it_preserves_the_ability_to_continue_chaining_laravel_test_response_calls(): void
public function test_it_preserves_the_ability_to_continue_chaining_laravel_test_response_calls(): void
{
$response = $this->makeMockRequest(
Inertia::render('foo')
Expand All @@ -37,8 +35,7 @@ public function it_preserves_the_ability_to_continue_chaining_laravel_test_respo
);
}

/** @test */
public function it_can_retrieve_the_inertia_page(): void
public function test_it_can_retrieve_the_inertia_page(): void
{
$response = $this->makeMockRequest(
Inertia::render('foo', ['bar' => 'baz'])
Expand Down
Loading