Add a disabled unit test for bug #50784. We need to add support for XSSF Colours to pull details from their parent theme where there is one

git-svn-id: https://svn.apache.org/repos/asf/poi/trunk@1072027 13f79535-47bb-0310-9956-ffa450edef68

2 files changed, 31 insertions, 0 deletions

diff --git a/src/ooxml/testcases/org/apache/poi/xssf/usermodel/TestXSSFBugs.java b/src/ooxml/testcases/org/apache/poi/xssf/usermodel/TestXSSFBugs.java
index abc9fc73bc..9f8909ea61 100644
--- a/src/ooxml/testcases/org/apache/poi/xssf/usermodel/TestXSSFBugs.java
+++ b/src/ooxml/testcases/org/apache/poi/xssf/usermodel/TestXSSFBugs.java
@@ -672,4 +672,35 @@ public final class TestXSSFBugs extends BaseTestBugzillaIssues {
        comment = cellWithoutComment.getCellComment();
        assertEquals(exp, comment.getString().getString());
     }
+    
+    /**
+     * Fonts where their colours come from the theme rather
+     *  then being set explicitly still should allow the
+     *  fetching of the RGB 
+     */
+    public void DISABLEDtest50784() throws Exception {
+       XSSFWorkbook wb = XSSFTestDataSamples.openSampleWorkbook("50784-font_theme_colours.xlsx");
+       XSSFSheet s = wb.getSheetAt(0);
+       XSSFRow r = s.getRow(0);
+       
+       // Column 1 has a font with regular colours
+       XSSFCell cr = r.getCell(1);
+       XSSFFont fr = wb.getFontAt( cr.getCellStyle().getFontIndex() );
+       XSSFColor colr =  fr.getXSSFColor();
+       // No theme, has colours
+       assertEquals(0, colr.getTheme());
+       assertNotNull( colr.getRgb() );
+       
+       // Column 0 has a font with colours from a theme
+       XSSFCell ct = r.getCell(0);
+       XSSFFont ft = wb.getFontAt( ct.getCellStyle().getFontIndex() );
+       XSSFColor colt =  ft.getXSSFColor();
+       // Has a theme, which has the colours on it
+       assertEquals(9, colt.getTheme());
+       XSSFColor themeC = wb.getTheme().getThemeColor(colt.getTheme());
+       assertNotNull( themeC.getRgb() );
+       // TODO Fix it so this works
+       assertNotNull( colt.getRgb() );
+       assertEquals( themeC.getRgb(), colt.getRgb() ); // The same colour
+    }
 }
diff --git a/test-data/spreadsheet/50784-font_theme_colours.xlsx b/test-data/spreadsheet/50784-font_theme_colours.xlsx
new file mode 100644
index 0000000000..72e2dbbf14
--- /dev/null
+++ b/test-data/spreadsheet/50784-font_theme_colours.xlsx