Segment auto-suggest dropdown now behaves as expected for Download URL, outlink URL and Page URL

 + tests

5 files changed, 61 insertions, 60 deletions

diff --git a/plugins/UserCountry/tests/System/AttributeHistoricalDataWithLocationsTest.php b/plugins/UserCountry/tests/System/AttributeHistoricalDataWithLocationsTest.php
index 5fb3bf7c92..9b0429c7c9 100644
--- a/plugins/UserCountry/tests/System/AttributeHistoricalDataWithLocationsTest.php
+++ b/plugins/UserCountry/tests/System/AttributeHistoricalDataWithLocationsTest.php
@@ -23,6 +23,7 @@ use Symfony\Component\Console\Tester\CommandTester;
  * @package Piwik\Plugins\UserCountry\Test\Integration
  *
  * @group UserCountry
+ * @group AttributeHistoricalDataWithLocations
  */
 class AttributeHistoricalDataWithLocationsTest extends IntegrationTestCase
 {
diff --git a/plugins/UserCountry/tests/System/expected/test_AttributeHistoricalDataWithLocationsTest_testExecute_ShouldReturnLogAfterWorkingWithSomeData__UserCountry.getCity_month.xml b/plugins/UserCountry/tests/System/expected/test_AttributeHistoricalDataWithLocationsTest_testExecute_ShouldReturnLogAfterWorkingWithSomeData__UserCountry.getCity_month.xml
index cd97a16b89..1a9caf8a5a 100644
--- a/plugins/UserCountry/tests/System/expected/test_AttributeHistoricalDataWithLocationsTest_testExecute_ShouldReturnLogAfterWorkingWithSomeData__UserCountry.getCity_month.xml
+++ b/plugins/UserCountry/tests/System/expected/test_AttributeHistoricalDataWithLocationsTest_testExecute_ShouldReturnLogAfterWorkingWithSomeData__UserCountry.getCity_month.xml
@@ -3,9 +3,9 @@
 	<row>
 		<label>Unknown</label>
 		<nb_visits>22</nb_visits>
-		<nb_actions>39</nb_actions>
-		<max_actions>3</max_actions>
-		<sum_visit_length>13871</sum_visit_length>
+		<nb_actions>61</nb_actions>
+		<max_actions>5</max_actions>
+		<sum_visit_length>17831</sum_visit_length>
 		<bounce_count>11</bounce_count>
 		<goals>
 			<row idgoal='1'>
@@ -34,9 +34,9 @@
 	<row>
 		<label>Vancouver, British Columbia, Canada</label>
 		<nb_visits>6</nb_visits>
-		<nb_actions>10</nb_actions>
-		<max_actions>3</max_actions>
-		<sum_visit_length>3783</sum_visit_length>
+		<nb_actions>16</nb_actions>
+		<max_actions>5</max_actions>
+		<sum_visit_length>4863</sum_visit_length>
 		<bounce_count>3</bounce_count>
 		<goals>
 			<row idgoal='1'>
@@ -67,9 +67,9 @@
 	<row>
 		<label>Besançon, Franche-Comte, France</label>
 		<nb_visits>3</nb_visits>
-		<nb_actions>5</nb_actions>
-		<max_actions>3</max_actions>
-		<sum_visit_length>1261</sum_visit_length>
+		<nb_actions>7</nb_actions>
+		<max_actions>5</max_actions>
+		<sum_visit_length>1621</sum_visit_length>
 		<bounce_count>2</bounce_count>
 		<goals>
 			<row idgoal='1'>
@@ -100,9 +100,9 @@
 	<row>
 		<label>Lhasa, Unknown, Tibet</label>
 		<nb_visits>2</nb_visits>
-		<nb_actions>3</nb_actions>
-		<max_actions>2</max_actions>
-		<sum_visit_length>1261</sum_visit_length>
+		<nb_actions>5</nb_actions>
+		<max_actions>4</max_actions>
+		<sum_visit_length>1621</sum_visit_length>
 		<bounce_count>1</bounce_count>
 		<goals>
 			<row idgoal='1'>
@@ -133,9 +133,9 @@
 	<row>
 		<label>Rome, Lazio, Italy</label>
 		<nb_visits>2</nb_visits>
-		<nb_actions>4</nb_actions>
-		<max_actions>3</max_actions>
-		<sum_visit_length>1261</sum_visit_length>
+		<nb_actions>6</nb_actions>
+		<max_actions>5</max_actions>
+		<sum_visit_length>1621</sum_visit_length>
 		<bounce_count>1</bounce_count>
 		<goals>
 			<row idgoal='1'>
diff --git a/plugins/UserCountry/tests/System/expected/test_AttributeHistoricalDataWithLocationsTest_testExecute_ShouldReturnLogAfterWorkingWithSomeData__UserCountry.getContinent_month.xml b/plugins/UserCountry/tests/System/expected/test_AttributeHistoricalDataWithLocationsTest_testExecute_ShouldReturnLogAfterWorkingWithSomeData__UserCountry.getContinent_month.xml
index ea3cb0fc36..d410c25319 100644
--- a/plugins/UserCountry/tests/System/expected/test_AttributeHistoricalDataWithLocationsTest_testExecute_ShouldReturnLogAfterWorkingWithSomeData__UserCountry.getContinent_month.xml
+++ b/plugins/UserCountry/tests/System/expected/test_AttributeHistoricalDataWithLocationsTest_testExecute_ShouldReturnLogAfterWorkingWithSomeData__UserCountry.getContinent_month.xml
@@ -3,9 +3,9 @@
 	<row>
 		<label>Asia</label>
 		<nb_visits>22</nb_visits>
-		<nb_actions>38</nb_actions>
-		<max_actions>3</max_actions>
-		<sum_visit_length>13871</sum_visit_length>
+		<nb_actions>60</nb_actions>
+		<max_actions>5</max_actions>
+		<sum_visit_length>17831</sum_visit_length>
 		<bounce_count>11</bounce_count>
 		<goals>
 			<row idgoal='1'>
@@ -28,9 +28,9 @@
 	<row>
 		<label>North America</label>
 		<nb_visits>8</nb_visits>
-		<nb_actions>14</nb_actions>
-		<max_actions>3</max_actions>
-		<sum_visit_length>5044</sum_visit_length>
+		<nb_actions>22</nb_actions>
+		<max_actions>5</max_actions>
+		<sum_visit_length>6484</sum_visit_length>
 		<bounce_count>4</bounce_count>
 		<goals>
 			<row idgoal='1'>
@@ -53,9 +53,9 @@
 	<row>
 		<label>Europe</label>
 		<nb_visits>5</nb_visits>
-		<nb_actions>9</nb_actions>
-		<max_actions>3</max_actions>
-		<sum_visit_length>2522</sum_visit_length>
+		<nb_actions>13</nb_actions>
+		<max_actions>5</max_actions>
+		<sum_visit_length>3242</sum_visit_length>
 		<bounce_count>3</bounce_count>
 		<goals>
 			<row idgoal='1'>
diff --git a/plugins/UserCountry/tests/System/expected/test_AttributeHistoricalDataWithLocationsTest_testExecute_ShouldReturnLogAfterWorkingWithSomeData__UserCountry.getCountry_month.xml b/plugins/UserCountry/tests/System/expected/test_AttributeHistoricalDataWithLocationsTest_testExecute_ShouldReturnLogAfterWorkingWithSomeData__UserCountry.getCountry_month.xml
index f9dc03dca9..faa38d2190 100644
--- a/plugins/UserCountry/tests/System/expected/test_AttributeHistoricalDataWithLocationsTest_testExecute_ShouldReturnLogAfterWorkingWithSomeData__UserCountry.getCountry_month.xml
+++ b/plugins/UserCountry/tests/System/expected/test_AttributeHistoricalDataWithLocationsTest_testExecute_ShouldReturnLogAfterWorkingWithSomeData__UserCountry.getCountry_month.xml
@@ -3,9 +3,9 @@
 	<row>
 		<label>China</label>
 		<nb_visits>18</nb_visits>
-		<nb_actions>32</nb_actions>
-		<max_actions>3</max_actions>
-		<sum_visit_length>11349</sum_visit_length>
+		<nb_actions>50</nb_actions>
+		<max_actions>5</max_actions>
+		<sum_visit_length>14589</sum_visit_length>
 		<bounce_count>9</bounce_count>
 		<goals>
 			<row idgoal='1'>
@@ -32,9 +32,9 @@
 	<row>
 		<label>Canada</label>
 		<nb_visits>6</nb_visits>
-		<nb_actions>10</nb_actions>
-		<max_actions>3</max_actions>
-		<sum_visit_length>3783</sum_visit_length>
+		<nb_actions>16</nb_actions>
+		<max_actions>5</max_actions>
+		<sum_visit_length>4863</sum_visit_length>
 		<bounce_count>3</bounce_count>
 		<goals>
 			<row idgoal='1'>
@@ -61,9 +61,9 @@
 	<row>
 		<label>France</label>
 		<nb_visits>3</nb_visits>
-		<nb_actions>5</nb_actions>
-		<max_actions>3</max_actions>
-		<sum_visit_length>1261</sum_visit_length>
+		<nb_actions>7</nb_actions>
+		<max_actions>5</max_actions>
+		<sum_visit_length>1621</sum_visit_length>
 		<bounce_count>2</bounce_count>
 		<goals>
 			<row idgoal='1'>
@@ -90,9 +90,9 @@
 	<row>
 		<label>Indonesia</label>
 		<nb_visits>2</nb_visits>
-		<nb_actions>3</nb_actions>
-		<max_actions>2</max_actions>
-		<sum_visit_length>1261</sum_visit_length>
+		<nb_actions>5</nb_actions>
+		<max_actions>4</max_actions>
+		<sum_visit_length>1621</sum_visit_length>
 		<bounce_count>1</bounce_count>
 		<goals>
 			<row idgoal='1'>
@@ -119,9 +119,9 @@
 	<row>
 		<label>Italy</label>
 		<nb_visits>2</nb_visits>
-		<nb_actions>4</nb_actions>
-		<max_actions>3</max_actions>
-		<sum_visit_length>1261</sum_visit_length>
+		<nb_actions>6</nb_actions>
+		<max_actions>5</max_actions>
+		<sum_visit_length>1621</sum_visit_length>
 		<bounce_count>1</bounce_count>
 		<goals>
 			<row idgoal='1'>
@@ -148,9 +148,9 @@
 	<row>
 		<label>Tibet</label>
 		<nb_visits>2</nb_visits>
-		<nb_actions>3</nb_actions>
-		<max_actions>2</max_actions>
-		<sum_visit_length>1261</sum_visit_length>
+		<nb_actions>5</nb_actions>
+		<max_actions>4</max_actions>
+		<sum_visit_length>1621</sum_visit_length>
 		<bounce_count>1</bounce_count>
 		<goals>
 			<row idgoal='1'>
@@ -177,9 +177,9 @@
 	<row>
 		<label>United States</label>
 		<nb_visits>2</nb_visits>
-		<nb_actions>4</nb_actions>
-		<max_actions>3</max_actions>
-		<sum_visit_length>1261</sum_visit_length>
+		<nb_actions>6</nb_actions>
+		<max_actions>5</max_actions>
+		<sum_visit_length>1621</sum_visit_length>
 		<bounce_count>1</bounce_count>
 		<goals>
 			<row idgoal='1'>
diff --git a/plugins/UserCountry/tests/System/expected/test_AttributeHistoricalDataWithLocationsTest_testExecute_ShouldReturnLogAfterWorkingWithSomeData__UserCountry.getRegion_month.xml b/plugins/UserCountry/tests/System/expected/test_AttributeHistoricalDataWithLocationsTest_testExecute_ShouldReturnLogAfterWorkingWithSomeData__UserCountry.getRegion_month.xml
index 1885070cfd..107dc47a28 100644
--- a/plugins/UserCountry/tests/System/expected/test_AttributeHistoricalDataWithLocationsTest_testExecute_ShouldReturnLogAfterWorkingWithSomeData__UserCountry.getRegion_month.xml
+++ b/plugins/UserCountry/tests/System/expected/test_AttributeHistoricalDataWithLocationsTest_testExecute_ShouldReturnLogAfterWorkingWithSomeData__UserCountry.getRegion_month.xml
@@ -3,9 +3,9 @@
 	<row>
 		<label>Unknown</label>
 		<nb_visits>22</nb_visits>
-		<nb_actions>39</nb_actions>
-		<max_actions>3</max_actions>
-		<sum_visit_length>13871</sum_visit_length>
+		<nb_actions>61</nb_actions>
+		<max_actions>5</max_actions>
+		<sum_visit_length>17831</sum_visit_length>
 		<bounce_count>11</bounce_count>
 		<goals>
 			<row idgoal='1'>
@@ -32,9 +32,9 @@
 	<row>
 		<label>British Columbia, Canada</label>
 		<nb_visits>6</nb_visits>
-		<nb_actions>10</nb_actions>
-		<max_actions>3</max_actions>
-		<sum_visit_length>3783</sum_visit_length>
+		<nb_actions>16</nb_actions>
+		<max_actions>5</max_actions>
+		<sum_visit_length>4863</sum_visit_length>
 		<bounce_count>3</bounce_count>
 		<goals>
 			<row idgoal='1'>
@@ -62,9 +62,9 @@
 	<row>
 		<label>Franche-Comte, France</label>
 		<nb_visits>3</nb_visits>
-		<nb_actions>5</nb_actions>
-		<max_actions>3</max_actions>
-		<sum_visit_length>1261</sum_visit_length>
+		<nb_actions>7</nb_actions>
+		<max_actions>5</max_actions>
+		<sum_visit_length>1621</sum_visit_length>
 		<bounce_count>2</bounce_count>
 		<goals>
 			<row idgoal='1'>
@@ -92,9 +92,9 @@
 	<row>
 		<label>Lazio, Italy</label>
 		<nb_visits>2</nb_visits>
-		<nb_actions>4</nb_actions>
-		<max_actions>3</max_actions>
-		<sum_visit_length>1261</sum_visit_length>
+		<nb_actions>6</nb_actions>
+		<max_actions>5</max_actions>
+		<sum_visit_length>1621</sum_visit_length>
 		<bounce_count>1</bounce_count>
 		<goals>
 			<row idgoal='1'>
@@ -122,9 +122,9 @@
 	<row>
 		<label>Unknown, Tibet</label>
 		<nb_visits>2</nb_visits>
-		<nb_actions>3</nb_actions>
-		<max_actions>2</max_actions>
-		<sum_visit_length>1261</sum_visit_length>
+		<nb_actions>5</nb_actions>
+		<max_actions>4</max_actions>
+		<sum_visit_length>1621</sum_visit_length>
 		<bounce_count>1</bounce_count>
 		<goals>
 			<row idgoal='1'>