From 23dcd8e5bff6a2e366b085ac373638206652f055 Mon Sep 17 00:00:00 2001 From: Bernhard Posselt Date: Thu, 6 Nov 2014 17:36:54 +0100 Subject: fix mysql --- tests/integration/db/ItemMapperTest.php | 3 +- tests/integration/fixtures/items.json | 38 +++++----- tests/unit/db/MapperFactoryTest.php | 50 ++++++++++++++ tests/unit/db/mysql/ItemMapperTest.php | 119 ++++++++++++++++++++++++++++++++ 4 files changed, 190 insertions(+), 20 deletions(-) create mode 100644 tests/unit/db/MapperFactoryTest.php create mode 100644 tests/unit/db/mysql/ItemMapperTest.php (limited to 'tests') diff --git a/tests/integration/db/ItemMapperTest.php b/tests/integration/db/ItemMapperTest.php index 53ade4c79..b0b979f8f 100644 --- a/tests/integration/db/ItemMapperTest.php +++ b/tests/integration/db/ItemMapperTest.php @@ -54,7 +54,8 @@ class ItemMapperTest extends NewsIntegrationTest { $this->userId); $this->itemMapper->find($this->items['a title2']->getId(), $this->userId); - $this->itemMapper->find($this->items['a title3']->getId(), $this->userId); + $this->itemMapper->find($this->items['a title3']->getId(), + $this->userId); $this->itemMapper->find($this->items['del3']->getId(), $this->userId); $this->itemMapper->find($this->items['del4']->getId(), $this->userId); } diff --git a/tests/integration/fixtures/items.json b/tests/integration/fixtures/items.json index 780796afe..7f6b6c9c4 100644 --- a/tests/integration/fixtures/items.json +++ b/tests/integration/fixtures/items.json @@ -1,11 +1,11 @@ { "first feed": [ { - "status": 2, + "status": 4, "body": "this is a body", - "title": "a title1", + "title": "a title2", "author": "my author", - "guid": "abc", + "guid": "def", "url": "http://google.de", "pubDate": 2323, "lastModified": 113, @@ -13,11 +13,11 @@ "enclosureLink": "http://google.de/web.webm" }, { - "status": 0, + "status": 6, "body": "this is a body", - "title": "del1", + "title": "a title3", "author": "my author", - "guid": "del1", + "guid": "gih", "url": "http://google.de", "pubDate": 2323, "lastModified": 113, @@ -25,11 +25,11 @@ "enclosureLink": "http://google.de/web.webm" }, { - "status": 0, + "status": 2, "body": "this is a body", - "title": "del2", + "title": "a title1", "author": "my author", - "guid": "del2", + "guid": "abc", "url": "http://google.de", "pubDate": 2323, "lastModified": 113, @@ -39,9 +39,9 @@ { "status": 0, "body": "this is a body", - "title": "del3", + "title": "del1", "author": "my author", - "guid": "del3", + "guid": "del1", "url": "http://google.de", "pubDate": 2323, "lastModified": 113, @@ -51,9 +51,9 @@ { "status": 0, "body": "this is a body", - "title": "del4", + "title": "del2", "author": "my author", - "guid": "del4", + "guid": "del2", "url": "http://google.de", "pubDate": 2323, "lastModified": 113, @@ -61,11 +61,11 @@ "enclosureLink": "http://google.de/web.webm" }, { - "status": 4, + "status": 0, "body": "this is a body", - "title": "a title2", + "title": "del3", "author": "my author", - "guid": "def", + "guid": "del3", "url": "http://google.de", "pubDate": 2323, "lastModified": 113, @@ -73,11 +73,11 @@ "enclosureLink": "http://google.de/web.webm" }, { - "status": 6, + "status": 0, "body": "this is a body", - "title": "a title3", + "title": "del4", "author": "my author", - "guid": "gih", + "guid": "del4", "url": "http://google.de", "pubDate": 2323, "lastModified": 113, diff --git a/tests/unit/db/MapperFactoryTest.php b/tests/unit/db/MapperFactoryTest.php new file mode 100644 index 000000000..cb84e78f5 --- /dev/null +++ b/tests/unit/db/MapperFactoryTest.php @@ -0,0 +1,50 @@ + + * @author Bernhard Posselt + * @copyright Alessandro Cosentino 2012 + * @copyright Bernhard Posselt 2012, 2014 + */ + +namespace OCA\News\Db; + + +class MapperFactoryTest extends \PHPUnit_Framework_TestCase { + + private $db; + private $settings; + + public function setUp() { + $this->db = $this->getMockBuilder('\OCP\IDb') + ->disableOriginalConstructor() + ->getMock(); + } + + + public function testGetItemMapperSqlite() { + $factory = new MapperFactory('sqlite', $this->db); + + $this->assertTrue($factory->getItemMapper() instanceof ItemMapper); + } + + + public function testGetItemMapperPostgres() { + $factory = new MapperFactory('pgsql', $this->db); + + $this->assertTrue($factory->getItemMapper() instanceof ItemMapper); + } + + + public function testGetItemMapperMysql() { + $factory = new MapperFactory('mysql', $this->db); + + $this->assertTrue($factory->getItemMapper() instanceof \OCA\News\Db\Mysql\ItemMapper); + } + + +} \ No newline at end of file diff --git a/tests/unit/db/mysql/ItemMapperTest.php b/tests/unit/db/mysql/ItemMapperTest.php new file mode 100644 index 000000000..84c9f5dc3 --- /dev/null +++ b/tests/unit/db/mysql/ItemMapperTest.php @@ -0,0 +1,119 @@ + + * @author Bernhard Posselt + * @copyright Alessandro Cosentino 2012 + * @copyright Bernhard Posselt 2012, 2014 + */ + +namespace OCA\News\Db\Mysql; + +use \OCA\News\Db\Item; +use \OCA\News\Db\StatusFlag; + + +class ItemMapperTest extends \Test\AppFramework\Db\MapperTestUtility { + + private $mapper; + private $items; + private $newestItemId; + private $limit; + private $user; + private $offset; + private $updatedSince; + private $status; + + + public function setUp() { + parent::setUp(); + + $this->mapper = new ItemMapper($this->db); + + // create mock items + $item1 = new Item(); + $item2 = new Item(); + + $this->items = [$item1, $item2]; + + $this->userId = 'john'; + $this->id = 3; + $this->folderId = 2; + + $this->row = [['id' => $this->items[0]->getId()]]; + + $this->rows = [ + ['id' => $this->items[0]->getId()], + ['id' => $this->items[1]->getId()] + ]; + + $this->user = 'john'; + $this->limit = 10; + $this->offset = 3; + $this->id = 11; + $this->status = 333; + $this->updatedSince = 323; + $this->newestItemId = 2; + + } + + + public function testDeleteReadOlderThanThresholdDoesNotDeleteBelow(){ + $status = StatusFlag::STARRED | StatusFlag::UNREAD; + $sql = 'SELECT (COUNT(*) - `feeds`.`articles_per_update`) AS `size`' . + ', `feeds`.`id` AS `feed_id`, `feeds`.`articles_per_update` ' . + 'FROM `*PREFIX*news_items` `items` ' . + 'JOIN `*PREFIX*news_feeds` `feeds` ' . + 'ON `feeds`.`id` = `items`.`feed_id` ' . + 'AND NOT ((`items`.`status` & ?) > 0) ' . + 'GROUP BY `feeds`.`id`, `feeds`.`articles_per_update` ' . + 'HAVING COUNT(*) > ?'; + + $threshold = 10; + $rows = [['feed_id' => 30, 'size' => 9]]; + $params = [$status, $threshold]; + + $this->setMapperResult($sql, $params, $rows); + $this->mapper->deleteReadOlderThanThreshold($threshold); + + + } + + + public function testDeleteReadOlderThanThreshold(){ + $threshold = 10; + $status = StatusFlag::STARRED | StatusFlag::UNREAD; + + $sql1 = 'SELECT (COUNT(*) - `feeds`.`articles_per_update`) AS `size`' . + ', `feeds`.`id` AS `feed_id`, `feeds`.`articles_per_update` ' . + 'FROM `*PREFIX*news_items` `items` ' . + 'JOIN `*PREFIX*news_feeds` `feeds` ' . + 'ON `feeds`.`id` = `items`.`feed_id` ' . + 'AND NOT ((`items`.`status` & ?) > 0) ' . + 'GROUP BY `feeds`.`id`, `feeds`.`articles_per_update` ' . + 'HAVING COUNT(*) > ?'; + $params1 = [$status, $threshold]; + + + $row = ['feed_id' => 30, 'size' => 11]; + + $sql2 = 'DELETE FROM `*PREFIX*news_items` ' . + 'WHERE NOT ((`status` & ?) > 0) ' . + 'AND `feed_id` = ? ' . + 'ORDER BY `id` ASC ' . + 'LIMIT ?'; + $params2 = [$status, 30, 1]; + + + $this->setMapperResult($sql1, $params1, [$row]); + $this->setMapperResult($sql2, $params2); + + $this->mapper->deleteReadOlderThanThreshold($threshold); + } + + +} \ No newline at end of file -- cgit v1.2.3