Merge pull request #75 from omise/fix-search-with-perpage-attr

Be able to limit items that will be shown at the Search Object

lib/omise/OmiseSearch.php

@@ -98,6 +98,18 @@ public function page($page)
         return $this->mergeAttributes('page', $page);
     }
 
+    /**
+     * Update `per_page` parameter.
+     *
+     * @param  int $limit   Number of items that will be shown per page.
+     *
+     * @return OmiseSearch  This instance.
+     */
+    public function per_page($limit)
+    {
+        return $this->mergeAttributes('per_page', $limit);
+    }
+
     /**
      * Update `order` parameter.
      *

tests/fixtures/api.omise.co/search-c2NvcGU9Y2hhcmdlJnF1ZXJ5PWRlbW8mcGVyX3BhZ2U9MiZvcmRlcj1yZXZlcnNlX2Nocm9ub2xvZ2ljYWw-get.json

@@ -0,0 +1,16 @@
+{
+  "object": "search",
+  "order": "reverse_chronological",
+  "scope": "charge",
+  "query": "demo",
+  "filters": {
+  },
+  "page": 1,
+  "per_page": 2,
+  "location": "/search",
+  "total_pages": 0,
+  "total": 0,
+  "data": [
+
+  ]
+}

tests/omise/SearchTest.php

@@ -111,4 +111,22 @@ public function testRetrieveSearchObjectBySpecificScopeAndQueryAndFilterAndPageA
         $this->assertEquals('demo', $search['query']);
         $this->assertEquals(array('captured' => 'true'), $search['filters']);
     }
+
+    /**
+     * Assert that items of search object can be shown at a specific amount
+     * given by 'per_page' number.
+     */
+    public function testSetLimit()
+    {
+        $search = OmiseSearch::scope('charge')
+            ->query('demo')
+            ->per_page(2)
+            ->order('reverse_chronological');
+
+        $this->assertArrayHasKey('object', $search);
+        $this->assertEquals('search', $search['object']);
+        $this->assertEquals('charge', $search['scope']);
+        $this->assertEquals('demo', $search['query']);
+        $this->assertEquals(2, $search['per_page']);
+    }
 }