From 4094ea47712efbe58624ff74daeb1f77c9b0edcf Mon Sep 17 00:00:00 2001 From: Jeremy Benoist Date: Tue, 12 Apr 2016 11:36:01 +0200 Subject: Convert array + phpDoc Thanks for https://github.com/thomasbachem/php-short-array-syntax-converter --- .../Tests/Controller/ConfigControllerTest.php | 222 ++++++++++----------- .../Tests/Controller/EntryControllerTest.php | 72 +++---- .../Tests/Controller/RssControllerTest.php | 16 +- .../Tests/Controller/SecurityControllerTest.php | 4 +- .../Tests/Controller/TagControllerTest.php | 12 +- 5 files changed, 163 insertions(+), 163 deletions(-) (limited to 'src/Wallabag/CoreBundle/Tests/Controller') diff --git a/src/Wallabag/CoreBundle/Tests/Controller/ConfigControllerTest.php b/src/Wallabag/CoreBundle/Tests/Controller/ConfigControllerTest.php index f2d26750..1aa1a7ed 100644 --- a/src/Wallabag/CoreBundle/Tests/Controller/ConfigControllerTest.php +++ b/src/Wallabag/CoreBundle/Tests/Controller/ConfigControllerTest.php @@ -43,12 +43,12 @@ class ConfigControllerTest extends WallabagCoreTestCase $form = $crawler->filter('button[id=config_save]')->form(); - $data = array( + $data = [ 'config[theme]' => 'baggy', 'config[items_per_page]' => '30', 'config[reading_speed]' => '0.5', 'config[language]' => 'en', - ); + ]; $client->submit($form, $data); @@ -56,19 +56,19 @@ class ConfigControllerTest extends WallabagCoreTestCase $crawler = $client->followRedirect(); - $this->assertGreaterThan(1, $alert = $crawler->filter('div.messages.success')->extract(array('_text'))); + $this->assertGreaterThan(1, $alert = $crawler->filter('div.messages.success')->extract(['_text'])); $this->assertContains('flashes.config.notice.config_saved', $alert[0]); } public function dataForUpdateFailed() { - return array( - array(array( + return [ + [[ 'config[theme]' => 'baggy', 'config[items_per_page]' => '', 'config[language]' => 'en', - )), - ); + ]], + ]; } /** @@ -89,46 +89,46 @@ class ConfigControllerTest extends WallabagCoreTestCase $this->assertEquals(200, $client->getResponse()->getStatusCode()); - $this->assertGreaterThan(1, $alert = $crawler->filter('body')->extract(array('_text'))); + $this->assertGreaterThan(1, $alert = $crawler->filter('body')->extract(['_text'])); $this->assertContains('This value should not be blank', $alert[0]); } public function dataForChangePasswordFailed() { - return array( - array( - array( + return [ + [ + [ 'change_passwd[old_password]' => 'material', 'change_passwd[new_password][first]' => '', 'change_passwd[new_password][second]' => '', - ), + ], 'validator.password_wrong_value', - ), - array( - array( + ], + [ + [ 'change_passwd[old_password]' => 'mypassword', 'change_passwd[new_password][first]' => '', 'change_passwd[new_password][second]' => '', - ), + ], 'This value should not be blank', - ), - array( - array( + ], + [ + [ 'change_passwd[old_password]' => 'mypassword', 'change_passwd[new_password][first]' => 'hop', 'change_passwd[new_password][second]' => '', - ), + ], 'validator.password_must_match', - ), - array( - array( + ], + [ + [ 'change_passwd[old_password]' => 'mypassword', 'change_passwd[new_password][first]' => 'hop', 'change_passwd[new_password][second]' => 'hop', - ), + ], 'validator.password_too_short', - ), - ); + ], + ]; } /** @@ -149,7 +149,7 @@ class ConfigControllerTest extends WallabagCoreTestCase $this->assertEquals(200, $client->getResponse()->getStatusCode()); - $this->assertGreaterThan(1, $alert = $crawler->filter('body')->extract(array('_text'))); + $this->assertGreaterThan(1, $alert = $crawler->filter('body')->extract(['_text'])); $this->assertContains($expectedMessage, $alert[0]); } @@ -164,11 +164,11 @@ class ConfigControllerTest extends WallabagCoreTestCase $form = $crawler->filter('button[id=change_passwd_save]')->form(); - $data = array( + $data = [ 'change_passwd[old_password]' => 'mypassword', 'change_passwd[new_password][first]' => 'mypassword', 'change_passwd[new_password][second]' => 'mypassword', - ); + ]; $client->submit($form, $data); @@ -176,28 +176,28 @@ class ConfigControllerTest extends WallabagCoreTestCase $crawler = $client->followRedirect(); - $this->assertGreaterThan(1, $alert = $crawler->filter('div.messages.success')->extract(array('_text'))); + $this->assertGreaterThan(1, $alert = $crawler->filter('div.messages.success')->extract(['_text'])); $this->assertContains('flashes.config.notice.password_updated', $alert[0]); } public function dataForUserFailed() { - return array( - array( - array( + return [ + [ + [ 'update_user[name]' => '', 'update_user[email]' => '', - ), + ], 'fos_user.email.blank', - ), - array( - array( + ], + [ + [ 'update_user[name]' => '', 'update_user[email]' => 'test', - ), + ], 'fos_user.email.invalid', - ), - ); + ], + ]; } /** @@ -218,7 +218,7 @@ class ConfigControllerTest extends WallabagCoreTestCase $this->assertEquals(200, $client->getResponse()->getStatusCode()); - $this->assertGreaterThan(1, $alert = $crawler->filter('body')->extract(array('_text'))); + $this->assertGreaterThan(1, $alert = $crawler->filter('body')->extract(['_text'])); $this->assertContains($expectedMessage, $alert[0]); } @@ -233,10 +233,10 @@ class ConfigControllerTest extends WallabagCoreTestCase $form = $crawler->filter('button[id=update_user_save]')->form(); - $data = array( + $data = [ 'update_user[name]' => 'new name', 'update_user[email]' => 'admin@wallabag.io', - ); + ]; $client->submit($form, $data); @@ -244,59 +244,59 @@ class ConfigControllerTest extends WallabagCoreTestCase $crawler = $client->followRedirect(); - $this->assertGreaterThan(1, $alert = $crawler->filter('body')->extract(array('_text'))); + $this->assertGreaterThan(1, $alert = $crawler->filter('body')->extract(['_text'])); $this->assertContains('flashes.config.notice.user_updated', $alert[0]); } public function dataForNewUserFailed() { - return array( - array( - array( + return [ + [ + [ 'new_user[username]' => '', 'new_user[plainPassword][first]' => '', 'new_user[plainPassword][second]' => '', 'new_user[email]' => '', - ), + ], 'fos_user.username.blank', - ), - array( - array( + ], + [ + [ 'new_user[username]' => 'a', 'new_user[plainPassword][first]' => 'mypassword', 'new_user[plainPassword][second]' => 'mypassword', 'new_user[email]' => '', - ), + ], 'fos_user.username.short', - ), - array( - array( + ], + [ + [ 'new_user[username]' => 'wallace', 'new_user[plainPassword][first]' => 'mypassword', 'new_user[plainPassword][second]' => 'mypassword', 'new_user[email]' => 'test', - ), + ], 'fos_user.email.invalid', - ), - array( - array( + ], + [ + [ 'new_user[username]' => 'admin', 'new_user[plainPassword][first]' => 'wallacewallace', 'new_user[plainPassword][second]' => 'wallacewallace', 'new_user[email]' => 'wallace@wallace.me', - ), + ], 'fos_user.username.already_used', - ), - array( - array( + ], + [ + [ 'new_user[username]' => 'wallace', 'new_user[plainPassword][first]' => 'mypassword1', 'new_user[plainPassword][second]' => 'mypassword2', 'new_user[email]' => 'wallace@wallace.me', - ), + ], 'validator.password_must_match', - ), - ); + ], + ]; } /** @@ -317,7 +317,7 @@ class ConfigControllerTest extends WallabagCoreTestCase $this->assertEquals(200, $client->getResponse()->getStatusCode()); - $this->assertGreaterThan(1, $alert = $crawler->filter('body')->extract(array('_text'))); + $this->assertGreaterThan(1, $alert = $crawler->filter('body')->extract(['_text'])); $this->assertContains($expectedMessage, $alert[0]); } @@ -332,12 +332,12 @@ class ConfigControllerTest extends WallabagCoreTestCase $form = $crawler->filter('button[id=new_user_save]')->form(); - $data = array( + $data = [ 'new_user[username]' => 'wallace', 'new_user[plainPassword][first]' => 'wallace1', 'new_user[plainPassword][second]' => 'wallace1', 'new_user[email]' => 'wallace@wallace.me', - ); + ]; $client->submit($form, $data); @@ -345,7 +345,7 @@ class ConfigControllerTest extends WallabagCoreTestCase $crawler = $client->followRedirect(); - $this->assertGreaterThan(1, $alert = $crawler->filter('div.messages.success')->extract(array('_text'))); + $this->assertGreaterThan(1, $alert = $crawler->filter('div.messages.success')->extract(['_text'])); $this->assertContains('flashes.config.notice.user_added', $alert[0]); $em = $client->getContainer()->get('doctrine.orm.entity_manager'); @@ -381,7 +381,7 @@ class ConfigControllerTest extends WallabagCoreTestCase $this->assertEquals(200, $client->getResponse()->getStatusCode()); - $this->assertGreaterThan(1, $body = $crawler->filter('body')->extract(array('_text'))); + $this->assertGreaterThan(1, $body = $crawler->filter('body')->extract(['_text'])); $this->assertContains('config.form_rss.no_token', $body[0]); $client->request('GET', '/generate-token'); @@ -389,7 +389,7 @@ class ConfigControllerTest extends WallabagCoreTestCase $crawler = $client->followRedirect(); - $this->assertGreaterThan(1, $body = $crawler->filter('body')->extract(array('_text'))); + $this->assertGreaterThan(1, $body = $crawler->filter('body')->extract(['_text'])); $this->assertNotContains('config.form_rss.no_token', $body[0]); } @@ -401,9 +401,9 @@ class ConfigControllerTest extends WallabagCoreTestCase $client->request( 'GET', '/generate-token', - array(), - array(), - array('HTTP_X-Requested-With' => 'XMLHttpRequest') + [], + [], + ['HTTP_X-Requested-With' => 'XMLHttpRequest'] ); $this->assertEquals(200, $client->getResponse()->getStatusCode()); @@ -422,9 +422,9 @@ class ConfigControllerTest extends WallabagCoreTestCase $form = $crawler->filter('button[id=rss_config_save]')->form(); - $data = array( + $data = [ 'rss_config[rss_limit]' => 12, - ); + ]; $client->submit($form, $data); @@ -432,26 +432,26 @@ class ConfigControllerTest extends WallabagCoreTestCase $crawler = $client->followRedirect(); - $this->assertGreaterThan(1, $alert = $crawler->filter('div.messages.success')->extract(array('_text'))); + $this->assertGreaterThan(1, $alert = $crawler->filter('div.messages.success')->extract(['_text'])); $this->assertContains('flashes.config.notice.rss_updated', $alert[0]); } public function dataForRssFailed() { - return array( - array( - array( + return [ + [ + [ 'rss_config[rss_limit]' => 0, - ), + ], 'This value should be 1 or more.', - ), - array( - array( + ], + [ + [ 'rss_config[rss_limit]' => 1000000000000, - ), + ], 'validator.rss_limit_too_hight', - ), - ); + ], + ]; } /** @@ -472,7 +472,7 @@ class ConfigControllerTest extends WallabagCoreTestCase $this->assertEquals(200, $client->getResponse()->getStatusCode()); - $this->assertGreaterThan(1, $alert = $crawler->filter('body')->extract(array('_text'))); + $this->assertGreaterThan(1, $alert = $crawler->filter('body')->extract(['_text'])); $this->assertContains($expectedMessage, $alert[0]); } @@ -487,10 +487,10 @@ class ConfigControllerTest extends WallabagCoreTestCase $form = $crawler->filter('button[id=tagging_rule_save]')->form(); - $data = array( + $data = [ 'tagging_rule[rule]' => 'readingTime <= 3', 'tagging_rule[tags]' => 'short reading', - ); + ]; $client->submit($form, $data); @@ -498,7 +498,7 @@ class ConfigControllerTest extends WallabagCoreTestCase $crawler = $client->followRedirect(); - $this->assertGreaterThan(1, $alert = $crawler->filter('div.messages.success')->extract(array('_text'))); + $this->assertGreaterThan(1, $alert = $crawler->filter('div.messages.success')->extract(['_text'])); $this->assertContains('flashes.config.notice.tagging_rules_updated', $alert[0]); $deleteLink = $crawler->filter('.delete')->last()->link(); @@ -507,34 +507,34 @@ class ConfigControllerTest extends WallabagCoreTestCase $this->assertEquals(302, $client->getResponse()->getStatusCode()); $crawler = $client->followRedirect(); - $this->assertGreaterThan(1, $alert = $crawler->filter('div.messages.success')->extract(array('_text'))); + $this->assertGreaterThan(1, $alert = $crawler->filter('div.messages.success')->extract(['_text'])); $this->assertContains('flashes.config.notice.tagging_rules_deleted', $alert[0]); } public function dataForTaggingRuleFailed() { - return array( - array( - array( + return [ + [ + [ 'tagging_rule[rule]' => 'unknownVar <= 3', 'tagging_rule[tags]' => 'cool tag', - ), - array( + ], + [ 'The variable', 'does not exist.', - ), - ), - array( - array( + ], + ], + [ + [ 'tagging_rule[rule]' => 'length(domainName) <= 42', 'tagging_rule[tags]' => 'cool tag', - ), - array( + ], + [ 'The operator', 'does not exist.', - ), - ), - ); + ], + ], + ]; } /** @@ -555,7 +555,7 @@ class ConfigControllerTest extends WallabagCoreTestCase $this->assertEquals(200, $client->getResponse()->getStatusCode()); - $this->assertGreaterThan(1, $body = $crawler->filter('body')->extract(array('_text'))); + $this->assertGreaterThan(1, $body = $crawler->filter('body')->extract(['_text'])); foreach ($messages as $message) { $this->assertContains($message, $body[0]); @@ -574,7 +574,7 @@ class ConfigControllerTest extends WallabagCoreTestCase $crawler = $client->request('GET', '/tagging-rule/delete/'.$rule->getId()); $this->assertEquals(403, $client->getResponse()->getStatusCode()); - $this->assertGreaterThan(1, $body = $crawler->filter('body')->extract(array('_text'))); + $this->assertGreaterThan(1, $body = $crawler->filter('body')->extract(['_text'])); $this->assertContains('You can not access this tagging rule', $body[0]); } @@ -593,11 +593,11 @@ class ConfigControllerTest extends WallabagCoreTestCase $form = $crawler->filter('button[id=change_passwd_save]')->form(); - $data = array( + $data = [ 'change_passwd[old_password]' => 'mypassword', 'change_passwd[new_password][first]' => 'mypassword', 'change_passwd[new_password][second]' => 'mypassword', - ); + ]; $client->submit($form, $data); diff --git a/src/Wallabag/CoreBundle/Tests/Controller/EntryControllerTest.php b/src/Wallabag/CoreBundle/Tests/Controller/EntryControllerTest.php index 46fbaf91..5260ca53 100644 --- a/src/Wallabag/CoreBundle/Tests/Controller/EntryControllerTest.php +++ b/src/Wallabag/CoreBundle/Tests/Controller/EntryControllerTest.php @@ -28,7 +28,7 @@ class EntryControllerTest extends WallabagCoreTestCase $crawler = $client->followRedirect(); $this->assertEquals(200, $client->getResponse()->getStatusCode()); - $this->assertGreaterThan(1, $body = $crawler->filter('body')->extract(array('_text'))); + $this->assertGreaterThan(1, $body = $crawler->filter('body')->extract(['_text'])); $this->assertContains('quickstart.intro.paragraph_1', $body[0]); // Test if quickstart is disabled when user has 1 entry @@ -38,16 +38,16 @@ class EntryControllerTest extends WallabagCoreTestCase $form = $crawler->filter('form[name=entry]')->form(); - $data = array( + $data = [ 'entry[url]' => $this->url, - ); + ]; $client->submit($form, $data); $this->assertEquals(302, $client->getResponse()->getStatusCode()); $client->followRedirect(); $crawler = $client->request('GET', '/unread/list'); - $this->assertGreaterThan(1, $body = $crawler->filter('body')->extract(array('_text'))); + $this->assertGreaterThan(1, $body = $crawler->filter('body')->extract(['_text'])); $this->assertContains('entry.list.number_on_the_page', $body[0]); } @@ -74,7 +74,7 @@ class EntryControllerTest extends WallabagCoreTestCase $this->assertCount(4, $crawler->filter('div[class=entry]')); // Good URL - $client->request('GET', '/bookmarklet', array('url' => $this->url)); + $client->request('GET', '/bookmarklet', ['url' => $this->url]); $this->assertEquals(302, $client->getResponse()->getStatusCode()); $client->followRedirect(); $crawler = $client->request('GET', '/'); @@ -103,7 +103,7 @@ class EntryControllerTest extends WallabagCoreTestCase $crawler = $client->submit($form); $this->assertEquals(200, $client->getResponse()->getStatusCode()); - $this->assertCount(1, $alert = $crawler->filter('form ul li')->extract(array('_text'))); + $this->assertCount(1, $alert = $crawler->filter('form ul li')->extract(['_text'])); $this->assertEquals('This value should not be blank.', $alert[0]); } @@ -121,9 +121,9 @@ class EntryControllerTest extends WallabagCoreTestCase $form = $crawler->filter('form[name=entry]')->form(); - $data = array( + $data = [ 'entry[url]' => $this->url, - ); + ]; $client->submit($form, $data); @@ -150,9 +150,9 @@ class EntryControllerTest extends WallabagCoreTestCase $form = $crawler->filter('form[name=entry]')->form(); - $data = array( + $data = [ 'entry[url]' => $this->url, - ); + ]; $client->submit($form, $data); @@ -174,9 +174,9 @@ class EntryControllerTest extends WallabagCoreTestCase $form = $crawler->filter('form[name=entry]')->form(); - $data = array( + $data = [ 'entry[url]' => $url = 'https://github.com/wallabag/wallabag', - ); + ]; $client->submit($form, $data); @@ -245,7 +245,7 @@ class EntryControllerTest extends WallabagCoreTestCase $crawler = $client->request('GET', '/view/'.$content->getId()); $this->assertEquals(200, $client->getResponse()->getStatusCode()); - $this->assertGreaterThan(1, $body = $crawler->filter('body')->extract(array('_text'))); + $this->assertGreaterThan(1, $body = $crawler->filter('body')->extract(['_text'])); $this->assertContains($content->getTitle(), $body[0]); } @@ -315,9 +315,9 @@ class EntryControllerTest extends WallabagCoreTestCase $form = $crawler->filter('button[type=submit]')->form(); - $data = array( + $data = [ 'entry[title]' => 'My updated title hehe :)', - ); + ]; $client->submit($form, $data); @@ -325,7 +325,7 @@ class EntryControllerTest extends WallabagCoreTestCase $crawler = $client->followRedirect(); - $this->assertGreaterThan(1, $alert = $crawler->filter('div[id=article] h1')->extract(array('_text'))); + $this->assertGreaterThan(1, $alert = $crawler->filter('div[id=article] h1')->extract(['_text'])); $this->assertContains('My updated title hehe :)', $alert[0]); } @@ -461,10 +461,10 @@ class EntryControllerTest extends WallabagCoreTestCase $form = $crawler->filter('button[id=submit-filter]')->form(); - $data = array( + $data = [ 'entry_filter[readingTime][right_number]' => 11, 'entry_filter[readingTime][left_number]' => 11, - ); + ]; $crawler = $client->submit($form, $data); @@ -480,28 +480,28 @@ class EntryControllerTest extends WallabagCoreTestCase $form = $crawler->filter('button[id=submit-filter]')->form(); - $data = array( + $data = [ 'entry_filter[createdAt][left_date]' => date('d/m/Y'), 'entry_filter[createdAt][right_date]' => date('d/m/Y', strtotime('+1 day')), - ); + ]; $crawler = $client->submit($form, $data); $this->assertCount(5, $crawler->filter('div[class=entry]')); - $data = array( + $data = [ 'entry_filter[createdAt][left_date]' => date('d/m/Y'), 'entry_filter[createdAt][right_date]' => date('d/m/Y'), - ); + ]; $crawler = $client->submit($form, $data); $this->assertCount(5, $crawler->filter('div[class=entry]')); - $data = array( + $data = [ 'entry_filter[createdAt][left_date]' => '01/01/1970', 'entry_filter[createdAt][right_date]' => '01/01/1970', - ); + ]; $crawler = $client->submit($form, $data); @@ -516,9 +516,9 @@ class EntryControllerTest extends WallabagCoreTestCase $form = $crawler->filter('button[id=config_save]')->form(); - $data = array( + $data = [ 'config[items_per_page]' => '1', - ); + ]; $client->submit($form, $data); @@ -531,9 +531,9 @@ class EntryControllerTest extends WallabagCoreTestCase // reset pagination $crawler = $client->request('GET', '/config'); $form = $crawler->filter('button[id=config_save]')->form(); - $data = array( + $data = [ 'config[items_per_page]' => '12', - ); + ]; $client->submit($form, $data); } @@ -544,17 +544,17 @@ class EntryControllerTest extends WallabagCoreTestCase $crawler = $client->request('GET', '/unread/list'); $form = $crawler->filter('button[id=submit-filter]')->form(); - $data = array( + $data = [ 'entry_filter[domainName]' => 'domain', - ); + ]; $crawler = $client->submit($form, $data); $this->assertCount(5, $crawler->filter('div[class=entry]')); $form = $crawler->filter('button[id=submit-filter]')->form(); - $data = array( + $data = [ 'entry_filter[domainName]' => 'wallabag', - ); + ]; $crawler = $client->submit($form, $data); $this->assertCount(0, $crawler->filter('div[class=entry]')); @@ -601,17 +601,17 @@ class EntryControllerTest extends WallabagCoreTestCase $crawler = $client->request('GET', '/unread/list'); $form = $crawler->filter('button[id=submit-filter]')->form(); - $data = array( + $data = [ 'entry_filter[language]' => 'fr', - ); + ]; $crawler = $client->submit($form, $data); $this->assertCount(2, $crawler->filter('div[class=entry]')); $form = $crawler->filter('button[id=submit-filter]')->form(); - $data = array( + $data = [ 'entry_filter[language]' => 'en', - ); + ]; $crawler = $client->submit($form, $data); $this->assertCount(2, $crawler->filter('div[class=entry]')); diff --git a/src/Wallabag/CoreBundle/Tests/Controller/RssControllerTest.php b/src/Wallabag/CoreBundle/Tests/Controller/RssControllerTest.php index 45a74c43..87a01b8e 100644 --- a/src/Wallabag/CoreBundle/Tests/Controller/RssControllerTest.php +++ b/src/Wallabag/CoreBundle/Tests/Controller/RssControllerTest.php @@ -34,17 +34,17 @@ class RssControllerTest extends WallabagCoreTestCase public function dataForBadUrl() { - return array( - array( + return [ + [ '/admin/YZIOAUZIAO/unread.xml', - ), - array( + ], + [ '/wallace/YZIOAUZIAO/starred.xml', - ), - array( + ], + [ '/wallace/YZIOAUZIAO/archives.xml', - ), - ); + ], + ]; } /** diff --git a/src/Wallabag/CoreBundle/Tests/Controller/SecurityControllerTest.php b/src/Wallabag/CoreBundle/Tests/Controller/SecurityControllerTest.php index ed292633..2910fa4f 100644 --- a/src/Wallabag/CoreBundle/Tests/Controller/SecurityControllerTest.php +++ b/src/Wallabag/CoreBundle/Tests/Controller/SecurityControllerTest.php @@ -13,7 +13,7 @@ class SecurityControllerTest extends WallabagCoreTestCase $client->followRedirects(); $crawler = $client->request('GET', '/config'); - $this->assertContains('config.form_rss.description', $crawler->filter('body')->extract(array('_text'))[0]); + $this->assertContains('config.form_rss.description', $crawler->filter('body')->extract(['_text'])[0]); } public function testLoginWith2Factor() @@ -38,7 +38,7 @@ class SecurityControllerTest extends WallabagCoreTestCase $this->logInAs('admin'); $crawler = $client->request('GET', '/config'); - $this->assertContains('scheb_two_factor.trusted', $crawler->filter('body')->extract(array('_text'))[0]); + $this->assertContains('scheb_two_factor.trusted', $crawler->filter('body')->extract(['_text'])[0]); // restore user $user = $em diff --git a/src/Wallabag/CoreBundle/Tests/Controller/TagControllerTest.php b/src/Wallabag/CoreBundle/Tests/Controller/TagControllerTest.php index 1457c807..8af37ea4 100644 --- a/src/Wallabag/CoreBundle/Tests/Controller/TagControllerTest.php +++ b/src/Wallabag/CoreBundle/Tests/Controller/TagControllerTest.php @@ -32,9 +32,9 @@ class TagControllerTest extends WallabagCoreTestCase $form = $crawler->filter('form[name=tag]')->form(); - $data = array( + $data = [ 'tag[label]' => $this->tagName, - ); + ]; $client->submit($form, $data); $this->assertEquals(302, $client->getResponse()->getStatusCode()); @@ -53,9 +53,9 @@ class TagControllerTest extends WallabagCoreTestCase $this->assertEquals(1, count($newEntry->getTags())); # tag already exists but still not assigned to this entry - $data = array( + $data = [ 'tag[label]' => 'foo', - ); + ]; $client->submit($form, $data); $this->assertEquals(302, $client->getResponse()->getStatusCode()); @@ -82,9 +82,9 @@ class TagControllerTest extends WallabagCoreTestCase $form = $crawler->filter('form[name=tag]')->form(); - $data = array( + $data = [ 'tag[label]' => 'foo2, bar2', - ); + ]; $client->submit($form, $data); $this->assertEquals(302, $client->getResponse()->getStatusCode()); -- cgit v1.2.3