From 3cff10827974cd773810faa2857ab81a6dcff0ee Mon Sep 17 00:00:00 2001 From: Tor Lillqvist Date: Thu, 11 Aug 2016 11:15:06 +0300 Subject: Follow-up to 38a3743e0c5d52f9386f74097fd512d3133fbbe3 Change-Id: If70b5754c161a0e1b262abeac8c54dde60515280 --- framework/qa/complex/XTitle/CheckXTitle.java | 13 ++++++------- 1 file changed, 6 insertions(+), 7 deletions(-) (limited to 'framework/qa') diff --git a/framework/qa/complex/XTitle/CheckXTitle.java b/framework/qa/complex/XTitle/CheckXTitle.java index 3e008649e806..1372cf3dc432 100644 --- a/framework/qa/complex/XTitle/CheckXTitle.java +++ b/framework/qa/complex/XTitle/CheckXTitle.java @@ -210,11 +210,12 @@ public class CheckXTitle xDisProv = null; } - /** @short checks loading of new document with readonly set to true + /** @short checks creation of new empty document with readonly set to true - @descr loads a new document with readonly mode set to true. - this should fail because new doc can not be opened in - read only mode + @descr creation of a new document with readonly property set + to true. this should not fail even if as such it can + be seen as a slightly silly thing to do. But existing + 3rd-party code depends on it to work. */ @Test public void checkLoadingWithReadOnly() { @@ -232,10 +233,8 @@ public class CheckXTitle try{ xDoc = m_xLoader.loadComponentFromURL("private:factory/swriter", "_default", 0, lArgs); } catch (Exception e) { - //exception is expected. do nothing } - - assertNull("Loading new document with read only mode true should have failed", xDoc); + assertNotNull("Creating a new document read with ReadOnly property true should work (even if slightly silly)", xDoc); } private XMultiServiceFactory getMSF() -- cgit