emfio/qa/cppunit/wmf/wmfimporttest.cxx |    5 -----
 1 file changed, 5 deletions(-)

New commits:
commit 45aac5b97192a97369c7785772ce7400d9ea8abc
Author:     Hossein <hoss...@libreoffice.org>
AuthorDate: Sun Apr 24 14:13:21 2022 +0200
Commit:     Hossein - <hoss...@libreoffice.org>
CommitDate: Sun Apr 24 14:56:24 2022 +0200

    Remove brittle test from WmfTest
    
    The WmfTest::testStockObject() contained test for the number of
    children in the metafile dump. This can be changed when unimplemented
    records are implemented.
    
    The problem was revelead while implementing SetPolyFillMode record:
    <https://ci.libreoffice.org/job/gerrit_linux_clang_dbgutil/112114/>
    
    xmltesttools.cxx:234:WmfTest::testStockObject
    equality assertion failed
    - Expected: 42
    - Actual  : 47
    - In <>, XPath '/metafile/push[2]' number of child-nodes is incorrect
    
    Change-Id: I627801b7ac535a2f0c736880d9675f3d0136136a
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/133353
    Tested-by: Jenkins
    Reviewed-by: Hossein - <hoss...@libreoffice.org>

diff --git a/emfio/qa/cppunit/wmf/wmfimporttest.cxx 
b/emfio/qa/cppunit/wmf/wmfimporttest.cxx
index d8a4ed82d0f0..a85876e03a7d 100644
--- a/emfio/qa/cppunit/wmf/wmfimporttest.cxx
+++ b/emfio/qa/cppunit/wmf/wmfimporttest.cxx
@@ -402,11 +402,6 @@ void WmfTest::testStockObject()
 
     CPPUNIT_ASSERT(pDoc);
 
-    //   Without the fix in place, this test would have failed with
-    // - Expected: 42
-    // - Actual  : 37
-    assertXPathChildren(pDoc, "/metafile/push[2]", 42);
-
     //   Without the fix in place, this test would have failed with
     // - Expected: 1
     // - Actual  : 0

Reply via email to