Add tests for search() method

[curquiza]

If I understand well, we don't have tests for the search() method:

meilisearch-laravel-scout/src/Engines/MeilisearchEngine.php

Lines 81 to 92 in a66ac4c

	/**
	* Perform the given search on the engine.
	*
	* @return mixed
	*/
	public function search(Builder $builder)
	{
	return $this->performSearch($builder, array_filter([
	'filters' => $this->filters($builder),
	'limit' => $builder->limit,
	]));
	}

We should add tests for this.

Tell me if I'm wrong 🙂

[shokme]

meilisearch-laravel-scout/tests/MeilisearchEngineTest.php

Lines 47 to 63 in 36d40eb

	/** @test */
	public function searchSendsCorrectParametersToMeilisearch()
	{
	$client = m::mock(Client::class);
	$client->shouldReceive('index')->with('table')->andReturn($index = m::mock(Indexes::class));
	$index->shouldReceive('search')->with('mustang', [
	'filters' => 'foo=1',
	]);
	$engine = new MeilisearchEngine($client);
	$builder = new Builder(new SearchableModel(), 'mustang', function ($meilisearch, $query, $options) {
	$options['filters'] = 'foo=1';
	return $meilisearch->search($query, $options);
	});
	$engine->search($builder);
	}

not sure to understand, this one is not enough?

[curquiza]

Maybe I badly understand I was surprised the tests (PHP ...) job did not break in this PR (#70) before I changed the search into rawSearch.
For me, there was a lack of test, but again, maybe I missed something.

[mmachatschek]

@curquiza I think this should be solved or at least be covered by #102

[curquiza]

Yes! :)
If anyone thinks there are missing tests, feel free to open a new issue!