Fix forgotten test, string majority/minority now are valid aggregates

qgis · Mar 30, 2020 · 27d99db · 27d99db
1 parent faaff32
commit 27d99db
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/tests/src/python/test_qgsaggregatecalculator.py b/tests/src/python/test_qgsaggregatecalculator.py
@@ -212,8 +212,6 @@ def testString(self):
                   QgsAggregateCalculator.StDev,
                   QgsAggregateCalculator.StDevSample,
                   QgsAggregateCalculator.Range,
-                  QgsAggregateCalculator.Minority,
-                  QgsAggregateCalculator.Majority,
                   QgsAggregateCalculator.FirstQuartile,
                   QgsAggregateCalculator.ThirdQuartile,
                   QgsAggregateCalculator.InterQuartileRange
@@ -232,6 +230,10 @@ def testString(self):
         self.assertEqual(val, ['', '', 'aaaa', 'aaaa', 'bbbbbbbb', 'cc', 'dddd', 'eeee', 'eeee'])
         val, ok = agg.calculate(QgsAggregateCalculator.StringConcatenate, 'fldstring')
         self.assertEqual(val, 'aaaaaaaabbbbbbbbccddddeeeeeeee')
+        val, ok = agg.calculate(QgsAggregateCalculator.Minority, 'fldstring')
+        self.assertEqual(val, 'bbbbbbbb')
+        val, ok = agg.calculate(QgsAggregateCalculator.Majority, 'fldstring')
+        self.assertEqual(val, '')
 
     def testDateTime(self):
         """ Test calculation of aggregates on date/datetime fields"""