From: Nick Burch Date: Fri, 18 Feb 2011 15:45:59 +0000 (+0000) Subject: Add a disabled unit test for bug #50784. We need to add support for XSSF Colours... X-Git-Tag: REL_3_8_BETA1~16 X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=abf0d84ead6326fba1cae643db29f67ec195141f;p=poi.git 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 --- 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 Binary files /dev/null and b/test-data/spreadsheet/50784-font_theme_colours.xlsx differ