diff options
| author | Mike Kaganski <mike.kaganski@collabora.com> | 2021-12-22 14:55:01 +0300 |
|---|---|---|
| committer | Mike Kaganski <mike.kaganski@collabora.com> | 2021-12-22 16:39:32 +0100 |
| commit | 822998f1dc202aba7a558767f641687aef0c1148 (patch) | |
| tree | 6474d7acf4527bcb6b8fb1a8b89e58554fd47cde /sc/qa | |
| parent | 771e721837e76bdf1f85dda164f607a3d3c6c9a2 (diff) | |
tdf#143582: Avoid error on clearing leftover return value of a method
SbxValue::Clear happens to not error out on clearing string and some
other types of objects, but for the rest, it calls SbxValue::Put, and
the latter errors out if not CanWrite(). The original test implemented
in commit 24d24debef4cda7de702c4b470a3707f1aae3ea3 only checked string
return value, so happened to miss this problem.
See similar code in SbiRuntime::FindElement.
Change-Id: I7c17137cc9e7ee3133ee86a9f701559df66e53b6
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/127248
Tested-by: Jenkins
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
Diffstat (limited to 'sc/qa')
| -rw-r--r-- | sc/qa/extras/macros-test.cxx | 6 | ||||
| -rw-r--r-- | sc/qa/extras/testdocuments/tdf143582.ods | bin | 12194 -> 8982 bytes |
2 files changed, 3 insertions, 3 deletions
diff --git a/sc/qa/extras/macros-test.cxx b/sc/qa/extras/macros-test.cxx index a68b39ce1cbc..b18bb7ab52a1 100644 --- a/sc/qa/extras/macros-test.cxx +++ b/sc/qa/extras/macros-test.cxx @@ -1061,9 +1061,9 @@ void ScMacrosTest::testTdf143582() aRet >>= aReturnValue; // Without the fix in place, this test would have failed with - // - Expected: Test - // - Actual : TeTest - CPPUNIT_ASSERT_EQUAL(OUString("Test"), aReturnValue); + // - Expected: Test6 + // - Actual : TeTest8 + CPPUNIT_ASSERT_EQUAL(OUString("Test6"), aReturnValue); css::uno::Reference<css::util::XCloseable> xCloseable(xComponent, css::uno::UNO_QUERY_THROW); xCloseable->close(true); diff --git a/sc/qa/extras/testdocuments/tdf143582.ods b/sc/qa/extras/testdocuments/tdf143582.ods Binary files differindex 05f4f07cea2c..1375bf5b9565 100644 --- a/sc/qa/extras/testdocuments/tdf143582.ods +++ b/sc/qa/extras/testdocuments/tdf143582.ods |