summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorXisco Fauli <xiscofauli@libreoffice.org>2022-11-15 21:04:49 +0100
committerXisco Fauli <xiscofauli@libreoffice.org>2022-11-15 22:04:03 +0100
commitd873aa4ddf617fb7c3440763efa71c97389ce738 (patch)
treeb5c2258703cc43b86b9963ba17e1bca698c427b5
parent3e1890ca6f66ccf675092f2841a1b99da0d85ec2 (diff)
subsequent_filters_test.cxx: use clang-format
Do it before changing the inheritance Change-Id: I03b1749b8c50aaf073cec9e8b25891bd9414b546 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/142749 Tested-by: Jenkins Reviewed-by: Xisco Fauli <xiscofauli@libreoffice.org>
-rw-r--r--sc/qa/unit/subsequent_filters_test.cxx954
-rw-r--r--solenv/clang-format/excludelist1
2 files changed, 529 insertions, 426 deletions
diff --git a/sc/qa/unit/subsequent_filters_test.cxx b/sc/qa/unit/subsequent_filters_test.cxx
index 327c39616b04..8b87ce6acd27 100644
--- a/sc/qa/unit/subsequent_filters_test.cxx
+++ b/sc/qa/unit/subsequent_filters_test.cxx
@@ -209,8 +209,8 @@ public:
CPPUNIT_TEST(testDatabaseRangesXLS);
CPPUNIT_TEST(testDatabaseRangesXLSX);
CPPUNIT_TEST(testFormatsODS);
-// CPPUNIT_TEST(testFormatsXLS); TODO: Fix this
-// CPPUNIT_TEST(testFormatsXLSX); TODO: Fix this
+ // CPPUNIT_TEST(testFormatsXLS); TODO: Fix this
+ // CPPUNIT_TEST(testFormatsXLSX); TODO: Fix this
CPPUNIT_TEST(testMatrixODS);
CPPUNIT_TEST(testMatrixXLS);
CPPUNIT_TEST(testDoubleThinBorder);
@@ -286,38 +286,41 @@ private:
void testPassword_Impl(std::u16string_view rFileNameBase);
};
-namespace {
-
+namespace
+{
void testRangeNameImpl(const ScDocument& rDoc)
{
//check one range data per sheet and one global more detailed
//add some more checks here
ScRangeData* pRangeData = rDoc.GetRangeName()->findByUpperName(OUString("GLOBAL1"));
CPPUNIT_ASSERT_MESSAGE("range name Global1 not found", pRangeData);
- double aValue = rDoc.GetValue(1,0,0);
+ double aValue = rDoc.GetValue(1, 0, 0);
CPPUNIT_ASSERT_EQUAL_MESSAGE("range name Global1 should reference Sheet1.A1", 1.0, aValue);
pRangeData = rDoc.GetRangeName(0)->findByUpperName(OUString("LOCAL1"));
CPPUNIT_ASSERT_MESSAGE("range name Sheet1.Local1 not found", pRangeData);
- aValue = rDoc.GetValue(1,2,0);
- CPPUNIT_ASSERT_EQUAL_MESSAGE("range name Sheet1.Local1 should reference Sheet1.A3", 3.0, aValue);
+ aValue = rDoc.GetValue(1, 2, 0);
+ CPPUNIT_ASSERT_EQUAL_MESSAGE("range name Sheet1.Local1 should reference Sheet1.A3", 3.0,
+ aValue);
pRangeData = rDoc.GetRangeName(1)->findByUpperName(OUString("LOCAL2"));
CPPUNIT_ASSERT_MESSAGE("range name Sheet2.Local2 not found", pRangeData);
- aValue = rDoc.GetValue(1,1,1);
- CPPUNIT_ASSERT_EQUAL_MESSAGE("range name Sheet2.Local2 should reference Sheet2.A2", 7.0, aValue);
+ aValue = rDoc.GetValue(1, 1, 1);
+ CPPUNIT_ASSERT_EQUAL_MESSAGE("range name Sheet2.Local2 should reference Sheet2.A2", 7.0,
+ aValue);
//check for correct results for the remaining formulas
- aValue = rDoc.GetValue(1,1,0);
+ aValue = rDoc.GetValue(1, 1, 0);
CPPUNIT_ASSERT_EQUAL_MESSAGE("=global2 should be 2", 2.0, aValue);
- aValue = rDoc.GetValue(1,3,0);
+ aValue = rDoc.GetValue(1, 3, 0);
CPPUNIT_ASSERT_EQUAL_MESSAGE("=local2 should be 4", 4.0, aValue);
- aValue = rDoc.GetValue(2,0,0);
+ aValue = rDoc.GetValue(2, 0, 0);
CPPUNIT_ASSERT_EQUAL_MESSAGE("=SUM(global3) should be 10", 10.0, aValue);
- aValue = rDoc.GetValue(1,0,1);
- CPPUNIT_ASSERT_EQUAL_MESSAGE("range name Sheet2.local1 should reference Sheet1.A5", 5.0, aValue);
+ aValue = rDoc.GetValue(1, 0, 1);
+ CPPUNIT_ASSERT_EQUAL_MESSAGE("range name Sheet2.local1 should reference Sheet1.A5", 5.0,
+ aValue);
// Test if Global5 ( which depends on Global6 ) is evaluated
- aValue = rDoc.GetValue(0,5,1);
- CPPUNIT_ASSERT_EQUAL_MESSAGE("formula Global5 should reference Global6 ( which is evaluated as local1 )", 5.0, aValue);
+ aValue = rDoc.GetValue(0, 5, 1);
+ CPPUNIT_ASSERT_EQUAL_MESSAGE(
+ "formula Global5 should reference Global6 ( which is evaluated as local1 )", 5.0, aValue);
}
-
}
void ScFiltersTest::testCondFormatOperatorsSameRangeXLSX()
@@ -364,23 +367,25 @@ void ScFiltersTest::testTdf119292()
// check the points of the polygon if the text is rotated 90 degrees
tools::Rectangle aMarkRect1(0, 0, 45, 3);
- tools::Polygon aPoly90degrees = SvxFont::DrawArrow(*pDev, aMarkRect1, aMarkSize, aArrowFillCol, true, true);
+ tools::Polygon aPoly90degrees
+ = SvxFont::DrawArrow(*pDev, aMarkRect1, aMarkSize, aArrowFillCol, true, true);
Point aPoly90Pos1 = aPoly90degrees.GetPoint(0);
Point aPoly90Pos2 = aPoly90degrees.GetPoint(1);
Point aPoly90Pos3 = aPoly90degrees.GetPoint(2);
- CPPUNIT_ASSERT_EQUAL(Point(19,3),aPoly90Pos1);
- CPPUNIT_ASSERT_EQUAL(Point(22,0),aPoly90Pos2);
- CPPUNIT_ASSERT_EQUAL(Point(25,3),aPoly90Pos3);
+ CPPUNIT_ASSERT_EQUAL(Point(19, 3), aPoly90Pos1);
+ CPPUNIT_ASSERT_EQUAL(Point(22, 0), aPoly90Pos2);
+ CPPUNIT_ASSERT_EQUAL(Point(25, 3), aPoly90Pos3);
// check the points of the polygon if the text is rotated 270 degrees
tools::Rectangle aMarkRect2(89, 62, 134, 57);
- tools::Polygon aPoly270degrees = SvxFont::DrawArrow(*pDev, aMarkRect2, aMarkSize, aArrowFillCol, false, true);
+ tools::Polygon aPoly270degrees
+ = SvxFont::DrawArrow(*pDev, aMarkRect2, aMarkSize, aArrowFillCol, false, true);
Point aPoly270Pos1 = aPoly270degrees.GetPoint(0);
Point aPoly270Pos2 = aPoly270degrees.GetPoint(1);
Point aPoly270Pos3 = aPoly270degrees.GetPoint(2);
- CPPUNIT_ASSERT_EQUAL(Point(108,54),aPoly270Pos1);
- CPPUNIT_ASSERT_EQUAL(Point(111,57),aPoly270Pos2);
- CPPUNIT_ASSERT_EQUAL(Point(114,54),aPoly270Pos3);
+ CPPUNIT_ASSERT_EQUAL(Point(108, 54), aPoly270Pos1);
+ CPPUNIT_ASSERT_EQUAL(Point(111, 57), aPoly270Pos2);
+ CPPUNIT_ASSERT_EQUAL(Point(114, 54), aPoly270Pos3);
xDocSh->DoClose();
}
@@ -532,7 +537,7 @@ void ScFiltersTest::testUpdateCircleInMergedCellODS()
// There should be four circle objects!
CPPUNIT_ASSERT_EQUAL(static_cast<size_t>(4), pPage->GetObjCount());
- ScCellMergeOption aCellMergeOption(0,0,1,1); // A1:B2
+ ScCellMergeOption aCellMergeOption(0, 0, 1, 1); // A1:B2
aCellMergeOption.maTabs.insert(0);
xDocSh->GetDocFunc().MergeCells(aCellMergeOption, false, true, true, false);
@@ -594,9 +599,8 @@ void ScFiltersTest::testBasicCellContentODS()
// Numeric value of 0.
ScRefCellValue aCell;
- aCell.assign(rDoc, ScAddress(1,4,0)); // B5
- CPPUNIT_ASSERT_EQUAL_MESSAGE(
- "This cell must be numeric.", CELLTYPE_VALUE, aCell.getType());
+ aCell.assign(rDoc, ScAddress(1, 4, 0)); // B5
+ CPPUNIT_ASSERT_EQUAL_MESSAGE("This cell must be numeric.", CELLTYPE_VALUE, aCell.getType());
CPPUNIT_ASSERT_EQUAL(0.0, aCell.getDouble());
xDocSh->DoClose();
@@ -637,7 +641,9 @@ void ScFiltersTest::testTdf143809()
aFormula = rDoc.GetFormula(0, 1, 0);
CPPUNIT_ASSERT_EQUAL(
- OUString("=SUMPRODUCT(IFERROR(CEILING(SUMIFS(DURATIONS,IDS,IDS),300)/COUNTIFS(IDS,IDS),0))"), aFormula);
+ OUString(
+ "=SUMPRODUCT(IFERROR(CEILING(SUMIFS(DURATIONS,IDS,IDS),300)/COUNTIFS(IDS,IDS),0))"),
+ aFormula);
CPPUNIT_ASSERT_EQUAL(OUString("51900"), rDoc.GetString(0, 1, 0));
xDocSh->DoClose();
@@ -670,9 +676,9 @@ void ScFiltersTest::testRangeNameXLS()
testRangeNameImpl(rDoc);
OUString aCSVPath;
- createCSVPath( "rangeExp_Sheet2.", aCSVPath );
+ createCSVPath("rangeExp_Sheet2.", aCSVPath);
// fdo#44587
- testFile( aCSVPath, rDoc, 1);
+ testFile(aCSVPath, rDoc, 1);
xDocSh->DoClose();
}
@@ -713,9 +719,9 @@ void ScFiltersTest::testHyperlinksXLSX()
ScDocShellRef xDocSh = loadDoc(u"hyperlinks.", FORMAT_XLSX);
ScDocument& rDoc = xDocSh->GetDocument();
- CPPUNIT_ASSERT_EQUAL(OUString("10:ABC10"), rDoc.GetString(ScAddress(0,1,0)));
- CPPUNIT_ASSERT_EQUAL(OUString("10:ABC11"), rDoc.GetString(ScAddress(0,2,0)));
- CPPUNIT_ASSERT_EQUAL(OUString("10:ABC12"), rDoc.GetString(ScAddress(0,3,0)));
+ CPPUNIT_ASSERT_EQUAL(OUString("10:ABC10"), rDoc.GetString(ScAddress(0, 1, 0)));
+ CPPUNIT_ASSERT_EQUAL(OUString("10:ABC11"), rDoc.GetString(ScAddress(0, 2, 0)));
+ CPPUNIT_ASSERT_EQUAL(OUString("10:ABC12"), rDoc.GetString(ScAddress(0, 3, 0)));
xDocSh->DoClose();
}
@@ -854,29 +860,30 @@ void ScFiltersTest::testCachedFormulaResultsODS()
//regardless of cached results. if you update the ods file, you must
//update the values here.
//if NOW() is recalculated, then it should never equal "01/25/13 01:06 PM"
- OUString sTodayRecalc(rDoc.GetString(0,0,1));
+ OUString sTodayRecalc(rDoc.GetString(0, 0, 1));
CPPUNIT_ASSERT("01/25/13 01:06 PM" != sTodayRecalc);
- OUString sTodayRecalcRef(rDoc.GetString(1,0,1));
+ OUString sTodayRecalcRef(rDoc.GetString(1, 0, 1));
CPPUNIT_ASSERT_EQUAL(sTodayRecalc, sTodayRecalcRef);
// make sure that error values are not being treated as string values
- for(SCCOL nCol = 0; nCol < 4; ++nCol)
+ for (SCCOL nCol = 0; nCol < 4; ++nCol)
{
- for(SCROW nRow = 0; nRow < 2; ++nRow)
+ for (SCROW nRow = 0; nRow < 2; ++nRow)
{
- OUString aFormula = "=ISERROR(" +
- OUStringChar(static_cast<char>('A'+nCol)) + OUString::number(nRow) +
- ")";
+ OUString aFormula = "=ISERROR(" + OUStringChar(static_cast<char>('A' + nCol))
+ + OUString::number(nRow) + ")";
rDoc.SetString(nCol, nRow + 2, 2, aFormula);
- CPPUNIT_ASSERT_EQUAL_MESSAGE(OUStringToOString(aFormula, RTL_TEXTENCODING_UTF8).getStr(), OUString("TRUE"), rDoc.GetString(nCol, nRow +2, 2));
+ CPPUNIT_ASSERT_EQUAL_MESSAGE(
+ OUStringToOString(aFormula, RTL_TEXTENCODING_UTF8).getStr(), OUString("TRUE"),
+ rDoc.GetString(nCol, nRow + 2, 2));
- OUString aIsTextFormula = "=ISTEXT(" +
- OUString::number(static_cast<char>('A'+nCol))+ OUString::number(nRow) +
- ")";
+ OUString aIsTextFormula = "=ISTEXT("
+ + OUString::number(static_cast<char>('A' + nCol))
+ + OUString::number(nRow) + ")";
rDoc.SetString(nCol, nRow + 4, 2, aIsTextFormula);
- CPPUNIT_ASSERT_EQUAL(OUString("FALSE"), rDoc.GetString(nCol, nRow +4, 2));
+ CPPUNIT_ASSERT_EQUAL(OUString("FALSE"), rDoc.GetString(nCol, nRow + 4, 2));
}
}
@@ -906,12 +913,12 @@ void ScFiltersTest::testCachedMatrixFormulaResultsODS()
//Import works around this by setting these cells as text cells so that
//the blank text is used for display instead of the number 0.
//If this is working properly, the following cell should NOT have value data.
- CPPUNIT_ASSERT_EQUAL(OUString(), rDoc.GetString(3,0,2));
+ CPPUNIT_ASSERT_EQUAL(OUString(), rDoc.GetString(3, 0, 2));
// fdo#59293 with cached value import error formulas require special
// treatment
rDoc.SetString(2, 5, 2, "=ISERROR(A6)");
- double nVal = rDoc.GetValue(2,5,2);
+ double nVal = rDoc.GetValue(2, 5, 2);
CPPUNIT_ASSERT_EQUAL(1.0, nVal);
xDocSh->DoClose();
@@ -925,17 +932,20 @@ void ScFiltersTest::testFormulaDepAcrossSheetsODS()
sc::AutoCalcSwitch aACSwitch(rDoc, true); // Make sure auto calc is turned on.
// Save the original values of A4:C4.
- double fA4 = rDoc.GetValue(ScAddress(0,3,2));
- double fB4 = rDoc.GetValue(ScAddress(1,3,2));
- double fC4 = rDoc.GetValue(ScAddress(2,3,2));
+ double fA4 = rDoc.GetValue(ScAddress(0, 3, 2));
+ double fB4 = rDoc.GetValue(ScAddress(1, 3, 2));
+ double fC4 = rDoc.GetValue(ScAddress(2, 3, 2));
// Change the value of D4. This should trigger A4:C4 to be recalculated.
- double fD4 = rDoc.GetValue(ScAddress(3,3,2));
- rDoc.SetValue(ScAddress(3,3,2), fD4+1.0);
+ double fD4 = rDoc.GetValue(ScAddress(3, 3, 2));
+ rDoc.SetValue(ScAddress(3, 3, 2), fD4 + 1.0);
- CPPUNIT_ASSERT_MESSAGE("The value must differ from the original.", fA4 != rDoc.GetValue(ScAddress(0,3,2)));
- CPPUNIT_ASSERT_MESSAGE("The value must differ from the original.", fB4 != rDoc.GetValue(ScAddress(1,3,2)));
- CPPUNIT_ASSERT_MESSAGE("The value must differ from the original.", fC4 != rDoc.GetValue(ScAddress(2,3,2)));
+ CPPUNIT_ASSERT_MESSAGE("The value must differ from the original.",
+ fA4 != rDoc.GetValue(ScAddress(0, 3, 2)));
+ CPPUNIT_ASSERT_MESSAGE("The value must differ from the original.",
+ fB4 != rDoc.GetValue(ScAddress(1, 3, 2)));
+ CPPUNIT_ASSERT_MESSAGE("The value must differ from the original.",
+ fC4 != rDoc.GetValue(ScAddress(2, 3, 2)));
xDocSh->DoClose();
}
@@ -948,33 +958,33 @@ void ScFiltersTest::testFormulaDepDeleteContentsODS()
sc::UndoSwitch aUndoSwitch(rDoc, true); // Enable undo.
sc::AutoCalcSwitch aACSwitch(rDoc, true); // Make sure auto calc is turned on.
- CPPUNIT_ASSERT_EQUAL(195.0, rDoc.GetValue(ScAddress(3,15,0))); // formula in D16
+ CPPUNIT_ASSERT_EQUAL(195.0, rDoc.GetValue(ScAddress(3, 15, 0))); // formula in D16
// Delete D2:D5.
ScDocFunc& rFunc = xDocSh->GetDocFunc();
- ScRange aRange(3,1,0,3,4,0);
+ ScRange aRange(3, 1, 0, 3, 4, 0);
ScMarkData aMark(rDoc.GetSheetLimits());
aMark.SetMarkArea(aRange);
aMark.MarkToMulti();
bool bGood = rFunc.DeleteContents(aMark, InsertDeleteFlags::ALL, true, true);
CPPUNIT_ASSERT(bGood);
- CPPUNIT_ASSERT_EQUAL(0.0, rDoc.GetValue(ScAddress(3,1,0)));
- CPPUNIT_ASSERT_EQUAL(0.0, rDoc.GetValue(ScAddress(3,2,0)));
- CPPUNIT_ASSERT_EQUAL(0.0, rDoc.GetValue(ScAddress(3,3,0)));
- CPPUNIT_ASSERT_EQUAL(0.0, rDoc.GetValue(ScAddress(3,4,0)));
+ CPPUNIT_ASSERT_EQUAL(0.0, rDoc.GetValue(ScAddress(3, 1, 0)));
+ CPPUNIT_ASSERT_EQUAL(0.0, rDoc.GetValue(ScAddress(3, 2, 0)));
+ CPPUNIT_ASSERT_EQUAL(0.0, rDoc.GetValue(ScAddress(3, 3, 0)));
+ CPPUNIT_ASSERT_EQUAL(0.0, rDoc.GetValue(ScAddress(3, 4, 0)));
- CPPUNIT_ASSERT_EQUAL(94.0, rDoc.GetValue(ScAddress(3,15,0))); // formula in D16
+ CPPUNIT_ASSERT_EQUAL(94.0, rDoc.GetValue(ScAddress(3, 15, 0))); // formula in D16
SfxUndoManager* pUndoMgr = rDoc.GetUndoManager();
CPPUNIT_ASSERT(pUndoMgr);
pUndoMgr->Undo();
- CPPUNIT_ASSERT_EQUAL(195.0, rDoc.GetValue(ScAddress(3,15,0))); // formula in D16
+ CPPUNIT_ASSERT_EQUAL(195.0, rDoc.GetValue(ScAddress(3, 15, 0))); // formula in D16
xDocSh->DoClose();
}
-namespace {
-
+namespace
+{
void testDBRanges_Impl(ScDocument& rDoc, sal_Int32 nFormat)
{
ScDBCollection* pDBCollection = rDoc.GetDBCollection();
@@ -1007,17 +1017,16 @@ void testDBRanges_Impl(ScDocument& rDoc, sal_Int32 nFormat)
CPPUNIT_ASSERT_EQUAL_MESSAGE("Sheet1: row 6-end should be visible", rDoc.MaxRow(), nRow2);
if (nFormat == FORMAT_ODS) //excel doesn't support named db ranges
{
- double aValue = rDoc.GetValue(0,10,1);
+ double aValue = rDoc.GetValue(0, 10, 1);
CPPUNIT_ASSERT_EQUAL_MESSAGE("Sheet2: A11: formula result is incorrect", 4.0, aValue);
aValue = rDoc.GetValue(1, 10, 1);
CPPUNIT_ASSERT_EQUAL_MESSAGE("Sheet2: B11: formula result is incorrect", 2.0, aValue);
}
- double aValue = rDoc.GetValue(3,10,1);
+ double aValue = rDoc.GetValue(3, 10, 1);
CPPUNIT_ASSERT_EQUAL_MESSAGE("Sheet2: D11: formula result is incorrect", 4.0, aValue);
aValue = rDoc.GetValue(4, 10, 1);
CPPUNIT_ASSERT_EQUAL_MESSAGE("Sheet2: E11: formula result is incorrect", 2.0, aValue);
}
-
}
void ScFiltersTest::testDatabaseRangesODS()
@@ -1116,7 +1125,7 @@ void ScFiltersTest::testMatrixXLS()
void ScFiltersTest::testDoubleThinBorder()
{
-// double-thin borders created with size less than 1.15 where invisible (and subsequently lost) on round-trips.
+ // double-thin borders created with size less than 1.15 where invisible (and subsequently lost) on round-trips.
ScDocShellRef xDocSh = loadDoc(u"tdf88827_borderDoubleThin.", FORMAT_ODS);
ScDocument& rDoc = xDocSh->GetDocument();
@@ -1126,10 +1135,10 @@ void ScFiltersTest::testDoubleThinBorder()
const editeng::SvxBorderLine* pRight = nullptr;
const editeng::SvxBorderLine* pBottom = nullptr;
- rDoc.GetBorderLines( 2, 2, 0, &pLeft, &pTop, &pRight, &pBottom );
+ rDoc.GetBorderLines(2, 2, 0, &pLeft, &pTop, &pRight, &pBottom);
CPPUNIT_ASSERT(pTop);
CPPUNIT_ASSERT(pRight);
- CPPUNIT_ASSERT_EQUAL( SvxBorderLineStyle::DOUBLE_THIN, pRight->GetBorderLineStyle() );
+ CPPUNIT_ASSERT_EQUAL(SvxBorderLineStyle::DOUBLE_THIN, pRight->GetBorderLineStyle());
xDocSh->DoClose();
}
@@ -1144,14 +1153,14 @@ void ScFiltersTest::testBorderODS()
const editeng::SvxBorderLine* pRight = nullptr;
const editeng::SvxBorderLine* pBottom = nullptr;
- rDoc.GetBorderLines( 0, 1, 0, &pLeft, &pTop, &pRight, &pBottom );
+ rDoc.GetBorderLines(0, 1, 0, &pLeft, &pTop, &pRight, &pBottom);
CPPUNIT_ASSERT(!pLeft);
CPPUNIT_ASSERT(!pTop);
CPPUNIT_ASSERT(!pBottom);
CPPUNIT_ASSERT(pRight);
CPPUNIT_ASSERT_EQUAL(SvxBorderLineStyle::SOLID, pRight->GetBorderLineStyle());
- rDoc.GetBorderLines( 2, 1, 0, &pLeft, &pTop, &pRight, &pBottom );
+ rDoc.GetBorderLines(2, 1, 0, &pLeft, &pTop, &pRight, &pBottom);
CPPUNIT_ASSERT(!pLeft);
CPPUNIT_ASSERT(!pTop);
CPPUNIT_ASSERT(!pBottom);
@@ -1160,7 +1169,7 @@ void ScFiltersTest::testBorderODS()
CPPUNIT_ASSERT_EQUAL(SvxBorderLineStyle::SOLID, pRight->GetBorderLineStyle());
CPPUNIT_ASSERT_EQUAL(tools::Long(20), pRight->GetWidth());
- rDoc.GetBorderLines( 2, 8, 0, &pLeft, &pTop, &pRight, &pBottom );
+ rDoc.GetBorderLines(2, 8, 0, &pLeft, &pTop, &pRight, &pBottom);
CPPUNIT_ASSERT(pLeft);
CPPUNIT_ASSERT(pTop);
@@ -1173,8 +1182,8 @@ void ScFiltersTest::testBorderODS()
xDocSh->DoClose();
}
-namespace {
-
+namespace
+{
struct Border
{
sal_Int16 column;
@@ -1200,15 +1209,34 @@ struct Border
SvxBorderLineStyle rStyle;
SvxBorderLineStyle bStyle;
// that's a monster
- Border(sal_Int16 col, sal_Int32 r, sal_Int32 lW, sal_Int32 tW, sal_Int32 rW, sal_Int32 bW, sal_uInt16 lOutW, sal_uInt16 lInW,
- sal_uInt16 lDist, sal_uInt16 tOutW, sal_uInt16 tInW, sal_uInt16 tDist, sal_uInt16 rOutW, sal_uInt16 rInW, sal_uInt16 rDist,
- sal_uInt16 bOutW, sal_uInt16 bInW, sal_uInt16 bDist, sal_Int16 lSt, sal_Int16 tSt, sal_Int16 rSt, sal_Int16 bSt):
- column(col), row(r), leftWidth(lW), topWidth(tW), rightWidth(rW), bottomWidth(bW), lOutWidth(lOutW), lInWidth(lInW), lDistance(lDist),
- tOutWidth(tOutW), tInWidth(tInW), tDistance(tDist), rOutWidth(rOutW), rInWidth(rInW), rDistance(rDist), bOutWidth(bOutW), bInWidth(bInW),
- bDistance(bDist),
- lStyle(static_cast<SvxBorderLineStyle>(lSt)), tStyle(static_cast<SvxBorderLineStyle>(tSt)), rStyle(static_cast<SvxBorderLineStyle>(rSt)), bStyle(static_cast<SvxBorderLineStyle>(bSt)) {};
+ Border(sal_Int16 col, sal_Int32 r, sal_Int32 lW, sal_Int32 tW, sal_Int32 rW, sal_Int32 bW,
+ sal_uInt16 lOutW, sal_uInt16 lInW, sal_uInt16 lDist, sal_uInt16 tOutW, sal_uInt16 tInW,
+ sal_uInt16 tDist, sal_uInt16 rOutW, sal_uInt16 rInW, sal_uInt16 rDist, sal_uInt16 bOutW,
+ sal_uInt16 bInW, sal_uInt16 bDist, sal_Int16 lSt, sal_Int16 tSt, sal_Int16 rSt,
+ sal_Int16 bSt)
+ : column(col)
+ , row(r)
+ , leftWidth(lW)
+ , topWidth(tW)
+ , rightWidth(rW)
+ , bottomWidth(bW)
+ , lOutWidth(lOutW)
+ , lInWidth(lInW)
+ , lDistance(lDist)
+ , tOutWidth(tOutW)
+ , tInWidth(tInW)
+ , tDistance(tDist)
+ , rOutWidth(rOutW)
+ , rInWidth(rInW)
+ , rDistance(rDist)
+ , bOutWidth(bOutW)
+ , bInWidth(bInW)
+ , bDistance(bDist)
+ , lStyle(static_cast<SvxBorderLineStyle>(lSt))
+ , tStyle(static_cast<SvxBorderLineStyle>(tSt))
+ , rStyle(static_cast<SvxBorderLineStyle>(rSt))
+ , bStyle(static_cast<SvxBorderLineStyle>(bSt)){};
};
-
}
void ScFiltersTest::testBordersOoo33()
@@ -1216,21 +1244,31 @@ void ScFiltersTest::testBordersOoo33()
std::vector<Border> borders;
borders.emplace_back(1, 1, 22, 22, 22, 22, 1, 1, 20, 1, 1, 20, 1, 1, 20, 1, 1, 20, 3, 3, 3, 3);
borders.emplace_back(1, 3, 52, 52, 52, 52, 1, 1, 50, 1, 1, 50, 1, 1, 50, 1, 1, 50, 3, 3, 3, 3);
- borders.emplace_back(1, 5, 60, 60, 60, 60, 20, 20, 20, 20, 20, 20, 20, 20, 20, 20, 20, 20, 3, 3, 3, 3);
- borders.emplace_back(1, 7, 150, 150, 150, 150, 50, 50, 50, 50, 50, 50, 50, 50, 50, 50, 50, 50, 3, 3, 3, 3);
- borders.emplace_back(1, 9, 71, 71, 71, 71, 20, 1, 50, 20, 1, 50, 20, 1, 50, 20, 1, 50, 3, 3, 3, 3);
- borders.emplace_back(1, 11, 101, 101, 101, 101, 50, 1, 50, 50, 1, 50, 50, 1, 50, 50, 1, 50, 3, 3, 3, 3);
- borders.emplace_back(1, 13, 131, 131, 131, 131, 80, 1, 50, 80, 1, 50, 80, 1, 50, 80, 1, 50, 3, 3, 3, 3);
- borders.emplace_back(1, 15, 120, 120, 120, 120, 50, 20, 50, 50, 20, 50, 50, 20, 50, 50, 20, 50, 3, 3, 3, 3);
- borders.emplace_back(1, 17, 90, 90, 90, 90, 20, 50, 20, 20, 50, 20, 20, 50, 20, 20, 50, 20, 3, 3, 3, 3);
- borders.emplace_back(1, 19, 180, 180, 180, 180, 80, 50, 50, 80, 50, 50, 80, 50, 50, 80, 50, 50, 3, 3, 3, 3);
- borders.emplace_back(1, 21, 180, 180, 180, 180, 50, 80, 50, 50, 80, 50, 50, 80, 50, 50, 80, 50, 3, 3, 3, 3);
+ borders.emplace_back(1, 5, 60, 60, 60, 60, 20, 20, 20, 20, 20, 20, 20, 20, 20, 20, 20, 20, 3, 3,
+ 3, 3);
+ borders.emplace_back(1, 7, 150, 150, 150, 150, 50, 50, 50, 50, 50, 50, 50, 50, 50, 50, 50, 50,
+ 3, 3, 3, 3);
+ borders.emplace_back(1, 9, 71, 71, 71, 71, 20, 1, 50, 20, 1, 50, 20, 1, 50, 20, 1, 50, 3, 3, 3,
+ 3);
+ borders.emplace_back(1, 11, 101, 101, 101, 101, 50, 1, 50, 50, 1, 50, 50, 1, 50, 50, 1, 50, 3,
+ 3, 3, 3);
+ borders.emplace_back(1, 13, 131, 131, 131, 131, 80, 1, 50, 80, 1, 50, 80, 1, 50, 80, 1, 50, 3,
+ 3, 3, 3);
+ borders.emplace_back(1, 15, 120, 120, 120, 120, 50, 20, 50, 50, 20, 50, 50, 20, 50, 50, 20, 50,
+ 3, 3, 3, 3);
+ borders.emplace_back(1, 17, 90, 90, 90, 90, 20, 50, 20, 20, 50, 20, 20, 50, 20, 20, 50, 20, 3,
+ 3, 3, 3);
+ borders.emplace_back(1, 19, 180, 180, 180, 180, 80, 50, 50, 80, 50, 50, 80, 50, 50, 80, 50, 50,
+ 3, 3, 3, 3);
+ borders.emplace_back(1, 21, 180, 180, 180, 180, 50, 80, 50, 50, 80, 50, 50, 80, 50, 50, 80, 50,
+ 3, 3, 3, 3);
borders.emplace_back(4, 1, 1, 1, 1, 1, 1, 0, 0, 1, 0, 0, 1, 0, 0, 1, 0, 0, 0, 0, 0, 0);
borders.emplace_back(4, 3, 10, 10, 10, 10, 10, 0, 0, 10, 0, 0, 10, 0, 0, 10, 0, 0, 0, 0, 0, 0);
borders.emplace_back(4, 5, 20, 20, 20, 20, 20, 0, 0, 20, 0, 0, 20, 0, 0, 20, 0, 0, 0, 0, 0, 0);
borders.emplace_back(4, 7, 50, 50, 50, 50, 50, 0, 0, 50, 0, 0, 50, 0, 0, 50, 0, 0, 0, 0, 0, 0);
borders.emplace_back(4, 9, 80, 80, 80, 80, 80, 0, 0, 80, 0, 0, 80, 0, 0, 80, 0, 0, 0, 0, 0, 0);
- borders.emplace_back(4, 11, 100, 100, 100, 100, 100, 0, 0, 100, 0, 0, 100, 0, 0, 100, 0, 0, 0, 0, 0, 0);
+ borders.emplace_back(4, 11, 100, 100, 100, 100, 100, 0, 0, 100, 0, 0, 100, 0, 0, 100, 0, 0, 0,
+ 0, 0, 0);
ScDocShellRef xDocSh = loadDoc(u"borders_ooo33.", FORMAT_ODS);
@@ -1241,12 +1279,12 @@ void ScFiltersTest::testBordersOoo33()
const editeng::SvxBorderLine* pRight = nullptr;
const editeng::SvxBorderLine* pBottom = nullptr;
sal_Int16 temp = 0;
- for(sal_Int16 i = 0; i<6; ++i)
+ for (sal_Int16 i = 0; i < 6; ++i)
{
- for(sal_Int32 j = 0; j<22; ++j)
+ for (sal_Int32 j = 0; j < 22; ++j)
{
- rDoc.GetBorderLines( i, j, 0, &pLeft, &pTop, &pRight, &pBottom );
- if(pLeft!=nullptr && pTop!=nullptr && pRight!=nullptr && pBottom!=nullptr)
+ rDoc.GetBorderLines(i, j, 0, &pLeft, &pTop, &pRight, &pBottom);
+ if (pLeft != nullptr && pTop != nullptr && pRight != nullptr && pBottom != nullptr)
{
CPPUNIT_ASSERT_EQUAL(borders[temp].column, i);
CPPUNIT_ASSERT_EQUAL(borders[temp].row, j);
@@ -1333,23 +1371,22 @@ void ScFiltersTest::testBugFixesXLSX()
xDocSh->DoClose();
}
-namespace {
-
-void checkMergedCells( ScDocument& rDoc, const ScAddress& rStartAddress,
- const ScAddress& rExpectedEndAddress )
+namespace
+{
+void checkMergedCells(ScDocument& rDoc, const ScAddress& rStartAddress,
+ const ScAddress& rExpectedEndAddress)
{
SCCOL nActualEndCol = rStartAddress.Col();
SCROW nActualEndRow = rStartAddress.Row();
- rDoc.ExtendMerge( rStartAddress.Col(), rStartAddress.Row(),
- nActualEndCol, nActualEndRow, rStartAddress.Tab() );
- OString sTab = OString::number( rStartAddress.Tab() + 1 );
+ rDoc.ExtendMerge(rStartAddress.Col(), rStartAddress.Row(), nActualEndCol, nActualEndRow,
+ rStartAddress.Tab());
+ OString sTab = OString::number(rStartAddress.Tab() + 1);
OString msg = "Merged cells are not correctly imported on sheet" + sTab;
OString msgCol = msg + "; end col";
OString msgRow = msg + "; end row";
- CPPUNIT_ASSERT_EQUAL_MESSAGE( msgCol.pData->buffer, rExpectedEndAddress.Col(), nActualEndCol );
- CPPUNIT_ASSERT_EQUAL_MESSAGE( msgRow.pData->buffer, rExpectedEndAddress.Row(), nActualEndRow );
+ CPPUNIT_ASSERT_EQUAL_MESSAGE(msgCol.pData->buffer, rExpectedEndAddress.Col(), nActualEndCol);
+ CPPUNIT_ASSERT_EQUAL_MESSAGE(msgRow.pData->buffer, rExpectedEndAddress.Row(), nActualEndRow);
}
-
}
void ScFiltersTest::testMergedCellsODS()
@@ -1363,9 +1400,9 @@ void ScFiltersTest::testMergedCellsODS()
testFile(aCSVFileName1, rDoc, 0);
//check sheet1 merged cells
- checkMergedCells( rDoc, ScAddress( 0, 0, 0 ), ScAddress( 5, 11, 0 ) );
- checkMergedCells( rDoc, ScAddress( 7, 2, 0 ), ScAddress( 9, 12, 0 ) );
- checkMergedCells( rDoc, ScAddress( 3, 15, 0 ), ScAddress( 7, 23, 0 ) );
+ checkMergedCells(rDoc, ScAddress(0, 0, 0), ScAddress(5, 11, 0));
+ checkMergedCells(rDoc, ScAddress(7, 2, 0), ScAddress(9, 12, 0));
+ checkMergedCells(rDoc, ScAddress(3, 15, 0), ScAddress(7, 23, 0));
//check sheet2 content
OUString aCSVFileName2;
@@ -1373,7 +1410,7 @@ void ScFiltersTest::testMergedCellsODS()
testFile(aCSVFileName2, rDoc, 1);
//check sheet2 merged cells
- checkMergedCells( rDoc, ScAddress( 4, 3, 1 ), ScAddress( 6, 15, 1 ) );
+ checkMergedCells(rDoc, ScAddress(4, 3, 1), ScAddress(6, 15, 1));
xDocSh->DoClose();
}
@@ -1396,8 +1433,8 @@ void ScFiltersTest::testRepeatedColumnsODS()
xDocSh->DoClose();
}
-namespace {
-
+namespace
+{
//for cleaner passing of parameters
struct ValDataTestParams
{
@@ -1412,40 +1449,44 @@ struct ValDataTestParams
ScValidErrorStyle eErrorStyle;
sal_uInt32 nExpectedIndex;
- ValDataTestParams( ScValidationMode eMode, ScConditionMode eOp,
- const OUString& aExpr1, const OUString& aExpr2, ScDocument& rDoc,
- const ScAddress& aPos, const OUString& aETitle, const OUString& aEMsg,
- ScValidErrorStyle eEStyle, sal_uInt32 nIndex ):
- eValMode(eMode), eCondOp(eOp), aStrVal1(aExpr1),
- aStrVal2(aExpr2), rDocument(rDoc), aPosition(aPos),
- aErrorTitle(aETitle), aErrorMessage(aEMsg),
- eErrorStyle(eEStyle), nExpectedIndex(nIndex) { };
+ ValDataTestParams(ScValidationMode eMode, ScConditionMode eOp, const OUString& aExpr1,
+ const OUString& aExpr2, ScDocument& rDoc, const ScAddress& aPos,
+ const OUString& aETitle, const OUString& aEMsg, ScValidErrorStyle eEStyle,
+ sal_uInt32 nIndex)
+ : eValMode(eMode)
+ , eCondOp(eOp)
+ , aStrVal1(aExpr1)
+ , aStrVal2(aExpr2)
+ , rDocument(rDoc)
+ , aPosition(aPos)
+ , aErrorTitle(aETitle)
+ , aErrorMessage(aEMsg)
+ , eErrorStyle(eEStyle)
+ , nExpectedIndex(nIndex){};
};
-void checkValiditationEntries( const ValDataTestParams& rVDTParams )
+void checkValiditationEntries(const ValDataTestParams& rVDTParams)
{
ScDocument& rDoc = rVDTParams.rDocument;
//create expected data validation entry
- ScValidationData aValData(
- rVDTParams.eValMode, rVDTParams.eCondOp, rVDTParams.aStrVal1,
- rVDTParams.aStrVal2, rDoc, rVDTParams.aPosition, OUString(),
- OUString(), rDoc.GetStorageGrammar(), rDoc.GetStorageGrammar()
- );
- aValData.SetIgnoreBlank( true );
- aValData.SetListType( 1 );
+ ScValidationData aValData(rVDTParams.eValMode, rVDTParams.eCondOp, rVDTParams.aStrVal1,
+ rVDTParams.aStrVal2, rDoc, rVDTParams.aPosition, OUString(),
+ OUString(), rDoc.GetStorageGrammar(), rDoc.GetStorageGrammar());
+ aValData.SetIgnoreBlank(true);
+ aValData.SetListType(1);
aValData.ResetInput();
- aValData.SetError( rVDTParams.aErrorTitle, rVDTParams.aErrorMessage, rVDTParams.eErrorStyle );
- aValData.SetSrcString( OUString() );
+ aValData.SetError(rVDTParams.aErrorTitle, rVDTParams.aErrorMessage, rVDTParams.eErrorStyle);
+ aValData.SetSrcString(OUString());
//get actual data validation entry from document
- const ScValidationData* pValDataTest = rDoc.GetValidationEntry( rVDTParams.nExpectedIndex );
+ const ScValidationData* pValDataTest = rDoc.GetValidationEntry(rVDTParams.nExpectedIndex);
- sal_Int32 nCol( static_cast<sal_Int32>(rVDTParams.aPosition.Col()) );
- sal_Int32 nRow( static_cast<sal_Int32>(rVDTParams.aPosition.Row()) );
- sal_Int32 nTab( static_cast<sal_Int32>(rVDTParams.aPosition.Tab()) );
- OString aMsgPrefix = "Data Validation Entry with base-cell-address: (" +
- OString::number(nCol) + "," + OString::number(nRow) + "," + OString::number(nTab) + ") ";
+ sal_Int32 nCol(static_cast<sal_Int32>(rVDTParams.aPosition.Col()));
+ sal_Int32 nRow(static_cast<sal_Int32>(rVDTParams.aPosition.Row()));
+ sal_Int32 nTab(static_cast<sal_Int32>(rVDTParams.aPosition.Tab()));
+ OString aMsgPrefix = "Data Validation Entry with base-cell-address: (" + OString::number(nCol)
+ + "," + OString::number(nRow) + "," + OString::number(nTab) + ") ";
OString aMsg = aMsgPrefix + "did not get imported at all.";
CPPUNIT_ASSERT_MESSAGE(aMsg.getStr(), pValDataTest);
@@ -1458,46 +1499,46 @@ void checkValiditationEntries( const ValDataTestParams& rVDTParams )
}
}
-void checkCellValidity( const ScAddress& rValBaseAddr, const ScRange& rRange, const ScDocument& rDoc )
+void checkCellValidity(const ScAddress& rValBaseAddr, const ScRange& rRange, const ScDocument& rDoc)
{
- SCCOL nBCol( rValBaseAddr.Col() );
- SCROW nBRow( rValBaseAddr.Row() );
- SCTAB nTab( static_cast<sal_Int32>(rValBaseAddr.Tab()) );
+ SCCOL nBCol(rValBaseAddr.Col());
+ SCROW nBRow(rValBaseAddr.Row());
+ SCTAB nTab(static_cast<sal_Int32>(rValBaseAddr.Tab()));
//get from the document the data validation entry we are checking against
const SfxUInt32Item* pItem = rDoc.GetAttr(nBCol, nBRow, nTab, ATTR_VALIDDATA);
- const ScValidationData* pValData = rDoc.GetValidationEntry( pItem->GetValue() );
+ const ScValidationData* pValData = rDoc.GetValidationEntry(pItem->GetValue());
CPPUNIT_ASSERT(pValData);
//check that each cell in the expected range is associated with the data validation entry
- for(SCCOL i = rRange.aStart.Col(); i <= rRange.aEnd.Col(); ++i)
+ for (SCCOL i = rRange.aStart.Col(); i <= rRange.aEnd.Col(); ++i)
{
- for(SCROW j = rRange.aStart.Row(); j <= rRange.aEnd.Row(); ++j)
+ for (SCROW j = rRange.aStart.Row(); j <= rRange.aEnd.Row(); ++j)
{
const SfxUInt32Item* pItemTest = rDoc.GetAttr(i, j, nTab, ATTR_VALIDDATA);
- const ScValidationData* pValDataTest = rDoc.GetValidationEntry( pItemTest->GetValue() );
+ const ScValidationData* pValDataTest = rDoc.GetValidationEntry(pItemTest->GetValue());
//prevent string operations for occurring unnecessarily
- if(!(pValDataTest && pValData->GetKey() == pValDataTest->GetKey()))
+ if (!(pValDataTest && pValData->GetKey() == pValDataTest->GetKey()))
{
sal_Int32 nCol = static_cast<sal_Int32>(i);
sal_Int32 nRow = static_cast<sal_Int32>(j);
sal_Int32 nTab32 = static_cast<sal_Int32>(nTab);
- OString sMsg = "\nData validation entry base-cell-address: (" +
- OString::number( static_cast<sal_Int32>(nBCol) ) + "," +
- OString::number( static_cast<sal_Int32>(nBRow) ) + "," +
- OString::number( nTab32 ) + ")\n"
- "Cell: (" + OString::number(nCol) + "," +
- OString::number(nRow) + "," +
- OString::number(nTab32) + ")";
+ OString sMsg = "\nData validation entry base-cell-address: ("
+ + OString::number(static_cast<sal_Int32>(nBCol)) + ","
+ + OString::number(static_cast<sal_Int32>(nBRow)) + ","
+ + OString::number(nTab32)
+ + ")\n"
+ "Cell: ("
+ + OString::number(nCol) + "," + OString::number(nRow) + ","
+ + OString::number(nTab32) + ")";
sal_uInt32 expectedKey(pValData->GetKey());
sal_uInt32 actualKey(0);
- if(pValDataTest)
+ if (pValDataTest)
actualKey = pValDataTest->GetKey();
CPPUNIT_ASSERT_EQUAL_MESSAGE(sMsg.getStr(), expectedKey, actualKey);
}
}
}
}
-
}
void ScFiltersTest::testDataValidityODS()
@@ -1505,45 +1546,38 @@ void ScFiltersTest::testDataValidityODS()
ScDocShellRef xDocSh = loadDoc(u"dataValidity.", FORMAT_ODS);
ScDocument& rDoc = xDocSh->GetDocument();
- ScAddress aValBaseAddr1( 2,6,0 ); //sheet1
- ScAddress aValBaseAddr2( 2,3,1 ); //sheet2
- ScAddress aValBaseAddr3( 2,2,2 ); //sheet3
+ ScAddress aValBaseAddr1(2, 6, 0); //sheet1
+ ScAddress aValBaseAddr2(2, 3, 1); //sheet2
+ ScAddress aValBaseAddr3(2, 2, 2); //sheet3
//sheet1's expected Data Validation Entry values
- ValDataTestParams aVDTParams1(
- SC_VALID_DECIMAL, ScConditionMode::Greater, "3.14", OUString(), rDoc,
- aValBaseAddr1, "Too small",
- "The number you are trying to enter is not greater than 3.14! Are you sure you want to enter it anyway?",
- SC_VALERR_WARNING, 1
- );
+ ValDataTestParams aVDTParams1(SC_VALID_DECIMAL, ScConditionMode::Greater, "3.14", OUString(),
+ rDoc, aValBaseAddr1, "Too small",
+ "The number you are trying to enter is not greater than 3.14! "
+ "Are you sure you want to enter it anyway?",
+ SC_VALERR_WARNING, 1);
//sheet2's expected Data Validation Entry values
- ValDataTestParams aVDTParams2(
- SC_VALID_WHOLE, ScConditionMode::Between, "1", "10", rDoc,
- aValBaseAddr2, "Error sheet 2",
- "Must be a whole number between 1 and 10.",
- SC_VALERR_STOP, 2
- );
+ ValDataTestParams aVDTParams2(SC_VALID_WHOLE, ScConditionMode::Between, "1", "10", rDoc,
+ aValBaseAddr2, "Error sheet 2",
+ "Must be a whole number between 1 and 10.", SC_VALERR_STOP, 2);
//sheet3's expected Data Validation Entry values
- ValDataTestParams aVDTParams3(
- SC_VALID_CUSTOM, ScConditionMode::Direct, "ISTEXT(C3)", OUString(), rDoc,
- aValBaseAddr3, "Error sheet 3",
- "Must not be a numerical value.",
- SC_VALERR_STOP, 3
- );
+ ValDataTestParams aVDTParams3(SC_VALID_CUSTOM, ScConditionMode::Direct, "ISTEXT(C3)",
+ OUString(), rDoc, aValBaseAddr3, "Error sheet 3",
+ "Must not be a numerical value.", SC_VALERR_STOP, 3);
//check each sheet's Data Validation Entries
- checkValiditationEntries( aVDTParams1 );
- checkValiditationEntries( aVDTParams2 );
- checkValiditationEntries( aVDTParams3 );
+ checkValiditationEntries(aVDTParams1);
+ checkValiditationEntries(aVDTParams2);
+ checkValiditationEntries(aVDTParams3);
//expected ranges to be associated with data validity
- ScRange aRange1( 2,2,0, 2,6,0 ); //sheet1
- ScRange aRange2( 2,3,1, 6,7,1 ); //sheet2
- ScRange aRange3( 2,2,2, 2,6,2 ); //sheet3
+ ScRange aRange1(2, 2, 0, 2, 6, 0); //sheet1
+ ScRange aRange2(2, 3, 1, 6, 7, 1); //sheet2
+ ScRange aRange3(2, 2, 2, 2, 6, 2); //sheet3
//check each sheet's cells for data validity
- checkCellValidity( aValBaseAddr1, aRange1, rDoc );
- checkCellValidity( aValBaseAddr2, aRange2, rDoc );
- checkCellValidity( aValBaseAddr3, aRange3, rDoc );
+ checkCellValidity(aValBaseAddr1, aRange1, rDoc);
+ checkCellValidity(aValBaseAddr2, aRange2, rDoc);
+ checkCellValidity(aValBaseAddr3, aRange3, rDoc);
//check each sheet's content
OUString aCSVFileName1;
@@ -1566,19 +1600,19 @@ void ScFiltersTest::testDataValidityXLSX()
ScDocShellRef xDocSh = loadDoc(u"dataValidity.", FORMAT_XLSX);
ScDocument& rDoc = xDocSh->GetDocument();
- ScAddress aValBaseAddr1( 2,6,0 ); //sheet1
- ScAddress aValBaseAddr2( 2,3,1 ); //sheet2
- ScAddress aValBaseAddr3( 2,2,2 ); //sheet3
+ ScAddress aValBaseAddr1(2, 6, 0); //sheet1
+ ScAddress aValBaseAddr2(2, 3, 1); //sheet2
+ ScAddress aValBaseAddr3(2, 2, 2); //sheet3
//expected ranges to be associated with data validity
- ScRange aRange1( 2,2,0, 2,6,0 ); //sheet1
- ScRange aRange2( 2,3,1, 6,7,1 ); //sheet2
- ScRange aRange3( 2,2,2, 2,6,2 ); //sheet3
+ ScRange aRange1(2, 2, 0, 2, 6, 0); //sheet1
+ ScRange aRange2(2, 3, 1, 6, 7, 1); //sheet2
+ ScRange aRange3(2, 2, 2, 2, 6, 2); //sheet3
//check each sheet's cells for data validity
- checkCellValidity( aValBaseAddr1, aRange1, rDoc );
- checkCellValidity( aValBaseAddr2, aRange2, rDoc );
- checkCellValidity( aValBaseAddr3, aRange3, rDoc );
+ checkCellValidity(aValBaseAddr1, aRange1, rDoc);
+ checkCellValidity(aValBaseAddr2, aRange2, rDoc);
+ checkCellValidity(aValBaseAddr3, aRange3, rDoc);
xDocSh->DoClose();
}
@@ -1591,20 +1625,29 @@ void ScFiltersTest::testDataTableMortgageXLS()
// One-variable table
- ASSERT_FORMULA_EQUAL(rDoc, ScAddress(3,1,0), "PMT(B3/12,B4,-B5)", "Wrong formula!");
- ASSERT_FORMULA_EQUAL(rDoc, ScAddress(3,2,0), "MULTIPLE.OPERATIONS(D$2,$B$3,$C3)", "Wrong formula!");
- ASSERT_FORMULA_EQUAL(rDoc, ScAddress(3,3,0), "MULTIPLE.OPERATIONS(D$2,$B$3,$C4)", "Wrong formula!");
- ASSERT_FORMULA_EQUAL(rDoc, ScAddress(3,4,0), "MULTIPLE.OPERATIONS(D$2,$B$3,$C5)", "Wrong formula!");
+ ASSERT_FORMULA_EQUAL(rDoc, ScAddress(3, 1, 0), "PMT(B3/12,B4,-B5)", "Wrong formula!");
+ ASSERT_FORMULA_EQUAL(rDoc, ScAddress(3, 2, 0), "MULTIPLE.OPERATIONS(D$2,$B$3,$C3)",
+ "Wrong formula!");
+ ASSERT_FORMULA_EQUAL(rDoc, ScAddress(3, 3, 0), "MULTIPLE.OPERATIONS(D$2,$B$3,$C4)",
+ "Wrong formula!");
+ ASSERT_FORMULA_EQUAL(rDoc, ScAddress(3, 4, 0), "MULTIPLE.OPERATIONS(D$2,$B$3,$C5)",
+ "Wrong formula!");
// Two-variable table
- ASSERT_FORMULA_EQUAL(rDoc, ScAddress(2,7,0), "PMT(B9/12,B10,-B11)", "Wrong formula!");
- ASSERT_FORMULA_EQUAL(rDoc, ScAddress(3,8,0), "MULTIPLE.OPERATIONS($C$8,$B$9,$C9,$B$10,D$8)", "Wrong formula!");
- ASSERT_FORMULA_EQUAL(rDoc, ScAddress(3,9,0), "MULTIPLE.OPERATIONS($C$8,$B$9,$C10,$B$10,D$8)", "Wrong formula!");
- ASSERT_FORMULA_EQUAL(rDoc, ScAddress(3,10,0), "MULTIPLE.OPERATIONS($C$8,$B$9,$C11,$B$10,D$8)", "Wrong formula!");
- ASSERT_FORMULA_EQUAL(rDoc, ScAddress(4,8,0), "MULTIPLE.OPERATIONS($C$8,$B$9,$C9,$B$10,E$8)", "Wrong formula!");
- ASSERT_FORMULA_EQUAL(rDoc, ScAddress(4,9,0), "MULTIPLE.OPERATIONS($C$8,$B$9,$C10,$B$10,E$8)", "Wrong formula!");
- ASSERT_FORMULA_EQUAL(rDoc, ScAddress(4,10,0), "MULTIPLE.OPERATIONS($C$8,$B$9,$C11,$B$10,E$8)", "Wrong formula!");
+ ASSERT_FORMULA_EQUAL(rDoc, ScAddress(2, 7, 0), "PMT(B9/12,B10,-B11)", "Wrong formula!");
+ ASSERT_FORMULA_EQUAL(rDoc, ScAddress(3, 8, 0), "MULTIPLE.OPERATIONS($C$8,$B$9,$C9,$B$10,D$8)",
+ "Wrong formula!");
+ ASSERT_FORMULA_EQUAL(rDoc, ScAddress(3, 9, 0), "MULTIPLE.OPERATIONS($C$8,$B$9,$C10,$B$10,D$8)",
+ "Wrong formula!");
+ ASSERT_FORMULA_EQUAL(rDoc, ScAddress(3, 10, 0), "MULTIPLE.OPERATIONS($C$8,$B$9,$C11,$B$10,D$8)",
+ "Wrong formula!");
+ ASSERT_FORMULA_EQUAL(rDoc, ScAddress(4, 8, 0), "MULTIPLE.OPERATIONS($C$8,$B$9,$C9,$B$10,E$8)",
+ "Wrong formula!");
+ ASSERT_FORMULA_EQUAL(rDoc, ScAddress(4, 9, 0), "MULTIPLE.OPERATIONS($C$8,$B$9,$C10,$B$10,E$8)",
+ "Wrong formula!");
+ ASSERT_FORMULA_EQUAL(rDoc, ScAddress(4, 10, 0), "MULTIPLE.OPERATIONS($C$8,$B$9,$C11,$B$10,E$8)",
+ "Wrong formula!");
xDocSh->DoClose();
}
@@ -1623,24 +1666,28 @@ void ScFiltersTest::testDataTableOneVarXLSX()
// B5:B11 should have multiple operations formula cells. Just check the
// top and bottom cells.
- ASSERT_FORMULA_EQUAL(rDoc, ScAddress(1,4,0), "MULTIPLE.OPERATIONS(B$4,$A$2,$A5)", "Wrong formula!");
+ ASSERT_FORMULA_EQUAL(rDoc, ScAddress(1, 4, 0), "MULTIPLE.OPERATIONS(B$4,$A$2,$A5)",
+ "Wrong formula!");
- CPPUNIT_ASSERT_EQUAL(2.0, rDoc.GetValue(ScAddress(1,4,0)));
+ CPPUNIT_ASSERT_EQUAL(2.0, rDoc.GetValue(ScAddress(1, 4, 0)));
- ASSERT_FORMULA_EQUAL(rDoc, ScAddress(1,10,0), "MULTIPLE.OPERATIONS(B$4,$A$2,$A11)", "Wrong formula!");
+ ASSERT_FORMULA_EQUAL(rDoc, ScAddress(1, 10, 0), "MULTIPLE.OPERATIONS(B$4,$A$2,$A11)",
+ "Wrong formula!");
- CPPUNIT_ASSERT_EQUAL(14.0, rDoc.GetValue(ScAddress(1,10,0)));
+ CPPUNIT_ASSERT_EQUAL(14.0, rDoc.GetValue(ScAddress(1, 10, 0)));
// Likewise, E5:I5 should have multiple operations formula cells. Just
// check the left- and right-most cells.
- ASSERT_FORMULA_EQUAL(rDoc, ScAddress(4,4,0), "MULTIPLE.OPERATIONS($D5,$B$2,E$4)", "Wrong formula!");
+ ASSERT_FORMULA_EQUAL(rDoc, ScAddress(4, 4, 0), "MULTIPLE.OPERATIONS($D5,$B$2,E$4)",
+ "Wrong formula!");
- CPPUNIT_ASSERT_EQUAL(10.0, rDoc.GetValue(ScAddress(4,4,0)));
+ CPPUNIT_ASSERT_EQUAL(10.0, rDoc.GetValue(ScAddress(4, 4, 0)));
- ASSERT_FORMULA_EQUAL(rDoc, ScAddress(8,4,0), "MULTIPLE.OPERATIONS($D5,$B$2,I$4)", "Wrong formula!");
+ ASSERT_FORMULA_EQUAL(rDoc, ScAddress(8, 4, 0), "MULTIPLE.OPERATIONS($D5,$B$2,I$4)",
+ "Wrong formula!");
- CPPUNIT_ASSERT_EQUAL(50.0, rDoc.GetValue(ScAddress(8,4,0)));
+ CPPUNIT_ASSERT_EQUAL(50.0, rDoc.GetValue(ScAddress(8, 4, 0)));
xDocSh->DoClose();
}
@@ -1659,38 +1706,44 @@ void ScFiltersTest::testDataTableMultiTableXLSX()
// B4:M15 should have multiple operations formula cells. We'll just check
// the top-left and bottom-right ones.
- ASSERT_FORMULA_EQUAL(rDoc, ScAddress(1,3,0), "MULTIPLE.OPERATIONS($A$3,$E$1,$A4,$D$1,B$3)", "Wrong formula!");
+ ASSERT_FORMULA_EQUAL(rDoc, ScAddress(1, 3, 0), "MULTIPLE.OPERATIONS($A$3,$E$1,$A4,$D$1,B$3)",
+ "Wrong formula!");
- CPPUNIT_ASSERT_EQUAL(1.0, rDoc.GetValue(ScAddress(1,3,0)));
+ CPPUNIT_ASSERT_EQUAL(1.0, rDoc.GetValue(ScAddress(1, 3, 0)));
- ASSERT_FORMULA_EQUAL(rDoc, ScAddress(12,14,0), "MULTIPLE.OPERATIONS($A$3,$E$1,$A15,$D$1,M$3)", "Wrong formula!");
+ ASSERT_FORMULA_EQUAL(rDoc, ScAddress(12, 14, 0), "MULTIPLE.OPERATIONS($A$3,$E$1,$A15,$D$1,M$3)",
+ "Wrong formula!");
- CPPUNIT_ASSERT_EQUAL(144.0, rDoc.GetValue(ScAddress(12,14,0)));
+ CPPUNIT_ASSERT_EQUAL(144.0, rDoc.GetValue(ScAddress(12, 14, 0)));
xDocSh->DoClose();
}
void ScFiltersTest::testBrokenQuotesCSV()
{
- OUString aFileExtension(getFileFormats()[FORMAT_CSV].pName, strlen(getFileFormats()[FORMAT_CSV].pName), RTL_TEXTENCODING_UTF8 );
- OUString aFilterName(getFileFormats()[FORMAT_CSV].pFilterName, strlen(getFileFormats()[FORMAT_CSV].pFilterName), RTL_TEXTENCODING_UTF8) ;
+ OUString aFileExtension(getFileFormats()[FORMAT_CSV].pName,
+ strlen(getFileFormats()[FORMAT_CSV].pName), RTL_TEXTENCODING_UTF8);
+ OUString aFilterName(getFileFormats()[FORMAT_CSV].pFilterName,
+ strlen(getFileFormats()[FORMAT_CSV].pFilterName), RTL_TEXTENCODING_UTF8);
OUString aFileName;
createFileURL(u"fdo48621_broken_quotes.", aFileExtension, aFileName);
- OUString aFilterType(getFileFormats()[FORMAT_CSV].pTypeName, strlen(getFileFormats()[FORMAT_CSV].pTypeName), RTL_TEXTENCODING_UTF8);
+ OUString aFilterType(getFileFormats()[FORMAT_CSV].pTypeName,
+ strlen(getFileFormats()[FORMAT_CSV].pTypeName), RTL_TEXTENCODING_UTF8);
std::cout << getFileFormats()[FORMAT_CSV].pName << " Test" << std::endl;
SfxFilterFlags nFormatType = getFileFormats()[FORMAT_CSV].nFormatType;
- SotClipboardFormatId nClipboardId = bool(nFormatType) ? SotClipboardFormatId::STARCALC_8 : SotClipboardFormatId::NONE;
+ SotClipboardFormatId nClipboardId
+ = bool(nFormatType) ? SotClipboardFormatId::STARCALC_8 : SotClipboardFormatId::NONE;
ScDocShellRef xDocSh = ScBootstrapFixture::load(aFileName, aFilterName, OUString(), aFilterType,
- nFormatType, nClipboardId);
+ nFormatType, nClipboardId);
CPPUNIT_ASSERT_MESSAGE("Failed to load fdo48621_broken_quotes.csv", xDocSh.is());
ScDocument& rDoc = xDocSh->GetDocument();
OUString aCSVPath;
- createCSVPath( "fdo48621_broken_quotes_exported.", aCSVPath );
+ createCSVPath("fdo48621_broken_quotes_exported.", aCSVPath);
// fdo#48621
- testFile( aCSVPath, rDoc, 0, StringType::PureString);
+ testFile(aCSVPath, rDoc, 0, StringType::PureString);
xDocSh->DoClose();
}
@@ -1698,24 +1751,28 @@ void ScFiltersTest::testBrokenQuotesCSV()
void ScFiltersTest::testCellValueXLSX()
{
static const OUStringLiteral aFileNameBase(u"cell-value.");
- OUString aFileExtension(getFileFormats()[FORMAT_XLSX].pName, strlen(getFileFormats()[FORMAT_XLSX].pName), RTL_TEXTENCODING_UTF8 );
- OUString aFilterName(getFileFormats()[FORMAT_XLSX].pFilterName, strlen(getFileFormats()[FORMAT_XLSX].pFilterName), RTL_TEXTENCODING_UTF8) ;
+ OUString aFileExtension(getFileFormats()[FORMAT_XLSX].pName,
+ strlen(getFileFormats()[FORMAT_XLSX].pName), RTL_TEXTENCODING_UTF8);
+ OUString aFilterName(getFileFormats()[FORMAT_XLSX].pFilterName,
+ strlen(getFileFormats()[FORMAT_XLSX].pFilterName), RTL_TEXTENCODING_UTF8);
OUString aFileName;
createFileURL(aFileNameBase, aFileExtension, aFileName);
- OUString aFilterType(getFileFormats()[FORMAT_XLSX].pTypeName, strlen(getFileFormats()[FORMAT_XLSX].pTypeName), RTL_TEXTENCODING_UTF8);
+ OUString aFilterType(getFileFormats()[FORMAT_XLSX].pTypeName,
+ strlen(getFileFormats()[FORMAT_XLSX].pTypeName), RTL_TEXTENCODING_UTF8);
std::cout << getFileFormats()[FORMAT_XLSX].pName << " Test" << std::endl;
SfxFilterFlags nFormatType = getFileFormats()[FORMAT_XLSX].nFormatType;
- SotClipboardFormatId nClipboardId = bool(nFormatType) ? SotClipboardFormatId::STARCALC_8 : SotClipboardFormatId::NONE;
- ScDocShellRef xDocSh = ScBootstrapFixture::load( aFileName, aFilterName, OUString(), aFilterType,
- nFormatType, nClipboardId);
+ SotClipboardFormatId nClipboardId
+ = bool(nFormatType) ? SotClipboardFormatId::STARCALC_8 : SotClipboardFormatId::NONE;
+ ScDocShellRef xDocSh = ScBootstrapFixture::load(aFileName, aFilterName, OUString(), aFilterType,
+ nFormatType, nClipboardId);
CPPUNIT_ASSERT_MESSAGE("Failed to load cell-value.xlsx", xDocSh.is());
ScDocument& rDoc = xDocSh->GetDocument();
OUString aCSVPath;
- createCSVPath( aFileNameBase, aCSVPath );
- testFile( aCSVPath, rDoc, 0 );
+ createCSVPath(aFileNameBase, aCSVPath);
+ testFile(aCSVPath, rDoc, 0);
xDocSh->DoClose();
}
@@ -1726,19 +1783,19 @@ void ScFiltersTest::testRowIndex1BasedXLSX()
ScDocument& rDoc = xDocSh->GetDocument();
// A1
- OUString aStr = rDoc.GetString(ScAddress(0,0,0));
+ OUString aStr = rDoc.GetString(ScAddress(0, 0, 0));
CPPUNIT_ASSERT_EQUAL(OUString("Action Plan.Name"), aStr);
// B1
- aStr = rDoc.GetString(ScAddress(1,0,0));
+ aStr = rDoc.GetString(ScAddress(1, 0, 0));
CPPUNIT_ASSERT_EQUAL(OUString("Action Plan.Description"), aStr);
// A2
- aStr = rDoc.GetString(ScAddress(0,1,0));
+ aStr = rDoc.GetString(ScAddress(0, 1, 0));
CPPUNIT_ASSERT_EQUAL(OUString("Jerry"), aStr);
// B2 - multi-line text.
- const EditTextObject* pText = rDoc.GetEditText(ScAddress(1,1,0));
+ const EditTextObject* pText = rDoc.GetEditText(ScAddress(1, 1, 0));
CPPUNIT_ASSERT(pText);
CPPUNIT_ASSERT_EQUAL(static_cast<sal_Int32>(3), pText->GetParagraphCount());
aStr = pText->GetText(0);
@@ -1753,7 +1810,7 @@ void ScFiltersTest::testRowIndex1BasedXLSX()
void ScFiltersTest::testImportCrash(std::u16string_view rFileName, sal_Int32 nFormat)
{
- ScDocShellRef xDocSh =loadDoc(rFileName, nFormat);
+ ScDocShellRef xDocSh = loadDoc(rFileName, nFormat);
ScDocument& rDoc = xDocSh->GetDocument();
rDoc.CalcAll(); // perform hard re-calculation.
@@ -1763,16 +1820,20 @@ void ScFiltersTest::testImportCrash(std::u16string_view rFileName, sal_Int32 nFo
void ScFiltersTest::testPassword_Impl(std::u16string_view aFileNameBase)
{
- OUString aFileExtension(getFileFormats()[0].pName, strlen(getFileFormats()[0].pName), RTL_TEXTENCODING_UTF8 );
- OUString aFilterName(getFileFormats()[0].pFilterName, strlen(getFileFormats()[0].pFilterName), RTL_TEXTENCODING_UTF8) ;
+ OUString aFileExtension(getFileFormats()[0].pName, strlen(getFileFormats()[0].pName),
+ RTL_TEXTENCODING_UTF8);
+ OUString aFilterName(getFileFormats()[0].pFilterName, strlen(getFileFormats()[0].pFilterName),
+ RTL_TEXTENCODING_UTF8);
OUString aFileName;
createFileURL(aFileNameBase, aFileExtension, aFileName);
- OUString aFilterType(getFileFormats()[0].pTypeName, strlen(getFileFormats()[0].pTypeName), RTL_TEXTENCODING_UTF8);
+ OUString aFilterType(getFileFormats()[0].pTypeName, strlen(getFileFormats()[0].pTypeName),
+ RTL_TEXTENCODING_UTF8);
SfxFilterFlags nFormatType = getFileFormats()[0].nFormatType;
OUString aPass("test");
ScDocShellRef xDocSh = ScBootstrapFixture::load(aFileName, aFilterName, OUString(), aFilterType,
- nFormatType, SotClipboardFormatId::STARCALC_8, SOFFICE_FILEFORMAT_CURRENT, &aPass);
+ nFormatType, SotClipboardFormatId::STARCALC_8,
+ SOFFICE_FILEFORMAT_CURRENT, &aPass);
CPPUNIT_ASSERT_MESSAGE("Failed to load password.ods", xDocSh.is());
xDocSh->DoClose();
@@ -1801,12 +1862,15 @@ void ScFiltersTest::testControlImport()
{
ScDocShellRef xDocSh = loadDoc(u"singlecontrol.", FORMAT_XLSX);
- uno::Reference< frame::XModel > xModel = xDocSh->GetModel();
- uno::Reference< sheet::XSpreadsheetDocument > xDoc(xModel, UNO_QUERY_THROW);
- uno::Reference< container::XIndexAccess > xIA(xDoc->getSheets(), UNO_QUERY_THROW);
- uno::Reference< drawing::XDrawPageSupplier > xDrawPageSupplier( xIA->getByIndex(0), UNO_QUERY_THROW);
- uno::Reference< container::XIndexAccess > xIA_DrawPage(xDrawPageSupplier->getDrawPage(), UNO_QUERY_THROW);
- uno::Reference< drawing::XControlShape > xControlShape(xIA_DrawPage->getByIndex(0), UNO_QUERY_THROW);
+ uno::Reference<frame::XModel> xModel = xDocSh->GetModel();
+ uno::Reference<sheet::XSpreadsheetDocument> xDoc(xModel, UNO_QUERY_THROW);
+ uno::Reference<container::XIndexAccess> xIA(xDoc->getSheets(), UNO_QUERY_THROW);
+ uno::Reference<drawing::XDrawPageSupplier> xDrawPageSupplier(xIA->getByIndex(0),
+ UNO_QUERY_THROW);
+ uno::Reference<container::XIndexAccess> xIA_DrawPage(xDrawPageSupplier->getDrawPage(),
+ UNO_QUERY_THROW);
+ uno::Reference<drawing::XControlShape> xControlShape(xIA_DrawPage->getByIndex(0),
+ UNO_QUERY_THROW);
xDocSh->DoClose();
}
@@ -1814,15 +1878,19 @@ void ScFiltersTest::testControlImport()
void ScFiltersTest::testActiveXOptionButtonGroup()
{
ScDocShellRef xDocSh = loadDoc(u"tdf111980_radioButtons.", FORMAT_XLSX);
- uno::Reference< frame::XModel > xModel = xDocSh->GetModel();
- uno::Reference< sheet::XSpreadsheetDocument > xDoc(xModel, UNO_QUERY_THROW);
- uno::Reference< container::XIndexAccess > xIA(xDoc->getSheets(), UNO_QUERY_THROW);
- uno::Reference< drawing::XDrawPageSupplier > xDrawPageSupplier( xIA->getByIndex(0), UNO_QUERY_THROW);
- uno::Reference< container::XIndexAccess > xIA_DrawPage(xDrawPageSupplier->getDrawPage(), UNO_QUERY_THROW);
+ uno::Reference<frame::XModel> xModel = xDocSh->GetModel();
+ uno::Reference<sheet::XSpreadsheetDocument> xDoc(xModel, UNO_QUERY_THROW);
+ uno::Reference<container::XIndexAccess> xIA(xDoc->getSheets(), UNO_QUERY_THROW);
+ uno::Reference<drawing::XDrawPageSupplier> xDrawPageSupplier(xIA->getByIndex(0),
+ UNO_QUERY_THROW);
+ uno::Reference<container::XIndexAccess> xIA_DrawPage(xDrawPageSupplier->getDrawPage(),
+ UNO_QUERY_THROW);
OUString sGroupName;
- uno::Reference< drawing::XControlShape > xControlShape(xIA_DrawPage->getByIndex(0), UNO_QUERY_THROW);
- uno::Reference<beans::XPropertySet> xPropertySet(xControlShape->getControl(), uno::UNO_QUERY_THROW);
+ uno::Reference<drawing::XControlShape> xControlShape(xIA_DrawPage->getByIndex(0),
+ UNO_QUERY_THROW);
+ uno::Reference<beans::XPropertySet> xPropertySet(xControlShape->getControl(),
+ uno::UNO_QUERY_THROW);
xPropertySet->getPropertyValue("GroupName") >>= sGroupName;
CPPUNIT_ASSERT_EQUAL(OUString("Sheet1"), sGroupName);
@@ -1832,55 +1900,55 @@ void ScFiltersTest::testActiveXOptionButtonGroup()
xControlShape.set(xIA_DrawPage->getByIndex(2), uno::UNO_QUERY_THROW);
xPropertySet.set(xControlShape->getControl(), uno::UNO_QUERY_THROW);
xPropertySet->getPropertyValue("GroupName") >>= sGroupName2;
- CPPUNIT_ASSERT_EQUAL( false, sGroupName2.isEmpty() );
+ CPPUNIT_ASSERT_EQUAL(false, sGroupName2.isEmpty());
OUString sGroupName3;
xControlShape.set(xIA_DrawPage->getByIndex(3), uno::UNO_QUERY_THROW);
xPropertySet.set(xControlShape->getControl(), uno::UNO_QUERY_THROW);
xPropertySet->getPropertyValue("GroupName") >>= sGroupName3;
- CPPUNIT_ASSERT_EQUAL( sGroupName2, sGroupName3 );
- CPPUNIT_ASSERT( sGroupName != sGroupName3 );
+ CPPUNIT_ASSERT_EQUAL(sGroupName2, sGroupName3);
+ CPPUNIT_ASSERT(sGroupName != sGroupName3);
OUString sGroupName4; //Form controls
xControlShape.set(xIA_DrawPage->getByIndex(4), uno::UNO_QUERY_THROW);
xPropertySet.set(xControlShape->getControl(), uno::UNO_QUERY_THROW);
xPropertySet->getPropertyValue("GroupName") >>= sGroupName4;
- CPPUNIT_ASSERT_EQUAL( false, sGroupName4.isEmpty() );
+ CPPUNIT_ASSERT_EQUAL(false, sGroupName4.isEmpty());
OUString sGroupName5;
xControlShape.set(xIA_DrawPage->getByIndex(5), uno::UNO_QUERY_THROW);
xPropertySet.set(xControlShape->getControl(), uno::UNO_QUERY_THROW);
xPropertySet->getPropertyValue("GroupName") >>= sGroupName5;
- CPPUNIT_ASSERT_EQUAL( sGroupName4, sGroupName5 );
- CPPUNIT_ASSERT( sGroupName2 != sGroupName5 );
- CPPUNIT_ASSERT( sGroupName != sGroupName5 );
+ CPPUNIT_ASSERT_EQUAL(sGroupName4, sGroupName5);
+ CPPUNIT_ASSERT(sGroupName2 != sGroupName5);
+ CPPUNIT_ASSERT(sGroupName != sGroupName5);
OUString sGroupName7; //Form radiobutton autogrouped by GroupBox
xControlShape.set(xIA_DrawPage->getByIndex(7), uno::UNO_QUERY_THROW);
xPropertySet.set(xControlShape->getControl(), uno::UNO_QUERY_THROW);
xPropertySet->getPropertyValue("GroupName") >>= sGroupName7;
- CPPUNIT_ASSERT_EQUAL( OUString("autoGroup_Group Box 7"), sGroupName7 );
+ CPPUNIT_ASSERT_EQUAL(OUString("autoGroup_Group Box 7"), sGroupName7);
OUString sGroupName8;
xControlShape.set(xIA_DrawPage->getByIndex(8), uno::UNO_QUERY_THROW);
xPropertySet.set(xControlShape->getControl(), uno::UNO_QUERY_THROW);
xPropertySet->getPropertyValue("GroupName") >>= sGroupName8;
- CPPUNIT_ASSERT_EQUAL( sGroupName7, sGroupName8 );
- CPPUNIT_ASSERT( sGroupName4 != sGroupName8 );
- CPPUNIT_ASSERT( sGroupName2 != sGroupName8 );
- CPPUNIT_ASSERT( sGroupName != sGroupName8 );
+ CPPUNIT_ASSERT_EQUAL(sGroupName7, sGroupName8);
+ CPPUNIT_ASSERT(sGroupName4 != sGroupName8);
+ CPPUNIT_ASSERT(sGroupName2 != sGroupName8);
+ CPPUNIT_ASSERT(sGroupName != sGroupName8);
OUString sGroupName9; //Form radiobutton not fully inside GroupBox
xControlShape.set(xIA_DrawPage->getByIndex(9), uno::UNO_QUERY_THROW);
xPropertySet.set(xControlShape->getControl(), uno::UNO_QUERY_THROW);
xPropertySet->getPropertyValue("GroupName") >>= sGroupName9;
- CPPUNIT_ASSERT_EQUAL( sGroupName4, sGroupName9 );
+ CPPUNIT_ASSERT_EQUAL(sGroupName4, sGroupName9);
OUString sGroupName10; //ActiveX unaffected by GroupBox
xControlShape.set(xIA_DrawPage->getByIndex(10), uno::UNO_QUERY_THROW);
xPropertySet.set(xControlShape->getControl(), uno::UNO_QUERY_THROW);
xPropertySet->getPropertyValue("GroupName") >>= sGroupName10;
- CPPUNIT_ASSERT_EQUAL( sGroupName, sGroupName10 );
+ CPPUNIT_ASSERT_EQUAL(sGroupName, sGroupName10);
xDocSh->DoClose();
}
@@ -1892,9 +1960,8 @@ void ScFiltersTest::testChartImportODS()
ScDocument& rDoc = xDocSh->GetDocument();
// Ensure that the document contains "Empty", "Chart", "Data" and "Title" sheets in this exact order.
- CPPUNIT_ASSERT_EQUAL_MESSAGE(
- "There should be 4 sheets in this document.", sal_Int16(4),
- rDoc.GetTableCount());
+ CPPUNIT_ASSERT_EQUAL_MESSAGE("There should be 4 sheets in this document.", sal_Int16(4),
+ rDoc.GetTableCount());
OUString aName;
rDoc.GetName(0, aName);
CPPUNIT_ASSERT_EQUAL(OUString("Empty"), aName);
@@ -1911,9 +1978,12 @@ void ScFiltersTest::testChartImportODS()
ScRangeList aRanges = getChartRanges(rDoc, *pOleObj);
- CPPUNIT_ASSERT_MESSAGE("Data series title cell not found.", aRanges.Contains(ScAddress(1,0,3))); // B1 on Title
- CPPUNIT_ASSERT_MESSAGE("Data series label range not found.", aRanges.Contains(ScRange(0,1,2,0,3,2))); // A2:A4 on Data
- CPPUNIT_ASSERT_MESSAGE("Data series value range not found.", aRanges.Contains(ScRange(1,1,2,1,3,2))); // B2:B4 on Data
+ CPPUNIT_ASSERT_MESSAGE("Data series title cell not found.",
+ aRanges.Contains(ScAddress(1, 0, 3))); // B1 on Title
+ CPPUNIT_ASSERT_MESSAGE("Data series label range not found.",
+ aRanges.Contains(ScRange(0, 1, 2, 0, 3, 2))); // A2:A4 on Data
+ CPPUNIT_ASSERT_MESSAGE("Data series value range not found.",
+ aRanges.Contains(ScRange(1, 1, 2, 1, 3, 2))); // B2:B4 on Data
xDocSh->DoClose();
}
@@ -1943,21 +2013,17 @@ void ScFiltersTest::testNumberFormatHTML()
ScDocument& rDoc = xDocSh->GetDocument();
// Check the header just in case.
- CPPUNIT_ASSERT_EQUAL_MESSAGE(
- "Cell value is not as expected", OUString("Product"),
- rDoc.GetString(0, 0, 0));
- CPPUNIT_ASSERT_EQUAL_MESSAGE(
- "Cell value is not as expected", OUString("Price"),
- rDoc.GetString(1, 0, 0));
- CPPUNIT_ASSERT_EQUAL_MESSAGE(
- "Cell value is not as expected", OUString("Note"),
- rDoc.GetString(2, 0, 0));
+ CPPUNIT_ASSERT_EQUAL_MESSAGE("Cell value is not as expected", OUString("Product"),
+ rDoc.GetString(0, 0, 0));
+ CPPUNIT_ASSERT_EQUAL_MESSAGE("Cell value is not as expected", OUString("Price"),
+ rDoc.GetString(1, 0, 0));
+ CPPUNIT_ASSERT_EQUAL_MESSAGE("Cell value is not as expected", OUString("Note"),
+ rDoc.GetString(2, 0, 0));
// B2 should be imported as a value cell.
bool bHasValue = rDoc.HasValueData(1, 1, 0);
CPPUNIT_ASSERT_MESSAGE("Fail to import number as a value cell.", bHasValue);
- CPPUNIT_ASSERT_EQUAL_MESSAGE(
- "Incorrect value.", 199.98, rDoc.GetValue(1, 1, 0));
+ CPPUNIT_ASSERT_EQUAL_MESSAGE("Incorrect value.", 199.98, rDoc.GetValue(1, 1, 0));
xDocSh->DoClose();
}
@@ -1969,21 +2035,17 @@ void ScFiltersTest::testNumberFormatCSV()
ScDocument& rDoc = xDocSh->GetDocument();
// Check the header just in case.
- CPPUNIT_ASSERT_EQUAL_MESSAGE(
- "Cell value is not as expected", OUString("Product"),
- rDoc.GetString(0, 0, 0));
- CPPUNIT_ASSERT_EQUAL_MESSAGE(
- "Cell value is not as expected", OUString("Price"),
- rDoc.GetString(1, 0, 0));
- CPPUNIT_ASSERT_EQUAL_MESSAGE(
- "Cell value is not as expected", OUString("Note"),
- rDoc.GetString(2, 0, 0));
+ CPPUNIT_ASSERT_EQUAL_MESSAGE("Cell value is not as expected", OUString("Product"),
+ rDoc.GetString(0, 0, 0));
+ CPPUNIT_ASSERT_EQUAL_MESSAGE("Cell value is not as expected", OUString("Price"),
+ rDoc.GetString(1, 0, 0));
+ CPPUNIT_ASSERT_EQUAL_MESSAGE("Cell value is not as expected", OUString("Note"),
+ rDoc.GetString(2, 0, 0));
// B2 should be imported as a value cell.
bool bHasValue = rDoc.HasValueData(1, 1, 0);
CPPUNIT_ASSERT_MESSAGE("Fail to import number as a value cell.", bHasValue);
- CPPUNIT_ASSERT_EQUAL_MESSAGE(
- "Incorrect value.", 199.98, rDoc.GetValue(1, 1, 0));
+ CPPUNIT_ASSERT_EQUAL_MESSAGE("Incorrect value.", 199.98, rDoc.GetValue(1, 1, 0));
xDocSh->DoClose();
}
@@ -2001,15 +2063,15 @@ void ScFiltersTest::testCellAnchoredShapesODS()
SdrPage* pPage = pDrawLayer->GetPage(0);
CPPUNIT_ASSERT_MESSAGE("draw page for sheet 1 should exist.", pPage);
const size_t nCount = pPage->GetObjCount();
- CPPUNIT_ASSERT_EQUAL_MESSAGE(
- "There should be 2 objects.", static_cast<size_t>(2), nCount);
+ CPPUNIT_ASSERT_EQUAL_MESSAGE("There should be 2 objects.", static_cast<size_t>(2), nCount);
for (size_t i = 0; i < nCount; ++i)
{
SdrObject* pObj = pPage->GetObj(i);
CPPUNIT_ASSERT_MESSAGE("Failed to get drawing object.", pObj);
ScDrawObjData* pData = ScDrawLayer::GetObjData(pObj);
CPPUNIT_ASSERT_MESSAGE("Failed to retrieve user data for this object.", pData);
- CPPUNIT_ASSERT_MESSAGE("Bounding rectangle should have been calculated upon import.", !pData->getShapeRect().IsEmpty());
+ CPPUNIT_ASSERT_MESSAGE("Bounding rectangle should have been calculated upon import.",
+ !pData->getShapeRect().IsEmpty());
}
xDocSh->DoClose();
@@ -2032,7 +2094,9 @@ void ScFiltersTest::testCellAnchoredHiddenShapesXLSX()
SdrObject* pObj = pPage->GetObj(1);
CPPUNIT_ASSERT_MESSAGE("Failed to get drawing object.", pObj);
- CPPUNIT_ASSERT_MESSAGE("The shape having same twocellanchor from and to attribute values, is visible.", !pObj->IsVisible());
+ CPPUNIT_ASSERT_MESSAGE(
+ "The shape having same twocellanchor from and to attribute values, is visible.",
+ !pObj->IsVisible());
xDocSh->DoClose();
}
@@ -2099,7 +2163,8 @@ void ScFiltersTest::testRichTextContentODS()
{
aPos.IncRow();
CPPUNIT_ASSERT_EQUAL(CELLTYPE_STRING, rDoc.GetCellType(aPos));
- CPPUNIT_ASSERT_EQUAL(OUString("All bold"), rDoc.GetString(aPos.Col(), aPos.Row(), aPos.Tab()));
+ CPPUNIT_ASSERT_EQUAL(OUString("All bold"),
+ rDoc.GetString(aPos.Col(), aPos.Row(), aPos.Tab()));
const ScPatternAttr* pAttr = rDoc.GetPattern(aPos.Col(), aPos.Row(), aPos.Tab());
CPPUNIT_ASSERT_MESSAGE("Failed to get cell attribute.", pAttr);
const SvxWeightItem& rWeightItem = pAttr->GetItem(ATTR_FONT_WEIGHT);
@@ -2128,7 +2193,8 @@ void ScFiltersTest::testRichTextContentODS()
CPPUNIT_ASSERT_EQUAL(static_cast<sal_Int32>(1), pEditText->GetParagraphCount());
aParaText = pEditText->GetText(0);
CPPUNIT_ASSERT_MESSAGE("Unexpected text.", aParaText.startsWith("Sheet name is "));
- CPPUNIT_ASSERT_MESSAGE("Sheet name field item not found.", pEditText->HasField(text::textfield::Type::TABLE));
+ CPPUNIT_ASSERT_MESSAGE("Sheet name field item not found.",
+ pEditText->HasField(text::textfield::Type::TABLE));
CPPUNIT_ASSERT_EQUAL(OUString("Sheet name is Test."), ScEditUtil::GetString(*pEditText, &rDoc));
CPPUNIT_ASSERT_EQUAL(OUString("Sheet name is ?."), ScEditUtil::GetString(*pEditText, nullptr));
@@ -2140,9 +2206,12 @@ void ScFiltersTest::testRichTextContentODS()
CPPUNIT_ASSERT_EQUAL(static_cast<sal_Int32>(1), pEditText->GetParagraphCount());
aParaText = pEditText->GetText(0);
CPPUNIT_ASSERT_MESSAGE("Unexpected text.", aParaText.startsWith("URL: "));
- CPPUNIT_ASSERT_MESSAGE("URL field item not found.", pEditText->HasField(text::textfield::Type::URL));
- CPPUNIT_ASSERT_EQUAL(OUString("URL: http://libreoffice.org"), ScEditUtil::GetString(*pEditText, &rDoc));
- CPPUNIT_ASSERT_EQUAL(OUString("URL: http://libreoffice.org"), ScEditUtil::GetString(*pEditText, nullptr));
+ CPPUNIT_ASSERT_MESSAGE("URL field item not found.",
+ pEditText->HasField(text::textfield::Type::URL));
+ CPPUNIT_ASSERT_EQUAL(OUString("URL: http://libreoffice.org"),
+ ScEditUtil::GetString(*pEditText, &rDoc));
+ CPPUNIT_ASSERT_EQUAL(OUString("URL: http://libreoffice.org"),
+ ScEditUtil::GetString(*pEditText, nullptr));
// Cell with Date field item.
aPos.IncRow();
@@ -2152,9 +2221,12 @@ void ScFiltersTest::testRichTextContentODS()
CPPUNIT_ASSERT_EQUAL(static_cast<sal_Int32>(1), pEditText->GetParagraphCount());
aParaText = pEditText->GetText(0);
CPPUNIT_ASSERT_MESSAGE("Unexpected text.", aParaText.startsWith("Date: "));
- CPPUNIT_ASSERT_MESSAGE("Date field item not found.", pEditText->HasField(text::textfield::Type::DATE));
- CPPUNIT_ASSERT_MESSAGE("Date field not resolved with rDoc.", ScEditUtil::GetString(*pEditText, &rDoc).indexOf("/20") > 0);
- CPPUNIT_ASSERT_MESSAGE("Date field not resolved with NULL.", ScEditUtil::GetString(*pEditText, nullptr).indexOf("/20") > 0);
+ CPPUNIT_ASSERT_MESSAGE("Date field item not found.",
+ pEditText->HasField(text::textfield::Type::DATE));
+ CPPUNIT_ASSERT_MESSAGE("Date field not resolved with rDoc.",
+ ScEditUtil::GetString(*pEditText, &rDoc).indexOf("/20") > 0);
+ CPPUNIT_ASSERT_MESSAGE("Date field not resolved with NULL.",
+ ScEditUtil::GetString(*pEditText, nullptr).indexOf("/20") > 0);
// Cell with DocInfo title field item.
aPos.IncRow();
@@ -2164,8 +2236,10 @@ void ScFiltersTest::testRichTextContentODS()
CPPUNIT_ASSERT_EQUAL(static_cast<sal_Int32>(1), pEditText->GetParagraphCount());
aParaText = pEditText->GetText(0);
CPPUNIT_ASSERT_MESSAGE("Unexpected text.", aParaText.startsWith("Title: "));
- CPPUNIT_ASSERT_MESSAGE("DocInfo title field item not found.", pEditText->HasField(text::textfield::Type::DOCINFO_TITLE));
- CPPUNIT_ASSERT_EQUAL(OUString("Title: Test Document"), ScEditUtil::GetString(*pEditText, &rDoc));
+ CPPUNIT_ASSERT_MESSAGE("DocInfo title field item not found.",
+ pEditText->HasField(text::textfield::Type::DOCINFO_TITLE));
+ CPPUNIT_ASSERT_EQUAL(OUString("Title: Test Document"),
+ ScEditUtil::GetString(*pEditText, &rDoc));
CPPUNIT_ASSERT_EQUAL(OUString("Title: ?"), ScEditUtil::GetString(*pEditText, nullptr));
// Cell with sentence with both bold and italic sequences.
@@ -2195,8 +2269,10 @@ void ScFiltersTest::testRichTextContentODS()
bHasItalic = (rItem.GetPosture() == ITALIC_NORMAL);
}
}
- CPPUNIT_ASSERT_MESSAGE("This sentence is expected to have both bold and italic sequences.", bHasBold);
- CPPUNIT_ASSERT_MESSAGE("This sentence is expected to have both bold and italic sequences.", bHasItalic);
+ CPPUNIT_ASSERT_MESSAGE("This sentence is expected to have both bold and italic sequences.",
+ bHasBold);
+ CPPUNIT_ASSERT_MESSAGE("This sentence is expected to have both bold and italic sequences.",
+ bHasItalic);
}
// Cell with multi-line content with formatting applied.
@@ -2215,24 +2291,33 @@ void ScFiltersTest::testRichTextContentODS()
// first line is bold.
pEditText->GetCharAttribs(0, aAttribs);
{
- bool bHasBold = std::any_of(aAttribs.begin(), aAttribs.end(), [](const EECharAttrib& rAttrib) {
- return rAttrib.pAttr->Which() == EE_CHAR_WEIGHT &&
- static_cast<const SvxWeightItem&>(*rAttrib.pAttr).GetWeight() == WEIGHT_BOLD; });
+ bool bHasBold
+ = std::any_of(aAttribs.begin(), aAttribs.end(), [](const EECharAttrib& rAttrib) {
+ return rAttrib.pAttr->Which() == EE_CHAR_WEIGHT
+ && static_cast<const SvxWeightItem&>(*rAttrib.pAttr).GetWeight()
+ == WEIGHT_BOLD;
+ });
CPPUNIT_ASSERT_MESSAGE("First line should be bold.", bHasBold);
}
// second line is italic.
pEditText->GetCharAttribs(1, aAttribs);
- bool bHasItalic = std::any_of(aAttribs.begin(), aAttribs.end(), [](const EECharAttrib& rAttrib) {
- return rAttrib.pAttr->Which() == EE_CHAR_ITALIC &&
- static_cast<const SvxPostureItem&>(*rAttrib.pAttr).GetPosture() == ITALIC_NORMAL; });
+ bool bHasItalic
+ = std::any_of(aAttribs.begin(), aAttribs.end(), [](const EECharAttrib& rAttrib) {
+ return rAttrib.pAttr->Which() == EE_CHAR_ITALIC
+ && static_cast<const SvxPostureItem&>(*rAttrib.pAttr).GetPosture()
+ == ITALIC_NORMAL;
+ });
CPPUNIT_ASSERT_MESSAGE("Second line should be italic.", bHasItalic);
// third line is underlined.
pEditText->GetCharAttribs(2, aAttribs);
- bool bHasUnderline = std::any_of(aAttribs.begin(), aAttribs.end(), [](const EECharAttrib& rAttrib) {
- return rAttrib.pAttr->Which() == EE_CHAR_UNDERLINE &&
- static_cast<const SvxUnderlineItem&>(*rAttrib.pAttr).GetLineStyle() == LINESTYLE_SINGLE; });
+ bool bHasUnderline
+ = std::any_of(aAttribs.begin(), aAttribs.end(), [](const EECharAttrib& rAttrib) {
+ return rAttrib.pAttr->Which() == EE_CHAR_UNDERLINE
+ && static_cast<const SvxUnderlineItem&>(*rAttrib.pAttr).GetLineStyle()
+ == LINESTYLE_SINGLE;
+ });
CPPUNIT_ASSERT_MESSAGE("Second line should be underlined.", bHasUnderline);
// URL with formats applied. For now, we'll check whether or not the
@@ -2241,25 +2326,29 @@ void ScFiltersTest::testRichTextContentODS()
aPos.IncRow();
pEditText = rDoc.GetEditText(aPos);
CPPUNIT_ASSERT_MESSAGE("Failed to retrieve edit text object.", pEditText);
- CPPUNIT_ASSERT_MESSAGE("URL field item not found.", pEditText->HasField(text::textfield::Type::URL));
+ CPPUNIT_ASSERT_MESSAGE("URL field item not found.",
+ pEditText->HasField(text::textfield::Type::URL));
// Sheet name with formats applied.
aPos.IncRow();
pEditText = rDoc.GetEditText(aPos);
CPPUNIT_ASSERT_MESSAGE("Failed to retrieve edit text object.", pEditText);
- CPPUNIT_ASSERT_MESSAGE("Sheet name field item not found.", pEditText->HasField(text::textfield::Type::TABLE));
+ CPPUNIT_ASSERT_MESSAGE("Sheet name field item not found.",
+ pEditText->HasField(text::textfield::Type::TABLE));
// Date with formats applied.
aPos.IncRow();
pEditText = rDoc.GetEditText(aPos);
CPPUNIT_ASSERT_MESSAGE("Failed to retrieve edit text object.", pEditText);
- CPPUNIT_ASSERT_MESSAGE("Date field item not found.", pEditText->HasField(text::textfield::Type::DATE));
+ CPPUNIT_ASSERT_MESSAGE("Date field item not found.",
+ pEditText->HasField(text::textfield::Type::DATE));
// Document title with formats applied.
aPos.IncRow();
pEditText = rDoc.GetEditText(aPos);
CPPUNIT_ASSERT_MESSAGE("Failed to retrieve edit text object.", pEditText);
- CPPUNIT_ASSERT_MESSAGE("Date field item not found.", pEditText->HasField(text::textfield::Type::DOCINFO_TITLE));
+ CPPUNIT_ASSERT_MESSAGE("Date field item not found.",
+ pEditText->HasField(text::textfield::Type::DOCINFO_TITLE));
// URL for a file in the same directory. It should be converted into an absolute URL on import.
aPos.IncRow();
@@ -2267,14 +2356,17 @@ void ScFiltersTest::testRichTextContentODS()
CPPUNIT_ASSERT_MESSAGE("Failed to retrieve edit text object.", pEditText);
const SvxFieldData* pData = pEditText->GetFieldData(0, 0, text::textfield::Type::URL);
CPPUNIT_ASSERT_MESSAGE("Failed to get the URL data.", pData);
- CPPUNIT_ASSERT_EQUAL_MESSAGE("Failed to get the URL data.", text::textfield::Type::URL, pData->GetClassId());
+ CPPUNIT_ASSERT_EQUAL_MESSAGE("Failed to get the URL data.", text::textfield::Type::URL,
+ pData->GetClassId());
const SvxURLField* pURLData = static_cast<const SvxURLField*>(pData);
- CPPUNIT_ASSERT_MESSAGE("URL is not absolute with respect to the file system.", pURLData->GetURL().startsWith("file:///"));
+ CPPUNIT_ASSERT_MESSAGE("URL is not absolute with respect to the file system.",
+ pURLData->GetURL().startsWith("file:///"));
// Embedded spaces as <text:s text:c='4' />, normal text
aPos.IncRow();
CPPUNIT_ASSERT_EQUAL(CELLTYPE_STRING, rDoc.GetCellType(aPos));
- CPPUNIT_ASSERT_EQUAL(OUString("one two"), rDoc.GetString(aPos.Col(), aPos.Row(), aPos.Tab()));
+ CPPUNIT_ASSERT_EQUAL(OUString("one two"),
+ rDoc.GetString(aPos.Col(), aPos.Row(), aPos.Tab()));
// Leading space as <text:s />.
aPos.IncRow();
@@ -2293,8 +2385,9 @@ void ScFiltersTest::testRichTextContentODS()
pEditText->GetCharAttribs(0, aAttribs);
{
auto it = std::find_if(aAttribs.begin(), aAttribs.end(), [](const EECharAttrib& rAttrib) {
- return rAttrib.pAttr->Which() == EE_CHAR_WEIGHT &&
- static_cast<const SvxWeightItem&>(*rAttrib.pAttr).GetWeight() == WEIGHT_BOLD; });
+ return rAttrib.pAttr->Which() == EE_CHAR_WEIGHT
+ && static_cast<const SvxWeightItem&>(*rAttrib.pAttr).GetWeight() == WEIGHT_BOLD;
+ });
bool bHasBold = (it != aAttribs.end());
if (bHasBold)
{
@@ -2356,7 +2449,7 @@ void ScFiltersTest::testNewCondFormatODS()
ScDocument& rDoc = xDocSh->GetDocument();
OUString aCSVPath;
- createCSVPath( "new_cond_format_test.", aCSVPath );
+ createCSVPath("new_cond_format_test.", aCSVPath);
testCondFile(aCSVPath, &rDoc, 0);
xDocSh->DoClose();
@@ -2369,7 +2462,7 @@ void ScFiltersTest::testNewCondFormatXLSX()
ScDocument& rDoc = xDocSh->GetDocument();
OUString aCSVPath;
- createCSVPath( "new_cond_format_test.", aCSVPath );
+ createCSVPath("new_cond_format_test.", aCSVPath);
testCondFile(aCSVPath, &rDoc, 0);
xDocSh->DoClose();
@@ -2390,20 +2483,20 @@ void ScFiltersTest::testCondFormatImportCellIs()
CPPUNIT_ASSERT_EQUAL(ScFormatEntry::Type::ExtCondition, pEntry->GetType());
const ScCondFormatEntry* pCondition = static_cast<const ScCondFormatEntry*>(pEntry);
- CPPUNIT_ASSERT_EQUAL( ScConditionMode::Equal, pCondition->GetOperation());
+ CPPUNIT_ASSERT_EQUAL(ScConditionMode::Equal, pCondition->GetOperation());
OUString aStr = pCondition->GetExpression(ScAddress(0, 0, 0), 0);
- CPPUNIT_ASSERT_EQUAL( OUString("$Sheet2.$A$2"), aStr );
+ CPPUNIT_ASSERT_EQUAL(OUString("$Sheet2.$A$2"), aStr);
pEntry = pFormat->GetEntry(1);
CPPUNIT_ASSERT(pEntry);
CPPUNIT_ASSERT_EQUAL(ScFormatEntry::Type::ExtCondition, pEntry->GetType());
pCondition = static_cast<const ScCondFormatEntry*>(pEntry);
- CPPUNIT_ASSERT_EQUAL( ScConditionMode::Equal, pCondition->GetOperation());
+ CPPUNIT_ASSERT_EQUAL(ScConditionMode::Equal, pCondition->GetOperation());
aStr = pCondition->GetExpression(ScAddress(0, 0, 0), 0);
- CPPUNIT_ASSERT_EQUAL( OUString("$Sheet2.$A$1"), aStr );
+ CPPUNIT_ASSERT_EQUAL(OUString("$Sheet2.$A$1"), aStr);
xDocSh->DoClose();
}
@@ -2465,8 +2558,8 @@ void ScFiltersTest::testCondFormatThemeColor2XLSX()
xDocSh->DoClose();
}
-namespace {
-
+namespace
+{
void checkDatabarPositiveColor(const ScConditionalFormat* pFormat, const Color& rColor)
{
CPPUNIT_ASSERT(pFormat);
@@ -2478,7 +2571,6 @@ void checkDatabarPositiveColor(const ScConditionalFormat* pFormat, const Color&
CPPUNIT_ASSERT_EQUAL(rColor, pDataBarFormatData->maPositiveColor);
}
-
}
void ScFiltersTest::testCondFormatThemeColor3XLSX()
@@ -2517,8 +2609,8 @@ void ScFiltersTest::testCondFormatThemeColor3XLSX()
xDocSh->DoClose();
}
-namespace {
-
+namespace
+{
void testComplexIconSetsXLSX_Impl(const ScDocument& rDoc, SCCOL nCol, ScIconSetType eType)
{
ScConditionalFormat* pFormat = rDoc.GetCondFormat(nCol, 1, 0);
@@ -2531,7 +2623,8 @@ void testComplexIconSetsXLSX_Impl(const ScDocument& rDoc, SCCOL nCol, ScIconSetT
CPPUNIT_ASSERT_EQUAL(eType, pIconSet->GetIconSetData()->eIconSetType);
}
-void testCustomIconSetsXLSX_Impl(const ScDocument& rDoc, SCCOL nCol, SCROW nRow, ScIconSetType eType, sal_Int32 nIndex)
+void testCustomIconSetsXLSX_Impl(const ScDocument& rDoc, SCCOL nCol, SCROW nRow,
+ ScIconSetType eType, sal_Int32 nIndex)
{
ScConditionalFormat* pFormat = rDoc.GetCondFormat(nCol, 1, 1);
CPPUNIT_ASSERT(pFormat);
@@ -2550,7 +2643,6 @@ void testCustomIconSetsXLSX_Impl(const ScDocument& rDoc, SCCOL nCol, SCROW nRow,
CPPUNIT_ASSERT_EQUAL(eType, pInfo->eIconSetType);
}
}
-
}
void ScFiltersTest::testComplexIconSetsXLSX()
@@ -2592,7 +2684,7 @@ void ScFiltersTest::testTdf101104()
CPPUNIT_ASSERT_EQUAL(ScFormatEntry::Type::Iconset, pEntry->GetType());
const ScIconSetFormat* pIconSet = static_cast<const ScIconSetFormat*>(pEntry);
- for(size_t i = 1; i < 10; ++i)
+ for (size_t i = 1; i < 10; ++i)
{
std::unique_ptr<ScIconSetInfo> pInfo(pIconSet->GetIconSetInfo(ScAddress(1, i, 0)));
@@ -2600,7 +2692,6 @@ void ScFiltersTest::testTdf101104()
CPPUNIT_ASSERT(pInfo);
CPPUNIT_ASSERT_EQUAL(sal_Int32(2), pInfo->nIconIndex);
CPPUNIT_ASSERT_EQUAL(IconSet_3Arrows, pInfo->eIconSetType);
-
}
xDocSh->DoClose();
@@ -2621,38 +2712,36 @@ void ScFiltersTest::testTdf64401()
CPPUNIT_ASSERT_EQUAL(ScFormatEntry::Type::Iconset, pEntry->GetType());
const ScIconSetFormat* pIconSet = static_cast<const ScIconSetFormat*>(pEntry);
- for(size_t i = 0; i < 10; ++i)
+ for (size_t i = 0; i < 10; ++i)
{
sal_Int32 nIndex = 0;
- if ( i >= 7 ) // B5 = 8
+ if (i >= 7) // B5 = 8
nIndex = 2;
- else if ( i >= 3 ) // B4 = 4
+ else if (i >= 3) // B4 = 4
nIndex = 1;
std::unique_ptr<ScIconSetInfo> pInfo(pIconSet->GetIconSetInfo(ScAddress(0, i, 0)));
CPPUNIT_ASSERT(pInfo);
CPPUNIT_ASSERT_EQUAL(nIndex, pInfo->nIconIndex);
CPPUNIT_ASSERT_EQUAL(IconSet_3Arrows, pInfo->eIconSetType);
-
}
// Update values in B4 and B5
- rDoc.SetValue(ScAddress(1,3,0), 2.0);
- rDoc.SetValue(ScAddress(1,4,0), 9.0);
+ rDoc.SetValue(ScAddress(1, 3, 0), 2.0);
+ rDoc.SetValue(ScAddress(1, 4, 0), 9.0);
- for(size_t i = 0; i < 10; ++i)
+ for (size_t i = 0; i < 10; ++i)
{
sal_Int32 nIndex = 0;
- if ( i >= 8 ) // B5 = 9
+ if (i >= 8) // B5 = 9
nIndex = 2;
- else if ( i >= 1 ) // B4 = 2
+ else if (i >= 1) // B4 = 2
nIndex = 1;
std::unique_ptr<ScIconSetInfo> pInfo(pIconSet->GetIconSetInfo(ScAddress(0, i, 0)));
CPPUNIT_ASSERT(pInfo);
CPPUNIT_ASSERT_EQUAL(nIndex, pInfo->nIconIndex);
CPPUNIT_ASSERT_EQUAL(IconSet_3Arrows, pInfo->eIconSetType);
-
}
xDocSh->DoClose();
@@ -2691,12 +2780,12 @@ void ScFiltersTest::testColorScaleNumWithRefXLSX()
CPPUNIT_ASSERT_EQUAL(ScFormatEntry::Type::Colorscale, pEntry->GetType());
- const ScColorScaleFormat* pColorScale= dynamic_cast<const ScColorScaleFormat*>(pEntry);
+ const ScColorScaleFormat* pColorScale = dynamic_cast<const ScColorScaleFormat*>(pEntry);
CPPUNIT_ASSERT(pColorScale);
const ScColorScaleEntry* pColorScaleEntry = pColorScale->GetEntry(1);
CPPUNIT_ASSERT_EQUAL(OUString("=$A$1"),
- pColorScaleEntry->GetFormula(formula::FormulaGrammar::GRAM_NATIVE));
+ pColorScaleEntry->GetFormula(formula::FormulaGrammar::GRAM_NATIVE));
xDocSh->DoClose();
}
@@ -2723,12 +2812,12 @@ void ScFiltersTest::testOrcusODSStyleInterface()
const SfxPoolItem* pItem = nullptr;
CPPUNIT_ASSERT_MESSAGE("Style Name1 : Doesn't have Attribute background, but it should have.",
- pStyleSheet->GetItemSet().HasItem(ATTR_BACKGROUND, &pItem));
+ pStyleSheet->GetItemSet().HasItem(ATTR_BACKGROUND, &pItem));
const SvxBrushItem* pBackground = static_cast<const SvxBrushItem*>(pItem);
CPPUNIT_ASSERT_EQUAL(Color(254, 255, 204), pBackground->GetColor());
CPPUNIT_ASSERT_MESSAGE("Style Name1 : Doesn't have Attribute border, but it should have.",
- pStyleSheet->GetItemSet().HasItem(ATTR_BORDER, &pItem));
+ pStyleSheet->GetItemSet().HasItem(ATTR_BORDER, &pItem));
const SvxBoxItem* pBoxItem = static_cast<const SvxBoxItem*>(pItem);
CPPUNIT_ASSERT_EQUAL(Color(255, 204, 18), pBoxItem->GetLeft()->GetColor());
CPPUNIT_ASSERT_EQUAL(Color(255, 204, 18), pBoxItem->GetRight()->GetColor());
@@ -2744,11 +2833,11 @@ void ScFiltersTest::testOrcusODSStyleInterface()
ASSERT_DOUBLES_EQUAL_MESSAGE("Error with bottom width", 1, pBoxItem->GetBottom()->GetWidth());
CPPUNIT_ASSERT_MESSAGE("Style Name1 : Has Attribute Protection, but it shouldn't.",
- !pStyleSheet->GetItemSet().HasItem(ATTR_PROTECTION, &pItem));
+ !pStyleSheet->GetItemSet().HasItem(ATTR_PROTECTION, &pItem));
CPPUNIT_ASSERT_MESSAGE("Style Name1 : Has Attribute font, but it shouldn't.",
- !pStyleSheet->GetItemSet().HasItem(ATTR_FONT, &pItem));
+ !pStyleSheet->GetItemSet().HasItem(ATTR_FONT, &pItem));
CPPUNIT_ASSERT_MESSAGE("Style Name1 : Has Attribute number format, but it shouldn't.",
- !pStyleSheet->GetItemSet().HasItem(ATTR_VALUE_FORMAT, &pItem));
+ !pStyleSheet->GetItemSet().HasItem(ATTR_VALUE_FORMAT, &pItem));
/* Test for Style "Name2"
* Has 4 sided borders + Diagonal borders.
@@ -2756,7 +2845,7 @@ void ScFiltersTest::testOrcusODSStyleInterface()
pStyleSheet = pStyleSheetPool->FindCaseIns("Name2", SfxStyleFamily::Para);
CPPUNIT_ASSERT_MESSAGE("Style Name2 : Doesn't have Attribute background, but it should have.",
- pStyleSheet->GetItemSet().HasItem(ATTR_BORDER, &pItem));
+ pStyleSheet->GetItemSet().HasItem(ATTR_BORDER, &pItem));
pBoxItem = static_cast<const SvxBoxItem*>(pItem);
CPPUNIT_ASSERT_EQUAL(Color(0, 0, 0), pBoxItem->GetLeft()->GetColor());
@@ -2766,138 +2855,152 @@ void ScFiltersTest::testOrcusODSStyleInterface()
ASSERT_DOUBLES_EQUAL_MESSAGE("Error with left width", 0, pBoxItem->GetLeft()->GetWidth());
ASSERT_DOUBLES_EQUAL_MESSAGE("Error with right width", 14, pBoxItem->GetRight()->GetWidth());
- CPPUNIT_ASSERT_MESSAGE("Style Name2 : Doesn't have Attribute diagonal(tl-br) border, but it should have.",
+ CPPUNIT_ASSERT_MESSAGE(
+ "Style Name2 : Doesn't have Attribute diagonal(tl-br) border, but it should have.",
pStyleSheet->GetItemSet().HasItem(ATTR_BORDER_TLBR, &pItem));
- const SvxLineItem* pTLBR= static_cast<const SvxLineItem*>(pItem);
+ const SvxLineItem* pTLBR = static_cast<const SvxLineItem*>(pItem);
CPPUNIT_ASSERT_EQUAL(Color(18, 0, 0), pTLBR->GetLine()->GetColor());
CPPUNIT_ASSERT_EQUAL(SvxBorderLineStyle::DASH_DOT, pTLBR->GetLine()->GetBorderLineStyle());
- ASSERT_DOUBLES_EQUAL_MESSAGE("Error with diagonal tl-br width", 14, pTLBR->GetLine()->GetWidth());
+ ASSERT_DOUBLES_EQUAL_MESSAGE("Error with diagonal tl-br width", 14,
+ pTLBR->GetLine()->GetWidth());
- CPPUNIT_ASSERT_MESSAGE("Style Name2 : Doesn't have Attribute diagonal(bl-tr) border, but it should have.",
+ CPPUNIT_ASSERT_MESSAGE(
+ "Style Name2 : Doesn't have Attribute diagonal(bl-tr) border, but it should have.",
pStyleSheet->GetItemSet().HasItem(ATTR_BORDER_BLTR, &pItem));
- const SvxLineItem* pBLTR= static_cast<const SvxLineItem*>(pItem);
+ const SvxLineItem* pBLTR = static_cast<const SvxLineItem*>(pItem);
CPPUNIT_ASSERT_EQUAL(Color(255, 204, 238), pBLTR->GetLine()->GetColor());
CPPUNIT_ASSERT_EQUAL(SvxBorderLineStyle::DASHED, pBLTR->GetLine()->GetBorderLineStyle());
- ASSERT_DOUBLES_EQUAL_MESSAGE("Error with diagonal tl-br width", 34, pBLTR->GetLine()->GetWidth());
+ ASSERT_DOUBLES_EQUAL_MESSAGE("Error with diagonal tl-br width", 34,
+ pBLTR->GetLine()->GetWidth());
CPPUNIT_ASSERT_MESSAGE("Style Name2 : Has Attribute background, but it shouldn't.",
- !pStyleSheet->GetItemSet().HasItem(ATTR_BACKGROUND, &pItem));
+ !pStyleSheet->GetItemSet().HasItem(ATTR_BACKGROUND, &pItem));
CPPUNIT_ASSERT_MESSAGE("Style Name2 : Has Attribute font, but it shouldn't.",
- !pStyleSheet->GetItemSet().HasItem(ATTR_FONT, &pItem));
+ !pStyleSheet->GetItemSet().HasItem(ATTR_FONT, &pItem));
CPPUNIT_ASSERT_MESSAGE("Style Name2 : Has Attribute number format, but it shouldn't.",
- !pStyleSheet->GetItemSet().HasItem(ATTR_VALUE_FORMAT, &pItem));
+ !pStyleSheet->GetItemSet().HasItem(ATTR_VALUE_FORMAT, &pItem));
/* Test for Style "Name3"
* Hidden, protected and content is printed.
*/
pStyleSheet = pStyleSheetPool->FindCaseIns("Name3", SfxStyleFamily::Para);
CPPUNIT_ASSERT_MESSAGE("Style Name3 : Doesn't have Attribute Protection, but it should have.",
- pStyleSheet->GetItemSet().HasItem(ATTR_PROTECTION, &pItem));
+ pStyleSheet->GetItemSet().HasItem(ATTR_PROTECTION, &pItem));
CPPUNIT_ASSERT_MESSAGE("Style Name 3 : Error with Protection attribute.",
- bool(ScProtectionAttr(true, false, true, true) == *pItem));
+ bool(ScProtectionAttr(true, false, true, true) == *pItem));
/* Test for Style "Name4"
* Hidden, protected and content is printed.
*/
pStyleSheet = pStyleSheetPool->FindCaseIns("Name4", SfxStyleFamily::Para);
CPPUNIT_ASSERT_MESSAGE("Style Name4 : Doesn't have Attribute Protection, but it should have.",
- pStyleSheet->GetItemSet().HasItem(ATTR_PROTECTION, &pItem));
+ pStyleSheet->GetItemSet().HasItem(ATTR_PROTECTION, &pItem));
CPPUNIT_ASSERT_MESSAGE("Style Name 4 : Error with Protection attribute.",
- bool(ScProtectionAttr(true, true, false, false) == *pItem));
+ bool(ScProtectionAttr(true, true, false, false) == *pItem));
/* Test for Style "Name5"
* Hidden, protected and content is printed.
*/
pStyleSheet = pStyleSheetPool->FindCaseIns("Name5", SfxStyleFamily::Para);
CPPUNIT_ASSERT_MESSAGE("Style Name5 : Doesn't have Attribute Protection, but it should have.",
- pStyleSheet->GetItemSet().HasItem(ATTR_PROTECTION, &pItem));
+ pStyleSheet->GetItemSet().HasItem(ATTR_PROTECTION, &pItem));
CPPUNIT_ASSERT_MESSAGE("Style Name5 : Error with Protection attribute.",
- bool(ScProtectionAttr(false, false, false, true) == *pItem));
+ bool(ScProtectionAttr(false, false, false, true) == *pItem));
CPPUNIT_ASSERT_MESSAGE("Style Name5 : Has Attribute Border, but it shouldn't.",
- !pStyleSheet->GetItemSet().HasItem(ATTR_BORDER, &pItem));
+ !pStyleSheet->GetItemSet().HasItem(ATTR_BORDER, &pItem));
CPPUNIT_ASSERT_MESSAGE("Style Name5 : Has Attribute background, but it shouldn't.",
- !pStyleSheet->GetItemSet().HasItem(ATTR_BACKGROUND, &pItem));
+ !pStyleSheet->GetItemSet().HasItem(ATTR_BACKGROUND, &pItem));
CPPUNIT_ASSERT_MESSAGE("Style Name5 : Has Attribute font, but it shouldn't.",
- !pStyleSheet->GetItemSet().HasItem(ATTR_FONT, &pItem));
+ !pStyleSheet->GetItemSet().HasItem(ATTR_FONT, &pItem));
CPPUNIT_ASSERT_MESSAGE("Style Name5 : Has Attribute number format, but it shouldn't.",
- !pStyleSheet->GetItemSet().HasItem(ATTR_VALUE_FORMAT, &pItem));
+ !pStyleSheet->GetItemSet().HasItem(ATTR_VALUE_FORMAT, &pItem));
/* Test for Style "Name6"
* Has Font name, posture, weight, color, height
*/
pStyleSheet = pStyleSheetPool->FindCaseIns("Name6", SfxStyleFamily::Para);
CPPUNIT_ASSERT_MESSAGE("Style Name6 : Doesn't have Attribute Font, but it should have.",
- pStyleSheet->GetItemSet().HasItem(ATTR_FONT, &pItem));
+ pStyleSheet->GetItemSet().HasItem(ATTR_FONT, &pItem));
- const SvxFontItem* pFontItem= static_cast<const SvxFontItem*>(pItem);
- CPPUNIT_ASSERT_EQUAL_MESSAGE("Style Name6 :Error with Font name", OUString("Liberation Sans"), pFontItem->GetStyleName());
+ const SvxFontItem* pFontItem = static_cast<const SvxFontItem*>(pItem);
+ CPPUNIT_ASSERT_EQUAL_MESSAGE("Style Name6 :Error with Font name", OUString("Liberation Sans"),
+ pFontItem->GetStyleName());
CPPUNIT_ASSERT_MESSAGE("Style Name6 : Doesn't have Attribute Font Height, but it should have.",
- pStyleSheet->GetItemSet().HasItem(ATTR_FONT_HEIGHT, &pItem));
+ pStyleSheet->GetItemSet().HasItem(ATTR_FONT_HEIGHT, &pItem));
- const SvxFontHeightItem* pFontHeightItem= static_cast<const SvxFontHeightItem*>(pItem);
- CPPUNIT_ASSERT_EQUAL_MESSAGE("Style Name6 :Error with Font Height", static_cast<sal_uInt32>(480), pFontHeightItem->GetHeight());
+ const SvxFontHeightItem* pFontHeightItem = static_cast<const SvxFontHeightItem*>(pItem);
+ CPPUNIT_ASSERT_EQUAL_MESSAGE("Style Name6 :Error with Font Height",
+ static_cast<sal_uInt32>(480), pFontHeightItem->GetHeight());
CPPUNIT_ASSERT_MESSAGE("Style Name6 : Doesn't have Attribute Font Posture, but it should have.",
- pStyleSheet->GetItemSet().HasItem(ATTR_FONT_POSTURE, &pItem));
+ pStyleSheet->GetItemSet().HasItem(ATTR_FONT_POSTURE, &pItem));
- const SvxPostureItem* pFontPostureItem= static_cast<const SvxPostureItem*>(pItem);
- CPPUNIT_ASSERT_EQUAL_MESSAGE("Style Name6 :Error with Font Posture", ITALIC_NORMAL, pFontPostureItem->GetPosture());
+ const SvxPostureItem* pFontPostureItem = static_cast<const SvxPostureItem*>(pItem);
+ CPPUNIT_ASSERT_EQUAL_MESSAGE("Style Name6 :Error with Font Posture", ITALIC_NORMAL,
+ pFontPostureItem->GetPosture());
CPPUNIT_ASSERT_MESSAGE("Style Name6 : Doesn't have Attribute Font Weight, but it should have.",
- pStyleSheet->GetItemSet().HasItem(ATTR_FONT_WEIGHT, &pItem));
+ pStyleSheet->GetItemSet().HasItem(ATTR_FONT_WEIGHT, &pItem));
- const SvxWeightItem* pFontWeightItem= static_cast<const SvxWeightItem*>(pItem);
- CPPUNIT_ASSERT_EQUAL_MESSAGE("Style Name6 :Error with Font Weight", WEIGHT_BOLD, pFontWeightItem->GetWeight());
+ const SvxWeightItem* pFontWeightItem = static_cast<const SvxWeightItem*>(pItem);
+ CPPUNIT_ASSERT_EQUAL_MESSAGE("Style Name6 :Error with Font Weight", WEIGHT_BOLD,
+ pFontWeightItem->GetWeight());
CPPUNIT_ASSERT_MESSAGE("Style Name6 : Doesn't have Attribute Font Color, but it should have.",
- pStyleSheet->GetItemSet().HasItem(ATTR_FONT_COLOR, &pItem));
+ pStyleSheet->GetItemSet().HasItem(ATTR_FONT_COLOR, &pItem));
- const SvxColorItem* pFontColorItem= static_cast<const SvxColorItem*>(pItem);
- CPPUNIT_ASSERT_EQUAL_MESSAGE("Style Name6 :Error with Font Color", Color(128, 128, 128), pFontColorItem->GetValue());
+ const SvxColorItem* pFontColorItem = static_cast<const SvxColorItem*>(pItem);
+ CPPUNIT_ASSERT_EQUAL_MESSAGE("Style Name6 :Error with Font Color", Color(128, 128, 128),
+ pFontColorItem->GetValue());
CPPUNIT_ASSERT_MESSAGE("Style Name6 : Doesn't have Attribute Underline, but it should have.",
- pStyleSheet->GetItemSet().HasItem(ATTR_FONT_UNDERLINE, &pItem));
+ pStyleSheet->GetItemSet().HasItem(ATTR_FONT_UNDERLINE, &pItem));
- const SvxUnderlineItem* pUnderlineItem= static_cast<const SvxUnderlineItem*>(pItem);
- CPPUNIT_ASSERT_EQUAL_MESSAGE("Style Name6 :Error with Font Underline Style", LINESTYLE_SINGLE, pUnderlineItem->GetLineStyle());
- CPPUNIT_ASSERT_EQUAL_MESSAGE("Style Name6 :Error with Font Underline Color", Color(128, 128, 128), pUnderlineItem->GetColor());
+ const SvxUnderlineItem* pUnderlineItem = static_cast<const SvxUnderlineItem*>(pItem);
+ CPPUNIT_ASSERT_EQUAL_MESSAGE("Style Name6 :Error with Font Underline Style", LINESTYLE_SINGLE,
+ pUnderlineItem->GetLineStyle());
+ CPPUNIT_ASSERT_EQUAL_MESSAGE("Style Name6 :Error with Font Underline Color",
+ Color(128, 128, 128), pUnderlineItem->GetColor());
/* Test for Style Name "7"
* Has strikethrough single
*/
pStyleSheet = pStyleSheetPool->FindCaseIns("Name7", SfxStyleFamily::Para);
CPPUNIT_ASSERT_MESSAGE("Style Name7 : Doesn't have Attribute Strikeout, but it should have.",
- pStyleSheet->GetItemSet().HasItem(ATTR_FONT_CROSSEDOUT, &pItem));
+ pStyleSheet->GetItemSet().HasItem(ATTR_FONT_CROSSEDOUT, &pItem));
const SvxCrossedOutItem* pCrossedOutItem = static_cast<const SvxCrossedOutItem*>(pItem);
- CPPUNIT_ASSERT_EQUAL_MESSAGE("Style Name7 :Error with Strikeout", STRIKEOUT_SINGLE, pCrossedOutItem->GetStrikeout());
+ CPPUNIT_ASSERT_EQUAL_MESSAGE("Style Name7 :Error with Strikeout", STRIKEOUT_SINGLE,
+ pCrossedOutItem->GetStrikeout());
/* Test for Style Name "8"
* Has strikethrough bold
*/
pStyleSheet = pStyleSheetPool->FindCaseIns("Name8", SfxStyleFamily::Para);
CPPUNIT_ASSERT_MESSAGE("Style Name8 : Doesn't have Attribute Strikeout, but it should have.",
- pStyleSheet->GetItemSet().HasItem(ATTR_FONT_CROSSEDOUT, &pItem));
+ pStyleSheet->GetItemSet().HasItem(ATTR_FONT_CROSSEDOUT, &pItem));
pCrossedOutItem = static_cast<const SvxCrossedOutItem*>(pItem);
- CPPUNIT_ASSERT_EQUAL_MESSAGE("Style Name7 :Error with Strikeout", STRIKEOUT_BOLD, pCrossedOutItem->GetStrikeout());
+ CPPUNIT_ASSERT_EQUAL_MESSAGE("Style Name7 :Error with Strikeout", STRIKEOUT_BOLD,
+ pCrossedOutItem->GetStrikeout());
/* Test for Style Name "9"
* Has strikethrough slash
*/
pStyleSheet = pStyleSheetPool->FindCaseIns("Name9", SfxStyleFamily::Para);
CPPUNIT_ASSERT_MESSAGE("Style Name9 : Doesn't have Attribute Strikeout, but it should have.",
- pStyleSheet->GetItemSet().HasItem(ATTR_FONT_CROSSEDOUT, &pItem));
+ pStyleSheet->GetItemSet().HasItem(ATTR_FONT_CROSSEDOUT, &pItem));
pCrossedOutItem = static_cast<const SvxCrossedOutItem*>(pItem);
- CPPUNIT_ASSERT_EQUAL_MESSAGE("Style Name9 :Error with Strikeout", STRIKEOUT_SLASH, pCrossedOutItem->GetStrikeout());
+ CPPUNIT_ASSERT_EQUAL_MESSAGE("Style Name9 :Error with Strikeout", STRIKEOUT_SLASH,
+ pCrossedOutItem->GetStrikeout());
/* Test for Style Name "10"
* Has ver align, and hor align
@@ -2905,20 +3008,21 @@ void ScFiltersTest::testOrcusODSStyleInterface()
pStyleSheet = pStyleSheetPool->FindCaseIns("Name10", SfxStyleFamily::Para);
CPPUNIT_ASSERT_MESSAGE("Style Name10 : Doesn't have Attribute hor justify, but it should have.",
- pStyleSheet->GetItemSet().HasItem(ATTR_HOR_JUSTIFY, &pItem));
+ pStyleSheet->GetItemSet().HasItem(ATTR_HOR_JUSTIFY, &pItem));
const SvxHorJustifyItem* pHorJustify = static_cast<const SvxHorJustifyItem*>(pItem);
- CPPUNIT_ASSERT_EQUAL_MESSAGE("Style Name10 :Error with hor justify", SvxCellHorJustify::Right, pHorJustify->GetValue());
+ CPPUNIT_ASSERT_EQUAL_MESSAGE("Style Name10 :Error with hor justify", SvxCellHorJustify::Right,
+ pHorJustify->GetValue());
pStyleSheet = pStyleSheetPool->FindCaseIns("Name10", SfxStyleFamily::Para);
CPPUNIT_ASSERT_MESSAGE("Style Name10 : Doesn't have Attribute ver justify, but it should have.",
- pStyleSheet->GetItemSet().HasItem(ATTR_VER_JUSTIFY, &pItem));
+ pStyleSheet->GetItemSet().HasItem(ATTR_VER_JUSTIFY, &pItem));
const SvxVerJustifyItem* pVerJustify = static_cast<const SvxVerJustifyItem*>(pItem);
- CPPUNIT_ASSERT_EQUAL_MESSAGE("Style Name10 :Error with ver justify", SvxCellVerJustify::Center, pVerJustify->GetValue());
+ CPPUNIT_ASSERT_EQUAL_MESSAGE("Style Name10 :Error with ver justify", SvxCellVerJustify::Center,
+ pVerJustify->GetValue());
- auto checkFontWeight = [pStyleSheetPool](const OUString& rName, FontWeight eExpected)
- {
+ auto checkFontWeight = [pStyleSheetPool](const OUString& rName, FontWeight eExpected) {
ScStyleSheet* pStyle = pStyleSheetPool->FindCaseIns(rName, SfxStyleFamily::Para);
CPPUNIT_ASSERT(pStyle);
@@ -2934,8 +3038,8 @@ void ScFiltersTest::testOrcusODSStyleInterface()
FontWeight eActual = pWeight->GetWeight();
{
std::ostringstream os;
- os << "Wrong font weight value for style named '" << rName << "': expected="
- << eExpected << "; actual=" << eActual;
+ os << "Wrong font weight value for style named '" << rName
+ << "': expected=" << eExpected << "; actual=" << eActual;
CPPUNIT_ASSERT_EQUAL_MESSAGE(os.str(), eExpected, eActual);
}
};
@@ -2953,7 +3057,7 @@ void ScFiltersTest::testLiteralInFormulaXLS()
ScDocument& rDoc = xDocSh->GetDocument();
rDoc.CalcAll();
- CPPUNIT_ASSERT_EQUAL(2.0, rDoc.GetValue(ScAddress(0,0,0)));
+ CPPUNIT_ASSERT_EQUAL(2.0, rDoc.GetValue(ScAddress(0, 0, 0)));
xDocSh->DoClose();
}
@@ -2965,10 +3069,10 @@ void ScFiltersTest::testFormulaDependency()
ScDocument& rDoc = xDocSh->GetDocument();
// check if formula in A1 changes value
- double nVal = rDoc.GetValue(0,0,0);
+ double nVal = rDoc.GetValue(0, 0, 0);
CPPUNIT_ASSERT_DOUBLES_EQUAL(nVal, 1.0, 1e-10);
- rDoc.SetValue(0,1,0, 0.0);
- nVal = rDoc.GetValue(0,0,0);
+ rDoc.SetValue(0, 1, 0, 0.0);
+ nVal = rDoc.GetValue(0, 0, 0);
CPPUNIT_ASSERT_DOUBLES_EQUAL(nVal, 2.0, 1e-10);
// check that the number format is implicitly inherited
@@ -2994,7 +3098,8 @@ void ScFiltersTest::testTdf151046()
xDocSh->DoClose();
}
-void ScFiltersTest::testImportCrashes() {
+void ScFiltersTest::testImportCrashes()
+{
testImportCrash(u"tdf149752.", FORMAT_ODS);
testImportCrash(u"tdf122643.", FORMAT_ODS);
testImportCrash(u"tdf132278.", FORMAT_ODS);
@@ -3079,7 +3184,7 @@ void ScFiltersTest::testTdf144758_DBDataDefaultOrientation()
}
ScFiltersTest::ScFiltersTest()
- : ScBootstrapFixture( "sc/qa/unit/data" )
+ : ScBootstrapFixture("sc/qa/unit/data")
{
}
@@ -3088,4 +3193,3 @@ CPPUNIT_TEST_SUITE_REGISTRATION(ScFiltersTest);
CPPUNIT_PLUGIN_IMPLEMENT();
/* vim:set shiftwidth=4 softtabstop=4 expandtab: */
-
diff --git a/solenv/clang-format/excludelist b/solenv/clang-format/excludelist
index aaa00004f80b..a1da3970e967 100644
--- a/solenv/clang-format/excludelist
+++ b/solenv/clang-format/excludelist
@@ -8040,7 +8040,6 @@ sc/qa/unit/parallelism.cxx
sc/qa/unit/range.cxx
sc/qa/unit/rangelst_test.cxx
sc/qa/unit/screenshots/screenshots.cxx
-sc/qa/unit/subsequent_filters_test.cxx
sc/qa/unit/tiledrendering/tiledrendering.cxx
sc/qa/unit/ucalc.cxx
sc/qa/unit/ucalc_condformat.cxx