PHPUnit testing named routes not working #4295
Closed
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
I don't like the fact we have to add
```
<script>
window.Laravel = {!! json_encode([
'csrfToken' => csrf_token(),
]) !!};
</script>
```
To have it working, when the docs suggest adding only the meta tag. This will get the token from the meta tag.
Add missing IPv4/IPv6 validation messages
Update CHANGELOG.md
Simplify test suite names
Dynamic changing locale
Note that I left your dang `const` as it is, even though it's stupid. `let` for life. 💃💃💃💃
Simplify mix require
Sass compilers can sometimes be weird if import urls like this aren't quoted. They'll spit out junk like ``` Error: Encountered invalid @import syntax. ``` Anyways, pretty common practice to use quotes for all imports. I mean, seriously, how much more do you want me to write for a dang PR that adds a pair of quotes? I've got work to do, gah.
Require Laravel Mix 1.0
|
@alex-petrea hey, I've tried to replicate this, but I need some more info. I get 200 for each request with phpunit. I have noticed that in your question you specify different paths. the first one you are testing goes to /app/invoices and the second one is only / is that exacly what you were testing or did you test: Route::get('/app/invoices', [ and you get error not found? |
|
@edenreich I mistyped it. I was testing Route::get('/app/invoices', [ nevermind |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
So I'm running the following test which does a post request to the login controller.
public function test_login_post()
{
$response = $this->post('/login', [
'email' => $this->user->email,
'password' => $this->user->password,
'_token' => csrf_token()
]);
dd($response);
}
Once the login is successful then the user should get redirected to the /app/invoices
If I set the /app/invoices route like this and run the test
Route::get('/app/invoices', 'InvoicesController@index')->name('account.invoices');
I get an error in the response that route('account.invoices') was not found.
If I name the route this way:
Route::get('/', [
'as' => 'account.invoices',
'uses' => 'InvoicesController@index'
]);
I get no errors.
I don't know if this is a bug or I'm doing something wrong