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.

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

Failed test on igintion #18

Open
sandersjj opened this issue Oct 7, 2024 · 0 comments
Open

Failed test on igintion #18

sandersjj opened this issue Oct 7, 2024 · 0 comments

Comments

@sandersjj
Copy link

sandersjj commented Oct 7, 2024

Hello,

First of all thanks for making this package available.

I have tested this package in my codebase.
I am running a Laravel FilamentPhp application

PHP 8.2
Laravel v11.23
filament/filament ^3.2

I have tried to run this test

routeTesting('all routes')
    ->exclude(['admin*', 'livewire*', 'up', '/'])
    ->assertSuccessful();

However I am getting this error:
  ─────────────────────────────────────────────────────────────────────────────────────────
   FAILED  P\Tests\Pest > pest evaluable all routes with data set "(' g e t', ' ignition/health-check
  Expected response status code [>=200, <300] but received 404.
Failed asserting that false is true.

  at vendor/spatie/pest-plugin-route-testing/src/RouteTest.php:49
     45▕             })
     46▕             ->expect(fn (string $method, string $uri) => $this->{$method}($this->url));
     47▕
     48▕         foreach ($assertions as [$method, $parameters]) {
  ➜  49▕             $testResponse->{$method}(...$parameters);
     50▕         }
     51▕     }
     52▕ }
     53



  Tests:    1 failed, 7 skipped, 27 passed (52 assertions)
  Duration: 4.56s
  Parallel: 10 processes

Note the spaces in the first line in 'get'

I find this weird since the ignition routes are explicitly excluded in the constructor of the RouteTestingTestCall

I am not sure if there is a bug in here or that I am doing something wrong.

Thanks in advance for your reaction.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant