Cache parser for assertContainsElement

[jackbayliss]

I had a thought, if we constantly chained this method it would be nice to cache the parser to save time so it saves the content having to be parsed over and over.

It could be nice doing this for assertElementExists... But I guess people tend to do assertions within the callback.

[sinnbeck]

Looks great, but I wonder if this might break in a future PHP version due to that error? Perhaps it would be better to put it in the container? I believe it will flush after each test.

if (! app()->has('dom-assertions.parser')) {
    app()->bind('dom-assertions.parser', DomParser::new($this->getContent()));
}

//and
app()->get('dom-assertions.parser')->query($selector);

Laravel uses the container as well in TestResponse

$session = app('session.store');

[jackbayliss]

@sinnbeck I've used the container now, thanks for the suggestion 👍🏻 - not sure if it'll affect things if tests are running in parallel but I'm guessing not!

[sinnbeck]

I tested it in parallel and it seems to work as expected :)

Tests: 190 passed (876 assertions)
Duration: 0.37s
Parallel: 22 processes