diff options
| author | Nathan Guse <nathaniel.guse@gmail.com> | 2014-02-18 14:25:46 -0600 |
|---|---|---|
| committer | Nathan Guse <nathaniel.guse@gmail.com> | 2014-02-18 14:25:46 -0600 |
| commit | eaf9a41ed8bd016880b11f480f92f1b0cfb32f84 (patch) | |
| tree | 27214d0a52c9a690c2c22dbdfa7c31b75e34c546 /tests/pagination | |
| parent | 7015fc8a00da40212e934ca4a499554e9289488a (diff) | |
| parent | 5e1db72532aa58991a9c1d7ddba6bc5dd45ae93c (diff) | |
| download | forums-eaf9a41ed8bd016880b11f480f92f1b0cfb32f84.tar forums-eaf9a41ed8bd016880b11f480f92f1b0cfb32f84.tar.gz forums-eaf9a41ed8bd016880b11f480f92f1b0cfb32f84.tar.bz2 forums-eaf9a41ed8bd016880b11f480f92f1b0cfb32f84.tar.xz forums-eaf9a41ed8bd016880b11f480f92f1b0cfb32f84.zip | |
Merge pull request #1994 from prototech/ticket/12158
[ticket/12158] The pagination start value should be 0 when there are no items.
Diffstat (limited to 'tests/pagination')
| -rw-r--r-- | tests/pagination/pagination_test.php | 19 |
1 files changed, 17 insertions, 2 deletions
diff --git a/tests/pagination/pagination_test.php b/tests/pagination/pagination_test.php index 4c658cbd56..b7a4f101aa 100644 --- a/tests/pagination/pagination_test.php +++ b/tests/pagination/pagination_test.php @@ -159,23 +159,38 @@ class phpbb_pagination_pagination_test extends phpbb_template_template_test_case { return array( array( + 0, + 0, + 0, + ), + array( -1, + 20, + 0, + ), + array( + 20, + -30, 0, ), array( 0, + 20, 0, ), array( 10, + 20, 10, ), array( 20, + 20, 10, ), array( 30, + 20, 10, ), ); @@ -184,9 +199,9 @@ class phpbb_pagination_pagination_test extends phpbb_template_template_test_case /** * @dataProvider validate_start_data */ - public function test_validate_start($start, $expect) + public function test_validate_start($start, $num_items, $expect) { - $this->assertEquals($expect, $this->pagination->validate_start($start, 10, 20)); + $this->assertEquals($expect, $this->pagination->validate_start($start, 10, $num_items)); } public function reverse_start_data() |