#7437 Prevent percentages to go over 100% for browser plugin support

1 files changed, 25 insertions, 2 deletions

diff --git a/tests/PHPUnit/Unit/DataTable/Filter/RangeCheckTest.php b/tests/PHPUnit/Unit/DataTable/Filter/RangeCheckTest.php
index 0deccd217f..4002424436 100644
--- a/tests/PHPUnit/Unit/DataTable/Filter/RangeCheckTest.php
+++ b/tests/PHPUnit/Unit/DataTable/Filter/RangeCheckTest.php
@@ -11,13 +11,13 @@ namespace Piwik\Tests\Unit\DataTable\Filter;
 use Piwik\DataTable\Filter\RangeCheck;
 use Piwik\DataTable;
 use Piwik\DataTable\Row;
+use Piwik\Plugins\CoreHome\Columns\Metrics\VisitsPercent;
 
 /**
  * @group DataTableTest
  */
 class DataTable_Filter_RangeCheckTest extends \PHPUnit_Framework_TestCase
 {
-
     public function testRangeCheckNormalDataTable()
     {
         $table = new DataTable();
@@ -35,7 +35,6 @@ class DataTable_Filter_RangeCheckTest extends \PHPUnit_Framework_TestCase
         $this->assertEquals($expectedOrder, $table->getColumn('count'));
     }
 
-
     public function testRangeCheckNormalDataTableNonIntegerValues()
     {
         $table = new DataTable();
@@ -53,4 +52,28 @@ class DataTable_Filter_RangeCheckTest extends \PHPUnit_Framework_TestCase
 
         $this->assertEquals($expectedOrder, $table->getColumn('count'));
     }
+
+    public function testRangeCheckOnMetadata()
+    {
+        $table = new DataTable();
+        $table->addRowsFromArray(array(
+            array(
+                Row::COLUMNS  => array('label' => 'foo'),
+                Row::METADATA => array('count' => 5),
+            ),
+            array(
+                Row::COLUMNS  => array('label' => 'bar'),
+                Row::METADATA => array('count' => 10),
+            ),
+            array(
+                Row::COLUMNS  => array('label' => 'bar'),
+                Row::METADATA => array('count' => 15),
+            ),
+        ));
+
+        $filter = new RangeCheck($table, 'count', 0, 10);
+        $filter->filter($table);
+
+        $this->assertEquals(array(5, 10, 10), $table->getRowsMetadata('count'));
+    }
 }