test: fix flaky test in definition properties

bpmn-io · Oct 16, 2024 · 7ed847e · 7ed847e
1 parent cb8ef20
commit 7ed847e
Show file tree

Hide file tree

Showing 3 changed files with 16 additions and 19 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/packages/dmn-js-drd/package.json b/packages/dmn-js-drd/package.json
@@ -36,6 +36,7 @@
     "tiny-svg": "^3.1.2"
   },
   "devDependencies": {
+    "@testing-library/dom": "^10.4.0",
     "dmn-font": "^0.6.2"
   }
 }
diff --git a/packages/dmn-js-drd/test/spec/features/definition-properties/DefinitionPropertiesSpec.js b/packages/dmn-js-drd/test/spec/features/definition-properties/DefinitionPropertiesSpec.js
@@ -3,6 +3,8 @@ import {
   query as domQuery
 } from 'min-dom';
 
+import { waitFor } from '@testing-library/dom';
+
 import { getBusinessObject } from 'dmn-js-shared/lib/util/ModelUtil';
 
 import {
@@ -309,8 +311,8 @@ describe('features/definition-properties', function() {
       ));
 
 
-      skipFF()('should clear error message on blur', inject(
-        function(definitionPropertiesEdit, definitionPropertiesView) {
+      it('should clear error message on blur', inject(
+        async function(definitionPropertiesEdit, definitionPropertiesView) {
 
           // given
           var idContainer = domQuery(
@@ -326,13 +328,15 @@ describe('features/definition-properties', function() {
           idContainer.blur();
 
           // then
-          var errorMessage = domQuery(
-            '.dmn-definitions-error-message',
-            definitionPropertiesView._container
-          );
-
-          expect(errorMessage).not.to.exist;
-          expect(domClasses(idContainer).has('dmn-definitions-error')).to.be.false;
+          await waitFor(() => {
+            var errorMessage = domQuery(
+              '.dmn-definitions-error-message',
+              definitionPropertiesView._container
+            );
+
+            expect(errorMessage).not.to.exist;
+            expect(domClasses(idContainer).has('dmn-definitions-error')).to.be.false;
+          });
         }
       ));
 
@@ -341,13 +345,3 @@ describe('features/definition-properties', function() {
   });
 
 });
-
-// helpers //////////////////
-
-function isFirefox() {
-  return /Firefox/.test(window.navigator.userAgent);
-}
-
-function skipFF() {
-  return isFirefox() ? it.only : it;
-}