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

Fix vite manifest not found #33

Merged
merged 2 commits into from
May 10, 2023
Merged

Fix vite manifest not found #33

merged 2 commits into from
May 10, 2023

Conversation

hfagerlund
Copy link
Owner

@hfagerlund hfagerlund commented May 10, 2023

Fixes the error message displayed below (when running tests) -

ERROR:

$ ./vendor/bin/sail artisan test

 FAIL  Tests\Feature\ExampleTest
  ⨯ the application returns a successful response                 

Exception: Vite manifest not found at: .../public/build/manifest.json in .../vendor/laravel/framework/src/Illuminate/Foundation/Vite.php:684
Stack trace:
# ...

Changed

  • Use $this->withoutVite() helper in test suite, where assets have not been built.
    • (Similar functionality to Mix's $this->withoutMix() in older Laravel versions.)

* Generated by:
$ composer update
* Error:
 FAIL  Tests\Feature\ExampleTest
  ⨯ the application returns a successful response

Exception: Vite manifest not found at: .../public/build/manifest.json in .../vendor/laravel/framework/src/Illuminate/Foundation/Vite.php:684
Stack trace:
# ...

* Solution:
Use ->withoutVite() helper (similar to Mix's ->withoutMix() in older Laravel versions)
@hfagerlund hfagerlund self-assigned this May 10, 2023
@hfagerlund hfagerlund merged commit 1f176f5 into main May 10, 2023
@hfagerlund hfagerlund deleted the fix_vite-manifest-not-found branch May 10, 2023 22:44
Copy link
Owner Author

@hfagerlund hfagerlund left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Already resolved in PR #32.

@hfagerlund hfagerlund added this to the Testing milestone May 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

1 participant