Author: fanningpj
Date: Wed Apr 29 20:55:09 2020
New Revision: 1877176

URL: http://svn.apache.org/viewvc?rev=1877176&view=rev
Log:
[github-176] Remove limit on number of rules in XSSFSheetConditionalFormatting.

Modified:
    
poi/trunk/src/testcases/org/apache/poi/ss/usermodel/BaseTestConditionalFormatting.java

Modified: 
poi/trunk/src/testcases/org/apache/poi/ss/usermodel/BaseTestConditionalFormatting.java
URL: 
http://svn.apache.org/viewvc/poi/trunk/src/testcases/org/apache/poi/ss/usermodel/BaseTestConditionalFormatting.java?rev=1877176&r1=1877175&r2=1877176&view=diff
==============================================================================
--- 
poi/trunk/src/testcases/org/apache/poi/ss/usermodel/BaseTestConditionalFormatting.java
 (original)
+++ 
poi/trunk/src/testcases/org/apache/poi/ss/usermodel/BaseTestConditionalFormatting.java
 Wed Apr 29 20:55:09 2020
@@ -97,14 +97,10 @@ public abstract class BaseTestConditiona
                 assertTrue(e.getMessage().startsWith("cfRules must not be 
empty"));
             }
 
-            try {
-                sheetCF.addConditionalFormatting(
-                        new 
CellRangeAddress[]{CellRangeAddress.valueOf("A1:A3")},
-                        new ConditionalFormattingRule[]{rule1, rule2, rule3, 
rule4});
-                fail("expected exception");
-            } catch (IllegalArgumentException e) {
-                assertTrue(e.getMessage().startsWith("Number of rules must not 
exceed 3"));
-            }
+            //this is now allowed
+            sheetCF.addConditionalFormatting(
+                    new CellRangeAddress[]{CellRangeAddress.valueOf("A1:A3")},
+                    new ConditionalFormattingRule[]{rule1, rule2, rule3, 
rule4});
         }
     }
 



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@poi.apache.org
For additional commands, e-mail: commits-h...@poi.apache.org

Reply via email to