Author: fanningpj
Date: Tue Aug 30 20:31:31 2022
New Revision: 1903781

URL: http://svn.apache.org/viewvc?rev=1903781&view=rev
Log:
broken test

Added:
    
poi/trunk/poi-ooxml/src/test/java/org/apache/poi/ss/tests/formula/functions/TestProperXSSF.java
      - copied, changed from r1903780, 
poi/trunk/poi-ooxml/src/test/java/org/apache/poi/ss/tests/formula/functions/TestProper.java
    poi/trunk/poi/src/test/java/org/apache/poi/ss/formula/TestProperHSSF.java
      - copied, changed from r1903726, 
poi/trunk/poi-ooxml/src/test/java/org/apache/poi/ss/tests/formula/functions/TestProper.java
Removed:
    
poi/trunk/poi-ooxml/src/test/java/org/apache/poi/ss/tests/formula/functions/TestProper.java
Modified:
    poi/trunk/poi/src/main/java/org/apache/poi/ss/formula/ptg/StringPtg.java
    poi/trunk/poi/src/main/java/org/apache/poi/util/StringUtil.java

Copied: 
poi/trunk/poi-ooxml/src/test/java/org/apache/poi/ss/tests/formula/functions/TestProperXSSF.java
 (from r1903780, 
poi/trunk/poi-ooxml/src/test/java/org/apache/poi/ss/tests/formula/functions/TestProper.java)
URL: 
http://svn.apache.org/viewvc/poi/trunk/poi-ooxml/src/test/java/org/apache/poi/ss/tests/formula/functions/TestProperXSSF.java?p2=poi/trunk/poi-ooxml/src/test/java/org/apache/poi/ss/tests/formula/functions/TestProperXSSF.java&p1=poi/trunk/poi-ooxml/src/test/java/org/apache/poi/ss/tests/formula/functions/TestProper.java&r1=1903780&r2=1903781&rev=1903781&view=diff
==============================================================================
--- 
poi/trunk/poi-ooxml/src/test/java/org/apache/poi/ss/tests/formula/functions/TestProper.java
 (original)
+++ 
poi/trunk/poi-ooxml/src/test/java/org/apache/poi/ss/tests/formula/functions/TestProperXSSF.java
 Tue Aug 30 20:31:31 2022
@@ -20,8 +20,6 @@ package org.apache.poi.ss.tests.formula.
 import static org.junit.jupiter.api.Assertions.assertEquals;
 import static org.junit.jupiter.api.Assertions.assertTrue;
 
-import org.apache.poi.hssf.usermodel.HSSFFormulaEvaluator;
-import org.apache.poi.hssf.usermodel.HSSFWorkbook;
 import org.apache.poi.ss.formula.eval.StringEval;
 import org.apache.poi.ss.formula.eval.ValueEval;
 import org.apache.poi.ss.formula.functions.TextFunction;
@@ -35,19 +33,11 @@ import org.apache.poi.xssf.usermodel.XSS
 import org.apache.poi.xssf.usermodel.XSSFWorkbook;
 import org.junit.jupiter.api.Test;
 
