X-Git-Url: https://git.mxchange.org/?a=blobdiff_plain;f=tests%2Finclude%2FApiTest.php;h=3410f3049df83a1660b9c7cd0c76ca1c773c653d;hb=0a82fe4211f73cf10107feb69fe38eaa85eb61f8;hp=df3a7e9401f9335e0fe96a49c64d0e0b0380373d;hpb=7e926ef6877db74696abf6c1f098bdb5c09732b8;p=friendica.git diff --git a/tests/include/ApiTest.php b/tests/include/ApiTest.php index df3a7e9401..3410f3049d 100644 --- a/tests/include/ApiTest.php +++ b/tests/include/ApiTest.php @@ -5,15 +5,16 @@ namespace Friendica\Test; +use Dice\Dice; use Friendica\App; -use Friendica\Core\Config; -use Friendica\Core\Config\Cache; -use Friendica\Core\PConfig; +use Friendica\BaseObject; +use Friendica\Core\Config\Configuration; +use Friendica\Core\Config\PConfiguration; use Friendica\Core\Protocol; use Friendica\Core\System; -use Friendica\Factory; +use Friendica\Database\Database; use Friendica\Network\HTTPException; -use Friendica\Util\BasePath; +use Friendica\Test\Util\Database\StaticDatabase; use Monolog\Handler\TestHandler; require_once __DIR__ . '/../../include/api.php'; @@ -31,9 +32,6 @@ class ApiTest extends DatabaseTest */ protected $logOutput; - /** @var App */ - protected $app; - /** @var array */ protected $selfUser; /** @var array */ @@ -43,39 +41,68 @@ class ApiTest extends DatabaseTest protected $wrongUserId; + /** @var App */ + protected $app; + + /** @var Configuration */ + protected $config; + + /** @var Dice */ + protected $dice; + /** * Create variables used by tests. */ - public function setUp() + protected function setUp() { - $basePath = BasePath::create(dirname(__DIR__) . '/../'); - $mode = new App\Mode($basePath); - $configLoader = new Cache\ConfigCacheLoader($basePath, $mode); - $configCache = Factory\ConfigFactory::createCache($configLoader); - $profiler = Factory\ProfilerFactory::create($configCache); - Factory\DBFactory::init($basePath, $configCache, $profiler, $_SERVER); - $config = Factory\ConfigFactory::createConfig($configCache); - Factory\ConfigFactory::createPConfig($configCache); - $logger = Factory\LoggerFactory::create('test', $config, $profiler); - $this->app = new App($basePath, $config, $mode, $logger, $profiler, false); - parent::setUp(); + $this->dice = (new Dice()) + ->addRules(include __DIR__ . '/../../static/dependencies.config.php') + ->addRule(Database::class, ['instanceOf' => StaticDatabase::class, 'shared' => true]); + BaseObject::setDependencyInjection($this->dice); + + /** @var Database $dba */ + $dba = $this->dice->create(Database::class); + + /** @var Configuration $config */ + $this->config = $this->dice->create(Configuration::class); + + $this->config->set('system', 'url', 'http://localhost'); + $this->config->set('system', 'hostname', 'localhost'); + $this->config->set('system', 'worker_dont_fork', true); + + // Default config + $this->config->set('config', 'hostname', 'localhost'); + $this->config->set('system', 'throttle_limit_day', 100); + $this->config->set('system', 'throttle_limit_week', 100); + $this->config->set('system', 'throttle_limit_month', 100); + $this->config->set('system', 'theme', 'system_theme'); + + // Load the API dataset for the whole API + $this->loadFixture(__DIR__ . '/../datasets/api.fixture.php', $dba); + + /** @var App app */ + $this->app = BaseObject::getApp(); + + $this->app->argc = 1; + $this->app->argv = ['home']; + // User data that the test database is populated with - $this->selfUser = [ - 'id' => 42, + $this->selfUser = [ + 'id' => 42, 'name' => 'Self contact', 'nick' => 'selfcontact', 'nurl' => 'http://localhost/profile/selfcontact' ]; $this->friendUser = [ - 'id' => 44, + 'id' => 44, 'name' => 'Friend contact', 'nick' => 'friendcontact', 'nurl' => 'http://localhost/profile/friendcontact' ]; - $this->otherUser = [ - 'id' => 43, + $this->otherUser = [ + 'id' => 43, 'name' => 'othercontact', 'nick' => 'othercontact', 'nurl' => 'http://localhost/profile/othercontact' @@ -86,37 +113,21 @@ class ApiTest extends DatabaseTest // Most API require login so we force the session $_SESSION = [ - 'allow_api' => true, + 'allow_api' => true, 'authenticated' => true, - 'uid' => $this->selfUser['id'] + 'uid' => $this->selfUser['id'] ]; - Config::set('system', 'url', 'http://localhost'); - Config::set('system', 'hostname', 'localhost'); - Config::set('system', 'worker_dont_fork', true); - - // Default config - Config::set('config', 'hostname', 'localhost'); - Config::set('system', 'throttle_limit_day', 100); - Config::set('system', 'throttle_limit_week', 100); - Config::set('system', 'throttle_limit_month', 100); - Config::set('system', 'theme', 'system_theme'); - } - - /** - * Cleanup variables used by tests. - */ - protected function tearDown() - { - parent::tearDown(); - - $this->app->argc = 1; - $this->app->argv = ['home']; + $_POST = []; + $_GET = []; + $_SERVER = []; } /** * Assert that an user array contains expected keys. + * * @param array $user User array + * * @return void */ private function assertSelfUser(array $user) @@ -133,7 +144,9 @@ class ApiTest extends DatabaseTest /** * Assert that an user array contains expected keys. + * * @param array $user User array + * * @return void */ private function assertOtherUser(array $user) @@ -148,7 +161,9 @@ class ApiTest extends DatabaseTest /** * Assert that a status array contains expected keys. + * * @param array $status Status array + * * @return void */ private function assertStatus(array $status) @@ -160,7 +175,9 @@ class ApiTest extends DatabaseTest /** * Assert that a list array contains expected keys. + * * @param array $list List array + * * @return void */ private function assertList(array $list) @@ -174,19 +191,22 @@ class ApiTest extends DatabaseTest /** * Assert that the string is XML and contain the root element. + * * @param string $result XML string * @param string $root_element Root element name + * * @return void */ private function assertXml($result, $root_element) { $this->assertStringStartsWith('', $result); - $this->assertContains('<'.$root_element, $result); + $this->assertContains('<' . $root_element, $result); // We could probably do more checks here. } /** * Get the path to a temporary empty PNG image. + * * @return string Path */ private function getTempImage() @@ -195,7 +215,7 @@ class ApiTest extends DatabaseTest file_put_contents( $tmpFile, base64_decode( - // Empty 1x1 px PNG image + // Empty 1x1 px PNG image 'iVBORw0KGgoAAAANSUhEUgAAAAEAAAABCAYAAAAfFcSJAAAADUlEQVR42mP8/5+hHgAHggJ/PchI7wAAAABJRU5ErkJggg==' ) ); @@ -205,6 +225,7 @@ class ApiTest extends DatabaseTest /** * Test the api_user() function. + * * @return void */ public function testApiUser() @@ -214,6 +235,7 @@ class ApiTest extends DatabaseTest /** * Test the api_user() function with an unallowed user. + * * @return void */ public function testApiUserWithUnallowedUser() @@ -224,6 +246,7 @@ class ApiTest extends DatabaseTest /** * Test the api_source() function. + * * @return void */ public function testApiSource() @@ -233,6 +256,7 @@ class ApiTest extends DatabaseTest /** * Test the api_source() function with a Twidere user agent. + * * @return void */ public function testApiSourceWithTwidere() @@ -243,6 +267,7 @@ class ApiTest extends DatabaseTest /** * Test the api_source() function with a GET parameter. + * * @return void */ public function testApiSourceWithGet() @@ -253,6 +278,7 @@ class ApiTest extends DatabaseTest /** * Test the api_date() function. + * * @return void */ public function testApiDate() @@ -262,6 +288,7 @@ class ApiTest extends DatabaseTest /** * Test the api_register_func() function. + * * @return void */ public function testApiRegisterFunc() @@ -283,6 +310,7 @@ class ApiTest extends DatabaseTest /** * Test the api_login() function without any login. + * * @return void * @runInSeparateProcess * @expectedException Friendica\Network\HTTPException\UnauthorizedException @@ -294,6 +322,7 @@ class ApiTest extends DatabaseTest /** * Test the api_login() function with a bad login. + * * @return void * @runInSeparateProcess * @expectedException Friendica\Network\HTTPException\UnauthorizedException @@ -306,6 +335,7 @@ class ApiTest extends DatabaseTest /** * Test the api_login() function with oAuth. + * * @return void */ public function testApiLoginWithOauth() @@ -315,6 +345,7 @@ class ApiTest extends DatabaseTest /** * Test the api_login() function with authentication provided by an addon. + * * @return void */ public function testApiLoginWithAddonAuth() @@ -324,18 +355,20 @@ class ApiTest extends DatabaseTest /** * Test the api_login() function with a correct login. + * * @return void * @runInSeparateProcess */ public function testApiLoginWithCorrectLogin() { $_SERVER['PHP_AUTH_USER'] = 'Test user'; - $_SERVER['PHP_AUTH_PW'] = 'password'; + $_SERVER['PHP_AUTH_PW'] = 'password'; api_login($this->app); } /** * Test the api_login() function with a remote user. + * * @return void * @runInSeparateProcess * @expectedException Friendica\Network\HTTPException\UnauthorizedException @@ -348,6 +381,7 @@ class ApiTest extends DatabaseTest /** * Test the api_check_method() function. + * * @return void */ public function testApiCheckMethod() @@ -357,6 +391,7 @@ class ApiTest extends DatabaseTest /** * Test the api_check_method() function with a correct method. + * * @return void */ public function testApiCheckMethodWithCorrectMethod() @@ -367,6 +402,7 @@ class ApiTest extends DatabaseTest /** * Test the api_check_method() function with a wildcard. + * * @return void */ public function testApiCheckMethodWithWildcard() @@ -376,20 +412,21 @@ class ApiTest extends DatabaseTest /** * Test the api_call() function. + * * @return void * @runInSeparateProcess */ public function testApiCall() { global $API; - $API['api_path'] = [ + $API['api_path'] = [ 'method' => 'method', - 'func' => function () { + 'func' => function () { return ['data' => ['some_data']]; } ]; $_SERVER['REQUEST_METHOD'] = 'method'; - $_GET['callback'] = 'callback_name'; + $_GET['callback'] = 'callback_name'; $this->app->query_string = 'api_path'; $this->assertEquals( @@ -400,27 +437,28 @@ class ApiTest extends DatabaseTest /** * Test the api_call() function with the profiled enabled. + * * @return void * @runInSeparateProcess */ public function testApiCallWithProfiler() { global $API; - $API['api_path'] = [ + $API['api_path'] = [ 'method' => 'method', - 'func' => function () { + 'func' => function () { return ['data' => ['some_data']]; } ]; $_SERVER['REQUEST_METHOD'] = 'method'; - Config::set('system', 'profiler', true); - Config::set('rendertime', 'callstack', true); + $this->config->set('system', 'profiler', true); + $this->config->set('rendertime', 'callstack', true); $this->app->callstack = [ - 'database' => ['some_function' => 200], + 'database' => ['some_function' => 200], 'database_write' => ['some_function' => 200], - 'cache' => ['some_function' => 200], - 'cache_write' => ['some_function' => 200], - 'network' => ['some_function' => 200] + 'cache' => ['some_function' => 200], + 'cache_write' => ['some_function' => 200], + 'network' => ['some_function' => 200] ]; $this->app->query_string = 'api_path'; @@ -432,15 +470,16 @@ class ApiTest extends DatabaseTest /** * Test the api_call() function without any result. + * * @return void * @runInSeparateProcess */ public function testApiCallWithNoResult() { global $API; - $API['api_path'] = [ + $API['api_path'] = [ 'method' => 'method', - 'func' => function () { + 'func' => function () { return false; } ]; @@ -455,6 +494,7 @@ class ApiTest extends DatabaseTest /** * Test the api_call() function with an unimplemented API. + * * @return void * @runInSeparateProcess */ @@ -468,15 +508,16 @@ class ApiTest extends DatabaseTest /** * Test the api_call() function with a JSON result. + * * @return void * @runInSeparateProcess */ public function testApiCallWithJson() { global $API; - $API['api_path'] = [ + $API['api_path'] = [ 'method' => 'method', - 'func' => function () { + 'func' => function () { return ['data' => ['some_data']]; } ]; @@ -491,15 +532,16 @@ class ApiTest extends DatabaseTest /** * Test the api_call() function with an XML result. + * * @return void * @runInSeparateProcess */ public function testApiCallWithXml() { global $API; - $API['api_path'] = [ + $API['api_path'] = [ 'method' => 'method', - 'func' => function () { + 'func' => function () { return 'some_data'; } ]; @@ -514,15 +556,16 @@ class ApiTest extends DatabaseTest /** * Test the api_call() function with an RSS result. + * * @return void * @runInSeparateProcess */ public function testApiCallWithRss() { global $API; - $API['api_path'] = [ + $API['api_path'] = [ 'method' => 'method', - 'func' => function () { + 'func' => function () { return 'some_data'; } ]; @@ -530,23 +573,24 @@ class ApiTest extends DatabaseTest $this->app->query_string = 'api_path.rss'; $this->assertEquals( - ''."\n". - 'some_data', + '' . "\n" . + 'some_data', api_call($this->app) ); } /** * Test the api_call() function with an Atom result. + * * @return void * @runInSeparateProcess */ public function testApiCallWithAtom() { global $API; - $API['api_path'] = [ + $API['api_path'] = [ 'method' => 'method', - 'func' => function () { + 'func' => function () { return 'some_data'; } ]; @@ -554,14 +598,15 @@ class ApiTest extends DatabaseTest $this->app->query_string = 'api_path.atom'; $this->assertEquals( - ''."\n". - 'some_data', + '' . "\n" . + 'some_data', api_call($this->app) ); } /** * Test the api_call() function with an unallowed method. + * * @return void * @runInSeparateProcess */ @@ -579,15 +624,16 @@ class ApiTest extends DatabaseTest /** * Test the api_call() function with an unauthorized user. + * * @return void * @runInSeparateProcess */ public function testApiCallWithWrongAuth() { global $API; - $API['api_path'] = [ + $API['api_path'] = [ 'method' => 'method', - 'auth' => true + 'auth' => true ]; $_SERVER['REQUEST_METHOD'] = 'method'; $_SESSION['authenticated'] = false; @@ -601,85 +647,90 @@ class ApiTest extends DatabaseTest /** * Test the api_error() function with a JSON result. + * * @return void * @runInSeparateProcess */ public function testApiErrorWithJson() { $this->assertEquals( - '{"status":{"error":"error_message","code":"200 Friendica\\\\Network\\\\HTTP","request":""}}', - api_error('json', new HTTPException('error_message')) + '{"status":{"error":"error_message","code":"200 OK","request":""}}', + api_error('json', new HTTPException\OKException('error_message')) ); } /** * Test the api_error() function with an XML result. + * * @return void * @runInSeparateProcess */ public function testApiErrorWithXml() { $this->assertEquals( - ''."\n". - ''."\n". - ' error_message'."\n". - ' 200 Friendica\Network\HTTP'."\n". - ' '."\n". - ''."\n", - api_error('xml', new HTTPException('error_message')) + '' . "\n" . + '' . "\n" . + ' error_message' . "\n" . + ' 200 OK' . "\n" . + ' ' . "\n" . + '' . "\n", + api_error('xml', new HTTPException\OKException('error_message')) ); } /** * Test the api_error() function with an RSS result. + * * @return void * @runInSeparateProcess */ public function testApiErrorWithRss() { $this->assertEquals( - ''."\n". - ''."\n". - ' error_message'."\n". - ' 200 Friendica\Network\HTTP'."\n". - ' '."\n". - ''."\n", - api_error('rss', new HTTPException('error_message')) + '' . "\n" . + '' . "\n" . + ' error_message' . "\n" . + ' 200 OK' . "\n" . + ' ' . "\n" . + '' . "\n", + api_error('rss', new HTTPException\OKException('error_message')) ); } /** * Test the api_error() function with an Atom result. + * * @return void * @runInSeparateProcess */ public function testApiErrorWithAtom() { $this->assertEquals( - ''."\n". - ''."\n". - ' error_message'."\n". - ' 200 Friendica\Network\HTTP'."\n". - ' '."\n". - ''."\n", - api_error('atom', new HTTPException('error_message')) + '' . "\n" . + '' . "\n" . + ' error_message' . "\n" . + ' 200 OK' . "\n" . + ' ' . "\n" . + '' . "\n", + api_error('atom', new HTTPException\OKException('error_message')) ); } /** * Test the api_rss_extra() function. + * * @return void */ public function testApiRssExtra() { $user_info = ['url' => 'user_url', 'lang' => 'en']; - $result = api_rss_extra($this->app, [], $user_info); + $result = api_rss_extra($this->app, [], $user_info); $this->assertEquals($user_info, $result['$user']); $this->assertEquals($user_info['url'], $result['$rss']['alternate']); $this->assertArrayHasKey('self', $result['$rss']); @@ -692,8 +743,8 @@ class ApiTest extends DatabaseTest /** * Test the api_rss_extra() function without any user info. + * * @return void - * @runInSeparateProcess */ public function testApiRssExtraWithoutUserInfo() { @@ -710,6 +761,7 @@ class ApiTest extends DatabaseTest /** * Test the api_unique_id_to_nurl() function. + * * @return void */ public function testApiUniqueIdToNurl() @@ -719,6 +771,7 @@ class ApiTest extends DatabaseTest /** * Test the api_unique_id_to_nurl() function with a correct ID. + * * @return void */ public function testApiUniqueIdToNurlWithCorrectId() @@ -728,8 +781,8 @@ class ApiTest extends DatabaseTest /** * Test the api_get_user() function. + * * @return void - * @runInSeparateProcess */ public function testApiGetUser() { @@ -742,12 +795,13 @@ class ApiTest extends DatabaseTest /** * Test the api_get_user() function with a Frio schema. + * * @return void - * @runInSeparateProcess */ public function testApiGetUserWithFrioSchema() { - PConfig::set($this->selfUser['id'], 'frio', 'schema', 'red'); + $pConfig = $this->dice->create(PConfiguration::class); + $pConfig->set($this->selfUser['id'], 'frio', 'schema', 'red'); $user = api_get_user($this->app); $this->assertSelfUser($user); $this->assertEquals('708fa0', $user['profile_sidebar_fill_color']); @@ -757,15 +811,16 @@ class ApiTest extends DatabaseTest /** * Test the api_get_user() function with a custom Frio schema. + * * @return void - * @runInSeparateProcess */ public function testApiGetUserWithCustomFrioSchema() { - $ret1 = PConfig::set($this->selfUser['id'], 'frio', 'schema', '---'); - $ret2 = PConfig::set($this->selfUser['id'], 'frio', 'nav_bg', '#123456'); - $ret3 = PConfig::set($this->selfUser['id'], 'frio', 'link_color', '#123456'); - $ret4 = PConfig::set($this->selfUser['id'], 'frio', 'background_color', '#123456'); + $pConfig = $this->dice->create(PConfiguration::class); + $pConfig->set($this->selfUser['id'], 'frio', 'schema', '---'); + $pConfig->set($this->selfUser['id'], 'frio', 'nav_bg', '#123456'); + $pConfig->set($this->selfUser['id'], 'frio', 'link_color', '#123456'); + $pConfig->set($this->selfUser['id'], 'frio', 'background_color', '#123456'); $user = api_get_user($this->app); $this->assertSelfUser($user); $this->assertEquals('123456', $user['profile_sidebar_fill_color']); @@ -775,12 +830,13 @@ class ApiTest extends DatabaseTest /** * Test the api_get_user() function with an empty Frio schema. + * * @return void - * @runInSeparateProcess */ public function testApiGetUserWithEmptyFrioSchema() { - PConfig::set($this->selfUser['id'], 'frio', 'schema', '---'); + $pConfig = $this->dice->create(PConfiguration::class); + $pConfig->set($this->selfUser['id'], 'frio', 'schema', '---'); $user = api_get_user($this->app); $this->assertSelfUser($user); $this->assertEquals('708fa0', $user['profile_sidebar_fill_color']); @@ -790,21 +846,22 @@ class ApiTest extends DatabaseTest /** * Test the api_get_user() function with an user that is not allowed to use the API. + * * @return void * @runInSeparateProcess */ public function testApiGetUserWithoutApiUser() { $_SERVER['PHP_AUTH_USER'] = 'Test user'; - $_SERVER['PHP_AUTH_PW'] = 'password'; - $_SESSION['allow_api'] = false; + $_SERVER['PHP_AUTH_PW'] = 'password'; + $_SESSION['allow_api'] = false; $this->assertFalse(api_get_user($this->app)); } /** * Test the api_get_user() function with an user ID in a GET parameter. + * * @return void - * @runInSeparateProcess */ public function testApiGetUserWithGetId() { @@ -814,8 +871,8 @@ class ApiTest extends DatabaseTest /** * Test the api_get_user() function with a wrong user ID in a GET parameter. + * * @return void - * @runInSeparateProcess * @expectedException Friendica\Network\HTTPException\BadRequestException */ public function testApiGetUserWithWrongGetId() @@ -826,8 +883,8 @@ class ApiTest extends DatabaseTest /** * Test the api_get_user() function with an user name in a GET parameter. + * * @return void - * @runInSeparateProcess */ public function testApiGetUserWithGetName() { @@ -837,8 +894,8 @@ class ApiTest extends DatabaseTest /** * Test the api_get_user() function with a profile URL in a GET parameter. + * * @return void - * @runInSeparateProcess */ public function testApiGetUserWithGetUrl() { @@ -848,21 +905,21 @@ class ApiTest extends DatabaseTest /** * Test the api_get_user() function with an user ID in the API path. + * * @return void - * @runInSeparateProcess */ public function testApiGetUserWithNumericCalledApi() { global $called_api; - $called_api = ['api_path']; - $this->app->argv[1] = $this->otherUser['id'].'.json'; + $called_api = ['api_path']; + $this->app->argv[1] = $this->otherUser['id'] . '.json'; $this->assertOtherUser(api_get_user($this->app)); } /** * Test the api_get_user() function with the $called_api global variable. + * * @return void - * @runInSeparateProcess */ public function testApiGetUserWithCalledApi() { @@ -873,8 +930,8 @@ class ApiTest extends DatabaseTest /** * Test the api_get_user() function with a valid user. + * * @return void - * @runInSeparateProcess */ public function testApiGetUserWithCorrectUser() { @@ -883,8 +940,8 @@ class ApiTest extends DatabaseTest /** * Test the api_get_user() function with a wrong user ID. + * * @return void - * @runInSeparateProcess * @expectedException Friendica\Network\HTTPException\BadRequestException */ public function testApiGetUserWithWrongUser() @@ -894,8 +951,8 @@ class ApiTest extends DatabaseTest /** * Test the api_get_user() function with a 0 user ID. + * * @return void - * @runInSeparateProcess */ public function testApiGetUserWithZeroUser() { @@ -904,8 +961,8 @@ class ApiTest extends DatabaseTest /** * Test the api_item_get_user() function. + * * @return void - * @runInSeparateProcess */ public function testApiItemGetUser() { @@ -915,6 +972,7 @@ class ApiTest extends DatabaseTest /** * Test the api_item_get_user() function with a different item parent. + * * @return void */ public function testApiItemGetUserWithDifferentParent() @@ -926,6 +984,7 @@ class ApiTest extends DatabaseTest /** * Test the api_walk_recursive() function. + * * @return void */ public function testApiWalkRecursive() @@ -945,6 +1004,7 @@ class ApiTest extends DatabaseTest /** * Test the api_walk_recursive() function with an array. + * * @return void */ public function testApiWalkRecursiveWithArray() @@ -964,74 +1024,80 @@ class ApiTest extends DatabaseTest /** * Test the api_reformat_xml() function. + * * @return void */ public function testApiReformatXml() { $item = true; - $key = ''; + $key = ''; $this->assertTrue(api_reformat_xml($item, $key)); $this->assertEquals('true', $item); } /** * Test the api_reformat_xml() function with a statusnet_api key. + * * @return void */ public function testApiReformatXmlWithStatusnetKey() { $item = ''; - $key = 'statusnet_api'; + $key = 'statusnet_api'; $this->assertTrue(api_reformat_xml($item, $key)); $this->assertEquals('statusnet:api', $key); } /** * Test the api_reformat_xml() function with a friendica_api key. + * * @return void */ public function testApiReformatXmlWithFriendicaKey() { $item = ''; - $key = 'friendica_api'; + $key = 'friendica_api'; $this->assertTrue(api_reformat_xml($item, $key)); $this->assertEquals('friendica:api', $key); } /** * Test the api_create_xml() function. + * * @return void */ public function testApiCreateXml() { $this->assertEquals( - ''."\n". - ''."\n". - ' some_data'."\n". - ''."\n", + '' . "\n" . + '' . "\n" . + ' some_data' . "\n" . + '' . "\n", api_create_xml(['data' => ['some_data']], 'root_element') ); } /** * Test the api_create_xml() function without any XML namespace. + * * @return void */ public function testApiCreateXmlWithoutNamespaces() { $this->assertEquals( - ''."\n". - ''."\n". - ' some_data'."\n". - ''."\n", + '' . "\n" . + '' . "\n" . + ' some_data' . "\n" . + '' . "\n", api_create_xml(['data' => ['some_data']], 'ok') ); } /** * Test the api_format_data() function. + * * @return void */ public function testApiFormatData() @@ -1042,23 +1108,25 @@ class ApiTest extends DatabaseTest /** * Test the api_format_data() function with an XML result. + * * @return void */ public function testApiFormatDataWithXml() { $this->assertEquals( - ''."\n". - ''."\n". - ' some_data'."\n". - ''."\n", + '' . "\n" . + '' . "\n" . + ' some_data' . "\n" . + '' . "\n", api_format_data('root_element', 'xml', ['data' => ['some_data']]) ); } /** * Test the api_account_verify_credentials() function. + * * @return void */ public function testApiAccountVerifyCredentials() @@ -1068,6 +1136,7 @@ class ApiTest extends DatabaseTest /** * Test the api_account_verify_credentials() function without an authenticated user. + * * @return void * @expectedException Friendica\Network\HTTPException\ForbiddenException */ @@ -1079,6 +1148,7 @@ class ApiTest extends DatabaseTest /** * Test the requestdata() function. + * * @return void */ public function testRequestdata() @@ -1088,6 +1158,7 @@ class ApiTest extends DatabaseTest /** * Test the requestdata() function with a POST parameter. + * * @return void */ public function testRequestdataWithPost() @@ -1098,6 +1169,7 @@ class ApiTest extends DatabaseTest /** * Test the requestdata() function with a GET parameter. + * * @return void */ public function testRequestdataWithGet() @@ -1108,21 +1180,22 @@ class ApiTest extends DatabaseTest /** * Test the api_statuses_mediap() function. + * * @return void */ public function testApiStatusesMediap() { $this->app->argc = 2; - $_FILES = [ + $_FILES = [ 'media' => [ - 'id' => 666, - 'size' => 666, - 'width' => 666, - 'height' => 666, + 'id' => 666, + 'size' => 666, + 'width' => 666, + 'height' => 666, 'tmp_name' => $this->getTempImage(), - 'name' => 'spacer.png', - 'type' => 'image/png' + 'name' => 'spacer.png', + 'type' => 'image/png' ] ]; $_GET['status'] = 'Status content'; @@ -1133,6 +1206,7 @@ class ApiTest extends DatabaseTest /** * Test the api_statuses_mediap() function without an authenticated user. + * * @return void * @expectedException Friendica\Network\HTTPException\ForbiddenException */ @@ -1144,23 +1218,24 @@ class ApiTest extends DatabaseTest /** * Test the api_statuses_update() function. + * * @return void */ public function testApiStatusesUpdate() { - $_GET['status'] = 'Status content #friendica'; + $_GET['status'] = 'Status content #friendica'; $_GET['in_reply_to_status_id'] = -1; - $_GET['lat'] = 48; - $_GET['long'] = 7; - $_FILES = [ + $_GET['lat'] = 48; + $_GET['long'] = 7; + $_FILES = [ 'media' => [ - 'id' => 666, - 'size' => 666, - 'width' => 666, - 'height' => 666, + 'id' => 666, + 'size' => 666, + 'width' => 666, + 'height' => 666, 'tmp_name' => $this->getTempImage(), - 'name' => 'spacer.png', - 'type' => 'image/png' + 'name' => 'spacer.png', + 'type' => 'image/png' ] ]; @@ -1170,6 +1245,7 @@ class ApiTest extends DatabaseTest /** * Test the api_statuses_update() function with an HTML status. + * * @return void */ public function testApiStatusesUpdateWithHtml() @@ -1182,6 +1258,7 @@ class ApiTest extends DatabaseTest /** * Test the api_statuses_update() function without an authenticated user. + * * @return void * @expectedException Friendica\Network\HTTPException\ForbiddenException */ @@ -1193,6 +1270,7 @@ class ApiTest extends DatabaseTest /** * Test the api_statuses_update() function with a parent status. + * * @return void */ public function testApiStatusesUpdateWithParent() @@ -1202,6 +1280,7 @@ class ApiTest extends DatabaseTest /** * Test the api_statuses_update() function with a media_ids parameter. + * * @return void */ public function testApiStatusesUpdateWithMediaIds() @@ -1211,6 +1290,7 @@ class ApiTest extends DatabaseTest /** * Test the api_statuses_update() function with the throttle limit reached. + * * @return void */ public function testApiStatusesUpdateWithDayThrottleReached() @@ -1220,6 +1300,7 @@ class ApiTest extends DatabaseTest /** * Test the api_media_upload() function. + * * @return void * @expectedException Friendica\Network\HTTPException\BadRequestException */ @@ -1230,6 +1311,7 @@ class ApiTest extends DatabaseTest /** * Test the api_media_upload() function without an authenticated user. + * * @return void * @expectedException Friendica\Network\HTTPException\ForbiddenException */ @@ -1241,6 +1323,7 @@ class ApiTest extends DatabaseTest /** * Test the api_media_upload() function with an invalid uploaded media. + * * @return void * @expectedException Friendica\Network\HTTPException\InternalServerErrorException */ @@ -1248,7 +1331,7 @@ class ApiTest extends DatabaseTest { $_FILES = [ 'media' => [ - 'id' => 666, + 'id' => 666, 'tmp_name' => 'tmp_name' ] ]; @@ -1257,22 +1340,23 @@ class ApiTest extends DatabaseTest /** * Test the api_media_upload() function with an valid uploaded media. + * * @return void */ public function testApiMediaUploadWithValidMedia() { - $_FILES = [ + $_FILES = [ 'media' => [ - 'id' => 666, - 'size' => 666, - 'width' => 666, - 'height' => 666, + 'id' => 666, + 'size' => 666, + 'width' => 666, + 'height' => 666, 'tmp_name' => $this->getTempImage(), - 'name' => 'spacer.png', - 'type' => 'image/png' + 'name' => 'spacer.png', + 'type' => 'image/png' ] ]; - $app = \get_app(); + $app = \get_app(); $app->argc = 2; $result = api_media_upload(); @@ -1312,6 +1396,7 @@ class ApiTest extends DatabaseTest /** * Test the api_users_show() function. + * * @return void */ public function testApiUsersShow() @@ -1328,6 +1413,7 @@ class ApiTest extends DatabaseTest /** * Test the api_users_show() function with an XML result. + * * @return void */ public function testApiUsersShowWithXml() @@ -1338,28 +1424,31 @@ class ApiTest extends DatabaseTest /** * Test the api_users_search() function. + * * @return void */ public function testApiUsersSearch() { $_GET['q'] = 'othercontact'; - $result = api_users_search('json'); + $result = api_users_search('json'); $this->assertOtherUser($result['users'][0]); } /** * Test the api_users_search() function with an XML result. + * * @return void */ public function testApiUsersSearchWithXml() { $_GET['q'] = 'othercontact'; - $result = api_users_search('xml'); + $result = api_users_search('xml'); $this->assertXml($result, 'users'); } /** * Test the api_users_search() function without a GET q parameter. + * * @return void * @expectedException Friendica\Network\HTTPException\BadRequestException */ @@ -1370,6 +1459,7 @@ class ApiTest extends DatabaseTest /** * Test the api_users_lookup() function. + * * @return void * @expectedException Friendica\Network\HTTPException\NotFoundException */ @@ -1380,24 +1470,26 @@ class ApiTest extends DatabaseTest /** * Test the api_users_lookup() function with an user ID. + * * @return void */ public function testApiUsersLookupWithUserId() { $_REQUEST['user_id'] = $this->otherUser['id']; - $result = api_users_lookup('json'); + $result = api_users_lookup('json'); $this->assertOtherUser($result['users'][0]); } /** * Test the api_search() function. + * * @return void */ public function testApiSearch() { - $_REQUEST['q'] = 'reply'; + $_REQUEST['q'] = 'reply'; $_REQUEST['max_id'] = 10; - $result = api_search('json'); + $result = api_search('json'); foreach ($result['status'] as $status) { $this->assertStatus($status); $this->assertContains('reply', $status['text'], null, true); @@ -1406,13 +1498,14 @@ class ApiTest extends DatabaseTest /** * Test the api_search() function a count parameter. + * * @return void */ public function testApiSearchWithCount() { - $_REQUEST['q'] = 'reply'; + $_REQUEST['q'] = 'reply'; $_REQUEST['count'] = 20; - $result = api_search('json'); + $result = api_search('json'); foreach ($result['status'] as $status) { $this->assertStatus($status); $this->assertContains('reply', $status['text'], null, true); @@ -1421,13 +1514,14 @@ class ApiTest extends DatabaseTest /** * Test the api_search() function with an rpp parameter. + * * @return void */ public function testApiSearchWithRpp() { - $_REQUEST['q'] = 'reply'; + $_REQUEST['q'] = 'reply'; $_REQUEST['rpp'] = 20; - $result = api_search('json'); + $result = api_search('json'); foreach ($result['status'] as $status) { $this->assertStatus($status); $this->assertContains('reply', $status['text'], null, true); @@ -1436,12 +1530,13 @@ class ApiTest extends DatabaseTest /** * Test the api_search() function with an q parameter contains hashtag. + * * @return void */ public function testApiSearchWithHashtag() { $_REQUEST['q'] = '%23friendica'; - $result = api_search('json'); + $result = api_search('json'); foreach ($result['status'] as $status) { $this->assertStatus($status); $this->assertContains('#friendica', $status['text'], null, true); @@ -1450,14 +1545,15 @@ class ApiTest extends DatabaseTest /** * Test the api_search() function with an exclude_replies parameter. + * * @return void */ public function testApiSearchWithExcludeReplies() { - $_REQUEST['max_id'] = 10; + $_REQUEST['max_id'] = 10; $_REQUEST['exclude_replies'] = true; - $_REQUEST['q'] = 'friendica'; - $result = api_search('json'); + $_REQUEST['q'] = 'friendica'; + $result = api_search('json'); foreach ($result['status'] as $status) { $this->assertStatus($status); } @@ -1465,18 +1561,20 @@ class ApiTest extends DatabaseTest /** * Test the api_search() function without an authenticated user. + * * @return void * @expectedException Friendica\Network\HTTPException\ForbiddenException */ public function testApiSearchWithUnallowedUser() { $_SESSION['allow_api'] = false; - $_GET['screen_name'] = $this->selfUser['nick']; + $_GET['screen_name'] = $this->selfUser['nick']; api_search('json'); } /** * Test the api_search() function without any GET query parameter. + * * @return void * @expectedException Friendica\Network\HTTPException\BadRequestException */ @@ -1487,14 +1585,15 @@ class ApiTest extends DatabaseTest /** * Test the api_statuses_home_timeline() function. + * * @return void */ public function testApiStatusesHomeTimeline() { - $_REQUEST['max_id'] = 10; + $_REQUEST['max_id'] = 10; $_REQUEST['exclude_replies'] = true; $_REQUEST['conversation_id'] = 1; - $result = api_statuses_home_timeline('json'); + $result = api_statuses_home_timeline('json'); $this->assertNotEmpty($result['status']); foreach ($result['status'] as $status) { $this->assertStatus($status); @@ -1503,12 +1602,13 @@ class ApiTest extends DatabaseTest /** * Test the api_statuses_home_timeline() function with a negative page parameter. + * * @return void */ public function testApiStatusesHomeTimelineWithNegativePage() { $_REQUEST['page'] = -2; - $result = api_statuses_home_timeline('json'); + $result = api_statuses_home_timeline('json'); $this->assertNotEmpty($result['status']); foreach ($result['status'] as $status) { $this->assertStatus($status); @@ -1517,18 +1617,20 @@ class ApiTest extends DatabaseTest /** * Test the api_statuses_home_timeline() with an unallowed user. + * * @return void * @expectedException Friendica\Network\HTTPException\ForbiddenException */ public function testApiStatusesHomeTimelineWithUnallowedUser() { $_SESSION['allow_api'] = false; - $_GET['screen_name'] = $this->selfUser['nick']; + $_GET['screen_name'] = $this->selfUser['nick']; api_statuses_home_timeline('json'); } /** * Test the api_statuses_home_timeline() function with an RSS result. + * * @return void */ public function testApiStatusesHomeTimelineWithRss() @@ -1539,13 +1641,14 @@ class ApiTest extends DatabaseTest /** * Test the api_statuses_public_timeline() function. + * * @return void */ public function testApiStatusesPublicTimeline() { - $_REQUEST['max_id'] = 10; + $_REQUEST['max_id'] = 10; $_REQUEST['conversation_id'] = 1; - $result = api_statuses_public_timeline('json'); + $result = api_statuses_public_timeline('json'); $this->assertNotEmpty($result['status']); foreach ($result['status'] as $status) { $this->assertStatus($status); @@ -1554,13 +1657,14 @@ class ApiTest extends DatabaseTest /** * Test the api_statuses_public_timeline() function with the exclude_replies parameter. + * * @return void */ public function testApiStatusesPublicTimelineWithExcludeReplies() { - $_REQUEST['max_id'] = 10; + $_REQUEST['max_id'] = 10; $_REQUEST['exclude_replies'] = true; - $result = api_statuses_public_timeline('json'); + $result = api_statuses_public_timeline('json'); $this->assertNotEmpty($result['status']); foreach ($result['status'] as $status) { $this->assertStatus($status); @@ -1569,12 +1673,13 @@ class ApiTest extends DatabaseTest /** * Test the api_statuses_public_timeline() function with a negative page parameter. + * * @return void */ public function testApiStatusesPublicTimelineWithNegativePage() { $_REQUEST['page'] = -2; - $result = api_statuses_public_timeline('json'); + $result = api_statuses_public_timeline('json'); $this->assertNotEmpty($result['status']); foreach ($result['status'] as $status) { $this->assertStatus($status); @@ -1583,18 +1688,20 @@ class ApiTest extends DatabaseTest /** * Test the api_statuses_public_timeline() function with an unallowed user. + * * @return void * @expectedException Friendica\Network\HTTPException\ForbiddenException */ public function testApiStatusesPublicTimelineWithUnallowedUser() { $_SESSION['allow_api'] = false; - $_GET['screen_name'] = $this->selfUser['nick']; + $_GET['screen_name'] = $this->selfUser['nick']; api_statuses_public_timeline('json'); } /** * Test the api_statuses_public_timeline() function with an RSS result. + * * @return void */ public function testApiStatusesPublicTimelineWithRss() @@ -1605,12 +1712,13 @@ class ApiTest extends DatabaseTest /** * Test the api_statuses_networkpublic_timeline() function. + * * @return void */ public function testApiStatusesNetworkpublicTimeline() { $_REQUEST['max_id'] = 10; - $result = api_statuses_networkpublic_timeline('json'); + $result = api_statuses_networkpublic_timeline('json'); $this->assertNotEmpty($result['status']); foreach ($result['status'] as $status) { $this->assertStatus($status); @@ -1619,12 +1727,13 @@ class ApiTest extends DatabaseTest /** * Test the api_statuses_networkpublic_timeline() function with a negative page parameter. + * * @return void */ public function testApiStatusesNetworkpublicTimelineWithNegativePage() { $_REQUEST['page'] = -2; - $result = api_statuses_networkpublic_timeline('json'); + $result = api_statuses_networkpublic_timeline('json'); $this->assertNotEmpty($result['status']); foreach ($result['status'] as $status) { $this->assertStatus($status); @@ -1633,18 +1742,20 @@ class ApiTest extends DatabaseTest /** * Test the api_statuses_networkpublic_timeline() function with an unallowed user. + * * @return void * @expectedException Friendica\Network\HTTPException\ForbiddenException */ public function testApiStatusesNetworkpublicTimelineWithUnallowedUser() { $_SESSION['allow_api'] = false; - $_GET['screen_name'] = $this->selfUser['nick']; + $_GET['screen_name'] = $this->selfUser['nick']; api_statuses_networkpublic_timeline('json'); } /** * Test the api_statuses_networkpublic_timeline() function with an RSS result. + * * @return void */ public function testApiStatusesNetworkpublicTimelineWithRss() @@ -1655,6 +1766,7 @@ class ApiTest extends DatabaseTest /** * Test the api_statuses_show() function. + * * @return void * @expectedException Friendica\Network\HTTPException\BadRequestException */ @@ -1665,24 +1777,26 @@ class ApiTest extends DatabaseTest /** * Test the api_statuses_show() function with an ID. + * * @return void */ public function testApiStatusesShowWithId() { $this->app->argv[3] = 1; - $result = api_statuses_show('json'); + $result = api_statuses_show('json'); $this->assertStatus($result['status']); } /** * Test the api_statuses_show() function with the conversation parameter. + * * @return void */ public function testApiStatusesShowWithConversation() { - $this->app->argv[3] = 1; + $this->app->argv[3] = 1; $_REQUEST['conversation'] = 1; - $result = api_statuses_show('json'); + $result = api_statuses_show('json'); $this->assertNotEmpty($result['status']); foreach ($result['status'] as $status) { $this->assertStatus($status); @@ -1691,18 +1805,20 @@ class ApiTest extends DatabaseTest /** * Test the api_statuses_show() function with an unallowed user. + * * @return void * @expectedException Friendica\Network\HTTPException\ForbiddenException */ public function testApiStatusesShowWithUnallowedUser() { $_SESSION['allow_api'] = false; - $_GET['screen_name'] = $this->selfUser['nick']; + $_GET['screen_name'] = $this->selfUser['nick']; api_statuses_show('json'); } /** * Test the api_conversation_show() function. + * * @return void * @expectedException Friendica\Network\HTTPException\BadRequestException */ @@ -1713,14 +1829,15 @@ class ApiTest extends DatabaseTest /** * Test the api_conversation_show() function with an ID. + * * @return void */ public function testApiConversationShowWithId() { $this->app->argv[3] = 1; $_REQUEST['max_id'] = 10; - $_REQUEST['page'] = -2; - $result = api_conversation_show('json'); + $_REQUEST['page'] = -2; + $result = api_conversation_show('json'); $this->assertNotEmpty($result['status']); foreach ($result['status'] as $status) { $this->assertStatus($status); @@ -1729,18 +1846,20 @@ class ApiTest extends DatabaseTest /** * Test the api_conversation_show() function with an unallowed user. + * * @return void * @expectedException Friendica\Network\HTTPException\ForbiddenException */ public function testApiConversationShowWithUnallowedUser() { $_SESSION['allow_api'] = false; - $_GET['screen_name'] = $this->selfUser['nick']; + $_GET['screen_name'] = $this->selfUser['nick']; api_conversation_show('json'); } /** * Test the api_statuses_repeat() function. + * * @return void * @expectedException Friendica\Network\HTTPException\ForbiddenException */ @@ -1751,6 +1870,7 @@ class ApiTest extends DatabaseTest /** * Test the api_statuses_repeat() function without an authenticated user. + * * @return void * @expectedException Friendica\Network\HTTPException\ForbiddenException */ @@ -1762,22 +1882,24 @@ class ApiTest extends DatabaseTest /** * Test the api_statuses_repeat() function with an ID. + * * @return void */ public function testApiStatusesRepeatWithId() { $this->app->argv[3] = 1; - $result = api_statuses_repeat('json'); + $result = api_statuses_repeat('json'); $this->assertStatus($result['status']); // Also test with a shared status $this->app->argv[3] = 5; - $result = api_statuses_repeat('json'); + $result = api_statuses_repeat('json'); $this->assertStatus($result['status']); } /** * Test the api_statuses_destroy() function. + * * @return void * @expectedException Friendica\Network\HTTPException\BadRequestException */ @@ -1788,6 +1910,7 @@ class ApiTest extends DatabaseTest /** * Test the api_statuses_destroy() function without an authenticated user. + * * @return void * @expectedException Friendica\Network\HTTPException\ForbiddenException */ @@ -1799,53 +1922,58 @@ class ApiTest extends DatabaseTest /** * Test the api_statuses_destroy() function with an ID. + * * @return void */ public function testApiStatusesDestroyWithId() { $this->app->argv[3] = 1; - $result = api_statuses_destroy('json'); + $result = api_statuses_destroy('json'); $this->assertStatus($result['status']); } /** * Test the api_statuses_mentions() function. + * * @return void */ public function testApiStatusesMentions() { - $this->app->user = ['nickname' => $this->selfUser['nick']]; + $this->app->user = ['nickname' => $this->selfUser['nick']]; $_REQUEST['max_id'] = 10; - $result = api_statuses_mentions('json'); + $result = api_statuses_mentions('json'); $this->assertEmpty($result['status']); // We should test with mentions in the database. } /** * Test the api_statuses_mentions() function with a negative page parameter. + * * @return void */ public function testApiStatusesMentionsWithNegativePage() { $_REQUEST['page'] = -2; - $result = api_statuses_mentions('json'); + $result = api_statuses_mentions('json'); $this->assertEmpty($result['status']); } /** * Test the api_statuses_mentions() function with an unallowed user. + * * @return void * @expectedException Friendica\Network\HTTPException\ForbiddenException */ public function testApiStatusesMentionsWithUnallowedUser() { $_SESSION['allow_api'] = false; - $_GET['screen_name'] = $this->selfUser['nick']; + $_GET['screen_name'] = $this->selfUser['nick']; api_statuses_mentions('json'); } /** * Test the api_statuses_mentions() function with an RSS result. + * * @return void */ public function testApiStatusesMentionsWithRss() @@ -1856,14 +1984,15 @@ class ApiTest extends DatabaseTest /** * Test the api_statuses_user_timeline() function. + * * @return void */ public function testApiStatusesUserTimeline() { - $_REQUEST['max_id'] = 10; + $_REQUEST['max_id'] = 10; $_REQUEST['exclude_replies'] = true; $_REQUEST['conversation_id'] = 1; - $result = api_statuses_user_timeline('json'); + $result = api_statuses_user_timeline('json'); $this->assertNotEmpty($result['status']); foreach ($result['status'] as $status) { $this->assertStatus($status); @@ -1872,12 +2001,13 @@ class ApiTest extends DatabaseTest /** * Test the api_statuses_user_timeline() function with a negative page parameter. + * * @return void */ public function testApiStatusesUserTimelineWithNegativePage() { $_REQUEST['page'] = -2; - $result = api_statuses_user_timeline('json'); + $result = api_statuses_user_timeline('json'); $this->assertNotEmpty($result['status']); foreach ($result['status'] as $status) { $this->assertStatus($status); @@ -1886,6 +2016,7 @@ class ApiTest extends DatabaseTest /** * Test the api_statuses_user_timeline() function with an RSS result. + * * @return void */ public function testApiStatusesUserTimelineWithRss() @@ -1896,18 +2027,20 @@ class ApiTest extends DatabaseTest /** * Test the api_statuses_user_timeline() function with an unallowed user. + * * @return void * @expectedException Friendica\Network\HTTPException\ForbiddenException */ public function testApiStatusesUserTimelineWithUnallowedUser() { $_SESSION['allow_api'] = false; - $_GET['screen_name'] = $this->selfUser['nick']; + $_GET['screen_name'] = $this->selfUser['nick']; api_statuses_user_timeline('json'); } /** * Test the api_favorites_create_destroy() function. + * * @return void * @expectedException Friendica\Network\HTTPException\BadRequestException */ @@ -1920,6 +2053,7 @@ class ApiTest extends DatabaseTest /** * Test the api_favorites_create_destroy() function with an invalid ID. + * * @return void * @expectedException Friendica\Network\HTTPException\BadRequestException */ @@ -1932,6 +2066,7 @@ class ApiTest extends DatabaseTest /** * Test the api_favorites_create_destroy() function with an invalid action. + * * @return void * @expectedException Friendica\Network\HTTPException\BadRequestException */ @@ -1939,71 +2074,76 @@ class ApiTest extends DatabaseTest { $this->app->argv = ['api', '1.1', 'favorites', 'change.json']; $this->app->argc = count($this->app->argv); - $_REQUEST['id'] = 1; + $_REQUEST['id'] = 1; api_favorites_create_destroy('json'); } /** * Test the api_favorites_create_destroy() function with the create action. + * * @return void */ public function testApiFavoritesCreateDestroyWithCreateAction() { $this->app->argv = ['api', '1.1', 'favorites', 'create.json']; $this->app->argc = count($this->app->argv); - $_REQUEST['id'] = 3; - $result = api_favorites_create_destroy('json'); + $_REQUEST['id'] = 3; + $result = api_favorites_create_destroy('json'); $this->assertStatus($result['status']); } /** * Test the api_favorites_create_destroy() function with the create action and an RSS result. + * * @return void */ public function testApiFavoritesCreateDestroyWithCreateActionAndRss() { $this->app->argv = ['api', '1.1', 'favorites', 'create.rss']; $this->app->argc = count($this->app->argv); - $_REQUEST['id'] = 3; - $result = api_favorites_create_destroy('rss'); + $_REQUEST['id'] = 3; + $result = api_favorites_create_destroy('rss'); $this->assertXml($result, 'status'); } /** * Test the api_favorites_create_destroy() function with the destroy action. + * * @return void */ public function testApiFavoritesCreateDestroyWithDestroyAction() { $this->app->argv = ['api', '1.1', 'favorites', 'destroy.json']; $this->app->argc = count($this->app->argv); - $_REQUEST['id'] = 3; - $result = api_favorites_create_destroy('json'); + $_REQUEST['id'] = 3; + $result = api_favorites_create_destroy('json'); $this->assertStatus($result['status']); } /** * Test the api_favorites_create_destroy() function without an authenticated user. + * * @return void * @expectedException Friendica\Network\HTTPException\ForbiddenException */ public function testApiFavoritesCreateDestroyWithoutAuthenticatedUser() { - $this->app->argv = ['api', '1.1', 'favorites', 'create.json']; - $this->app->argc = count($this->app->argv); + $this->app->argv = ['api', '1.1', 'favorites', 'create.json']; + $this->app->argc = count($this->app->argv); $_SESSION['authenticated'] = false; api_favorites_create_destroy('json'); } /** * Test the api_favorites() function. + * * @return void */ public function testApiFavorites() { - $_REQUEST['page'] = -1; + $_REQUEST['page'] = -1; $_REQUEST['max_id'] = 10; - $result = api_favorites('json'); + $result = api_favorites('json'); foreach ($result['status'] as $status) { $this->assertStatus($status); } @@ -2011,6 +2151,7 @@ class ApiTest extends DatabaseTest /** * Test the api_favorites() function with an RSS result. + * * @return void */ public function testApiFavoritesWithRss() @@ -2021,18 +2162,20 @@ class ApiTest extends DatabaseTest /** * Test the api_favorites() function with an unallowed user. + * * @return void * @expectedException Friendica\Network\HTTPException\ForbiddenException */ public function testApiFavoritesWithUnallowedUser() { $_SESSION['allow_api'] = false; - $_GET['screen_name'] = $this->selfUser['nick']; + $_GET['screen_name'] = $this->selfUser['nick']; api_favorites('json'); } /** * Test the api_format_messages() function. + * * @return void */ public function testApiFormatMessages() @@ -2042,7 +2185,7 @@ class ApiTest extends DatabaseTest ['id' => 2, 'screen_name' => 'recipient_name'], ['id' => 3, 'screen_name' => 'sender_name'] ); - $this->assertEquals('item_title'."\n".'item_body', $result['text']); + $this->assertEquals('item_title' . "\n" . 'item_body', $result['text']); $this->assertEquals(1, $result['id']); $this->assertEquals(2, $result['recipient_id']); $this->assertEquals(3, $result['sender_id']); @@ -2052,12 +2195,13 @@ class ApiTest extends DatabaseTest /** * Test the api_format_messages() function with HTML. + * * @return void */ public function testApiFormatMessagesWithHtmlText() { $_GET['getText'] = 'html'; - $result = api_format_messages( + $result = api_format_messages( ['id' => 1, 'title' => 'item_title', 'body' => '[b]item_body[/b]'], ['id' => 2, 'screen_name' => 'recipient_name'], ['id' => 3, 'screen_name' => 'sender_name'] @@ -2068,12 +2212,13 @@ class ApiTest extends DatabaseTest /** * Test the api_format_messages() function with plain text. + * * @return void */ public function testApiFormatMessagesWithPlainText() { $_GET['getText'] = 'plain'; - $result = api_format_messages( + $result = api_format_messages( ['id' => 1, 'title' => 'item_title', 'body' => '[b]item_body[/b]'], ['id' => 2, 'screen_name' => 'recipient_name'], ['id' => 3, 'screen_name' => 'sender_name'] @@ -2084,12 +2229,13 @@ class ApiTest extends DatabaseTest /** * Test the api_format_messages() function with the getUserObjects GET parameter set to false. + * * @return void */ public function testApiFormatMessagesWithoutUserObjects() { $_GET['getUserObjects'] = 'false'; - $result = api_format_messages( + $result = api_format_messages( ['id' => 1, 'title' => 'item_title', 'body' => '[b]item_body[/b]'], ['id' => 2, 'screen_name' => 'recipient_name'], ['id' => 3, 'screen_name' => 'sender_name'] @@ -2100,6 +2246,7 @@ class ApiTest extends DatabaseTest /** * Test the api_convert_item() function. + * * @return void */ public function testApiConvertItem() @@ -2107,32 +2254,32 @@ class ApiTest extends DatabaseTest $result = api_convert_item( [ 'network' => 'feed', - 'title' => 'item_title', + 'title' => 'item_title', // We need a long string to test that it is correctly cut - 'body' => 'perspiciatis impedit voluptatem quis molestiae ea qui '. - 'reiciendis dolorum aut ducimus sunt consequatur inventore dolor '. - 'officiis pariatur doloremque nemo culpa aut quidem qui dolore '. - 'laudantium atque commodi alias voluptatem non possimus aperiam '. - 'ipsum rerum consequuntur aut amet fugit quia aliquid praesentium '. - 'repellendus quibusdam et et inventore mollitia rerum sit autem '. - 'pariatur maiores ipsum accusantium perferendis vel sit possimus '. - 'veritatis nihil distinctio qui eum repellat officia illum quos '. - 'impedit quam iste esse unde qui suscipit aut facilis ut inventore '. - 'omnis exercitationem quo magnam consequatur maxime aut illum '. - 'soluta quaerat natus unde aspernatur et sed beatae nihil ullam '. - 'temporibus corporis ratione blanditiis perspiciatis impedit '. - 'voluptatem quis molestiae ea qui reiciendis dolorum aut ducimus '. - 'sunt consequatur inventore dolor officiis pariatur doloremque '. - 'nemo culpa aut quidem qui dolore laudantium atque commodi alias '. - 'voluptatem non possimus aperiam ipsum rerum consequuntur aut '. - 'amet fugit quia aliquid praesentium repellendus quibusdam et et '. - 'inventore mollitia rerum sit autem pariatur maiores ipsum accusantium '. - 'perferendis vel sit possimus veritatis nihil distinctio qui eum '. - 'repellat officia illum quos impedit quam iste esse unde qui '. - 'suscipit aut facilis ut inventore omnis exercitationem quo magnam '. - 'consequatur maxime aut illum soluta quaerat natus unde aspernatur '. - 'et sed beatae nihil ullam temporibus corporis ratione blanditiis', - 'plink' => 'item_plink' + 'body' => 'perspiciatis impedit voluptatem quis molestiae ea qui ' . + 'reiciendis dolorum aut ducimus sunt consequatur inventore dolor ' . + 'officiis pariatur doloremque nemo culpa aut quidem qui dolore ' . + 'laudantium atque commodi alias voluptatem non possimus aperiam ' . + 'ipsum rerum consequuntur aut amet fugit quia aliquid praesentium ' . + 'repellendus quibusdam et et inventore mollitia rerum sit autem ' . + 'pariatur maiores ipsum accusantium perferendis vel sit possimus ' . + 'veritatis nihil distinctio qui eum repellat officia illum quos ' . + 'impedit quam iste esse unde qui suscipit aut facilis ut inventore ' . + 'omnis exercitationem quo magnam consequatur maxime aut illum ' . + 'soluta quaerat natus unde aspernatur et sed beatae nihil ullam ' . + 'temporibus corporis ratione blanditiis perspiciatis impedit ' . + 'voluptatem quis molestiae ea qui reiciendis dolorum aut ducimus ' . + 'sunt consequatur inventore dolor officiis pariatur doloremque ' . + 'nemo culpa aut quidem qui dolore laudantium atque commodi alias ' . + 'voluptatem non possimus aperiam ipsum rerum consequuntur aut ' . + 'amet fugit quia aliquid praesentium repellendus quibusdam et et ' . + 'inventore mollitia rerum sit autem pariatur maiores ipsum accusantium ' . + 'perferendis vel sit possimus veritatis nihil distinctio qui eum ' . + 'repellat officia illum quos impedit quam iste esse unde qui ' . + 'suscipit aut facilis ut inventore omnis exercitationem quo magnam ' . + 'consequatur maxime aut illum soluta quaerat natus unde aspernatur ' . + 'et sed beatae nihil ullam temporibus corporis ratione blanditiis', + 'plink' => 'item_plink' ] ); $this->assertStringStartsWith('item_title', $result['text']); @@ -2141,6 +2288,7 @@ class ApiTest extends DatabaseTest /** * Test the api_convert_item() function with an empty item body. + * * @return void */ public function testApiConvertItemWithoutBody() @@ -2148,9 +2296,9 @@ class ApiTest extends DatabaseTest $result = api_convert_item( [ 'network' => 'feed', - 'title' => 'item_title', - 'body' => '', - 'plink' => 'item_plink' + 'title' => 'item_title', + 'body' => '', + 'plink' => 'item_plink' ] ); $this->assertEquals('item_title', $result['text']); @@ -2159,6 +2307,7 @@ class ApiTest extends DatabaseTest /** * Test the api_convert_item() function with the title in the body. + * * @return void */ public function testApiConvertItemWithTitleInBody() @@ -2166,7 +2315,7 @@ class ApiTest extends DatabaseTest $result = api_convert_item( [ 'title' => 'item_title', - 'body' => 'item_title item_body' + 'body' => 'item_title item_body' ] ); $this->assertEquals('item_title item_body', $result['text']); @@ -2175,6 +2324,7 @@ class ApiTest extends DatabaseTest /** * Test the api_get_attachments() function. + * * @return void */ public function testApiGetAttachments() @@ -2185,6 +2335,7 @@ class ApiTest extends DatabaseTest /** * Test the api_get_attachments() function with an img tag. + * * @return void */ public function testApiGetAttachmentsWithImage() @@ -2195,17 +2346,19 @@ class ApiTest extends DatabaseTest /** * Test the api_get_attachments() function with an img tag and an AndStatus user agent. + * * @return void */ public function testApiGetAttachmentsWithImageAndAndStatus() { $_SERVER['HTTP_USER_AGENT'] = 'AndStatus'; - $body = '[img]http://via.placeholder.com/1x1.png[/img]'; + $body = '[img]http://via.placeholder.com/1x1.png[/img]'; $this->assertInternalType('array', api_get_attachments($body)); } /** * Test the api_get_entitities() function. + * * @return void */ public function testApiGetEntitities() @@ -2216,13 +2369,14 @@ class ApiTest extends DatabaseTest /** * Test the api_get_entitities() function with the include_entities parameter. + * * @return void */ public function testApiGetEntititiesWithIncludeEntities() { $_REQUEST['include_entities'] = 'true'; - $text = 'text'; - $result = api_get_entitities($text, 'bbcode'); + $text = 'text'; + $result = api_get_entitities($text, 'bbcode'); $this->assertInternalType('array', $result['hashtags']); $this->assertInternalType('array', $result['symbols']); $this->assertInternalType('array', $result['urls']); @@ -2231,6 +2385,7 @@ class ApiTest extends DatabaseTest /** * Test the api_format_items_embeded_images() function. + * * @return void */ public function testApiFormatItemsEmbededImages() @@ -2243,6 +2398,7 @@ class ApiTest extends DatabaseTest /** * Test the api_contactlink_to_array() function. + * * @return void */ public function testApiContactlinkToArray() @@ -2250,7 +2406,7 @@ class ApiTest extends DatabaseTest $this->assertEquals( [ 'name' => 'text', - 'url' => '', + 'url' => '', ], api_contactlink_to_array('text') ); @@ -2258,6 +2414,7 @@ class ApiTest extends DatabaseTest /** * Test the api_contactlink_to_array() function with an URL. + * * @return void */ public function testApiContactlinkToArrayWithUrl() @@ -2265,7 +2422,7 @@ class ApiTest extends DatabaseTest $this->assertEquals( [ 'name' => ['link_text'], - 'url' => ['url'], + 'url' => ['url'], ], api_contactlink_to_array('text link_text') ); @@ -2273,11 +2430,12 @@ class ApiTest extends DatabaseTest /** * Test the api_format_items_activities() function. + * * @return void */ public function testApiFormatItemsActivities() { - $item = ['uid' => 0, 'uri' => '']; + $item = ['uid' => 0, 'uri' => '']; $result = api_format_items_activities($item); $this->assertArrayHasKey('like', $result); $this->assertArrayHasKey('dislike', $result); @@ -2288,11 +2446,12 @@ class ApiTest extends DatabaseTest /** * Test the api_format_items_activities() function with an XML result. + * * @return void */ public function testApiFormatItemsActivitiesWithXml() { - $item = ['uid' => 0, 'uri' => '']; + $item = ['uid' => 0, 'uri' => '']; $result = api_format_items_activities($item, 'xml'); $this->assertArrayHasKey('friendica:like', $result); $this->assertArrayHasKey('friendica:dislike', $result); @@ -2303,94 +2462,95 @@ class ApiTest extends DatabaseTest /** * Test the api_format_items_profiles() function. + * * @return void */ public function testApiFormatItemsProfiles() { $profile_row = [ - 'id' => 'profile_id', + 'id' => 'profile_id', 'profile-name' => 'profile_name', - 'is-default' => true, + 'is-default' => true, 'hide-friends' => true, - 'photo' => 'profile_photo', - 'thumb' => 'profile_thumb', - 'publish' => true, - 'net-publish' => true, - 'pdesc' => 'description', - 'dob' => 'date_of_birth', - 'address' => 'address', - 'locality' => 'city', - 'region' => 'region', - 'postal-code' => 'postal_code', + 'photo' => 'profile_photo', + 'thumb' => 'profile_thumb', + 'publish' => true, + 'net-publish' => true, + 'pdesc' => 'description', + 'dob' => 'date_of_birth', + 'address' => 'address', + 'locality' => 'city', + 'region' => 'region', + 'postal-code' => 'postal_code', 'country-name' => 'country', - 'hometown' => 'hometown', - 'gender' => 'gender', - 'marital' => 'marital', - 'with' => 'marital_with', - 'howlong' => 'marital_since', - 'sexual' => 'sexual', - 'politic' => 'politic', - 'religion' => 'religion', + 'hometown' => 'hometown', + 'gender' => 'gender', + 'marital' => 'marital', + 'with' => 'marital_with', + 'howlong' => 'marital_since', + 'sexual' => 'sexual', + 'politic' => 'politic', + 'religion' => 'religion', 'pub_keywords' => 'public_keywords', 'prv_keywords' => 'private_keywords', - 'likes' => 'likes', - 'dislikes' => 'dislikes', - 'about' => 'about', - 'music' => 'music', - 'book' => 'book', - 'tv' => 'tv', - 'film' => 'film', - 'interest' => 'interest', - 'romance' => 'romance', - 'work' => 'work', + 'likes' => 'likes', + 'dislikes' => 'dislikes', + 'about' => 'about', + 'music' => 'music', + 'book' => 'book', + 'tv' => 'tv', + 'film' => 'film', + 'interest' => 'interest', + 'romance' => 'romance', + 'work' => 'work', 'education' => 'education', - 'contact' => 'social_networks', - 'homepage' => 'homepage' + 'contact' => 'social_networks', + 'homepage' => 'homepage' ]; - $result = api_format_items_profiles($profile_row); + $result = api_format_items_profiles($profile_row); $this->assertEquals( [ - 'profile_id' => 'profile_id', - 'profile_name' => 'profile_name', - 'is_default' => true, - 'hide_friends' => true, - 'profile_photo' => 'profile_photo', - 'profile_thumb' => 'profile_thumb', - 'publish' => true, - 'net_publish' => true, - 'description' => 'description', - 'date_of_birth' => 'date_of_birth', - 'address' => 'address', - 'city' => 'city', - 'region' => 'region', - 'postal_code' => 'postal_code', - 'country' => 'country', - 'hometown' => 'hometown', - 'gender' => 'gender', - 'marital' => 'marital', - 'marital_with' => 'marital_with', - 'marital_since' => 'marital_since', - 'sexual' => 'sexual', - 'politic' => 'politic', - 'religion' => 'religion', - 'public_keywords' => 'public_keywords', + 'profile_id' => 'profile_id', + 'profile_name' => 'profile_name', + 'is_default' => true, + 'hide_friends' => true, + 'profile_photo' => 'profile_photo', + 'profile_thumb' => 'profile_thumb', + 'publish' => true, + 'net_publish' => true, + 'description' => 'description', + 'date_of_birth' => 'date_of_birth', + 'address' => 'address', + 'city' => 'city', + 'region' => 'region', + 'postal_code' => 'postal_code', + 'country' => 'country', + 'hometown' => 'hometown', + 'gender' => 'gender', + 'marital' => 'marital', + 'marital_with' => 'marital_with', + 'marital_since' => 'marital_since', + 'sexual' => 'sexual', + 'politic' => 'politic', + 'religion' => 'religion', + 'public_keywords' => 'public_keywords', 'private_keywords' => 'private_keywords', - 'likes' => 'likes', - 'dislikes' => 'dislikes', - 'about' => 'about', - 'music' => 'music', - 'book' => 'book', - 'tv' => 'tv', - 'film' => 'film', - 'interest' => 'interest', - 'romance' => 'romance', - 'work' => 'work', - 'education' => 'education', + 'likes' => 'likes', + 'dislikes' => 'dislikes', + 'about' => 'about', + 'music' => 'music', + 'book' => 'book', + 'tv' => 'tv', + 'film' => 'film', + 'interest' => 'interest', + 'romance' => 'romance', + 'work' => 'work', + 'education' => 'education', 'social_networks' => 'social_networks', - 'homepage' => 'homepage', - 'users' => null + 'homepage' => 'homepage', + 'users' => null ], $result ); @@ -2398,21 +2558,22 @@ class ApiTest extends DatabaseTest /** * Test the api_format_items() function. + * * @return void */ public function testApiFormatItems() { - $items = [ + $items = [ [ - 'item_network' => 'item_network', - 'source' => 'web', - 'coord' => '5 7', - 'body' => '', - 'verb' => '', - 'author-id' => 43, + 'item_network' => 'item_network', + 'source' => 'web', + 'coord' => '5 7', + 'body' => '', + 'verb' => '', + 'author-id' => 43, 'author-network' => Protocol::DFRN, - 'author-link' => 'http://localhost/profile/othercontact', - 'plink' => '', + 'author-link' => 'http://localhost/profile/othercontact', + 'plink' => '', ] ]; $result = api_format_items($items, ['id' => 0], true); @@ -2423,19 +2584,20 @@ class ApiTest extends DatabaseTest /** * Test the api_format_items() function with an XML result. + * * @return void */ public function testApiFormatItemsWithXml() { - $items = [ + $items = [ [ - 'coord' => '5 7', - 'body' => '', - 'verb' => '', - 'author-id' => 43, + 'coord' => '5 7', + 'body' => '', + 'verb' => '', + 'author-id' => 43, 'author-network' => Protocol::DFRN, - 'author-link' => 'http://localhost/profile/othercontact', - 'plink' => '', + 'author-link' => 'http://localhost/profile/othercontact', + 'plink' => '', ] ]; $result = api_format_items($items, ['id' => 0], true, 'xml'); @@ -2446,6 +2608,7 @@ class ApiTest extends DatabaseTest /** * Test the api_format_items() function. + * * @return void */ public function testApiAccountRateLimitStatus() @@ -2458,6 +2621,7 @@ class ApiTest extends DatabaseTest /** * Test the api_format_items() function with an XML result. + * * @return void */ public function testApiAccountRateLimitStatusWithXml() @@ -2468,6 +2632,7 @@ class ApiTest extends DatabaseTest /** * Test the api_help_test() function. + * * @return void */ public function testApiHelpTest() @@ -2478,6 +2643,7 @@ class ApiTest extends DatabaseTest /** * Test the api_help_test() function with an XML result. + * * @return void */ public function testApiHelpTestWithXml() @@ -2488,6 +2654,7 @@ class ApiTest extends DatabaseTest /** * Test the api_lists_list() function. + * * @return void */ public function testApiListsList() @@ -2498,6 +2665,7 @@ class ApiTest extends DatabaseTest /** * Test the api_lists_ownerships() function. + * * @return void */ public function testApiListsOwnerships() @@ -2510,6 +2678,7 @@ class ApiTest extends DatabaseTest /** * Test the api_lists_ownerships() function without an authenticated user. + * * @return void * @expectedException Friendica\Network\HTTPException\ForbiddenException */ @@ -2521,6 +2690,7 @@ class ApiTest extends DatabaseTest /** * Test the api_lists_statuses() function. + * * @expectedException Friendica\Network\HTTPException\BadRequestException * @return void */ @@ -2531,14 +2701,15 @@ class ApiTest extends DatabaseTest /** * Test the api_lists_statuses() function with a list ID. + * * @return void */ public function testApiListsStatusesWithListId() { $_REQUEST['list_id'] = 1; - $_REQUEST['page'] = -1; - $_REQUEST['max_id'] = 10; - $result = api_lists_statuses('json'); + $_REQUEST['page'] = -1; + $_REQUEST['max_id'] = 10; + $result = api_lists_statuses('json'); foreach ($result['status'] as $status) { $this->assertStatus($status); } @@ -2546,40 +2717,44 @@ class ApiTest extends DatabaseTest /** * Test the api_lists_statuses() function with a list ID and a RSS result. + * * @return void */ public function testApiListsStatusesWithListIdAndRss() { $_REQUEST['list_id'] = 1; - $result = api_lists_statuses('rss'); + $result = api_lists_statuses('rss'); $this->assertXml($result, 'statuses'); } /** * Test the api_lists_statuses() function with an unallowed user. + * * @return void * @expectedException Friendica\Network\HTTPException\ForbiddenException */ public function testApiListsStatusesWithUnallowedUser() { $_SESSION['allow_api'] = false; - $_GET['screen_name'] = $this->selfUser['nick']; + $_GET['screen_name'] = $this->selfUser['nick']; api_lists_statuses('json'); } /** * Test the api_statuses_f() function. + * * @return void */ public function testApiStatusesFWithFriends() { $_GET['page'] = -1; - $result = api_statuses_f('friends'); + $result = api_statuses_f('friends'); $this->assertArrayHasKey('user', $result); } /** * Test the api_statuses_f() function. + * * @return void */ public function testApiStatusesFWithFollowers() @@ -2590,6 +2765,7 @@ class ApiTest extends DatabaseTest /** * Test the api_statuses_f() function. + * * @return void */ public function testApiStatusesFWithBlocks() @@ -2600,6 +2776,7 @@ class ApiTest extends DatabaseTest /** * Test the api_statuses_f() function. + * * @return void */ public function testApiStatusesFWithIncoming() @@ -2610,6 +2787,7 @@ class ApiTest extends DatabaseTest /** * Test the api_statuses_f() function an undefined cursor GET variable. + * * @return void */ public function testApiStatusesFWithUndefinedCursor() @@ -2620,6 +2798,7 @@ class ApiTest extends DatabaseTest /** * Test the api_statuses_friends() function. + * * @return void */ public function testApiStatusesFriends() @@ -2630,6 +2809,7 @@ class ApiTest extends DatabaseTest /** * Test the api_statuses_friends() function an undefined cursor GET variable. + * * @return void */ public function testApiStatusesFriendsWithUndefinedCursor() @@ -2640,6 +2820,7 @@ class ApiTest extends DatabaseTest /** * Test the api_statuses_followers() function. + * * @return void */ public function testApiStatusesFollowers() @@ -2650,6 +2831,7 @@ class ApiTest extends DatabaseTest /** * Test the api_statuses_followers() function an undefined cursor GET variable. + * * @return void */ public function testApiStatusesFollowersWithUndefinedCursor() @@ -2660,6 +2842,7 @@ class ApiTest extends DatabaseTest /** * Test the api_blocks_list() function. + * * @return void */ public function testApiBlocksList() @@ -2670,6 +2853,7 @@ class ApiTest extends DatabaseTest /** * Test the api_blocks_list() function an undefined cursor GET variable. + * * @return void */ public function testApiBlocksListWithUndefinedCursor() @@ -2680,6 +2864,7 @@ class ApiTest extends DatabaseTest /** * Test the api_friendships_incoming() function. + * * @return void */ public function testApiFriendshipsIncoming() @@ -2690,6 +2875,7 @@ class ApiTest extends DatabaseTest /** * Test the api_friendships_incoming() function an undefined cursor GET variable. + * * @return void */ public function testApiFriendshipsIncomingWithUndefinedCursor() @@ -2700,6 +2886,7 @@ class ApiTest extends DatabaseTest /** * Test the api_statusnet_config() function. + * * @return void */ public function testApiStatusnetConfig() @@ -2719,6 +2906,7 @@ class ApiTest extends DatabaseTest /** * Test the api_statusnet_version() function. + * * @return void */ public function testApiStatusnetVersion() @@ -2729,6 +2917,7 @@ class ApiTest extends DatabaseTest /** * Test the api_ff_ids() function. + * * @return void */ public function testApiFfIds() @@ -2739,6 +2928,7 @@ class ApiTest extends DatabaseTest /** * Test the api_ff_ids() function with a result. + * * @return void */ public function testApiFfIdsWithResult() @@ -2748,6 +2938,7 @@ class ApiTest extends DatabaseTest /** * Test the api_ff_ids() function without an authenticated user. + * * @return void * @expectedException Friendica\Network\HTTPException\ForbiddenException */ @@ -2759,6 +2950,7 @@ class ApiTest extends DatabaseTest /** * Test the api_friends_ids() function. + * * @return void */ public function testApiFriendsIds() @@ -2769,6 +2961,7 @@ class ApiTest extends DatabaseTest /** * Test the api_followers_ids() function. + * * @return void */ public function testApiFollowersIds() @@ -2779,6 +2972,7 @@ class ApiTest extends DatabaseTest /** * Test the api_direct_messages_new() function. + * * @return void */ public function testApiDirectMessagesNew() @@ -2789,6 +2983,7 @@ class ApiTest extends DatabaseTest /** * Test the api_direct_messages_new() function without an authenticated user. + * * @return void * @expectedException Friendica\Network\HTTPException\ForbiddenException */ @@ -2800,26 +2995,27 @@ class ApiTest extends DatabaseTest /** * Test the api_direct_messages_new() function with an user ID. + * * @return void */ public function testApiDirectMessagesNewWithUserId() { - $_POST['text'] = 'message_text'; + $_POST['text'] = 'message_text'; $_POST['user_id'] = $this->otherUser['id']; - $result = api_direct_messages_new('json'); + $result = api_direct_messages_new('json'); $this->assertEquals(['direct_message' => ['error' => -1]], $result); } /** * Test the api_direct_messages_new() function with a screen name. + * * @return void */ public function testApiDirectMessagesNewWithScreenName() { - $_POST['text'] = 'message_text'; + $_POST['text'] = 'message_text'; $_POST['screen_name'] = $this->friendUser['nick']; - $result = api_direct_messages_new('json'); - $this->assertEquals(1, $result['direct_message']['id']); + $result = api_direct_messages_new('json'); $this->assertContains('message_text', $result['direct_message']['text']); $this->assertEquals('selfcontact', $result['direct_message']['sender_screen_name']); $this->assertEquals(1, $result['direct_message']['friendica_seen']); @@ -2827,15 +3023,15 @@ class ApiTest extends DatabaseTest /** * Test the api_direct_messages_new() function with a title. + * * @return void */ public function testApiDirectMessagesNewWithTitle() { - $_POST['text'] = 'message_text'; + $_POST['text'] = 'message_text'; $_POST['screen_name'] = $this->friendUser['nick']; - $_REQUEST['title'] = 'message_title'; - $result = api_direct_messages_new('json'); - $this->assertEquals(1, $result['direct_message']['id']); + $_REQUEST['title'] = 'message_title'; + $result = api_direct_messages_new('json'); $this->assertContains('message_text', $result['direct_message']['text']); $this->assertContains('message_title', $result['direct_message']['text']); $this->assertEquals('selfcontact', $result['direct_message']['sender_screen_name']); @@ -2844,18 +3040,20 @@ class ApiTest extends DatabaseTest /** * Test the api_direct_messages_new() function with an RSS result. + * * @return void */ public function testApiDirectMessagesNewWithRss() { - $_POST['text'] = 'message_text'; + $_POST['text'] = 'message_text'; $_POST['screen_name'] = $this->friendUser['nick']; - $result = api_direct_messages_new('rss'); + $result = api_direct_messages_new('rss'); $this->assertXml($result, 'direct-messages'); } /** * Test the api_direct_messages_destroy() function. + * * @return void * @expectedException Friendica\Network\HTTPException\BadRequestException */ @@ -2866,16 +3064,17 @@ class ApiTest extends DatabaseTest /** * Test the api_direct_messages_destroy() function with the friendica_verbose GET param. + * * @return void */ public function testApiDirectMessagesDestroyWithVerbose() { $_GET['friendica_verbose'] = 'true'; - $result = api_direct_messages_destroy('json'); + $result = api_direct_messages_destroy('json'); $this->assertEquals( [ '$result' => [ - 'result' => 'error', + 'result' => 'error', 'message' => 'message id or parenturi not specified' ] ], @@ -2885,6 +3084,7 @@ class ApiTest extends DatabaseTest /** * Test the api_direct_messages_destroy() function without an authenticated user. + * * @return void * @expectedException Friendica\Network\HTTPException\ForbiddenException */ @@ -2896,6 +3096,7 @@ class ApiTest extends DatabaseTest /** * Test the api_direct_messages_destroy() function with a non-zero ID. + * * @return void * @expectedException Friendica\Network\HTTPException\BadRequestException */ @@ -2907,18 +3108,19 @@ class ApiTest extends DatabaseTest /** * Test the api_direct_messages_destroy() with a non-zero ID and the friendica_verbose GET param. + * * @return void */ public function testApiDirectMessagesDestroyWithIdAndVerbose() { - $_REQUEST['id'] = 1; + $_REQUEST['id'] = 1; $_REQUEST['friendica_parenturi'] = 'parent_uri'; - $_GET['friendica_verbose'] = 'true'; - $result = api_direct_messages_destroy('json'); + $_GET['friendica_verbose'] = 'true'; + $result = api_direct_messages_destroy('json'); $this->assertEquals( [ '$result' => [ - 'result' => 'error', + 'result' => 'error', 'message' => 'message id not in database' ] ], @@ -2928,6 +3130,7 @@ class ApiTest extends DatabaseTest /** * Test the api_direct_messages_destroy() function with a non-zero ID. + * * @return void */ public function testApiDirectMessagesDestroyWithCorrectId() @@ -2937,18 +3140,20 @@ class ApiTest extends DatabaseTest /** * Test the api_direct_messages_box() function. + * * @return void */ public function testApiDirectMessagesBoxWithSentbox() { - $_REQUEST['page'] = -1; + $_REQUEST['page'] = -1; $_REQUEST['max_id'] = 10; - $result = api_direct_messages_box('json', 'sentbox', 'false'); + $result = api_direct_messages_box('json', 'sentbox', 'false'); $this->assertArrayHasKey('direct_message', $result); } /** * Test the api_direct_messages_box() function. + * * @return void */ public function testApiDirectMessagesBoxWithConversation() @@ -2959,6 +3164,7 @@ class ApiTest extends DatabaseTest /** * Test the api_direct_messages_box() function. + * * @return void */ public function testApiDirectMessagesBoxWithAll() @@ -2969,6 +3175,7 @@ class ApiTest extends DatabaseTest /** * Test the api_direct_messages_box() function. + * * @return void */ public function testApiDirectMessagesBoxWithInbox() @@ -2979,6 +3186,7 @@ class ApiTest extends DatabaseTest /** * Test the api_direct_messages_box() function. + * * @return void */ public function testApiDirectMessagesBoxWithVerbose() @@ -2987,7 +3195,7 @@ class ApiTest extends DatabaseTest $this->assertEquals( [ '$result' => [ - 'result' => 'error', + 'result' => 'error', 'message' => 'no mails available' ] ], @@ -2997,6 +3205,7 @@ class ApiTest extends DatabaseTest /** * Test the api_direct_messages_box() function with a RSS result. + * * @return void */ public function testApiDirectMessagesBoxWithRss() @@ -3007,18 +3216,20 @@ class ApiTest extends DatabaseTest /** * Test the api_direct_messages_box() function without an authenticated user. + * * @return void * @expectedException Friendica\Network\HTTPException\ForbiddenException */ public function testApiDirectMessagesBoxWithUnallowedUser() { $_SESSION['allow_api'] = false; - $_GET['screen_name'] = $this->selfUser['nick']; + $_GET['screen_name'] = $this->selfUser['nick']; api_direct_messages_box('json', 'sentbox', 'false'); } /** * Test the api_direct_messages_sentbox() function. + * * @return void */ public function testApiDirectMessagesSentbox() @@ -3029,6 +3240,7 @@ class ApiTest extends DatabaseTest /** * Test the api_direct_messages_inbox() function. + * * @return void */ public function testApiDirectMessagesInbox() @@ -3039,6 +3251,7 @@ class ApiTest extends DatabaseTest /** * Test the api_direct_messages_all() function. + * * @return void */ public function testApiDirectMessagesAll() @@ -3049,6 +3262,7 @@ class ApiTest extends DatabaseTest /** * Test the api_direct_messages_conversation() function. + * * @return void */ public function testApiDirectMessagesConversation() @@ -3059,6 +3273,7 @@ class ApiTest extends DatabaseTest /** * Test the api_oauth_request_token() function. + * * @return void */ public function testApiOauthRequestToken() @@ -3068,6 +3283,7 @@ class ApiTest extends DatabaseTest /** * Test the api_oauth_access_token() function. + * * @return void */ public function testApiOauthAccessToken() @@ -3077,6 +3293,7 @@ class ApiTest extends DatabaseTest /** * Test the api_fr_photoalbum_delete() function. + * * @return void * @expectedException Friendica\Network\HTTPException\BadRequestException */ @@ -3087,6 +3304,7 @@ class ApiTest extends DatabaseTest /** * Test the api_fr_photoalbum_delete() function with an album name. + * * @return void * @expectedException Friendica\Network\HTTPException\BadRequestException */ @@ -3098,6 +3316,7 @@ class ApiTest extends DatabaseTest /** * Test the api_fr_photoalbum_delete() function with an album name. + * * @return void */ public function testApiFrPhotoalbumDeleteWithValidAlbum() @@ -3107,6 +3326,7 @@ class ApiTest extends DatabaseTest /** * Test the api_fr_photoalbum_delete() function. + * * @return void * @expectedException Friendica\Network\HTTPException\BadRequestException */ @@ -3117,6 +3337,7 @@ class ApiTest extends DatabaseTest /** * Test the api_fr_photoalbum_delete() function with an album name. + * * @return void * @expectedException Friendica\Network\HTTPException\BadRequestException */ @@ -3128,18 +3349,20 @@ class ApiTest extends DatabaseTest /** * Test the api_fr_photoalbum_delete() function with an album name. + * * @return void * @expectedException Friendica\Network\HTTPException\BadRequestException */ public function testApiFrPhotoalbumUpdateWithAlbumAndNewAlbum() { - $_REQUEST['album'] = 'album_name'; + $_REQUEST['album'] = 'album_name'; $_REQUEST['album_new'] = 'album_name'; api_fr_photoalbum_update('json'); } /** * Test the api_fr_photoalbum_update() function without an authenticated user. + * * @return void * @expectedException Friendica\Network\HTTPException\ForbiddenException */ @@ -3151,6 +3374,7 @@ class ApiTest extends DatabaseTest /** * Test the api_fr_photoalbum_delete() function with an album name. + * * @return void */ public function testApiFrPhotoalbumUpdateWithValidAlbum() @@ -3160,6 +3384,7 @@ class ApiTest extends DatabaseTest /** * Test the api_fr_photos_list() function. + * * @return void */ public function testApiFrPhotosList() @@ -3170,6 +3395,7 @@ class ApiTest extends DatabaseTest /** * Test the api_fr_photos_list() function without an authenticated user. + * * @return void * @expectedException Friendica\Network\HTTPException\ForbiddenException */ @@ -3181,6 +3407,7 @@ class ApiTest extends DatabaseTest /** * Test the api_fr_photo_create_update() function. + * * @return void * @expectedException Friendica\Network\HTTPException\BadRequestException */ @@ -3191,6 +3418,7 @@ class ApiTest extends DatabaseTest /** * Test the api_fr_photo_create_update() function without an authenticated user. + * * @return void * @expectedException Friendica\Network\HTTPException\ForbiddenException */ @@ -3202,6 +3430,7 @@ class ApiTest extends DatabaseTest /** * Test the api_fr_photo_create_update() function with an album name. + * * @return void * @expectedException Friendica\Network\HTTPException\BadRequestException */ @@ -3213,6 +3442,7 @@ class ApiTest extends DatabaseTest /** * Test the api_fr_photo_create_update() function with the update mode. + * * @return void */ public function testApiFrPhotoCreateUpdateWithUpdate() @@ -3222,6 +3452,7 @@ class ApiTest extends DatabaseTest /** * Test the api_fr_photo_create_update() function with an uploaded file. + * * @return void */ public function testApiFrPhotoCreateUpdateWithFile() @@ -3231,6 +3462,7 @@ class ApiTest extends DatabaseTest /** * Test the api_fr_photo_delete() function. + * * @return void * @expectedException Friendica\Network\HTTPException\BadRequestException */ @@ -3241,6 +3473,7 @@ class ApiTest extends DatabaseTest /** * Test the api_fr_photo_delete() function without an authenticated user. + * * @return void * @expectedException Friendica\Network\HTTPException\ForbiddenException */ @@ -3252,6 +3485,7 @@ class ApiTest extends DatabaseTest /** * Test the api_fr_photo_delete() function with a photo ID. + * * @return void * @expectedException Friendica\Network\HTTPException\BadRequestException */ @@ -3263,6 +3497,7 @@ class ApiTest extends DatabaseTest /** * Test the api_fr_photo_delete() function with a correct photo ID. + * * @return void */ public function testApiFrPhotoDeleteWithCorrectPhotoId() @@ -3272,6 +3507,7 @@ class ApiTest extends DatabaseTest /** * Test the api_fr_photo_detail() function. + * * @return void * @expectedException Friendica\Network\HTTPException\BadRequestException */ @@ -3282,6 +3518,7 @@ class ApiTest extends DatabaseTest /** * Test the api_fr_photo_detail() function without an authenticated user. + * * @return void * @expectedException Friendica\Network\HTTPException\ForbiddenException */ @@ -3293,6 +3530,7 @@ class ApiTest extends DatabaseTest /** * Test the api_fr_photo_detail() function with a photo ID. + * * @return void * @expectedException Friendica\Network\HTTPException\NotFoundException */ @@ -3304,6 +3542,7 @@ class ApiTest extends DatabaseTest /** * Test the api_fr_photo_detail() function with a correct photo ID. + * * @return void */ public function testApiFrPhotoDetailCorrectPhotoId() @@ -3313,6 +3552,7 @@ class ApiTest extends DatabaseTest /** * Test the api_account_update_profile_image() function. + * * @return void * @expectedException Friendica\Network\HTTPException\BadRequestException */ @@ -3323,6 +3563,7 @@ class ApiTest extends DatabaseTest /** * Test the api_account_update_profile_image() function without an authenticated user. + * * @return void * @expectedException Friendica\Network\HTTPException\ForbiddenException */ @@ -3334,6 +3575,7 @@ class ApiTest extends DatabaseTest /** * Test the api_account_update_profile_image() function with an uploaded file. + * * @return void * @expectedException Friendica\Network\HTTPException\BadRequestException */ @@ -3345,13 +3587,14 @@ class ApiTest extends DatabaseTest /** * Test the api_account_update_profile() function. + * * @return void */ public function testApiAccountUpdateProfile() { - $_POST['name'] = 'new_name'; + $_POST['name'] = 'new_name'; $_POST['description'] = 'new_description'; - $result = api_account_update_profile('json'); + $result = api_account_update_profile('json'); // We can't use assertSelfUser() here because the user object is missing some properties. $this->assertEquals($this->selfUser['id'], $result['user']['cid']); $this->assertEquals('DFRN', $result['user']['location']); @@ -3363,6 +3606,7 @@ class ApiTest extends DatabaseTest /** * Test the check_acl_input() function. + * * @return void */ public function testCheckAclInput() @@ -3374,6 +3618,7 @@ class ApiTest extends DatabaseTest /** * Test the check_acl_input() function with an empty ACL string. + * * @return void */ public function testCheckAclInputWithEmptyAclString() @@ -3384,6 +3629,7 @@ class ApiTest extends DatabaseTest /** * Test the save_media_to_database() function. + * * @return void */ public function testSaveMediaToDatabase() @@ -3393,6 +3639,7 @@ class ApiTest extends DatabaseTest /** * Test the post_photo_item() function. + * * @return void */ public function testPostPhotoItem() @@ -3402,6 +3649,7 @@ class ApiTest extends DatabaseTest /** * Test the prepare_photo_data() function. + * * @return void */ public function testPreparePhotoData() @@ -3411,6 +3659,7 @@ class ApiTest extends DatabaseTest /** * Test the api_friendica_remoteauth() function. + * * @return void * @expectedException Friendica\Network\HTTPException\BadRequestException */ @@ -3421,41 +3670,45 @@ class ApiTest extends DatabaseTest /** * Test the api_friendica_remoteauth() function with an URL. + * * @return void * @expectedException Friendica\Network\HTTPException\BadRequestException */ public function testApiFriendicaRemoteauthWithUrl() { - $_GET['url'] = 'url'; + $_GET['url'] = 'url'; $_GET['c_url'] = 'url'; api_friendica_remoteauth(); } /** * Test the api_friendica_remoteauth() function with a correct URL. + * * @return void */ public function testApiFriendicaRemoteauthWithCorrectUrl() { $this->markTestIncomplete("We can't use an assertion here because of App->redirect()."); - $_GET['url'] = 'url'; + $_GET['url'] = 'url'; $_GET['c_url'] = $this->selfUser['nurl']; api_friendica_remoteauth(); } /** * Test the api_share_as_retweet() function. + * * @return void */ public function testApiShareAsRetweet() { - $item = ['body' => '', 'author-id' => 1, 'owner-id' => 1]; + $item = ['body' => '', 'author-id' => 1, 'owner-id' => 1]; $result = api_share_as_retweet($item); $this->assertFalse($result); } /** * Test the api_share_as_retweet() function with a valid item. + * * @return void */ public function testApiShareAsRetweetWithValidItem() @@ -3465,6 +3718,7 @@ class ApiTest extends DatabaseTest /** * Test the api_get_nick() function. + * * @return void */ public function testApiGetNick() @@ -3475,6 +3729,7 @@ class ApiTest extends DatabaseTest /** * Test the api_get_nick() function with a wrong URL. + * * @return void */ public function testApiGetNickWithWrongUrl() @@ -3485,6 +3740,7 @@ class ApiTest extends DatabaseTest /** * Test the api_in_reply_to() function. + * * @return void */ public function testApiInReplyTo() @@ -3499,6 +3755,7 @@ class ApiTest extends DatabaseTest /** * Test the api_in_reply_to() function with a valid item. + * * @return void */ public function testApiInReplyToWithValidItem() @@ -3508,17 +3765,19 @@ class ApiTest extends DatabaseTest /** * Test the api_clean_plain_items() function. + * * @return void */ public function testApiCleanPlainItems() { $_REQUEST['include_entities'] = 'true'; - $result = api_clean_plain_items('some_text [url="some_url"]some_text[/url]'); + $result = api_clean_plain_items('some_text [url="some_url"]some_text[/url]'); $this->assertEquals('some_text [url="some_url"]"some_url"[/url]', $result); } /** * Test the api_clean_attachments() function. + * * @return void */ public function testApiCleanAttachments() @@ -3528,17 +3787,19 @@ class ApiTest extends DatabaseTest /** * Test the api_best_nickname() function. + * * @return void */ public function testApiBestNickname() { $contacts = []; - $result = api_best_nickname($contacts); + $result = api_best_nickname($contacts); $this->assertNull($result); } /** * Test the api_best_nickname() function with contacts. + * * @return void */ public function testApiBestNicknameWithContacts() @@ -3548,6 +3809,7 @@ class ApiTest extends DatabaseTest /** * Test the api_friendica_group_show() function. + * * @return void */ public function testApiFriendicaGroupShow() @@ -3557,6 +3819,7 @@ class ApiTest extends DatabaseTest /** * Test the api_friendica_group_delete() function. + * * @return void */ public function testApiFriendicaGroupDelete() @@ -3566,6 +3829,7 @@ class ApiTest extends DatabaseTest /** * Test the api_lists_destroy() function. + * * @return void */ public function testApiListsDestroy() @@ -3575,6 +3839,7 @@ class ApiTest extends DatabaseTest /** * Test the group_create() function. + * * @return void */ public function testGroupCreate() @@ -3584,6 +3849,7 @@ class ApiTest extends DatabaseTest /** * Test the api_friendica_group_create() function. + * * @return void */ public function testApiFriendicaGroupCreate() @@ -3593,6 +3859,7 @@ class ApiTest extends DatabaseTest /** * Test the api_lists_create() function. + * * @return void */ public function testApiListsCreate() @@ -3602,6 +3869,7 @@ class ApiTest extends DatabaseTest /** * Test the api_friendica_group_update() function. + * * @return void */ public function testApiFriendicaGroupUpdate() @@ -3611,6 +3879,7 @@ class ApiTest extends DatabaseTest /** * Test the api_lists_update() function. + * * @return void */ public function testApiListsUpdate() @@ -3620,6 +3889,7 @@ class ApiTest extends DatabaseTest /** * Test the api_friendica_activity() function. + * * @return void */ public function testApiFriendicaActivity() @@ -3629,6 +3899,7 @@ class ApiTest extends DatabaseTest /** * Test the api_friendica_notification() function. + * * @return void * @expectedException Friendica\Network\HTTPException\BadRequestException */ @@ -3639,6 +3910,7 @@ class ApiTest extends DatabaseTest /** * Test the api_friendica_notification() function without an authenticated user. + * * @return void * @expectedException Friendica\Network\HTTPException\ForbiddenException */ @@ -3650,30 +3922,33 @@ class ApiTest extends DatabaseTest /** * Test the api_friendica_notification() function with an argument count. + * * @return void */ public function testApiFriendicaNotificationWithArgumentCount() { $this->app->argv = ['api', 'friendica', 'notification']; $this->app->argc = count($this->app->argv); - $result = api_friendica_notification('json'); + $result = api_friendica_notification('json'); $this->assertEquals(['note' => false], $result); } /** * Test the api_friendica_notification() function with an XML result. + * * @return void */ public function testApiFriendicaNotificationWithXmlResult() { $this->app->argv = ['api', 'friendica', 'notification']; $this->app->argc = count($this->app->argv); - $result = api_friendica_notification('xml'); + $result = api_friendica_notification('xml'); $this->assertXml($result, 'notes'); } /** * Test the api_friendica_notification_seen() function. + * * @return void */ public function testApiFriendicaNotificationSeen() @@ -3683,6 +3958,7 @@ class ApiTest extends DatabaseTest /** * Test the api_friendica_direct_messages_setseen() function. + * * @return void */ public function testApiFriendicaDirectMessagesSetseen() @@ -3692,6 +3968,7 @@ class ApiTest extends DatabaseTest /** * Test the api_friendica_direct_messages_search() function. + * * @return void */ public function testApiFriendicaDirectMessagesSearch() @@ -3701,6 +3978,7 @@ class ApiTest extends DatabaseTest /** * Test the api_friendica_profile_show() function. + * * @return void */ public function testApiFriendicaProfileShow() @@ -3718,6 +3996,7 @@ class ApiTest extends DatabaseTest /** * Test the api_friendica_profile_show() function with a profile ID. + * * @return void */ public function testApiFriendicaProfileShowWithProfileId() @@ -3727,6 +4006,7 @@ class ApiTest extends DatabaseTest /** * Test the api_friendica_profile_show() function with a wrong profile ID. + * * @return void * @expectedException Friendica\Network\HTTPException\BadRequestException */ @@ -3738,6 +4018,7 @@ class ApiTest extends DatabaseTest /** * Test the api_friendica_profile_show() function without an authenticated user. + * * @return void * @expectedException Friendica\Network\HTTPException\ForbiddenException */ @@ -3749,6 +4030,7 @@ class ApiTest extends DatabaseTest /** * Test the api_saved_searches_list() function. + * * @return void */ public function testApiSavedSearchesList()