зеркало из https://github.com/nextcloud/cookbook.git
Updated constructor tests
Signed-off-by: Christian Wolf <github@christianwolf.email>
This commit is contained in:
Christian Wolf
Родитель
27178d341e
Коммит
c1f991f8cb
|
|
@ -2,20 +2,15 @@
|
||||||
|
|
||||||
namespace OCA\Cookbook\tests\Unit\Controller;
|
namespace OCA\Cookbook\tests\Unit\Controller;
|
||||||
|
|
||||||
use Exception;
|
|
||||||
use OCP\IRequest;
|
use OCP\IRequest;
|
||||||
use OCP\Files\File;
|
|
||||||
use OCP\IURLGenerator;
|
use OCP\IURLGenerator;
|
||||||
use ReflectionProperty;
|
use ReflectionProperty;
|
||||||
use PHPUnit\Framework\TestCase;
|
use PHPUnit\Framework\TestCase;
|
||||||
use OCA\Cookbook\Service\RecipeService;
|
use OCA\Cookbook\Service\RecipeService;
|
||||||
use OCP\AppFramework\Http\DataResponse;
|
|
||||||
use OCP\AppFramework\Http\JSONResponse;
|
|
||||||
use OCA\Cookbook\Service\DbCacheService;
|
use OCA\Cookbook\Service\DbCacheService;
|
||||||
use OCA\Cookbook\Controller\MainController;
|
use OCA\Cookbook\Controller\MainController;
|
||||||
use OCA\Cookbook\Helper\RestParameterParser;
|
use OCA\Cookbook\Helper\RestParameterParser;
|
||||||
use PHPUnit\Framework\MockObject\MockObject;
|
use PHPUnit\Framework\MockObject\MockObject;
|
||||||
use OCA\Cookbook\Exception\RecipeExistsException;
|
|
||||||
use OCA\Cookbook\Exception\UserFolderNotWritableException;
|
use OCA\Cookbook\Exception\UserFolderNotWritableException;
|
||||||
|
|
||||||
/**
|
/**
|
||||||
|
|
@ -56,17 +51,15 @@ class MainControllerTest extends TestCase {
|
||||||
$this->restParser = $this->createMock(RestParameterParser::class);
|
$this->restParser = $this->createMock(RestParameterParser::class);
|
||||||
$request = $this->createStub(IRequest::class);
|
$request = $this->createStub(IRequest::class);
|
||||||
|
|
||||||
$this->sut = new MainController('cookbook', $request, $this->recipeService, $this->dbCacheService, $this->urlGenerator, $this->restParser);
|
$this->sut = new MainController('cookbook', $request, $this->recipeService, $this->dbCacheService);
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* @covers ::__construct
|
* @covers ::__construct
|
||||||
*/
|
*/
|
||||||
public function testConstructor(): void {
|
public function testConstructor(): void {
|
||||||
$this->ensurePropertyIsCorrect('urlGenerator', $this->urlGenerator);
|
|
||||||
$this->ensurePropertyIsCorrect('service', $this->recipeService);
|
$this->ensurePropertyIsCorrect('service', $this->recipeService);
|
||||||
$this->ensurePropertyIsCorrect('dbCacheService', $this->dbCacheService);
|
$this->ensurePropertyIsCorrect('dbCacheService', $this->dbCacheService);
|
||||||
$this->ensurePropertyIsCorrect('restParser', $this->restParser);
|
|
||||||
}
|
}
|
||||||
|
|
||||||
private function ensurePropertyIsCorrect(string $name, &$val) {
|
private function ensurePropertyIsCorrect(string $name, &$val) {
|
||||||
|
|
@ -114,553 +107,4 @@ class MainControllerTest extends TestCase {
|
||||||
$this->assertTrue(isset($retData['api_version']['major']));
|
$this->assertTrue(isset($retData['api_version']['major']));
|
||||||
$this->assertTrue(isset($retData['api_version']['minor']));
|
$this->assertTrue(isset($retData['api_version']['minor']));
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
|
||||||
* @covers ::categories
|
|
||||||
*/
|
|
||||||
public function testGetCategories(): void {
|
|
||||||
$this->ensureCacheCheckTriggered();
|
|
||||||
|
|
||||||
$cat = ['Foo', 'Bar', 'Baz'];
|
|
||||||
$this->recipeService->expects($this->once())->method('getAllCategoriesInSearchIndex')->willReturn($cat);
|
|
||||||
|
|
||||||
$ret = $this->sut->categories();
|
|
||||||
$this->assertEquals(200, $ret->getStatus());
|
|
||||||
$this->assertEquals($cat, $ret->getData());
|
|
||||||
}
|
|
||||||
|
|
||||||
/**
|
|
||||||
* @covers ::keywords
|
|
||||||
*/
|
|
||||||
public function testGetKeywords(): void {
|
|
||||||
$this->ensureCacheCheckTriggered();
|
|
||||||
|
|
||||||
$kw = ['Foo', 'Bar', 'Baz'];
|
|
||||||
$this->recipeService->expects($this->once())->method('getAllKeywordsInSearchIndex')->willReturn($kw);
|
|
||||||
|
|
||||||
$ret = $this->sut->keywords();
|
|
||||||
$this->assertEquals(200, $ret->getStatus());
|
|
||||||
$this->assertEquals($kw, $ret->getData());
|
|
||||||
}
|
|
||||||
|
|
||||||
/**
|
|
||||||
* @covers ::new
|
|
||||||
* @dataProvider dataProviderNew
|
|
||||||
*/
|
|
||||||
public function testNew($data, $id): void {
|
|
||||||
$this->ensureCacheCheckTriggered();
|
|
||||||
|
|
||||||
$this->restParser->method('getParameters')->willReturn($data);
|
|
||||||
$file = $this->createMock(File::class);
|
|
||||||
$file->method('getParent')->willReturnSelf();
|
|
||||||
$file->method('getId')->willReturn($id);
|
|
||||||
$this->recipeService->expects($this->once())->method('addRecipe')->with($data)->willReturn($file);
|
|
||||||
$this->dbCacheService->expects($this->once())->method('addRecipe')->with($file);
|
|
||||||
|
|
||||||
/**
|
|
||||||
* @var DataResponse $ret
|
|
||||||
*/
|
|
||||||
$ret = $this->sut->new();
|
|
||||||
|
|
||||||
$this->assertEquals(200, $ret->getStatus());
|
|
||||||
$this->assertEquals($id, $ret->getData());
|
|
||||||
}
|
|
||||||
|
|
||||||
public function dataProviderNew() {
|
|
||||||
return [
|
|
||||||
'success' => [
|
|
||||||
['some', 'recipe', 'data'],
|
|
||||||
10
|
|
||||||
],
|
|
||||||
];
|
|
||||||
}
|
|
||||||
|
|
||||||
/**
|
|
||||||
* @covers ::new
|
|
||||||
* @dataProvider dataProviderNew
|
|
||||||
*/
|
|
||||||
public function testNewFailed($data, $id): void {
|
|
||||||
$this->ensureCacheCheckTriggered();
|
|
||||||
|
|
||||||
$errMsg = 'My error message';
|
|
||||||
|
|
||||||
$this->restParser->method('getParameters')->willReturn($data);
|
|
||||||
$this->recipeService->expects($this->once())->method('addRecipe')->with($data)->willThrowException(new Exception($errMsg));
|
|
||||||
|
|
||||||
/**
|
|
||||||
* @var DataResponse $ret
|
|
||||||
*/
|
|
||||||
$ret = $this->sut->new();
|
|
||||||
|
|
||||||
$this->assertEquals(500, $ret->getStatus());
|
|
||||||
$this->assertEquals($errMsg, $ret->getData());
|
|
||||||
}
|
|
||||||
|
|
||||||
/**
|
|
||||||
* @covers ::update
|
|
||||||
* @dataProvider dataProviderUpdate
|
|
||||||
*/
|
|
||||||
public function testUpdate($data, $id): void {
|
|
||||||
$this->ensureCacheCheckTriggered();
|
|
||||||
|
|
||||||
$this->restParser->method('getParameters')->willReturn($data);
|
|
||||||
$file = $this->createMock(File::class);
|
|
||||||
$file->method('getParent')->willReturnSelf();
|
|
||||||
$file->method('getId')->willReturn($id);
|
|
||||||
$this->recipeService->expects($this->once())->method('addRecipe')->with($data)->willReturn($file);
|
|
||||||
$this->dbCacheService->expects($this->once())->method('addRecipe')->with($file);
|
|
||||||
|
|
||||||
/**
|
|
||||||
* @var DataResponse $ret
|
|
||||||
*/
|
|
||||||
$ret = $this->sut->update($id);
|
|
||||||
|
|
||||||
$this->assertEquals(200, $ret->getStatus());
|
|
||||||
$this->assertEquals($id, $ret->getData());
|
|
||||||
}
|
|
||||||
|
|
||||||
public function dataProviderUpdate() {
|
|
||||||
return [
|
|
||||||
'success' => [
|
|
||||||
['some', 'recipe', 'data', 'id' => 10],
|
|
||||||
10
|
|
||||||
],
|
|
||||||
];
|
|
||||||
}
|
|
||||||
|
|
||||||
/**
|
|
||||||
* @covers ::update
|
|
||||||
* @dataProvider dataProviderUpdate
|
|
||||||
*/
|
|
||||||
public function testUpdateFailed($data, $id): void {
|
|
||||||
$this->ensureCacheCheckTriggered();
|
|
||||||
|
|
||||||
$errMsg = 'My error message';
|
|
||||||
|
|
||||||
$this->restParser->method('getParameters')->willReturn($data);
|
|
||||||
$this->recipeService->expects($this->once())->method('addRecipe')->with($data)->willThrowException(new Exception($errMsg));
|
|
||||||
|
|
||||||
/**
|
|
||||||
* @var DataResponse $ret
|
|
||||||
*/
|
|
||||||
$ret = $this->sut->update($id);
|
|
||||||
|
|
||||||
$this->assertEquals(500, $ret->getStatus());
|
|
||||||
$this->assertEquals($errMsg, $ret->getData());
|
|
||||||
}
|
|
||||||
|
|
||||||
/**
|
|
||||||
* @covers ::import
|
|
||||||
*/
|
|
||||||
public function testImportFailed(): void {
|
|
||||||
$this->ensureCacheCheckTriggered();
|
|
||||||
|
|
||||||
$this->restParser->method('getParameters')->willReturn([]);
|
|
||||||
|
|
||||||
/**
|
|
||||||
* @var DataResponse $ret
|
|
||||||
*/
|
|
||||||
$ret = $this->sut->import();
|
|
||||||
|
|
||||||
$this->assertEquals(400, $ret->getStatus());
|
|
||||||
}
|
|
||||||
|
|
||||||
/**
|
|
||||||
* @covers ::import
|
|
||||||
*/
|
|
||||||
public function testImport(): void {
|
|
||||||
$this->ensureCacheCheckTriggered();
|
|
||||||
|
|
||||||
$url = 'http://example.com/Recipe.html';
|
|
||||||
$file = $this->createStub(File::class);
|
|
||||||
$json = [
|
|
||||||
'id' => 123,
|
|
||||||
'name' => 'The recipe name',
|
|
||||||
];
|
|
||||||
|
|
||||||
$this->restParser->method('getParameters')->willReturn([ 'url' => $url ]);
|
|
||||||
$this->recipeService->expects($this->once())->method('downloadRecipe')->with($url)->willReturn($file);
|
|
||||||
$this->recipeService->expects($this->once())->method('parseRecipeFile')->with($file)->willReturn($json);
|
|
||||||
$this->dbCacheService->expects($this->once())->method('addRecipe')->with($file);
|
|
||||||
|
|
||||||
/**
|
|
||||||
* @var DataResponse $ret
|
|
||||||
*/
|
|
||||||
$ret = $this->sut->import();
|
|
||||||
|
|
||||||
$this->assertEquals(200, $ret->getStatus());
|
|
||||||
$this->assertEquals($json, $ret->getData());
|
|
||||||
}
|
|
||||||
|
|
||||||
/**
|
|
||||||
* @covers ::import
|
|
||||||
*/
|
|
||||||
public function testImportExisting(): void {
|
|
||||||
$this->ensureCacheCheckTriggered();
|
|
||||||
|
|
||||||
$url = 'http://example.com/Recipe.html';
|
|
||||||
$errorMsg = 'The error message';
|
|
||||||
$ex = new RecipeExistsException($errorMsg);
|
|
||||||
|
|
||||||
$this->restParser->method('getParameters')->willReturn([ 'url' => $url ]);
|
|
||||||
$this->recipeService->expects($this->once())->method('downloadRecipe')->with($url)->willThrowException($ex);
|
|
||||||
|
|
||||||
/**
|
|
||||||
* @var JSONResponse $ret
|
|
||||||
*/
|
|
||||||
$ret = $this->sut->import();
|
|
||||||
|
|
||||||
$expected = [
|
|
||||||
'msg' => $ex->getMessage(),
|
|
||||||
'line' => $ex->getLine(),
|
|
||||||
'file' => $ex->getFile(),
|
|
||||||
];
|
|
||||||
|
|
||||||
$this->assertEquals(409, $ret->getStatus());
|
|
||||||
$this->assertEquals($expected, $ret->getData());
|
|
||||||
}
|
|
||||||
|
|
||||||
/**
|
|
||||||
* @covers ::import
|
|
||||||
*/
|
|
||||||
public function testImportOther(): void {
|
|
||||||
$this->ensureCacheCheckTriggered();
|
|
||||||
|
|
||||||
$url = 'http://example.com/Recipe.html';
|
|
||||||
$errorMsg = 'The error message';
|
|
||||||
$ex = new Exception($errorMsg);
|
|
||||||
|
|
||||||
$this->restParser->method('getParameters')->willReturn([ 'url' => $url ]);
|
|
||||||
$this->recipeService->expects($this->once())->method('downloadRecipe')->with($url)->willThrowException($ex);
|
|
||||||
|
|
||||||
/**
|
|
||||||
* @var JSONResponse $ret
|
|
||||||
*/
|
|
||||||
$ret = $this->sut->import();
|
|
||||||
|
|
||||||
|
|
||||||
$this->assertEquals(400, $ret->getStatus());
|
|
||||||
$this->assertEquals($errorMsg, $ret->getData());
|
|
||||||
}
|
|
||||||
|
|
||||||
/**
|
|
||||||
* @covers ::category
|
|
||||||
* @dataProvider dataProviderCategory
|
|
||||||
*/
|
|
||||||
public function testCategory($cat, $recipes): void {
|
|
||||||
$this->ensureCacheCheckTriggered();
|
|
||||||
|
|
||||||
$this->recipeService->method('getRecipesByCategory')->with($cat)->willReturn($recipes);
|
|
||||||
|
|
||||||
$expected = $this->getExpectedRecipes($recipes);
|
|
||||||
|
|
||||||
/**
|
|
||||||
* @var DataResponse $ret
|
|
||||||
*/
|
|
||||||
$ret = $this->sut->category(urlencode($cat));
|
|
||||||
|
|
||||||
$this->assertEquals(200, $ret->getStatus());
|
|
||||||
$this->assertEquals($expected, $ret->getData());
|
|
||||||
}
|
|
||||||
|
|
||||||
private function getExpectedRecipes($recipes): array {
|
|
||||||
$ret = $recipes;
|
|
||||||
|
|
||||||
$ids = [];
|
|
||||||
for ($i = 0; $i < count($recipes); $i++) {
|
|
||||||
$id = $recipes[$i]['recipe_id'];
|
|
||||||
$ids[] = $id;
|
|
||||||
$ret[$i]['imageUrl'] = "/path/to/image/$id/thumb";
|
|
||||||
$ret[$i]['imagePlaceholderUrl'] = "/path/to/image/$id/thumb16";
|
|
||||||
}
|
|
||||||
|
|
||||||
$this->urlGenerator->method('linkToRoute')->with(
|
|
||||||
'cookbook.recipe.image',
|
|
||||||
$this->callback(function ($p) use ($ids) {
|
|
||||||
return isset($p['id']) && isset($p['size']) && false !== array_search($p['id'], $ids);
|
|
||||||
})
|
|
||||||
)->willReturnCallback(function ($name, $p) use ($ret) {
|
|
||||||
// return $ret[$idx[$p['id']]];
|
|
||||||
$id = $p['id'];
|
|
||||||
$size = $p['size'];
|
|
||||||
return "/path/to/image/$id/$size";
|
|
||||||
});
|
|
||||||
|
|
||||||
return $ret;
|
|
||||||
}
|
|
||||||
|
|
||||||
public function dataProviderCategory(): array {
|
|
||||||
return [
|
|
||||||
'noRecipes' => [
|
|
||||||
'My category',
|
|
||||||
[]
|
|
||||||
],
|
|
||||||
'someRecipes' => [
|
|
||||||
'My category',
|
|
||||||
[
|
|
||||||
[
|
|
||||||
'name' => 'My recipe 1',
|
|
||||||
'recipe_id' => 123,
|
|
||||||
],
|
|
||||||
[
|
|
||||||
'name' => 'My recipe 2',
|
|
||||||
'recipe_id' => 122,
|
|
||||||
],
|
|
||||||
]
|
|
||||||
],
|
|
||||||
];
|
|
||||||
}
|
|
||||||
|
|
||||||
/**
|
|
||||||
* @covers ::category
|
|
||||||
*/
|
|
||||||
public function testCategoryFailed(): void {
|
|
||||||
$this->ensureCacheCheckTriggered();
|
|
||||||
|
|
||||||
$cat = 'My category';
|
|
||||||
$errorMsg = 'The error is found.';
|
|
||||||
$this->recipeService->method('getRecipesByCategory')->with($cat)->willThrowException(new Exception($errorMsg));
|
|
||||||
|
|
||||||
/**
|
|
||||||
* @var DataResponse $ret
|
|
||||||
*/
|
|
||||||
$ret = $this->sut->category(urlencode($cat));
|
|
||||||
|
|
||||||
$this->assertEquals(500, $ret->getStatus());
|
|
||||||
$this->assertEquals($errorMsg, $ret->getData());
|
|
||||||
}
|
|
||||||
|
|
||||||
/**
|
|
||||||
* @covers ::tags
|
|
||||||
* @dataProvider dataProviderTags
|
|
||||||
*/
|
|
||||||
public function testTags($keywords, $recipes): void {
|
|
||||||
$this->ensureCacheCheckTriggered();
|
|
||||||
|
|
||||||
$this->recipeService->method('getRecipesByKeywords')->with($keywords)->willReturn($recipes);
|
|
||||||
|
|
||||||
$expected = $this->getExpectedRecipes($recipes);
|
|
||||||
|
|
||||||
/**
|
|
||||||
* @var DataResponse $ret
|
|
||||||
*/
|
|
||||||
$ret = $this->sut->tags(urlencode($keywords));
|
|
||||||
|
|
||||||
$this->assertEquals(200, $ret->getStatus());
|
|
||||||
$this->assertEquals($expected, $ret->getData());
|
|
||||||
}
|
|
||||||
|
|
||||||
public function dataProviderTags(): array {
|
|
||||||
return [
|
|
||||||
'noTag' => [
|
|
||||||
'*',
|
|
||||||
[
|
|
||||||
[
|
|
||||||
'name' => 'My recipe 1',
|
|
||||||
'recipe_id' => 123,
|
|
||||||
],
|
|
||||||
[
|
|
||||||
'name' => 'My recipe 2',
|
|
||||||
'recipe_id' => 122,
|
|
||||||
],
|
|
||||||
]
|
|
||||||
],
|
|
||||||
'noRecipes' => [
|
|
||||||
'Tag A,Tag B',
|
|
||||||
[]
|
|
||||||
],
|
|
||||||
'someRecipes' => [
|
|
||||||
'Tag A, Tag B',
|
|
||||||
[
|
|
||||||
[
|
|
||||||
'name' => 'My recipe 1',
|
|
||||||
'recipe_id' => 123,
|
|
||||||
],
|
|
||||||
[
|
|
||||||
'name' => 'My recipe 2',
|
|
||||||
'recipe_id' => 122,
|
|
||||||
],
|
|
||||||
]
|
|
||||||
],
|
|
||||||
];
|
|
||||||
}
|
|
||||||
|
|
||||||
/**
|
|
||||||
* @covers ::tags
|
|
||||||
*/
|
|
||||||
public function testTagsFailed(): void {
|
|
||||||
$this->ensureCacheCheckTriggered();
|
|
||||||
|
|
||||||
$keywords = 'Tag 1,Tag B';
|
|
||||||
$errorMsg = 'The error is found.';
|
|
||||||
$this->recipeService->method('getRecipesByKeywords')->with($keywords)->willThrowException(new Exception($errorMsg));
|
|
||||||
|
|
||||||
/**
|
|
||||||
* @var DataResponse $ret
|
|
||||||
*/
|
|
||||||
$ret = $this->sut->tags(urlencode($keywords));
|
|
||||||
|
|
||||||
$this->assertEquals(500, $ret->getStatus());
|
|
||||||
$this->assertEquals($errorMsg, $ret->getData());
|
|
||||||
}
|
|
||||||
|
|
||||||
/**
|
|
||||||
* @covers ::search
|
|
||||||
* @dataProvider dpSearch
|
|
||||||
* @todo no implementation in controller
|
|
||||||
*/
|
|
||||||
public function testSearch($query, $recipes): void {
|
|
||||||
$this->ensureCacheCheckTriggered();
|
|
||||||
|
|
||||||
$this->recipeService->expects($this->once())->method('findRecipesInSearchIndex')->with($query)->willReturn($recipes);
|
|
||||||
|
|
||||||
$expected = $this->getExpectedRecipes($recipes);
|
|
||||||
|
|
||||||
/**
|
|
||||||
* @var DataResponse $res
|
|
||||||
*/
|
|
||||||
$res = $this->sut->search(urlencode($query));
|
|
||||||
|
|
||||||
$this->assertEquals(200, $res->getStatus());
|
|
||||||
$this->assertEquals($expected, $res->getData());
|
|
||||||
}
|
|
||||||
|
|
||||||
public function dpSearch() {
|
|
||||||
return [
|
|
||||||
'noRecipes' => [
|
|
||||||
'some query',
|
|
||||||
[],
|
|
||||||
],
|
|
||||||
'someRecipes' => [
|
|
||||||
'some query',
|
|
||||||
[
|
|
||||||
[
|
|
||||||
'name' => 'First recipe',
|
|
||||||
'recipe_id' => 123,
|
|
||||||
],
|
|
||||||
],
|
|
||||||
],
|
|
||||||
];
|
|
||||||
}
|
|
||||||
|
|
||||||
/**
|
|
||||||
* @covers ::search
|
|
||||||
*/
|
|
||||||
public function testSearchFailed(): void {
|
|
||||||
$this->ensureCacheCheckTriggered();
|
|
||||||
|
|
||||||
$query = 'some query';
|
|
||||||
$errorMsg = 'Could not search for recipes';
|
|
||||||
$this->recipeService->expects($this->once())->method('findRecipesInSearchIndex')->with($query)->willThrowException(new Exception($errorMsg));
|
|
||||||
|
|
||||||
/**
|
|
||||||
* @var DataResponse $res
|
|
||||||
*/
|
|
||||||
$res = $this->sut->search(urlencode($query));
|
|
||||||
|
|
||||||
$this->assertEquals(500, $res->getStatus());
|
|
||||||
$this->assertEquals($errorMsg, $res->getData());
|
|
||||||
}
|
|
||||||
|
|
||||||
/**
|
|
||||||
* @covers ::categoryUpdate
|
|
||||||
* @dataProvider dataProviderCategoryUpdateNoName
|
|
||||||
*/
|
|
||||||
public function testCategoryUpdateNoName($requestParams): void {
|
|
||||||
$this->ensureCacheCheckTriggered();
|
|
||||||
|
|
||||||
$this->restParser->expects($this->once())->method('getParameters')->willReturn($requestParams);
|
|
||||||
|
|
||||||
$ret = $this->sut->categoryUpdate('');
|
|
||||||
|
|
||||||
$this->assertEquals(400, $ret->getStatus());
|
|
||||||
}
|
|
||||||
|
|
||||||
public function dataProviderCategoryUpdateNoName() {
|
|
||||||
yield [[]];
|
|
||||||
yield [[
|
|
||||||
'some', 'variable'
|
|
||||||
]];
|
|
||||||
yield [['name' => null]];
|
|
||||||
yield [['name' => '']];
|
|
||||||
}
|
|
||||||
|
|
||||||
/**
|
|
||||||
* @covers ::categoryUpdate
|
|
||||||
* @dataProvider dpCategoryUpdate
|
|
||||||
* @todo No business logic in controller
|
|
||||||
*/
|
|
||||||
public function testCategoryUpdate($cat, $oldCat, $recipes): void {
|
|
||||||
$this->ensureCacheCheckTriggered();
|
|
||||||
|
|
||||||
$this->recipeService->expects($this->once())->method('getRecipesByCategory')->with($oldCat)->willReturn($recipes);
|
|
||||||
$this->dbCacheService->expects($this->once())->method('updateCache');
|
|
||||||
|
|
||||||
$this->restParser->expects($this->once())->method('getParameters')->willReturn(['name' => $cat]);
|
|
||||||
|
|
||||||
$n = count($recipes);
|
|
||||||
$indices = array_map(function ($v) {
|
|
||||||
return [$v['recipe_id']];
|
|
||||||
}, $recipes);
|
|
||||||
$this->recipeService->expects($this->exactly($n))->method('getRecipeById')->withConsecutive(...$indices);
|
|
||||||
$this->recipeService->expects($this->exactly($n))->method('addRecipe')->with($this->callback(function ($p) use ($cat) {
|
|
||||||
return $p['recipeCategory'] === $cat;
|
|
||||||
}));
|
|
||||||
|
|
||||||
/**
|
|
||||||
* @var DataResponse $ret
|
|
||||||
*/
|
|
||||||
$ret = $this->sut->categoryUpdate(urlencode($oldCat));
|
|
||||||
|
|
||||||
$this->assertEquals(200, $ret->getStatus());
|
|
||||||
$this->assertEquals($cat, $ret->getData());
|
|
||||||
}
|
|
||||||
|
|
||||||
public function dpCategoryUpdate() {
|
|
||||||
return [
|
|
||||||
'noRecipes' => [
|
|
||||||
'new Category Name',
|
|
||||||
'Old category',
|
|
||||||
[]
|
|
||||||
],
|
|
||||||
'someRecipes' => [
|
|
||||||
'new Category Name',
|
|
||||||
'Old category',
|
|
||||||
[
|
|
||||||
[
|
|
||||||
'name' => 'First recipe',
|
|
||||||
'recipeCategory' => 'some fancy category',
|
|
||||||
'recipe_id' => 123,
|
|
||||||
],
|
|
||||||
[
|
|
||||||
'name' => 'Second recipe',
|
|
||||||
'recipeCategory' => 'some fancy category',
|
|
||||||
'recipe_id' => 124,
|
|
||||||
],
|
|
||||||
]
|
|
||||||
],
|
|
||||||
];
|
|
||||||
}
|
|
||||||
|
|
||||||
/**
|
|
||||||
* @covers ::categoryUpdate
|
|
||||||
*/
|
|
||||||
public function testCategoryUpdateFailure(): void {
|
|
||||||
$this->ensureCacheCheckTriggered();
|
|
||||||
|
|
||||||
$this->restParser->expects($this->once())->method('getParameters')->willReturn(['name' => 'New category']);
|
|
||||||
|
|
||||||
$errorMsg = 'Something bad has happened.';
|
|
||||||
$oldCat = 'Old category';
|
|
||||||
|
|
||||||
$this->recipeService->expects($this->once())->method('getRecipesByCategory')->with($oldCat)->willThrowException(new Exception($errorMsg));
|
|
||||||
|
|
||||||
/**
|
|
||||||
* @var DataResponse $ret
|
|
||||||
*/
|
|
||||||
$ret = $this->sut->categoryUpdate(urlencode($oldCat));
|
|
||||||
|
|
||||||
$this->assertEquals(500, $ret->getStatus());
|
|
||||||
$this->assertEquals($errorMsg, $ret->getData());
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
|
|
|
||||||
|
|
@ -1,6 +1,6 @@
|
||||||
<?php
|
<?php
|
||||||
|
|
||||||
namespace OCA\Cookbook\tests\Unit\Controller;
|
namespace OCA\Cookbook\tests\Unit\Controller\v1;
|
||||||
|
|
||||||
use OCP\IRequest;
|
use OCP\IRequest;
|
||||||
use PHPUnit\Framework\TestCase;
|
use PHPUnit\Framework\TestCase;
|
||||||
|
|
@ -8,13 +8,13 @@ use OCA\Cookbook\Service\RecipeService;
|
||||||
use OCA\Cookbook\Service\DbCacheService;
|
use OCA\Cookbook\Service\DbCacheService;
|
||||||
use OCA\Cookbook\Helper\RestParameterParser;
|
use OCA\Cookbook\Helper\RestParameterParser;
|
||||||
use PHPUnit\Framework\MockObject\MockObject;
|
use PHPUnit\Framework\MockObject\MockObject;
|
||||||
use OCA\Cookbook\Controller\ConfigController;
|
use OCA\Cookbook\Controller\v1\ConfigController;
|
||||||
use OCP\AppFramework\Http\DataResponse;
|
use OCP\AppFramework\Http\DataResponse;
|
||||||
use OCP\AppFramework\Http\Response;
|
use OCP\AppFramework\Http\Response;
|
||||||
use ReflectionProperty;
|
use ReflectionProperty;
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* @coversDefaultClass OCA\Cookbook\Controller\ConfigController
|
* @coversDefaultClass OCA\Cookbook\Controller\v1\ConfigController
|
||||||
* @covers ::<private>
|
* @covers ::<private>
|
||||||
* @covers ::<protected>
|
* @covers ::<protected>
|
||||||
*/
|
*/
|
||||||
|
|
@ -0,0 +1,629 @@
|
||||||
|
<?php
|
||||||
|
|
||||||
|
namespace OCA\Cookbook\tests\Unit\Controller\v1;
|
||||||
|
|
||||||
|
use Exception;
|
||||||
|
use OCP\IRequest;
|
||||||
|
use OCP\Files\File;
|
||||||
|
use OCP\IURLGenerator;
|
||||||
|
use ReflectionProperty;
|
||||||
|
use PHPUnit\Framework\TestCase;
|
||||||
|
use OCA\Cookbook\Service\RecipeService;
|
||||||
|
use OCP\AppFramework\Http\DataResponse;
|
||||||
|
use OCP\AppFramework\Http\JSONResponse;
|
||||||
|
use OCA\Cookbook\Service\DbCacheService;
|
||||||
|
use OCA\Cookbook\Controller\v1\MainController;
|
||||||
|
use OCA\Cookbook\Helper\RestParameterParser;
|
||||||
|
use PHPUnit\Framework\MockObject\MockObject;
|
||||||
|
use OCA\Cookbook\Exception\RecipeExistsException;
|
||||||
|
|
||||||
|
/**
|
||||||
|
* @coversDefaultClass \OCA\Cookbook\Controller\v1\MainController
|
||||||
|
* @covers ::<private>
|
||||||
|
* @covers ::<protected>
|
||||||
|
*/
|
||||||
|
class MainControllerTest extends TestCase {
|
||||||
|
|
||||||
|
/**
|
||||||
|
* @var MockObject|RecipeService
|
||||||
|
*/
|
||||||
|
private $recipeService;
|
||||||
|
/**
|
||||||
|
* @var IURLGenerator|MockObject
|
||||||
|
*/
|
||||||
|
private $urlGenerator;
|
||||||
|
/**
|
||||||
|
* @var DbCacheService|MockObject
|
||||||
|
*/
|
||||||
|
private $dbCacheService;
|
||||||
|
/**
|
||||||
|
* @var RestParameterParser|MockObject
|
||||||
|
*/
|
||||||
|
private $restParser;
|
||||||
|
|
||||||
|
/**
|
||||||
|
* @var MainController
|
||||||
|
*/
|
||||||
|
private $sut;
|
||||||
|
|
||||||
|
public function setUp(): void {
|
||||||
|
parent::setUp();
|
||||||
|
|
||||||
|
$this->recipeService = $this->createMock(RecipeService::class);
|
||||||
|
$this->urlGenerator = $this->createMock(IURLGenerator::class);
|
||||||
|
$this->dbCacheService = $this->createMock(DbCacheService::class);
|
||||||
|
$this->restParser = $this->createMock(RestParameterParser::class);
|
||||||
|
$request = $this->createStub(IRequest::class);
|
||||||
|
|
||||||
|
$this->sut = new MainController('cookbook', $request, $this->recipeService, $this->dbCacheService, $this->urlGenerator, $this->restParser);
|
||||||
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* @covers ::__construct
|
||||||
|
*/
|
||||||
|
public function testConstructor(): void {
|
||||||
|
$this->ensurePropertyIsCorrect('urlGenerator', $this->urlGenerator);
|
||||||
|
$this->ensurePropertyIsCorrect('service', $this->recipeService);
|
||||||
|
$this->ensurePropertyIsCorrect('dbCacheService', $this->dbCacheService);
|
||||||
|
$this->ensurePropertyIsCorrect('restParser', $this->restParser);
|
||||||
|
}
|
||||||
|
|
||||||
|
private function ensurePropertyIsCorrect(string $name, &$val) {
|
||||||
|
$property = new ReflectionProperty(MainController::class, $name);
|
||||||
|
$property->setAccessible(true);
|
||||||
|
$this->assertSame($val, $property->getValue($this->sut));
|
||||||
|
}
|
||||||
|
|
||||||
|
private function ensureCacheCheckTriggered(): void {
|
||||||
|
$this->dbCacheService->expects($this->once())->method('triggerCheck');
|
||||||
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* @covers ::categories
|
||||||
|
*/
|
||||||
|
public function testGetCategories(): void {
|
||||||
|
$this->ensureCacheCheckTriggered();
|
||||||
|
|
||||||
|
$cat = ['Foo', 'Bar', 'Baz'];
|
||||||
|
$this->recipeService->expects($this->once())->method('getAllCategoriesInSearchIndex')->willReturn($cat);
|
||||||
|
|
||||||
|
$ret = $this->sut->categories();
|
||||||
|
$this->assertEquals(200, $ret->getStatus());
|
||||||
|
$this->assertEquals($cat, $ret->getData());
|
||||||
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* @covers ::keywords
|
||||||
|
*/
|
||||||
|
public function testGetKeywords(): void {
|
||||||
|
$this->ensureCacheCheckTriggered();
|
||||||
|
|
||||||
|
$kw = ['Foo', 'Bar', 'Baz'];
|
||||||
|
$this->recipeService->expects($this->once())->method('getAllKeywordsInSearchIndex')->willReturn($kw);
|
||||||
|
|
||||||
|
$ret = $this->sut->keywords();
|
||||||
|
$this->assertEquals(200, $ret->getStatus());
|
||||||
|
$this->assertEquals($kw, $ret->getData());
|
||||||
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* @covers ::new
|
||||||
|
* @dataProvider dataProviderNew
|
||||||
|
*/
|
||||||
|
public function testNew($data, $id): void {
|
||||||
|
$this->ensureCacheCheckTriggered();
|
||||||
|
|
||||||
|
$this->restParser->method('getParameters')->willReturn($data);
|
||||||
|
$file = $this->createMock(File::class);
|
||||||
|
$file->method('getParent')->willReturnSelf();
|
||||||
|
$file->method('getId')->willReturn($id);
|
||||||
|
$this->recipeService->expects($this->once())->method('addRecipe')->with($data)->willReturn($file);
|
||||||
|
$this->dbCacheService->expects($this->once())->method('addRecipe')->with($file);
|
||||||
|
|
||||||
|
/**
|
||||||
|
* @var DataResponse $ret
|
||||||
|
*/
|
||||||
|
$ret = $this->sut->new();
|
||||||
|
|
||||||
|
$this->assertEquals(200, $ret->getStatus());
|
||||||
|
$this->assertEquals($id, $ret->getData());
|
||||||
|
}
|
||||||
|
|
||||||
|
public function dataProviderNew() {
|
||||||
|
return [
|
||||||
|
'success' => [
|
||||||
|
['some', 'recipe', 'data'],
|
||||||
|
10
|
||||||
|
],
|
||||||
|
];
|
||||||
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* @covers ::new
|
||||||
|
* @dataProvider dataProviderNew
|
||||||
|
*/
|
||||||
|
public function testNewFailed($data, $id): void {
|
||||||
|
$this->ensureCacheCheckTriggered();
|
||||||
|
|
||||||
|
$errMsg = 'My error message';
|
||||||
|
|
||||||
|
$this->restParser->method('getParameters')->willReturn($data);
|
||||||
|
$this->recipeService->expects($this->once())->method('addRecipe')->with($data)->willThrowException(new Exception($errMsg));
|
||||||
|
|
||||||
|
/**
|
||||||
|
* @var DataResponse $ret
|
||||||
|
*/
|
||||||
|
$ret = $this->sut->new();
|
||||||
|
|
||||||
|
$this->assertEquals(500, $ret->getStatus());
|
||||||
|
$this->assertEquals($errMsg, $ret->getData());
|
||||||
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* @covers ::update
|
||||||
|
* @dataProvider dataProviderUpdate
|
||||||
|
*/
|
||||||
|
public function testUpdate($data, $id): void {
|
||||||
|
$this->ensureCacheCheckTriggered();
|
||||||
|
|
||||||
|
$this->restParser->method('getParameters')->willReturn($data);
|
||||||
|
$file = $this->createMock(File::class);
|
||||||
|
$file->method('getParent')->willReturnSelf();
|
||||||
|
$file->method('getId')->willReturn($id);
|
||||||
|
$this->recipeService->expects($this->once())->method('addRecipe')->with($data)->willReturn($file);
|
||||||
|
$this->dbCacheService->expects($this->once())->method('addRecipe')->with($file);
|
||||||
|
|
||||||
|
/**
|
||||||
|
* @var DataResponse $ret
|
||||||
|
*/
|
||||||
|
$ret = $this->sut->update($id);
|
||||||
|
|
||||||
|
$this->assertEquals(200, $ret->getStatus());
|
||||||
|
$this->assertEquals($id, $ret->getData());
|
||||||
|
}
|
||||||
|
|
||||||
|
public function dataProviderUpdate() {
|
||||||
|
return [
|
||||||
|
'success' => [
|
||||||
|
['some', 'recipe', 'data', 'id' => 10],
|
||||||
|
10
|
||||||
|
],
|
||||||
|
];
|
||||||
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* @covers ::update
|
||||||
|
* @dataProvider dataProviderUpdate
|
||||||
|
*/
|
||||||
|
public function testUpdateFailed($data, $id): void {
|
||||||
|
$this->ensureCacheCheckTriggered();
|
||||||
|
|
||||||
|
$errMsg = 'My error message';
|
||||||
|
|
||||||
|
$this->restParser->method('getParameters')->willReturn($data);
|
||||||
|
$this->recipeService->expects($this->once())->method('addRecipe')->with($data)->willThrowException(new Exception($errMsg));
|
||||||
|
|
||||||
|
/**
|
||||||
|
* @var DataResponse $ret
|
||||||
|
*/
|
||||||
|
$ret = $this->sut->update($id);
|
||||||
|
|
||||||
|
$this->assertEquals(500, $ret->getStatus());
|
||||||
|
$this->assertEquals($errMsg, $ret->getData());
|
||||||
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* @covers ::import
|
||||||
|
*/
|
||||||
|
public function testImportFailed(): void {
|
||||||
|
$this->ensureCacheCheckTriggered();
|
||||||
|
|
||||||
|
$this->restParser->method('getParameters')->willReturn([]);
|
||||||
|
|
||||||
|
/**
|
||||||
|
* @var DataResponse $ret
|
||||||
|
*/
|
||||||
|
$ret = $this->sut->import();
|
||||||
|
|
||||||
|
$this->assertEquals(400, $ret->getStatus());
|
||||||
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* @covers ::import
|
||||||
|
*/
|
||||||
|
public function testImport(): void {
|
||||||
|
$this->ensureCacheCheckTriggered();
|
||||||
|
|
||||||
|
$url = 'http://example.com/Recipe.html';
|
||||||
|
$file = $this->createStub(File::class);
|
||||||
|
$json = [
|
||||||
|
'id' => 123,
|
||||||
|
'name' => 'The recipe name',
|
||||||
|
];
|
||||||
|
|
||||||
|
$this->restParser->method('getParameters')->willReturn([ 'url' => $url ]);
|
||||||
|
$this->recipeService->expects($this->once())->method('downloadRecipe')->with($url)->willReturn($file);
|
||||||
|
$this->recipeService->expects($this->once())->method('parseRecipeFile')->with($file)->willReturn($json);
|
||||||
|
$this->dbCacheService->expects($this->once())->method('addRecipe')->with($file);
|
||||||
|
|
||||||
|
/**
|
||||||
|
* @var DataResponse $ret
|
||||||
|
*/
|
||||||
|
$ret = $this->sut->import();
|
||||||
|
|
||||||
|
$this->assertEquals(200, $ret->getStatus());
|
||||||
|
$this->assertEquals($json, $ret->getData());
|
||||||
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* @covers ::import
|
||||||
|
*/
|
||||||
|
public function testImportExisting(): void {
|
||||||
|
$this->ensureCacheCheckTriggered();
|
||||||
|
|
||||||
|
$url = 'http://example.com/Recipe.html';
|
||||||
|
$errorMsg = 'The error message';
|
||||||
|
$ex = new RecipeExistsException($errorMsg);
|
||||||
|
|
||||||
|
$this->restParser->method('getParameters')->willReturn([ 'url' => $url ]);
|
||||||
|
$this->recipeService->expects($this->once())->method('downloadRecipe')->with($url)->willThrowException($ex);
|
||||||
|
|
||||||
|
/**
|
||||||
|
* @var JSONResponse $ret
|
||||||
|
*/
|
||||||
|
$ret = $this->sut->import();
|
||||||
|
|
||||||
|
$expected = [
|
||||||
|
'msg' => $ex->getMessage(),
|
||||||
|
'line' => $ex->getLine(),
|
||||||
|
'file' => $ex->getFile(),
|
||||||
|
];
|
||||||
|
|
||||||
|
$this->assertEquals(409, $ret->getStatus());
|
||||||
|
$this->assertEquals($expected, $ret->getData());
|
||||||
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* @covers ::import
|
||||||
|
*/
|
||||||
|
public function testImportOther(): void {
|
||||||
|
$this->ensureCacheCheckTriggered();
|
||||||
|
|
||||||
|
$url = 'http://example.com/Recipe.html';
|
||||||
|
$errorMsg = 'The error message';
|
||||||
|
$ex = new Exception($errorMsg);
|
||||||
|
|
||||||
|
$this->restParser->method('getParameters')->willReturn([ 'url' => $url ]);
|
||||||
|
$this->recipeService->expects($this->once())->method('downloadRecipe')->with($url)->willThrowException($ex);
|
||||||
|
|
||||||
|
/**
|
||||||
|
* @var JSONResponse $ret
|
||||||
|
*/
|
||||||
|
$ret = $this->sut->import();
|
||||||
|
|
||||||
|
|
||||||
|
$this->assertEquals(400, $ret->getStatus());
|
||||||
|
$this->assertEquals($errorMsg, $ret->getData());
|
||||||
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* @covers ::category
|
||||||
|
* @dataProvider dataProviderCategory
|
||||||
|
*/
|
||||||
|
public function testCategory($cat, $recipes): void {
|
||||||
|
$this->ensureCacheCheckTriggered();
|
||||||
|
|
||||||
|
$this->recipeService->method('getRecipesByCategory')->with($cat)->willReturn($recipes);
|
||||||
|
|
||||||
|
$expected = $this->getExpectedRecipes($recipes);
|
||||||
|
|
||||||
|
/**
|
||||||
|
* @var DataResponse $ret
|
||||||
|
*/
|
||||||
|
$ret = $this->sut->category(urlencode($cat));
|
||||||
|
|
||||||
|
$this->assertEquals(200, $ret->getStatus());
|
||||||
|
$this->assertEquals($expected, $ret->getData());
|
||||||
|
}
|
||||||
|
|
||||||
|
private function getExpectedRecipes($recipes): array {
|
||||||
|
$ret = $recipes;
|
||||||
|
|
||||||
|
$ids = [];
|
||||||
|
for ($i = 0; $i < count($recipes); $i++) {
|
||||||
|
$id = $recipes[$i]['recipe_id'];
|
||||||
|
$ids[] = $id;
|
||||||
|
$ret[$i]['imageUrl'] = "/path/to/image/$id/thumb";
|
||||||
|
$ret[$i]['imagePlaceholderUrl'] = "/path/to/image/$id/thumb16";
|
||||||
|
}
|
||||||
|
|
||||||
|
$this->urlGenerator->method('linkToRoute')->with(
|
||||||
|
'cookbook.recipe_v1.image',
|
||||||
|
$this->callback(function ($p) use ($ids) {
|
||||||
|
return isset($p['id']) && isset($p['size']) && false !== array_search($p['id'], $ids);
|
||||||
|
})
|
||||||
|
)->willReturnCallback(function ($name, $p) use ($ret) {
|
||||||
|
// return $ret[$idx[$p['id']]];
|
||||||
|
$id = $p['id'];
|
||||||
|
$size = $p['size'];
|
||||||
|
return "/path/to/image/$id/$size";
|
||||||
|
});
|
||||||
|
|
||||||
|
return $ret;
|
||||||
|
}
|
||||||
|
|
||||||
|
public function dataProviderCategory(): array {
|
||||||
|
return [
|
||||||
|
'noRecipes' => [
|
||||||
|
'My category',
|
||||||
|
[]
|
||||||
|
],
|
||||||
|
'someRecipes' => [
|
||||||
|
'My category',
|
||||||
|
[
|
||||||
|
[
|
||||||
|
'name' => 'My recipe 1',
|
||||||
|
'recipe_id' => 123,
|
||||||
|
],
|
||||||
|
[
|
||||||
|
'name' => 'My recipe 2',
|
||||||
|
'recipe_id' => 122,
|
||||||
|
],
|
||||||
|
]
|
||||||
|
],
|
||||||
|
];
|
||||||
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* @covers ::category
|
||||||
|
*/
|
||||||
|
public function testCategoryFailed(): void {
|
||||||
|
$this->ensureCacheCheckTriggered();
|
||||||
|
|
||||||
|
$cat = 'My category';
|
||||||
|
$errorMsg = 'The error is found.';
|
||||||
|
$this->recipeService->method('getRecipesByCategory')->with($cat)->willThrowException(new Exception($errorMsg));
|
||||||
|
|
||||||
|
/**
|
||||||
|
* @var DataResponse $ret
|
||||||
|
*/
|
||||||
|
$ret = $this->sut->category(urlencode($cat));
|
||||||
|
|
||||||
|
$this->assertEquals(500, $ret->getStatus());
|
||||||
|
$this->assertEquals($errorMsg, $ret->getData());
|
||||||
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* @covers ::tags
|
||||||
|
* @dataProvider dataProviderTags
|
||||||
|
*/
|
||||||
|
public function testTags($keywords, $recipes): void {
|
||||||
|
$this->ensureCacheCheckTriggered();
|
||||||
|
|
||||||
|
$this->recipeService->method('getRecipesByKeywords')->with($keywords)->willReturn($recipes);
|
||||||
|
|
||||||
|
$expected = $this->getExpectedRecipes($recipes);
|
||||||
|
|
||||||
|
/**
|
||||||
|
* @var DataResponse $ret
|
||||||
|
*/
|
||||||
|
$ret = $this->sut->tags(urlencode($keywords));
|
||||||
|
|
||||||
|
$this->assertEquals(200, $ret->getStatus());
|
||||||
|
$this->assertEquals($expected, $ret->getData());
|
||||||
|
}
|
||||||
|
|
||||||
|
public function dataProviderTags(): array {
|
||||||
|
return [
|
||||||
|
'noTag' => [
|
||||||
|
'*',
|
||||||
|
[
|
||||||
|
[
|
||||||
|
'name' => 'My recipe 1',
|
||||||
|
'recipe_id' => 123,
|
||||||
|
],
|
||||||
|
[
|
||||||
|
'name' => 'My recipe 2',
|
||||||
|
'recipe_id' => 122,
|
||||||
|
],
|
||||||
|
]
|
||||||
|
],
|
||||||
|
'noRecipes' => [
|
||||||
|
'Tag A,Tag B',
|
||||||
|
[]
|
||||||
|
],
|
||||||
|
'someRecipes' => [
|
||||||
|
'Tag A, Tag B',
|
||||||
|
[
|
||||||
|
[
|
||||||
|
'name' => 'My recipe 1',
|
||||||
|
'recipe_id' => 123,
|
||||||
|
],
|
||||||
|
[
|
||||||
|
'name' => 'My recipe 2',
|
||||||
|
'recipe_id' => 122,
|
||||||
|
],
|
||||||
|
]
|
||||||
|
],
|
||||||
|
];
|
||||||
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* @covers ::tags
|
||||||
|
*/
|
||||||
|
public function testTagsFailed(): void {
|
||||||
|
$this->ensureCacheCheckTriggered();
|
||||||
|
|
||||||
|
$keywords = 'Tag 1,Tag B';
|
||||||
|
$errorMsg = 'The error is found.';
|
||||||
|
$this->recipeService->method('getRecipesByKeywords')->with($keywords)->willThrowException(new Exception($errorMsg));
|
||||||
|
|
||||||
|
/**
|
||||||
|
* @var DataResponse $ret
|
||||||
|
*/
|
||||||
|
$ret = $this->sut->tags(urlencode($keywords));
|
||||||
|
|
||||||
|
$this->assertEquals(500, $ret->getStatus());
|
||||||
|
$this->assertEquals($errorMsg, $ret->getData());
|
||||||
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* @covers ::search
|
||||||
|
* @dataProvider dpSearch
|
||||||
|
* @todo no implementation in controller
|
||||||
|
*/
|
||||||
|
public function testSearch($query, $recipes): void {
|
||||||
|
$this->ensureCacheCheckTriggered();
|
||||||
|
|
||||||
|
$this->recipeService->expects($this->once())->method('findRecipesInSearchIndex')->with($query)->willReturn($recipes);
|
||||||
|
|
||||||
|
$expected = $this->getExpectedRecipes($recipes);
|
||||||
|
|
||||||
|
/**
|
||||||
|
* @var DataResponse $res
|
||||||
|
*/
|
||||||
|
$res = $this->sut->search(urlencode($query));
|
||||||
|
|
||||||
|
$this->assertEquals(200, $res->getStatus());
|
||||||
|
$this->assertEquals($expected, $res->getData());
|
||||||
|
}
|
||||||
|
|
||||||
|
public function dpSearch() {
|
||||||
|
return [
|
||||||
|
'noRecipes' => [
|
||||||
|
'some query',
|
||||||
|
[],
|
||||||
|
],
|
||||||
|
'someRecipes' => [
|
||||||
|
'some query',
|
||||||
|
[
|
||||||
|
[
|
||||||
|
'name' => 'First recipe',
|
||||||
|
'recipe_id' => 123,
|
||||||
|
],
|
||||||
|
],
|
||||||
|
],
|
||||||
|
];
|
||||||
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* @covers ::search
|
||||||
|
*/
|
||||||
|
public function testSearchFailed(): void {
|
||||||
|
$this->ensureCacheCheckTriggered();
|
||||||
|
|
||||||
|
$query = 'some query';
|
||||||
|
$errorMsg = 'Could not search for recipes';
|
||||||
|
$this->recipeService->expects($this->once())->method('findRecipesInSearchIndex')->with($query)->willThrowException(new Exception($errorMsg));
|
||||||
|
|
||||||
|
/**
|
||||||
|
* @var DataResponse $res
|
||||||
|
*/
|
||||||
|
$res = $this->sut->search(urlencode($query));
|
||||||
|
|
||||||
|
$this->assertEquals(500, $res->getStatus());
|
||||||
|
$this->assertEquals($errorMsg, $res->getData());
|
||||||
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* @covers ::categoryUpdate
|
||||||
|
* @dataProvider dataProviderCategoryUpdateNoName
|
||||||
|
*/
|
||||||
|
public function testCategoryUpdateNoName($requestParams): void {
|
||||||
|
$this->ensureCacheCheckTriggered();
|
||||||
|
|
||||||
|
$this->restParser->expects($this->once())->method('getParameters')->willReturn($requestParams);
|
||||||
|
|
||||||
|
$ret = $this->sut->categoryUpdate('');
|
||||||
|
|
||||||
|
$this->assertEquals(400, $ret->getStatus());
|
||||||
|
}
|
||||||
|
|
||||||
|
public function dataProviderCategoryUpdateNoName() {
|
||||||
|
yield [[]];
|
||||||
|
yield [[
|
||||||
|
'some', 'variable'
|
||||||
|
]];
|
||||||
|
yield [['name' => null]];
|
||||||
|
yield [['name' => '']];
|
||||||
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* @covers ::categoryUpdate
|
||||||
|
* @dataProvider dpCategoryUpdate
|
||||||
|
* @todo No business logic in controller
|
||||||
|
*/
|
||||||
|
public function testCategoryUpdate($cat, $oldCat, $recipes): void {
|
||||||
|
$this->ensureCacheCheckTriggered();
|
||||||
|
|
||||||
|
$this->recipeService->expects($this->once())->method('getRecipesByCategory')->with($oldCat)->willReturn($recipes);
|
||||||
|
$this->dbCacheService->expects($this->once())->method('updateCache');
|
||||||
|
|
||||||
|
$this->restParser->expects($this->once())->method('getParameters')->willReturn(['name' => $cat]);
|
||||||
|
|
||||||
|
$n = count($recipes);
|
||||||
|
$indices = array_map(function ($v) {
|
||||||
|
return [$v['recipe_id']];
|
||||||
|
}, $recipes);
|
||||||
|
$this->recipeService->expects($this->exactly($n))->method('getRecipeById')->withConsecutive(...$indices);
|
||||||
|
$this->recipeService->expects($this->exactly($n))->method('addRecipe')->with($this->callback(function ($p) use ($cat) {
|
||||||
|
return $p['recipeCategory'] === $cat;
|
||||||
|
}));
|
||||||
|
|
||||||
|
/**
|
||||||
|
* @var DataResponse $ret
|
||||||
|
*/
|
||||||
|
$ret = $this->sut->categoryUpdate(urlencode($oldCat));
|
||||||
|
|
||||||
|
$this->assertEquals(200, $ret->getStatus());
|
||||||
|
$this->assertEquals($cat, $ret->getData());
|
||||||
|
}
|
||||||
|
|
||||||
|
public function dpCategoryUpdate() {
|
||||||
|
return [
|
||||||
|
'noRecipes' => [
|
||||||
|
'new Category Name',
|
||||||
|
'Old category',
|
||||||
|
[]
|
||||||
|
],
|
||||||
|
'someRecipes' => [
|
||||||
|
'new Category Name',
|
||||||
|
'Old category',
|
||||||
|
[
|
||||||
|
[
|
||||||
|
'name' => 'First recipe',
|
||||||
|
'recipeCategory' => 'some fancy category',
|
||||||
|
'recipe_id' => 123,
|
||||||
|
],
|
||||||
|
[
|
||||||
|
'name' => 'Second recipe',
|
||||||
|
'recipeCategory' => 'some fancy category',
|
||||||
|
'recipe_id' => 124,
|
||||||
|
],
|
||||||
|
]
|
||||||
|
],
|
||||||
|
];
|
||||||
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* @covers ::categoryUpdate
|
||||||
|
*/
|
||||||
|
public function testCategoryUpdateFailure(): void {
|
||||||
|
$this->ensureCacheCheckTriggered();
|
||||||
|
|
||||||
|
$this->restParser->expects($this->once())->method('getParameters')->willReturn(['name' => 'New category']);
|
||||||
|
|
||||||
|
$errorMsg = 'Something bad has happened.';
|
||||||
|
$oldCat = 'Old category';
|
||||||
|
|
||||||
|
$this->recipeService->expects($this->once())->method('getRecipesByCategory')->with($oldCat)->willThrowException(new Exception($errorMsg));
|
||||||
|
|
||||||
|
/**
|
||||||
|
* @var DataResponse $ret
|
||||||
|
*/
|
||||||
|
$ret = $this->sut->categoryUpdate(urlencode($oldCat));
|
||||||
|
|
||||||
|
$this->assertEquals(500, $ret->getStatus());
|
||||||
|
$this->assertEquals($errorMsg, $ret->getData());
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
@ -1,6 +1,6 @@
|
||||||
<?php
|
<?php
|
||||||
|
|
||||||
namespace OCA\Cookbook\tests\Unit\Controller;
|
namespace OCA\Cookbook\tests\Unit\Controller\v1;
|
||||||
|
|
||||||
use Exception;
|
use Exception;
|
||||||
use OCP\IRequest;
|
use OCP\IRequest;
|
||||||
|
|
@ -15,12 +15,12 @@ use OCP\AppFramework\Http\DataResponse;
|
||||||
use OCA\Cookbook\Service\DbCacheService;
|
use OCA\Cookbook\Service\DbCacheService;
|
||||||
use OCA\Cookbook\Helper\RestParameterParser;
|
use OCA\Cookbook\Helper\RestParameterParser;
|
||||||
use PHPUnit\Framework\MockObject\MockObject;
|
use PHPUnit\Framework\MockObject\MockObject;
|
||||||
use OCA\Cookbook\Controller\RecipeController;
|
use OCA\Cookbook\Controller\v1\RecipeController;
|
||||||
use OCP\AppFramework\Http\FileDisplayResponse;
|
use OCP\AppFramework\Http\FileDisplayResponse;
|
||||||
use OCA\Cookbook\Exception\RecipeExistsException;
|
use OCA\Cookbook\Exception\RecipeExistsException;
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* @coversDefaultClass \OCA\Cookbook\Controller\RecipeController
|
* @coversDefaultClass \OCA\Cookbook\Controller\v1\RecipeController
|
||||||
* @covers ::<private>
|
* @covers ::<private>
|
||||||
* @covers ::<protected>
|
* @covers ::<protected>
|
||||||
*/
|
*/
|
||||||
|
|
@ -173,7 +173,7 @@ class RecipeControllerTest extends TestCase {
|
||||||
$imageUrl = "/path/to/image/of/id/123";
|
$imageUrl = "/path/to/image/of/id/123";
|
||||||
|
|
||||||
$this->urlGenerator->method('linkToRoute')->with(
|
$this->urlGenerator->method('linkToRoute')->with(
|
||||||
'cookbook.recipe.image',
|
'cookbook.recipe_v1.image',
|
||||||
$this->callback(function ($p) use ($id) {
|
$this->callback(function ($p) use ($id) {
|
||||||
return isset($p['size']) && $p['id'] === $id;
|
return isset($p['size']) && $p['id'] === $id;
|
||||||
})
|
})
|
||||||
|
|
@ -308,8 +308,8 @@ class RecipeControllerTest extends TestCase {
|
||||||
}
|
}
|
||||||
|
|
||||||
$this->urlGenerator->method('linkToRoute')->will($this->returnCallback(function ($name, $params) {
|
$this->urlGenerator->method('linkToRoute')->will($this->returnCallback(function ($name, $params) {
|
||||||
if ($name !== 'cookbook.recipe.image') {
|
if ($name !== 'cookbook.recipe_v1.image') {
|
||||||
throw new Exception('Must use correct controller');
|
throw new Exception('Must use correct image route');
|
||||||
}
|
}
|
||||||
|
|
||||||
$id = $params['id'];
|
$id = $params['id'];
|
||||||
Загрузка…
Ссылка в новой задаче