use Tests\Wallabag\CoreBundle\WallabagCoreTestCase;
use Wallabag\CoreBundle\Entity\Config;
use Wallabag\CoreBundle\Entity\Entry;
+use Wallabag\CoreBundle\Entity\SiteCredential;
+use Wallabag\CoreBundle\Entity\Tag;
+use Wallabag\CoreBundle\Helper\ContentProxy;
class EntryControllerTest extends WallabagCoreTestCase
{
+ const AN_URL_CONTAINING_AN_ARTICLE_WITH_IMAGE = 'https://www.lemonde.fr/judo/article/2017/11/11/judo-la-decima-de-teddy-riner_5213605_1556020.html';
public $downloadImagesEnabled = false;
- public $url = 'http://www.lemonde.fr/pixels/article/2015/03/28/plongee-dans-l-univers-d-ingress-le-jeu-de-google-aux-frontieres-du-reel_4601155_4408996.html';
+ public $url = 'https://www.lemonde.fr/pixels/article/2015/03/28/plongee-dans-l-univers-d-ingress-le-jeu-de-google-aux-frontieres-du-reel_4601155_4408996.html';
/**
* @after
$client->request('GET', '/new');
- $this->assertEquals(302, $client->getResponse()->getStatusCode());
+ $this->assertSame(302, $client->getResponse()->getStatusCode());
$this->assertContains('login', $client->getResponse()->headers->get('location'));
}
$client = $this->getClient();
$client->request('GET', '/unread/list');
+ $this->assertSame(302, $client->getResponse()->getStatusCode());
$crawler = $client->followRedirect();
- $this->assertEquals(200, $client->getResponse()->getStatusCode());
+ $this->assertSame(200, $client->getResponse()->getStatusCode());
$this->assertGreaterThan(1, $body = $crawler->filter('body')->extract(['_text']));
$this->assertContains('quickstart.intro.title', $body[0]);
// Test if quickstart is disabled when user has 1 entry
$crawler = $client->request('GET', '/new');
- $this->assertEquals(200, $client->getResponse()->getStatusCode());
+ $this->assertSame(200, $client->getResponse()->getStatusCode());
$form = $crawler->filter('form[name=entry]')->form();
];
$client->submit($form, $data);
- $this->assertEquals(302, $client->getResponse()->getStatusCode());
+ $this->assertSame(302, $client->getResponse()->getStatusCode());
$client->followRedirect();
$crawler = $client->request('GET', '/unread/list');
$crawler = $client->request('GET', '/new');
- $this->assertEquals(200, $client->getResponse()->getStatusCode());
+ $this->assertSame(200, $client->getResponse()->getStatusCode());
$this->assertCount(1, $crawler->filter('input[type=url]'));
$this->assertCount(1, $crawler->filter('form[name=entry]'));
// Good URL
$client->request('GET', '/bookmarklet', ['url' => $this->url]);
- $this->assertEquals(302, $client->getResponse()->getStatusCode());
+ $this->assertSame(302, $client->getResponse()->getStatusCode());
$client->followRedirect();
$crawler = $client->request('GET', '/');
$this->assertCount(5, $crawler->filter('div[class=entry]'));
$crawler = $client->request('GET', '/new');
- $this->assertEquals(200, $client->getResponse()->getStatusCode());
+ $this->assertSame(200, $client->getResponse()->getStatusCode());
$form = $crawler->filter('form[name=entry]')->form();
$crawler = $client->submit($form);
- $this->assertEquals(200, $client->getResponse()->getStatusCode());
+ $this->assertSame(200, $client->getResponse()->getStatusCode());
$this->assertCount(1, $alert = $crawler->filter('form ul li')->extract(['_text']));
- $this->assertEquals('This value should not be blank.', $alert[0]);
+ $this->assertSame('This value should not be blank.', $alert[0]);
}
/**
$this->logInAs('admin');
$client = $this->getClient();
+ $client->getContainer()->get('craue_config')->set('store_article_headers', 1);
+
$crawler = $client->request('GET', '/new');
- $this->assertEquals(200, $client->getResponse()->getStatusCode());
+ $this->assertSame(200, $client->getResponse()->getStatusCode());
$form = $crawler->filter('form[name=entry]')->form();
$client->submit($form, $data);
- $this->assertEquals(302, $client->getResponse()->getStatusCode());
+ $this->assertSame(302, $client->getResponse()->getStatusCode());
$content = $client->getContainer()
->get('doctrine.orm.entity_manager')
$author = $content->getPublishedBy();
$this->assertInstanceOf('Wallabag\CoreBundle\Entity\Entry', $content);
- $this->assertEquals($this->url, $content->getUrl());
+ $this->assertSame($this->url, $content->getUrl());
$this->assertContains('Google', $content->getTitle());
- $this->assertEquals('2015-03-28 15:37:39', $content->getPublishedAt()->format('Y-m-d H:i:s'));
- $this->assertEquals('Morgane Tual', $author[0]);
- $this->assertArrayHasKey('x-varnish1', $content->getHeaders());
+ $this->assertSame('fr', $content->getLanguage());
+ $this->assertSame('2016-04-07 19:01:35', $content->getPublishedAt()->format('Y-m-d H:i:s'));
+ $this->assertArrayHasKey('x-frame-options', $content->getHeaders());
+ $client->getContainer()->get('craue_config')->set('store_article_headers', 0);
}
public function testPostWithMultipleAuthors()
{
- $url = 'http://www.liberation.fr/planete/2017/04/05/donald-trump-et-xi-jinping-tentative-de-flirt-en-floride_1560768';
+ $url = 'https://www.liberation.fr/planete/2017/04/05/donald-trump-et-xi-jinping-tentative-de-flirt-en-floride_1560768';
$this->logInAs('admin');
$client = $this->getClient();
$crawler = $client->request('GET', '/new');
- $this->assertEquals(200, $client->getResponse()->getStatusCode());
+ $this->assertSame(200, $client->getResponse()->getStatusCode());
$form = $crawler->filter('form[name=entry]')->form();
$client->submit($form, $data);
- $this->assertEquals(302, $client->getResponse()->getStatusCode());
+ $this->assertSame(302, $client->getResponse()->getStatusCode());
$content = $client->getContainer()
->get('doctrine.orm.entity_manager')
->getRepository('WallabagCoreBundle:Entry')
->findByUrlAndUserId($url, $this->getLoggedInUserId());
+ $this->assertInstanceOf('Wallabag\CoreBundle\Entity\Entry', $content);
$authors = $content->getPublishedBy();
- $this->assertEquals('2017-04-05 19:26:13', $content->getPublishedAt()->format('Y-m-d H:i:s'));
- $this->assertEquals('Raphaël Balenieri, correspondant à Pékin', $authors[0]);
- $this->assertEquals('Frédéric Autran, correspondant à New York', $authors[1]);
+ $this->assertSame('2017-04-05 19:26:13', $content->getPublishedAt()->format('Y-m-d H:i:s'));
+ $this->assertSame('fr', $content->getLanguage());
+ $this->assertSame('Raphaël Balenieri, correspondant à Pékin', $authors[0]);
+ $this->assertSame('Frédéric Autran, correspondant à New York', $authors[1]);
}
public function testPostNewOkUrlExist()
$crawler = $client->request('GET', '/new');
- $this->assertEquals(200, $client->getResponse()->getStatusCode());
+ $this->assertSame(200, $client->getResponse()->getStatusCode());
$form = $crawler->filter('form[name=entry]')->form();
$client->submit($form, $data);
- $this->assertEquals(302, $client->getResponse()->getStatusCode());
+ $this->assertSame(302, $client->getResponse()->getStatusCode());
$this->assertContains('/view/', $client->getResponse()->getTargetUrl());
}
$crawler = $client->request('GET', '/new');
- $this->assertEquals(200, $client->getResponse()->getStatusCode());
+ $this->assertSame(200, $client->getResponse()->getStatusCode());
$form = $crawler->filter('form[name=entry]')->form();
$crawler = $client->request('GET', '/new');
- $this->assertEquals(200, $client->getResponse()->getStatusCode());
+ $this->assertSame(200, $client->getResponse()->getStatusCode());
$form = $crawler->filter('form[name=entry]')->form();
$client->submit($form, $data);
- $this->assertEquals(302, $client->getResponse()->getStatusCode());
+ $this->assertSame(302, $client->getResponse()->getStatusCode());
$this->assertContains('/view/', $client->getResponse()->getTargetUrl());
-
- $em = $client->getContainer()
- ->get('doctrine.orm.entity_manager');
- $entry = $em
- ->getRepository('WallabagCoreBundle:Entry')
- ->findOneByUrl(urldecode($url));
-
- $em->remove($entry);
- $em->flush();
}
/**
$crawler = $client->request('GET', '/new');
- $this->assertEquals(200, $client->getResponse()->getStatusCode());
+ $this->assertSame(200, $client->getResponse()->getStatusCode());
$form = $crawler->filter('form[name=entry]')->form();
$client->submit($form, $data);
- $this->assertEquals(302, $client->getResponse()->getStatusCode());
+ $this->assertSame(302, $client->getResponse()->getStatusCode());
$this->assertContains('/', $client->getResponse()->getTargetUrl());
$em = $client->getContainer()
$this->assertCount(2, $tags);
$this->assertContains('wallabag', $tags);
+ $this->assertSame('en', $entry->getLanguage());
$em->remove($entry);
$em->flush();
// related https://github.com/wallabag/wallabag/issues/2121
$crawler = $client->request('GET', '/new');
- $this->assertEquals(200, $client->getResponse()->getStatusCode());
+ $this->assertSame(200, $client->getResponse()->getStatusCode());
$form = $crawler->filter('form[name=entry]')->form();
$client->submit($form, $data);
- $this->assertEquals(302, $client->getResponse()->getStatusCode());
+ $this->assertSame(302, $client->getResponse()->getStatusCode());
$this->assertContains('/', $client->getResponse()->getTargetUrl());
$entry = $em
$client->request('GET', '/archive/list');
- $this->assertEquals(200, $client->getResponse()->getStatusCode());
+ $this->assertSame(200, $client->getResponse()->getStatusCode());
}
public function testUntagged()
$client->request('GET', '/untagged/list');
- $this->assertEquals(200, $client->getResponse()->getStatusCode());
+ $this->assertSame(200, $client->getResponse()->getStatusCode());
}
public function testStarred()
$client->request('GET', '/starred/list');
- $this->assertEquals(200, $client->getResponse()->getStatusCode());
+ $this->assertSame(200, $client->getResponse()->getStatusCode());
}
public function testRangeException()
$client->request('GET', '/all/list/900');
- $this->assertEquals(302, $client->getResponse()->getStatusCode());
- $this->assertEquals('/all/list', $client->getResponse()->getTargetUrl());
+ $this->assertSame(302, $client->getResponse()->getStatusCode());
+ $this->assertSame('/all/list', $client->getResponse()->getTargetUrl());
}
public function testView()
$this->getEntityManager()->persist($entry);
$this->getEntityManager()->flush();
- $crawler = $client->request('GET', '/view/'.$entry->getId());
+ $crawler = $client->request('GET', '/view/' . $entry->getId());
- $this->assertEquals(200, $client->getResponse()->getStatusCode());
+ $this->assertSame(200, $client->getResponse()->getStatusCode());
$this->assertGreaterThan(1, $body = $crawler->filter('body')->extract(['_text']));
$this->assertContains($entry->getTitle(), $body[0]);
}
/**
- * @depends testPostNewOk
- *
* This test will require an internet connection.
*/
public function testReload()
$this->getEntityManager()->flush();
$this->getEntityManager()->clear();
- $client->request('GET', '/reload/'.$entry->getId());
+ $client->request('GET', '/reload/' . $entry->getId());
- $this->assertEquals(302, $client->getResponse()->getStatusCode());
+ $this->assertSame(302, $client->getResponse()->getStatusCode());
$entry = $this->getEntityManager()
->getRepository('WallabagCoreBundle:Entry')
$this->assertNotEmpty($entry->getContent());
}
- /**
- * @depends testPostNewOk
- */
public function testReloadWithFetchingFailed()
{
$this->logInAs('admin');
$this->getEntityManager()->persist($entry);
$this->getEntityManager()->flush();
- $client->request('GET', '/reload/'.$entry->getId());
+ $client->request('GET', '/reload/' . $entry->getId());
- $this->assertEquals(302, $client->getResponse()->getStatusCode());
+ $this->assertSame(302, $client->getResponse()->getStatusCode());
// force EntityManager to clear previous entity
// otherwise, retrieve the same entity will retrieve change from the previous request :0
->getRepository('WallabagCoreBundle:Entry')
->find($entry->getId());
- $this->assertNotEquals($client->getContainer()->getParameter('wallabag_core.fetching_error_message'), $newContent->getContent());
+ $this->assertNotSame($client->getContainer()->getParameter('wallabag_core.fetching_error_message'), $newContent->getContent());
}
public function testEdit()
$this->getEntityManager()->persist($entry);
$this->getEntityManager()->flush();
- $crawler = $client->request('GET', '/edit/'.$entry->getId());
+ $crawler = $client->request('GET', '/edit/' . $entry->getId());
- $this->assertEquals(200, $client->getResponse()->getStatusCode());
+ $this->assertSame(200, $client->getResponse()->getStatusCode());
$this->assertCount(1, $crawler->filter('input[id=entry_title]'));
$this->assertCount(1, $crawler->filter('button[id=entry_save]'));
$this->getEntityManager()->persist($entry);
$this->getEntityManager()->flush();
- $crawler = $client->request('GET', '/edit/'.$entry->getId());
+ $crawler = $client->request('GET', '/edit/' . $entry->getId());
+
+ $this->assertSame(200, $client->getResponse()->getStatusCode());
+
+ $form = $crawler->filter('button[id=entry_save]')->form();
+
+ $data = [
+ 'entry[title]' => 'My updated title hehe :)',
+ 'entry[origin_url]' => 'https://example.io',
+ ];
+
+ $client->submit($form, $data);
+
+ $this->assertSame(302, $client->getResponse()->getStatusCode());
+
+ $crawler = $client->followRedirect();
+
+ $this->assertGreaterThan(1, $title = $crawler->filter('div[id=article] h1')->extract(['_text']));
+ $this->assertContains('My updated title hehe :)', $title[0]);
+ $this->assertGreaterThan(1, $stats = $crawler->filter('div[class=tools] ul[class=stats] li a[class=tool]')->extract(['_text']));
+ $this->assertContains('example.io', trim($stats[1]));
+ }
+
+ public function testEditRemoveOriginUrl()
+ {
+ $this->logInAs('admin');
+ $client = $this->getClient();
+
+ $entry = new Entry($this->getLoggedInUser());
+ $entry->setUrl($this->url);
+ $this->getEntityManager()->persist($entry);
+ $this->getEntityManager()->flush();
+
+ $crawler = $client->request('GET', '/edit/' . $entry->getId());
- $this->assertEquals(200, $client->getResponse()->getStatusCode());
+ $this->assertSame(200, $client->getResponse()->getStatusCode());
- $form = $crawler->filter('button[type=submit]')->form();
+ $form = $crawler->filter('button[id=entry_save]')->form();
$data = [
'entry[title]' => 'My updated title hehe :)',
+ 'entry[origin_url]' => '',
];
$client->submit($form, $data);
- $this->assertEquals(302, $client->getResponse()->getStatusCode());
+ $this->assertSame(302, $client->getResponse()->getStatusCode());
$crawler = $client->followRedirect();
- $this->assertGreaterThan(1, $alert = $crawler->filter('div[id=article] h1')->extract(['_text']));
- $this->assertContains('My updated title hehe :)', $alert[0]);
+ $title = $crawler->filter('div[id=article] h1')->extract(['_text']);
+ $this->assertGreaterThan(1, $title);
+ $this->assertContains('My updated title hehe :)', $title[0]);
+
+ $stats = $crawler->filter('div[class=tools] ul[class=stats] li a[class=tool]')->extract(['_text']);
+ $this->assertCount(1, $stats);
+ $this->assertNotContains('example.io', trim($stats[0]));
}
public function testToggleArchive()
$this->getEntityManager()->flush();
$this->getEntityManager()->clear();
- $client->request('GET', '/archive/'.$entry->getId());
+ $client->request('GET', '/archive/' . $entry->getId());
- $this->assertEquals(302, $client->getResponse()->getStatusCode());
+ $this->assertSame(302, $client->getResponse()->getStatusCode());
$res = $client->getContainer()
->get('doctrine.orm.entity_manager')
->getRepository('WallabagCoreBundle:Entry')
->find($entry->getId());
- $this->assertEquals($res->isArchived(), true);
+ $this->assertSame(1, $res->isArchived());
}
public function testToggleStar()
$this->getEntityManager()->flush();
$this->getEntityManager()->clear();
- $client->request('GET', '/star/'.$entry->getId());
+ $client->request('GET', '/star/' . $entry->getId());
- $this->assertEquals(302, $client->getResponse()->getStatusCode());
+ $this->assertSame(302, $client->getResponse()->getStatusCode());
$res = $client->getContainer()
->get('doctrine.orm.entity_manager')
->getRepository('WallabagCoreBundle:Entry')
->findOneById($entry->getId());
- $this->assertEquals($res->isStarred(), true);
+ $this->assertSame(1, $res->isStarred());
}
public function testDelete()
$this->getEntityManager()->persist($entry);
$this->getEntityManager()->flush();
- $client->request('GET', '/delete/'.$entry->getId());
+ $client->request('GET', '/delete/' . $entry->getId());
- $this->assertEquals(302, $client->getResponse()->getStatusCode());
+ $this->assertSame(302, $client->getResponse()->getStatusCode());
- $client->request('GET', '/delete/'.$entry->getId());
+ $client->request('GET', '/delete/' . $entry->getId());
- $this->assertEquals(404, $client->getResponse()->getStatusCode());
+ $this->assertSame(404, $client->getResponse()->getStatusCode());
}
/**
$content->setMimetype('text/html');
$content->setTitle('test title entry');
$content->setContent('This is my content /o/');
- $content->setArchived(true);
+ $content->updateArchived(true);
$content->setLanguage('fr');
$em->persist($content);
$em->flush();
- $client->request('GET', '/view/'.$content->getId());
- $this->assertEquals(200, $client->getResponse()->getStatusCode());
+ $client->request('GET', '/view/' . $content->getId());
+ $this->assertSame(200, $client->getResponse()->getStatusCode());
- $client->request('GET', '/delete/'.$content->getId());
- $this->assertEquals(302, $client->getResponse()->getStatusCode());
+ $client->request('GET', '/delete/' . $content->getId());
+ $this->assertSame(302, $client->getResponse()->getStatusCode());
$client->followRedirect();
- $this->assertEquals(200, $client->getResponse()->getStatusCode());
+ $this->assertSame(200, $client->getResponse()->getStatusCode());
}
public function testViewOtherUserEntry()
->getRepository('WallabagCoreBundle:Entry')
->findOneByUsernameAndNotArchived('bob');
- $client->request('GET', '/view/'.$content->getId());
+ $client->request('GET', '/view/' . $content->getId());
- $this->assertEquals(403, $client->getResponse()->getStatusCode());
+ $this->assertSame(403, $client->getResponse()->getStatusCode());
}
public function testFilterOnReadingTime()
$entry = new Entry($this->getLoggedInUser());
$entry->setUrl($this->url);
- $entry->setArchived(false);
+ $entry->updateArchived(false);
$this->getEntityManager()->persist($entry);
$this->getEntityManager()->flush();
$parameters = '?entry_filter%5BreadingTime%5D%5Bleft_number%5D=&entry_filter%5BreadingTime%5D%5Bright_number%5D=';
- $client->request('GET', 'unread/list'.$parameters);
+ $client->request('GET', 'unread/list' . $parameters);
$this->assertContains($parameters, $client->getResponse()->getContent());
$this->assertCount(1, $crawler->filter('div[class=entry]'));
}
+ public function testFilterOnIsPublic()
+ {
+ $this->logInAs('admin');
+ $this->useTheme('baggy');
+ $client = $this->getClient();
+
+ $crawler = $client->request('GET', '/unread/list');
+ $form = $crawler->filter('button[id=submit-filter]')->form();
+ $form['entry_filter[isPublic]']->tick();
+
+ $crawler = $client->submit($form);
+ $this->assertCount(0, $crawler->filter('div[class=entry]'));
+ }
+
public function testPreviewPictureFilter()
{
$this->logInAs('admin');
$this->getEntityManager()->clear();
// no uid
- $client->request('GET', '/share/'.$content->getUid());
- $this->assertEquals(404, $client->getResponse()->getStatusCode());
+ $client->request('GET', '/share/' . $content->getUid());
+ $this->assertSame(404, $client->getResponse()->getStatusCode());
// generating the uid
- $client->request('GET', '/share/'.$content->getId());
- $this->assertEquals(302, $client->getResponse()->getStatusCode());
+ $client->request('GET', '/share/' . $content->getId());
+ $this->assertSame(302, $client->getResponse()->getStatusCode());
- // follow link with uid
- $crawler = $client->followRedirect();
- $this->assertEquals(200, $client->getResponse()->getStatusCode());
+ $shareUrl = $client->getResponse()->getTargetUrl();
+
+ // use a new client to have a fresh empty session (instead of a logged one from the previous client)
+ $client->restart();
+
+ $client->request('GET', $shareUrl);
+
+ $this->assertSame(200, $client->getResponse()->getStatusCode());
$this->assertContains('max-age=25200', $client->getResponse()->headers->get('cache-control'));
$this->assertContains('public', $client->getResponse()->headers->get('cache-control'));
$this->assertContains('s-maxage=25200', $client->getResponse()->headers->get('cache-control'));
// sharing is now disabled
$client->getContainer()->get('craue_config')->set('share_public', 0);
- $client->request('GET', '/share/'.$content->getUid());
- $this->assertEquals(404, $client->getResponse()->getStatusCode());
-
- $client->request('GET', '/view/'.$content->getId());
- $this->assertContains('no-cache', $client->getResponse()->headers->get('cache-control'));
+ $client->request('GET', '/share/' . $content->getUid());
+ $this->assertSame(404, $client->getResponse()->getStatusCode());
// removing the share
- $client->request('GET', '/share/delete/'.$content->getId());
- $this->assertEquals(302, $client->getResponse()->getStatusCode());
+ $client->request('GET', '/share/delete/' . $content->getId());
+ $this->assertSame(302, $client->getResponse()->getStatusCode());
// share is now disable
- $client->request('GET', '/share/'.$content->getUid());
- $this->assertEquals(404, $client->getResponse()->getStatusCode());
+ $client->request('GET', '/share/' . $content->getUid());
+ $this->assertSame(404, $client->getResponse()->getStatusCode());
}
public function testNewEntryWithDownloadImagesEnabled()
$this->logInAs('admin');
$client = $this->getClient();
- $url = 'http://www.20minutes.fr/montpellier/1952003-20161030-video-car-tombe-panne-rugbymen-perpignan-improvisent-melee-route';
+ $url = self::AN_URL_CONTAINING_AN_ARTICLE_WITH_IMAGE;
$client->getContainer()->get('craue_config')->set('download_images_enabled', 1);
$crawler = $client->request('GET', '/new');
- $this->assertEquals(200, $client->getResponse()->getStatusCode());
+ $this->assertSame(200, $client->getResponse()->getStatusCode());
$form = $crawler->filter('form[name=entry]')->form();
$client->submit($form, $data);
- $this->assertEquals(302, $client->getResponse()->getStatusCode());
+ $this->assertSame(302, $client->getResponse()->getStatusCode());
$em = $client->getContainer()
->get('doctrine.orm.entity_manager');
->findByUrlAndUserId($url, $this->getLoggedInUserId());
$this->assertInstanceOf('Wallabag\CoreBundle\Entity\Entry', $entry);
- $this->assertEquals($url, $entry->getUrl());
- $this->assertContains('Perpignan', $entry->getTitle());
+ $this->assertSame($url, $entry->getUrl());
+ $this->assertContains('Judo', $entry->getTitle());
// instead of checking for the filename (which might change) check that the image is now local
- $this->assertContains('http://v2.wallabag.org/assets/images/', $entry->getContent());
+ $this->assertContains(rtrim($client->getContainer()->getParameter('domain_name'), '/') . '/assets/images/', $entry->getContent());
$client->getContainer()->get('craue_config')->set('download_images_enabled', 0);
}
$this->logInAs('admin');
$client = $this->getClient();
- $url = 'http://www.20minutes.fr/montpellier/1952003-20161030-video-car-tombe-panne-rugbymen-perpignan-improvisent-melee-route';
+ $url = self::AN_URL_CONTAINING_AN_ARTICLE_WITH_IMAGE;
$client->getContainer()->get('craue_config')->set('download_images_enabled', 1);
$crawler = $client->request('GET', '/new');
- $this->assertEquals(200, $client->getResponse()->getStatusCode());
+ $this->assertSame(200, $client->getResponse()->getStatusCode());
$form = $crawler->filter('form[name=entry]')->form();
$client->submit($form, $data);
- $this->assertEquals(302, $client->getResponse()->getStatusCode());
+ $this->assertSame(302, $client->getResponse()->getStatusCode());
$content = $client->getContainer()
->get('doctrine.orm.entity_manager')
->getRepository('WallabagCoreBundle:Entry')
->findByUrlAndUserId($url, $this->getLoggedInUserId());
- $client->request('GET', '/delete/'.$content->getId());
+ $client->request('GET', '/delete/' . $content->getId());
- $this->assertEquals(302, $client->getResponse()->getStatusCode());
+ $this->assertSame(302, $client->getResponse()->getStatusCode());
$client->getContainer()->get('craue_config')->set('download_images_enabled', 0);
}
$this->getEntityManager()->flush();
- $client->request('GET', '/view/'.$entry->getId());
- $client->request('GET', '/archive/'.$entry->getId());
+ $client->request('GET', '/view/' . $entry->getId());
+ $client->request('GET', '/archive/' . $entry->getId());
- $this->assertEquals(302, $client->getResponse()->getStatusCode());
- $this->assertEquals('/', $client->getResponse()->headers->get('location'));
+ $this->assertSame(302, $client->getResponse()->getStatusCode());
+ $this->assertSame('/', $client->getResponse()->headers->get('location'));
}
public function testRedirectToCurrentPage()
$this->getEntityManager()->flush();
- $client->request('GET', '/view/'.$entry->getId());
- $client->request('GET', '/archive/'.$entry->getId());
+ $client->request('GET', '/view/' . $entry->getId());
+ $client->request('GET', '/archive/' . $entry->getId());
- $this->assertEquals(302, $client->getResponse()->getStatusCode());
- $this->assertContains('/view/'.$entry->getId(), $client->getResponse()->headers->get('location'));
+ $this->assertSame(302, $client->getResponse()->getStatusCode());
+ $this->assertContains('/view/' . $entry->getId(), $client->getResponse()->headers->get('location'));
}
public function testFilterOnHttpStatus()
$client = $this->getClient();
$entry = new Entry($this->getLoggedInUser());
- $entry->setUrl('http://www.lemonde.fr/incorrect-url/');
+ $entry->setUrl('https://www.lemonde.fr/incorrect-url/');
$entry->setHttpStatus(404);
$this->getEntityManager()->persist($entry);
$entry = new Entry($this->getLoggedInUser());
$entry->setUrl('http://0.0.0.0/foo/baz/qux');
$entry->setTitle('Le manège');
- $entry->setArchived(true);
+ $entry->updateArchived(true);
$this->getEntityManager()->persist($entry);
$this->getEntityManager()->flush();
$crawler = $client->submit($form, $data);
$this->assertCount(1, $crawler->filter('div[class=entry]'));
- $client->request('GET', '/delete/'.$entry->getId());
+ $client->request('GET', '/delete/' . $entry->getId());
// test on list of all articles
$crawler = $client->request('GET', '/all/list');
$entry = new Entry($this->getLoggedInUser());
$entry->setUrl('http://domain/qux');
$entry->setTitle('Le manège');
- $entry->setArchived(true);
+ $entry->updateArchived(true);
$this->getEntityManager()->persist($entry);
$this->getEntityManager()->flush();
$this->assertCount(1, $crawler->filter('div[class=entry]'));
}
+
+ public function dataForLanguage()
+ {
+ return [
+ 'ru' => [
+ 'https://www.pravda.ru/world/09-06-2017/1337283-qatar-0/',
+ 'ru',
+ ],
+ 'fr' => [
+ 'https://fr.wikipedia.org/wiki/Wallabag',
+ 'fr',
+ ],
+ 'de' => [
+ 'https://www.bild.de/politik/ausland/theresa-may/wahlbeben-grossbritannien-analyse-52108924.bild.html',
+ 'de',
+ ],
+ 'it' => [
+ 'http://www.ansa.it/sito/notizie/mondo/europa/2017/06/08/voto-gb-seggi-aperti-misure-sicurezza-rafforzate_0cb71f7f-e23b-4d5f-95ca-bc12296419f0.html',
+ 'it',
+ ],
+ 'zh_CN' => [
+ 'http://www.hao123.com/shequ?__noscript__-=1',
+ 'zh_CN',
+ ],
+ 'pt_BR' => [
+ 'https://politica.estadao.com.br/noticias/eleicoes,campanha-catatonica,70002491983',
+ 'pt_BR',
+ ],
+ 'fucked_list_of_languages' => [
+ 'http://geocatalog.webservice-energy.org/geonetwork/srv/eng/main.home',
+ null,
+ ],
+ 'es-ES' => [
+ 'https://www.20minutos.es/noticia/3360685/0/gobierno-sanchez-primero-historia-mas-mujeres-que-hombres/',
+ 'es',
+ ],
+ ];
+ }
+
+ /**
+ * @dataProvider dataForLanguage
+ */
+ public function testLanguageValidation($url, $expectedLanguage)
+ {
+ $this->logInAs('admin');
+ $client = $this->getClient();
+
+ $crawler = $client->request('GET', '/new');
+
+ $this->assertSame(200, $client->getResponse()->getStatusCode());
+
+ $form = $crawler->filter('form[name=entry]')->form();
+
+ $data = [
+ 'entry[url]' => $url,
+ ];
+
+ $client->submit($form, $data);
+
+ $this->assertSame(302, $client->getResponse()->getStatusCode());
+
+ $content = $client->getContainer()
+ ->get('doctrine.orm.entity_manager')
+ ->getRepository('WallabagCoreBundle:Entry')
+ ->findByUrlAndUserId($url, $this->getLoggedInUserId());
+
+ $this->assertInstanceOf('Wallabag\CoreBundle\Entity\Entry', $content);
+ $this->assertSame($url, $content->getUrl());
+ $this->assertSame($expectedLanguage, $content->getLanguage());
+ }
+
+ /**
+ * This test will require an internet connection.
+ */
+ public function testRestrictedArticle()
+ {
+ $url = 'https://www.monde-diplomatique.fr/2017/05/BONNET/57475';
+ $this->logInAs('admin');
+ $client = $this->getClient();
+ $em = $client->getContainer()->get('doctrine.orm.entity_manager');
+
+ // enable restricted access
+ $client->getContainer()->get('craue_config')->set('restricted_access', 1);
+
+ // create a new site_credential
+ $user = $client->getContainer()->get('security.token_storage')->getToken()->getUser();
+ $credential = new SiteCredential($user);
+ $credential->setHost('monde-diplomatique.fr');
+ $credential->setUsername($client->getContainer()->get('wallabag_core.helper.crypto_proxy')->crypt('foo'));
+ $credential->setPassword($client->getContainer()->get('wallabag_core.helper.crypto_proxy')->crypt('bar'));
+
+ $em->persist($credential);
+ $em->flush();
+
+ $crawler = $client->request('GET', '/new');
+
+ $this->assertSame(200, $client->getResponse()->getStatusCode());
+
+ $form = $crawler->filter('form[name=entry]')->form();
+
+ $data = [
+ 'entry[url]' => $url,
+ ];
+
+ $client->submit($form, $data);
+
+ $this->assertSame(302, $client->getResponse()->getStatusCode());
+
+ $crawler = $client->followRedirect();
+
+ $this->assertSame(200, $client->getResponse()->getStatusCode());
+ $this->assertContains('flashes.entry.notice.entry_saved', $crawler->filter('body')->extract(['_text'])[0]);
+
+ $content = $em
+ ->getRepository('WallabagCoreBundle:Entry')
+ ->findByUrlAndUserId($url, $this->getLoggedInUserId());
+
+ $this->assertInstanceOf('Wallabag\CoreBundle\Entity\Entry', $content);
+ $this->assertSame('Crimes et réformes aux Philippines', $content->getTitle());
+
+ $client->getContainer()->get('craue_config')->set('restricted_access', 0);
+ }
+
+ public function testPostEntryWhenFetchFails()
+ {
+ $url = 'http://example.com/papers/email_tracking.pdf';
+ $this->logInAs('admin');
+ $client = $this->getClient();
+
+ $container = $client->getContainer();
+ $contentProxy = $this->getMockBuilder(ContentProxy::class)
+ ->disableOriginalConstructor()
+ ->setMethods(['updateEntry'])
+ ->getMock();
+ $contentProxy->expects($this->any())
+ ->method('updateEntry')
+ ->willThrowException(new \Exception('Test Fetch content fails'));
+
+ $crawler = $client->request('GET', '/new');
+
+ $this->assertSame(200, $client->getResponse()->getStatusCode());
+
+ $form = $crawler->filter('form[name=entry]')->form();
+
+ $data = [
+ 'entry[url]' => $url,
+ ];
+
+ /**
+ * We generate a new client to be able to use Mock ContentProxy
+ * Also we reinject the cookie from the previous client to keep the
+ * session.
+ */
+ $cookie = $client->getCookieJar()->all();
+ $client = $this->getNewClient();
+ $client->getCookieJar()->set($cookie[0]);
+ $client->getContainer()->set('wallabag_core.content_proxy', $contentProxy);
+ $client->submit($form, $data);
+
+ $this->assertSame(302, $client->getResponse()->getStatusCode());
+
+ $content = $client->getContainer()
+ ->get('doctrine.orm.entity_manager')
+ ->getRepository('WallabagCoreBundle:Entry')
+ ->findByUrlAndUserId($url, $this->getLoggedInUserId());
+
+ $authors = $content->getPublishedBy();
+ $this->assertSame('email_tracking.pdf', $content->getTitle());
+ $this->assertSame('example.com', $content->getDomainName());
+ }
+
+ public function testEntryDeleteTagLink()
+ {
+ $this->logInAs('admin');
+ $client = $this->getClient();
+
+ $em = $client->getContainer()->get('doctrine.orm.entity_manager');
+ $entry = $em->getRepository('WallabagCoreBundle:Entry')->findByUrlAndUserId('http://0.0.0.0/entry1', $this->getLoggedInUserId());
+ $tag = $entry->getTags()[0];
+
+ $crawler = $client->request('GET', '/view/' . $entry->getId());
+
+ // As long as the deletion link of a tag is following
+ // a link to the tag view, we take the second one to retrieve
+ // the deletion link of the first tag
+ $link = $crawler->filter('body div#article div.tools ul.tags li.chip a')->extract('href')[1];
+
+ $this->assertSame(sprintf('/remove-tag/%s/%s', $entry->getId(), $tag->getId()), $link);
+ }
}
projects / github / wallabag / wallabag.git / blobdiff