Test response view data doesn't contain shared props about inertia-laravel

[mattharcourt]

I just ran into this myself...

Not sure why, but in the test environment (not in the browser) it appears that the Inertia Facade returns a different instance of the Inertia\ResponseFactory when calling Inertia::render() than was returned when Inertia::share() was called when the AppServiceProvider was registered. As a result, the ResponseFactory used by the render method didn't have any shared props.

I was able to solve when I changed the Inertia Facade to always return the same instance by binding the ResponseFactory class into the app container as a singleton in the Inertia\ServiceProvider register method.

This has been proposed to solve a similar issue (#73) by @rojtjo in PR #75

[georgehanson]

Ignore that, I've just noticed #51 :)

[reinink]

Have you reviewed the tests in the PingCRM app? See here. Might be helpful. In particular, see the test macros here.

[chargefield]

I have copied those macros already :)

When i call $this->assertHasProp('auth') it doesn't exist. Even $this->assertHasProp('app.name')...

$this->original->getData() only outputs the data provided to the Inertia::render function. I'm not getting any of the shared data.

Only in my tests...

[georgehanson]

Just in addition to this, it might be useful to have those assertions on a trait which can be applied to a Laravel test suite - rather than having to create the macros yourself.

What do you think @reinink?

[rakk7]

Yeah, same issue here..

When calling $this->assertHasProp('auth') it doesn't exist.. the response doesn't contain the shared data.

But when I open some URL in the browser the "auth" prop does exist!

[Juhlinus]

Hello, @rakk7 !

Those methods does not exist outside of the PingCRM as of right now. If you want to use them then check out the macros:

https://github.com/inertiajs/pingcrm/blob/master/tests/TestCase.php#L18-L52

Or review my PR: #51

[rakk7]

@Juhlinus

Yeah, I already have these methods. I am facing the same issue as @chargefield.