-public final class TestProper {
+public final class TestProperXSSF {
     private Cell cell11;
     private FormulaEvaluator evaluator;
 
     @Test
-    void testValidHSSF() {
-        HSSFWorkbook wb = new HSSFWorkbook();
-        evaluator = new HSSFFormulaEvaluator(wb);
-
-        confirm(wb);
-    }
-
-    @Test
     void testValidXSSF() {
         XSSFWorkbook wb = new XSSFWorkbook();
         evaluator = new XSSFFormulaEvaluator(wb);

Modified: 
poi/trunk/poi/src/main/java/org/apache/poi/ss/formula/ptg/StringPtg.java
URL: 
http://svn.apache.org/viewvc/poi/trunk/poi/src/main/java/org/apache/poi/ss/formula/ptg/StringPtg.java?rev=1903781&r1=1903780&r2=1903781&view=diff
==============================================================================
--- poi/trunk/poi/src/main/java/org/apache/poi/ss/formula/ptg/StringPtg.java 
(original)
+++ poi/trunk/poi/src/main/java/org/apache/poi/ss/formula/ptg/StringPtg.java 
Tue Aug 30 20:31:31 2022
@@ -48,7 +48,7 @@ public final class StringPtg extends Sca
         if (_is16bitUnicode) {
             field_3_string = StringUtil.readUnicodeLE(in, nChars);
         } else {
-            field_3_string = StringUtil.readCompressedUnicode(in, nChars);
+            field_3_string = StringUtil.readCompressedLatinA(in, nChars);
         }
     }
 

Modified: poi/trunk/poi/src/main/java/org/apache/poi/util/StringUtil.java
URL: 
http://svn.apache.org/viewvc/poi/trunk/poi/src/main/java/org/apache/poi/util/StringUtil.java?rev=1903781&r1=1903780&r2=1903781&view=diff
==============================================================================
--- poi/trunk/poi/src/main/java/org/apache/poi/util/StringUtil.java (original)
+++ poi/trunk/poi/src/main/java/org/apache/poi/util/StringUtil.java Tue Aug 30 
20:31:31 2022
@@ -138,6 +138,11 @@ public final class StringUtil {
         return new String(string, offset, len_to_use, UTF8);
     }
 
+    /**
+     * @param in stream,
+     * @param nChars number pf chars
+     * @return UTF-8 encoded result
+     */
     public static String readCompressedUnicode(LittleEndianInput in, int 
nChars) {
         byte[] buf = IOUtils.safelyAllocate(nChars, MAX_RECORD_LENGTH);
         in.readFully(buf);
@@ -145,6 +150,17 @@ public final class StringUtil {
     }
 
     /**
+     * @param in stream,
+     * @param nChars number pf chars
+     * @return LATIN-A (ISO-8859-1) encoded result
+     */
+    public static String readCompressedLatinA(LittleEndianInput in, int 
nChars) {
+        byte[] buf = IOUtils.safelyAllocate(nChars, MAX_RECORD_LENGTH);
+        in.readFully(buf);
+        return new String(buf, ISO_8859_1);
+    }
+
+    /**
      * InputStream {@code in} is expected to contain:
      * <ol>
      * <li>ushort nChars</li>

Copied: 
poi/trunk/poi/src/test/java/org/apache/poi/ss/formula/TestProperHSSF.java (from 
r1903726, 
poi/trunk/poi-ooxml/src/test/java/org/apache/poi/ss/tests/formula/functions/TestProper.java)
URL: 
http://svn.apache.org/viewvc/poi/trunk/poi/src/test/java/org/apache/poi/ss/formula/TestProperHSSF.java?p2=poi/trunk/poi/src/test/java/org/apache/poi/ss/formula/TestProperHSSF.java&p1=poi/trunk/poi-ooxml/src/test/java/org/apache/poi/ss/tests/formula/functions/TestProper.java&r1=1903726&r2=1903781&rev=1903781&view=diff
==============================================================================
--- 
poi/trunk/poi-ooxml/src/test/java/org/apache/poi/ss/tests/formula/functions/TestProper.java
 (original)
+++ poi/trunk/poi/src/test/java/org/apache/poi/ss/formula/TestProperHSSF.java 
Tue Aug 30 20:31:31 2022
@@ -15,27 +15,20 @@
    limitations under the License.
 ==================================================================== */
 
-package org.apache.poi.ss.tests.formula.functions;
-
-import static org.junit.jupiter.api.Assertions.assertEquals;
-import static org.junit.jupiter.api.Assertions.assertTrue;
+package org.apache.poi.ss.formula;
 
 import org.apache.poi.hssf.usermodel.HSSFFormulaEvaluator;
 import org.apache.poi.hssf.usermodel.HSSFWorkbook;
 import org.apache.poi.ss.formula.eval.StringEval;
 import org.apache.poi.ss.formula.eval.ValueEval;
 import org.apache.poi.ss.formula.functions.TextFunction;
-import org.apache.poi.ss.usermodel.Cell;
-import org.apache.poi.ss.usermodel.CellType;
-import org.apache.poi.ss.usermodel.CellValue;
-import org.apache.poi.ss.usermodel.FormulaEvaluator;
-import org.apache.poi.ss.usermodel.Sheet;
-import org.apache.poi.ss.usermodel.Workbook;
-import org.apache.poi.xssf.usermodel.XSSFFormulaEvaluator;
-import org.apache.poi.xssf.usermodel.XSSFWorkbook;
+import org.apache.poi.ss.usermodel.*;
 import org.junit.jupiter.api.Test;
 
-public final class TestProper {
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertTrue;
+
+public final class TestProperHSSF {
     private Cell cell11;
     private FormulaEvaluator evaluator;
 
@@ -47,14 +40,6 @@ public final class TestProper {
         confirm(wb);
     }
 
-    @Test
-    void testValidXSSF() {
-        XSSFWorkbook wb = new XSSFWorkbook();
-        evaluator = new XSSFFormulaEvaluator(wb);
-
-        confirm(wb);
-    }
-
     private void confirm(Workbook wb) {
         Sheet sheet = wb.createSheet("new sheet");
         cell11 = sheet.createRow(0).createCell(0);
@@ -69,6 +54,10 @@ public final class TestProper {
         confirm("PROPER(\"/&%\")", "/&%"); //nothing happens with ascii 
punctuation that is not upper or lower case
         confirm("PROPER(\"Apache POI\")", "Apache Poi"); //acronyms are not 
special
         confirm("PROPER(\"  hello world\")", "  Hello World"); //leading 
whitespace is ignored
+        
//https://support.microsoft.com/en-us/office/proper-function-52a5a283-e8b2-49be-8506-b2887b889f94
+        confirm("PROPER(\"this is a TITLE\")", "This Is A Title");
+        confirm("PROPER(\"2-way street\")", "2-Way Street");
+        confirm("PROPER(\"76BudGet\")", "76Budget");
 
         final String scharfes = "\u00df"; //German lowercase eszett, scharfes 
s, sharp s
         confirm("PROPER(\"stra"+scharfes+"e\")", "Stra"+scharfes+"e");



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to