summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
Diffstat
-rw-r--r--chart2/qa/extras/chart2export.cxx14
-rw-r--r--chart2/qa/extras/charttest.hxx43
-rw-r--r--include/unotest/bootstrapfixturebase.hxx1
-rw-r--r--include/unotest/macros_test.hxx2
-rw-r--r--unotest/source/cpp/bootstrapfixturebase.cxx5
-rw-r--r--unotest/source/cpp/macros_test.cxx6
6 files changed, 35 insertions, 36 deletions
diff --git a/chart2/qa/extras/chart2export.cxx b/chart2/qa/extras/chart2export.cxx
index d122d8c0f147..276715874979 100644
--- a/chart2/qa/extras/chart2export.cxx
+++ b/chart2/qa/extras/chart2export.cxx
@@ -49,7 +49,7 @@ protected:
* xml stream, and asserting an XPath expression. This method returns the
* xml stream, so that you can do the asserting.
*/
- xmlDocPtr parseExport(const OUString& rStreamName);
+ xmlDocPtr parseExport(const OUString& rStreamName, const OUString& rFilterFormat);
/**
* Helper method to return nodes represented by rXPath.
@@ -83,11 +83,9 @@ void Chart2ExportTest::test()
reload("Calc Office Open XML");
}
-xmlDocPtr Chart2ExportTest::parseExport(const OUString& rStreamName)
+xmlDocPtr Chart2ExportTest::parseExport(const OUString& rStreamName, const OUString& rFilterFormat)
{
- if (!m_bExported)
- return 0;
- utl::TempFile aTempFile = reload("Office Open XML Text");
+ utl::TempFile aTempFile = reload(rFilterFormat);
// Read the XML stream we're interested in.
uno::Reference<packages::zip::XZipFileAccess2> xNameAccess = packages::zip::ZipFileAccess::createWithURL(comphelper::getComponentContext(m_xSFactory), aTempFile.GetURL());
@@ -374,7 +372,7 @@ void Chart2ExportTest::testStockChart()
*/
load("/chart2/qa/extras/data/docx/", "testStockChart.docx");
- xmlDocPtr pXmlDoc = parseExport("word/charts/chart1.xml");
+ xmlDocPtr pXmlDoc = parseExport("word/charts/chart1.xml", "Office Open XML Text");
if (!pXmlDoc)
return;
@@ -386,9 +384,9 @@ void Chart2ExportTest::testStockChart()
void Chart2ExportTest::testBarChart()
{
load("/chart2/qa/extras/data/docx/", "testBarChart.docx");
- xmlDocPtr pXmlDoc = parseExport("word/charts/chart1.xml");
+ xmlDocPtr pXmlDoc = parseExport("word/charts/chart1.xml", "Office Open XML Text");
if (!pXmlDoc)
- return;
+ return;
assertXPath(pXmlDoc, "/c:chartSpace/c:chart/c:plotArea/c:barChart/c:barDir", "val", "col");
}
diff --git a/chart2/qa/extras/charttest.hxx b/chart2/qa/extras/charttest.hxx
index fdabb9c0b654..b50f4fd6cd0f 100644
--- a/chart2/qa/extras/charttest.hxx
+++ b/chart2/qa/extras/charttest.hxx
@@ -50,15 +50,10 @@ using namespace com::sun::star::uno;
class ChartTest : public test::BootstrapFixture, public unotest::MacrosTest
{
public:
- ChartTest()
- : mServiceName(),
- m_bExported(false)
- {
- }
- void load( const char* pDir, const char* pName );
- utl::TempFile reload( const OUString& rFilterName );
+ void load( const OUString& rDir, const OUString& rFileName );
+ utl::TempFile reload( const OUString& rFileName );
uno::Sequence < OUString > getImpressChartColumnDescriptions( const char* pDir, const char* pName );
- OUString getFileExtension( const char* pName );
+ OUString getFileExtension( const OUString& rFileName );
void loadDocx(const char* pDir, const char* pName);
utl::TempFile reloadDocx();
@@ -67,29 +62,29 @@ public:
protected:
Reference< lang::XComponent > mxComponent;
- const char* mServiceName;
- bool m_bExported; ///< Does m_aTempFile already contain something useful?
-
+ OUString maServiceName;
};
-OUString ChartTest::getFileExtension( const char* pName )
+
+OUString ChartTest::getFileExtension( const OUString& aFileName )
{
- OUString fileName = OUString::createFromAscii(pName);
- sal_Int32 dotLocation = fileName.lastIndexOf(L'.');
- return fileName.copy(dotLocation);
+ sal_Int32 nDotLocation = aFileName.lastIndexOf('.');
+ CPPUNIT_ASSERT(nDotLocation != -1);
+ return aFileName.copy(nDotLocation);
}
-void ChartTest::load( const char* pDir, const char* pName )
+
+void ChartTest::load( const OUString& aDir, const OUString& aName )
{
- OUString extension = getFileExtension(pName);
- if(extension.equals("ods"))
+ OUString extension = getFileExtension(aName);
+ if(extension == "ods")
{
- mServiceName = "com.sun.star.sheet.SpreadsheetDocument";
+ maServiceName = "com.sun.star.sheet.SpreadsheetDocument";
}
- else if(extension.equals("docx"))
+ else if(extension == "docx")
{
- mServiceName = "com.sun.star.text.TextDocument";
-
+ maServiceName = "com.sun.star.text.TextDocument";
}
- mxComponent = loadFromDesktop(getURLFromSrc(pDir) + OUString::createFromAscii(pName), mServiceName);
+
+ mxComponent = loadFromDesktop(getURLFromSrc(aDir) + aName, maServiceName);
CPPUNIT_ASSERT(mxComponent.is());
}
@@ -103,7 +98,7 @@ utl::TempFile ChartTest::reload(const OUString& rFilterName)
aTempFile.EnableKillingFile();
xStorable->storeToURL(aTempFile.GetURL(), aArgs);
mxComponent->dispose();
- mxComponent = loadFromDesktop(aTempFile.GetURL(), mServiceName);
+ mxComponent = loadFromDesktop(aTempFile.GetURL(), maServiceName);
std::cout << aTempFile.GetURL();
CPPUNIT_ASSERT(mxComponent.is());
return aTempFile;
diff --git a/include/unotest/bootstrapfixturebase.hxx b/include/unotest/bootstrapfixturebase.hxx
index de6a6e73e1a5..c447f4fa626e 100644
--- a/include/unotest/bootstrapfixturebase.hxx
+++ b/include/unotest/bootstrapfixturebase.hxx
@@ -57,6 +57,7 @@ public:
// return a URL to a given c-str path from the source directory
OUString getURLFromSrc( const char *pPath );
+ OUString getURLFromSrc( const OUString& rPath );
// return a Path to a given c-str path from the source directory
OUString getPathFromSrc( const char *pPath );
diff --git a/include/unotest/macros_test.hxx b/include/unotest/macros_test.hxx
index f710645b10ba..d995246f5b93 100644
--- a/include/unotest/macros_test.hxx
+++ b/include/unotest/macros_test.hxx
@@ -27,7 +27,7 @@ namespace unotest {
class OOO_DLLPUBLIC_UNOTEST MacrosTest
{
public:
- com::sun::star::uno::Reference< com::sun::star::lang::XComponent > loadFromDesktop(const OUString& rURL, const char* pDocService = NULL);
+ com::sun::star::uno::Reference< com::sun::star::lang::XComponent > loadFromDesktop(const OUString& rURL, const OUString& rDocService = OUString() );
protected:
com::sun::star::uno::Reference< com::sun::star::frame::XDesktop2> mxDesktop;
diff --git a/unotest/source/cpp/bootstrapfixturebase.cxx b/unotest/source/cpp/bootstrapfixturebase.cxx
index fce13e364b97..988d9bd3172d 100644
--- a/unotest/source/cpp/bootstrapfixturebase.cxx
+++ b/unotest/source/cpp/bootstrapfixturebase.cxx
@@ -68,6 +68,11 @@ OUString test::BootstrapFixtureBase::getURLFromSrc( const char *pPath )
return m_aSrcRootURL + OUString::createFromAscii( pPath );
}
+OUString test::BootstrapFixtureBase::getURLFromSrc( const OUString& rPath )
+{
+ return m_aSrcRootURL + rPath;
+}
+
OUString test::BootstrapFixtureBase::getPathFromSrc( const char *pPath )
{
return m_aSrcRootPath + OUString::createFromAscii( pPath );
diff --git a/unotest/source/cpp/macros_test.cxx b/unotest/source/cpp/macros_test.cxx
index e0376ccb9bd1..9305a68b5a89 100644
--- a/unotest/source/cpp/macros_test.cxx
+++ b/unotest/source/cpp/macros_test.cxx
@@ -19,7 +19,7 @@ using namespace com::sun::star;
namespace unotest {
-uno::Reference< com::sun::star::lang::XComponent > MacrosTest::loadFromDesktop(const OUString& rURL, const char* pDocService)
+uno::Reference< com::sun::star::lang::XComponent > MacrosTest::loadFromDesktop(const OUString& rURL, const OUString& rDocService)
{
CPPUNIT_ASSERT_MESSAGE("no desktop", mxDesktop.is());
uno::Reference< com::sun::star::frame::XComponentLoader> xLoader = uno::Reference< com::sun::star::frame::XComponentLoader >( mxDesktop, uno::UNO_QUERY );
@@ -31,12 +31,12 @@ uno::Reference< com::sun::star::lang::XComponent > MacrosTest::loadFromDesktop(c
com::sun::star::document::MacroExecMode::ALWAYS_EXECUTE_NO_WARN;
args[0].State = com::sun::star::beans::PropertyState_DIRECT_VALUE;
- if (pDocService)
+ if (!rDocService.isEmpty())
{
args.realloc(2);
args[1].Name = "DocumentService";
args[1].Handle = -1;
- args[1].Value <<= OUString::createFromAscii(pDocService);
+ args[1].Value <<= rDocService;
args[1].State = com::sun::star::beans::PropertyState_DIRECT_VALUE;
}
generated by cgit (git 2.34.1) at 2025-02-16 07:54:33 +0000