unotest/source/embindtest/embindtest.js |   20 +++++++++++---------
 1 file changed, 11 insertions(+), 9 deletions(-)

New commits:
commit fbecf2d6b72ee2b5631bee4c0d6d0c5556aa9698
Author:     Stephan Bergmann <stephan.bergm...@allotropia.de>
AuthorDate: Fri Mar 8 16:27:03 2024 +0100
Commit:     Stephan Bergmann <stephan.bergm...@allotropia.de>
CommitDate: Sat Mar 9 15:47:08 2024 +0100

    Enable (and fix) some TODO test code
    
    ...now that all the relevant Any getter functionality is available
    
    Change-Id: I5824dd50d177506b0450506146df109ded7e20d6
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/164588
    Tested-by: Jenkins
    Reviewed-by: Stephan Bergmann <stephan.bergm...@allotropia.de>

diff --git a/unotest/source/embindtest/embindtest.js 
b/unotest/source/embindtest/embindtest.js
index 8568fa2acaa2..1c6cd5f2718e 100644
--- a/unotest/source/embindtest/embindtest.js
+++ b/unotest/source/embindtest/embindtest.js
@@ -461,12 +461,14 @@ Module.addOnPostRun(function() {
         let e1 = v.get(1);
         console.assert(e1.get() === undefined);
         e1.delete();
-        //TODO: let e2 = v.get(2);
-        //TODO: console.assert(e2.size() === 3);
-        //TODO: console.assert(e2.get(0) == 
uno.org.libreoffice.embindtest.Enum.E_2);
-        //TODO: console.assert(e2.get(1) == 
uno.org.libreoffice.embindtest.Enum.E3);
-        //TODO: console.assert(e2.get(2) == 
uno.org.libreoffice.embindtest.Enum.E_10);
-        //TODO: e2.delete();
+        let e2 = v.get(2);
+        let s = e2.get();
+        console.assert(s.size() === 3);
+        console.assert(s.get(0) === uno.org.libreoffice.embindtest.Enum.E_2);
+        console.assert(s.get(1) === uno.org.libreoffice.embindtest.Enum.E3);
+        console.assert(s.get(2) === uno.org.libreoffice.embindtest.Enum.E_10);
+        s.delete();
+        e2.delete();
         console.assert(test.isSequenceAny(v));
         v.delete();
     }
@@ -493,9 +495,9 @@ Module.addOnPostRun(function() {
         let v = test.getSequenceEnum();
         console.log(v);
         console.assert(v.size() === 3);
-        console.assert(v.get(0) == uno.org.libreoffice.embindtest.Enum.E_2);
-        console.assert(v.get(1) == uno.org.libreoffice.embindtest.Enum.E3);
-        console.assert(v.get(2) == uno.org.libreoffice.embindtest.Enum.E_10);
+        console.assert(v.get(0) === uno.org.libreoffice.embindtest.Enum.E_2);
+        console.assert(v.get(1) === uno.org.libreoffice.embindtest.Enum.E3);
+        console.assert(v.get(2) === uno.org.libreoffice.embindtest.Enum.E_10);
         console.assert(test.isSequenceEnum(v));
         v.delete();
     }

Reply via email to