bitPickups/friendica_2019-12_sharedHosting_hotFix
1
1
Fork 0
Code Pull requests Activity

Add tests to prove error https://github.com/friendica/friendica/issues/11023

Browse source 
- Use `$request` instead of `$post` in `BaseModule->run`
This commit is contained in:
Philipp Holzer 2021-11-28 14:52:13 +01:00 committed by Hypolite Petovan
parent 75f0c58c5f
commit d433c06632
5 changed files with 100 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

42
tests/datasets/photo/photo.fixture.php Normal file
View file

@ -0,0 +1,42 @@
<?php
return [
'storage' => [
[
'id' => 1,
'data' => file_get_contents(__DIR__ . '/test_photo.content'),
],
],
'photo' => [
[
"id" => 1,
"uid" => 42,
"contact-id" => 0,
"guid" => "71361a283a6aa6a2",
"resource-id" => "709057080661a283a6aa598501504178",
"hash" => "fc84f6c0ae79f626e4b7f65f438e60c1",
"created" => "2021-11-27 19:14:47",
"edited" => "2021-11-27 19:14:47",
"title" => "",
"desc" => "",
"album" => "test_album",
"photo-type" => 0,
"filename" => "traffic-signs-g1c8094e4f_1920.jpg",
"type" => "image/jpeg",
"height" => 226,
"width" => 320,
"datasize" => 96968,
"data" => "",
"scale" => 2,
"profile" => 0,
"allow_cid" => "",
"allow_gid" => "",
"deny_cid" => "",
"deny_gid" => "",
"accessible" => 0,
"backend-class" => "Database",
"backend-ref" => "1",
"updated" => "0001-01-01 00:00:00",
],
],
];

1
tests/datasets/photo/test_photo.content Normal file
View file

File diff suppressed because one or more lines are too long

33
tests/src/Module/Api/Friendica/Photo/DeleteTest.php
View file

@ -46,8 +46,37 @@ class DeleteTest extends ApiTest
(new Delete(DI::app(), DI::l10n(), DI::baseUrl(), DI::args(), DI::logger(), DI::profiler(), DI::apiResponse(), ['REQUEST_METHOD' => Router::POST]))->run(['photo_id' => 1]);
}
public function testWithCorrectPhotoId()
public function testValidWithPost()
{
self::markTestIncomplete('We need to add a dataset for this.');
$this->loadFixture(__DIR__ . '/../../../../../datasets/photo/photo.fixture.php', DI::dba());
$delete = new Delete(DI::app(), DI::l10n(), DI::baseUrl(), DI::args(), DI::logger(), DI::profiler(), DI::apiResponse(), ['REQUEST_METHOD' => Router::POST]);
$response = $delete->run([], ['photo_id' => '709057080661a283a6aa598501504178']);
$responseText = (string)$response->getBody();
self::assertJson($responseText);
$json = json_decode($responseText);
self::assertEquals('deleted', $json->result);
self::assertEquals('photo with id `709057080661a283a6aa598501504178` has been deleted from server.', $json->message);
}
public function testValidWithDelete()
{
$this->loadFixture(__DIR__ . '/../../../../../datasets/photo/photo.fixture.php', DI::dba());
$delete = new Delete(DI::app(), DI::l10n(), DI::baseUrl(), DI::args(), DI::logger(), DI::profiler(), DI::apiResponse(), ['REQUEST_METHOD' => Router::DELETE]);
$response = $delete->run([], ['photo_id' => '709057080661a283a6aa598501504178']);
$responseText = (string)$response->getBody();
self::assertJson($responseText);
$json = json_decode($responseText);
self::assertEquals('deleted', $json->result);
self::assertEquals('photo with id `709057080661a283a6aa598501504178` has been deleted from server.', $json->message);
}
}

16
tests/src/Module/Api/Friendica/Photoalbum/DeleteTest.php
View file

@ -42,8 +42,20 @@ class DeleteTest extends ApiTest
(new Delete(DI::app(), DI::l10n(), DI::baseUrl(), DI::args(), DI::logger(), DI::profiler(), DI::apiResponse(), ['REQUEST_METHOD' => Router::POST]))->run(['album' => 'album_name']);
}
public function testValid()
public function testValidWithDelete()
{
self::markTestIncomplete('We need to add a dataset for this.');
$this->loadFixture(__DIR__ . '/../../../../../datasets/photo/photo.fixture.php', DI::dba());
$delete = new Delete(DI::app(), DI::l10n(), DI::baseUrl(), DI::args(), DI::logger(), DI::profiler(), DI::apiResponse(), ['REQUEST_METHOD' => Router::DELETE]);
$response = $delete->run([], ['album' => 'test_album']);
$responseText = (string)$response->getBody();
self::assertJson($responseText);
$json = json_decode($responseText);
self::assertEquals('deleted', $json->result);
self::assertEquals('album `test_album` with all containing photos has been deleted.', $json->message);
}
}

13
tests/src/Module/Api/Friendica/Photoalbum/UpdateTest.php
View file

@ -54,6 +54,17 @@ class UpdateTest extends ApiTest
public function testValid()
{
self::markTestIncomplete('We need to add a dataset for this.');
$this->loadFixture(__DIR__ . '/../../../../../datasets/photo/photo.fixture.php', DI::dba());
$response = (new Update(DI::app(), DI::l10n(), DI::baseUrl(), DI::args(), DI::logger(), DI::profiler(), DI::apiResponse(), ['REQUEST_METHOD' => Router::POST]))->run([], ['album' => 'test_album', 'album_new' => 'test_album_2']);
$responseBody = (string)$response->getBody();
self::assertJson($responseBody);
$json = json_decode($responseBody);
self::assertEquals('updated', $json->result);
self::assertEquals('album `test_album` with all containing photos has been renamed to `test_album_2`.', $json->message);
}
}