Skip to content

Commit

Permalink
Remove RefreshDatabase trait from tests as tests handle in-package mi…
Browse files Browse the repository at this point in the history
…grations

Update github workflow matrix from php 8.0 and 8.1 -> 8.2 only, and add Laravel v10
  • Loading branch information
hayleyberryl2s committed Nov 19, 2024
1 parent f725e68 commit 64391f9
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 7 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/laravel-report-build-test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -16,8 +16,8 @@ jobs:
strategy:
fail-fast: true
matrix:
php: ["8.0", "8.1"]
laravel: ["^9.0"]
php: ["8.2"]
laravel: ["^9.0", "^10.0"]

name: PHP ${{ matrix.php }} - Laravel ${{ matrix.laravel }}

Expand Down
3 changes: 0 additions & 3 deletions tests/LaravelTestCase.php
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,6 @@
namespace MBLSolutions\Report\Tests;

use Illuminate\Foundation\Application;
use Illuminate\Foundation\Testing\RefreshDatabase;
use Illuminate\Support\Collection;
use Illuminate\Support\Str;
use Maatwebsite\Excel\ExcelServiceProvider;
Expand All @@ -17,8 +16,6 @@

class LaravelTestCase extends OTBTestCase
{
use RefreshDatabase;

/** {@inheritdoc} **/
protected function setUp(): void
{
Expand Down
2 changes: 0 additions & 2 deletions tests/Unit/Repositories/ScheduledReportRepositoryTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -14,8 +14,6 @@

class ScheduledReportRepositoryTest extends LaravelTestCase
{
use RefreshDatabase;

protected ScheduledReportRepository $repository;

protected Report $report;
Expand Down

0 comments on commit 64391f9

Please sign in to comment.