Author: liuzhe Date: Thu Sep 20 07:17:44 2012 New Revision: 1387890 URL: http://svn.apache.org/viewvc?rev=1387890&view=rev Log: Add parameter into pvt.uno.Conversion to control how to kill openoffice.
Modified: incubator/ooo/trunk/test/testcommon/source/org/openoffice/test/OpenOffice.java incubator/ooo/trunk/test/testcommon/source/org/openoffice/test/common/FileProvider.java incubator/ooo/trunk/test/testcommon/source/org/openoffice/test/common/Testspace.java incubator/ooo/trunk/test/testuno/source/pvt/uno/Conversion.java Modified: incubator/ooo/trunk/test/testcommon/source/org/openoffice/test/OpenOffice.java URL: http://svn.apache.org/viewvc/incubator/ooo/trunk/test/testcommon/source/org/openoffice/test/OpenOffice.java?rev=1387890&r1=1387889&r2=1387890&view=diff ============================================================================== --- incubator/ooo/trunk/test/testcommon/source/org/openoffice/test/OpenOffice.java (original) +++ incubator/ooo/trunk/test/testcommon/source/org/openoffice/test/OpenOffice.java Thu Sep 20 07:17:44 2012 @@ -74,7 +74,7 @@ public class OpenOffice { private Process process = null; - private String unoUrl = "socket,host=localhost,port=2002;urp"; + private String unoUrl = "socket,host=127.0.0.1,port=2002;urp"; private Properties versionProps = null; Modified: incubator/ooo/trunk/test/testcommon/source/org/openoffice/test/common/FileProvider.java URL: http://svn.apache.org/viewvc/incubator/ooo/trunk/test/testcommon/source/org/openoffice/test/common/FileProvider.java?rev=1387890&r1=1387889&r2=1387890&view=diff ============================================================================== --- incubator/ooo/trunk/test/testcommon/source/org/openoffice/test/common/FileProvider.java (original) +++ incubator/ooo/trunk/test/testcommon/source/org/openoffice/test/common/FileProvider.java Thu Sep 20 07:17:44 2012 @@ -176,7 +176,7 @@ public class FileProvider extends Suite for (int i = 0; i < list.size(); i++) { Object[] t = list.get(i); for (int j = 0; j < repeat; j++) { - TestClassRunnerForParameters runner = new TestClassRunnerForParameters(getTestClass().getJavaClass(), t, i); + TestClassRunnerForParameters runner = new TestClassRunnerForParameters(getTestClass().getJavaClass(), t, i * repeat + j); runners.add(runner); } } Modified: incubator/ooo/trunk/test/testcommon/source/org/openoffice/test/common/Testspace.java URL: http://svn.apache.org/viewvc/incubator/ooo/trunk/test/testcommon/source/org/openoffice/test/common/Testspace.java?rev=1387890&r1=1387889&r2=1387890&view=diff ============================================================================== --- incubator/ooo/trunk/test/testcommon/source/org/openoffice/test/common/Testspace.java (original) +++ incubator/ooo/trunk/test/testcommon/source/org/openoffice/test/common/Testspace.java Thu Sep 20 07:17:44 2012 @@ -24,6 +24,7 @@ package org.openoffice.test.common; import java.io.File; +import java.io.IOException; import java.io.InputStream; import java.net.URL; @@ -132,6 +133,16 @@ public class Testspace { } } + public static String getDataPath(String dataFilePath) { + File file = getDataFile(dataFilePath); + if (file == null) + return null; + try { + return file.getCanonicalPath(); + } catch (IOException e) { + return null; + } + } public static boolean deleteFile(String path) { return FileUtil.deleteFile(getPath(path)); Modified: incubator/ooo/trunk/test/testuno/source/pvt/uno/Conversion.java URL: http://svn.apache.org/viewvc/incubator/ooo/trunk/test/testuno/source/pvt/uno/Conversion.java?rev=1387890&r1=1387889&r2=1387890&view=diff ============================================================================== --- incubator/ooo/trunk/test/testuno/source/pvt/uno/Conversion.java (original) +++ incubator/ooo/trunk/test/testuno/source/pvt/uno/Conversion.java Thu Sep 20 07:17:44 2012 @@ -58,8 +58,7 @@ public class Conversion { public Logger log = Logger.getLogger(this); @FileRepos - public static String repos = getDataFile("pvt_conversion").getAbsolutePath(); - + public static String repos = System.getProperty("conversion.repos", getDataPath("conversion_pvt")); @FileFilter public static String filter = System.getProperty("conversion.filter", "-f .*\\.((doc)|(dot)|(odt)|(ott))$ writer_pdf_Export pdf " + "-f .*\\.((xls)|(xlt)|(ods)|(ots))$ calc_pdf_Export pdf " @@ -74,6 +73,8 @@ public class Conversion { @FileRepeat public static int repeat = Integer.parseInt(System.getProperty("conversion.repeat", "8")); + public static String clean = System.getProperty("conversion.clean", "file"); + private static UnoApp app = new UnoApp(); private static PrintStream result; @@ -89,7 +90,7 @@ public class Conversion { File resultFile = Testspace.getFile("output/conversion.csv"); resultFile.getParentFile().mkdirs(); result = new PrintStream(new FileOutputStream(resultFile)); - result.println("File,Scenario,After Close,After Save,After Load"); + result.println("File,Scenario,File Size,Time Consumed After Closing,Time Consumed After Saving,Time Consumed After Loading"); } @AfterClass @@ -123,27 +124,30 @@ public class Conversion { @Before public void before() throws Exception { - sourceFile = prepareDataFile(sourcePath); + sourceFile = new File(sourcePath); sourceFileUrl = FileUtil.getUrl(this.sourceFile); targetFile = getFile("classtemp/" + sourceFile.getName()+ "." + targetExtName); targetFileUrl = FileUtil.getUrl(this.targetFile); scenario = FileUtil.getFileExtName(sourceFile.getName()).toLowerCase() + " to " + FileUtil.getFileExtName(targetFile.getName()).toLowerCase(); - sourceFileId = sourceFile.getAbsolutePath().replace(new File(repos).getAbsolutePath(), "").replace("\\", "/"); - + String pathSource = sourceFile.getCanonicalPath().replace("\\", "/"); + String pathRepos = new File(repos).getCanonicalPath().replace("\\", "/") + "/"; + sourceFileId = pathSource.replace(pathRepos, ""); log.info("Start [File: " + sourceFileId + "] [Size: " + (sourceFile.length() / 1024) + "KB] [Scenario: " + scenario + "]"); app.start(); } @After public void after() throws Exception{ - result.println(sourceFileId + "," + scenario + "," + closeTime + "," + saveTime + "," + loadTime); + result.println(sourceFileId + "," + scenario + "," + sourceFile.length() + "," + closeTime + "," + saveTime + "," + loadTime); log.info("Result [After Closing: " + closeTime + "] [After Saving: " + saveTime + "] [After Loading: " + loadTime + "]"); - if (counter % repeat == 0) { + if (closeTime < 0) { + app.close(); + } else if ("file".equalsIgnoreCase(clean) && counter % repeat == 0) { app.close(); } } - + private PropertyValue propertyValue(String name, Object value) { PropertyValue p = new PropertyValue(); p.Name = name;