diff options
author | Noel Grandin <noel@peralex.com> | 2021-06-30 10:30:25 +0200 |
---|---|---|
committer | Noel Grandin <noel.grandin@collabora.co.uk> | 2021-06-30 11:30:49 +0200 |
commit | ecea9c42d5f1a5e63493fcd92944bd49fb91b338 (patch) | |
tree | 0fe5c6d7157a1d69580efdade16abe63172e717b /unotools/qa | |
parent | 8b3ff370ab0fe2e7593a69513c8ff3df35810c4e (diff) |
fix failure in JunitTest_unotools_complex
Temp files created from OTempFileService are always opened with StreamMode::SHARE_DENYALL.
So normally you can't open them twice.
But the JunitTest_unotools_complex unit test used to work because it exploited a pessimisation in OTempFileService,
where that code would close the file when we read to the end.
Which meant that the unit test could open it again and read it.
However, in
commit 218f36dd614cf828e949f605faaf6a6fd615da26
Date: Sun Jun 20 18:51:12 2021 +0200
tdf#135316 remove OTempFileService pessimisation
I removed that pessimisation.
So make the share mode a little more permissive.
Change-Id: I297a5c9c0505816b399fad29414077d03231ec72
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/118146
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
Diffstat (limited to 'unotools/qa')
-rw-r--r-- | unotools/qa/complex/tempfile/Test01.java | 2 | ||||
-rw-r--r-- | unotools/qa/complex/tempfile/TestHelper.java | 1 |
2 files changed, 3 insertions, 0 deletions
diff --git a/unotools/qa/complex/tempfile/Test01.java b/unotools/qa/complex/tempfile/Test01.java index f93b02186e8d..ca81e1801bef 100644 --- a/unotools/qa/complex/tempfile/Test01.java +++ b/unotools/qa/complex/tempfile/Test01.java @@ -51,6 +51,8 @@ public class Test01 { //compare the file name with the name in the URL. sFileURL = m_aTestHelper.GetTempFileURL( xTempFile ); sFileName = m_aTestHelper.GetTempFileName( xTempFile ); + m_aTestHelper.Message( "Tempfile URL: " + sFileURL ); + m_aTestHelper.Message( "Tempfile name: " + sFileName ); m_aTestHelper.CompareFileNameAndURL( sFileName, sFileURL ); //write to the stream using the service. diff --git a/unotools/qa/complex/tempfile/TestHelper.java b/unotools/qa/complex/tempfile/TestHelper.java index 821c33da054b..9ea42b102673 100644 --- a/unotools/qa/complex/tempfile/TestHelper.java +++ b/unotools/qa/complex/tempfile/TestHelper.java @@ -78,6 +78,7 @@ public class TestHelper { public void ReadDirectlyFromTempFile( byte [][] pBytes, int nBytes, XSimpleFileAccess xSFA, String sTempFileURL ) throws java.lang.Exception { + Message ( "Attempting to read directly from " + sTempFileURL ); XInputStream xInTemp = xSFA.openFileRead( sTempFileURL ); if ( xInTemp == null ) throw new java.lang.Exception("Cannot create input stream from URL."); |