@json-editor/json-editor 2.5.2 → 2.6.1

package/src/themes/bootstrap4.js

@@ -135,6 +135,12 @@ export class bootstrap4Theme extends AbstractTheme {
     const min = input.getAttribute('min')
     const max = input.getAttribute('max')
 
+    input.addEventListener('change', () => {
+      if (!input.getAttribute('initialized')) {
+        input.setAttribute('initialized', '1')
+      }
+    })
+
     minusBtn.addEventListener('click', () => {
       if (!input.getAttribute('initialized')) {
         initialize(input, min)
@@ -246,6 +252,7 @@ export class bootstrap4Theme extends AbstractTheme {
       if (window.jQuery && window.jQuery().tooltip) {
         window.jQuery(button).tooltip()
       } else {
+        // eslint-disable-next-line no-console
         console.warn('Could not find popper jQuery plugin of Bootstrap.')
       }
     } else if (this.options.tooltip === 'css') {
@@ -398,12 +405,13 @@ export class bootstrap4Theme extends AbstractTheme {
     return el
   }
 
-  getHeader (text) {
+  getHeader (text, pathDepth) {
     /* var cardHeader = document.createElement('div') */
     /* cardHeader.classList.add('card-header') */
 
     const el = document.createElement('h3')
     el.classList.add('card-title')
+    el.classList.add('level-' + pathDepth)
 
     if (typeof text === 'string') {
       el.textContent = text
@@ -475,7 +483,7 @@ export class bootstrap4Theme extends AbstractTheme {
   addInputError (input, text) {
     if (!input.controlgroup) return
 
-    input.classList.add('is-invalid')
+    input.controlgroup.classList.add('is-invalid')
 
     if (!input.errmsg) {
       input.errmsg = document.createElement('p')
@@ -491,7 +499,7 @@ export class bootstrap4Theme extends AbstractTheme {
   removeInputError (input) {
     if (!input.errmsg) return
     input.errmsg.style.display = 'none'
-    input.classList.remove('is-invalid')
+    input.controlgroup.classList.remove('is-invalid')
   }
 
   getTabHolder (propertyName) {

package/src/themes/html.js

@@ -43,10 +43,9 @@ export class htmlTheme extends AbstractTheme {
   }
 
   addInputError (input, text) {
-    input.style.borderColor = 'red'
+    const group = this.closest(input, '.form-control') || input.controlgroup
 
     if (!input.errmsg) {
-      const group = this.closest(input, '.form-control')
       input.errmsg = document.createElement('div')
       input.errmsg.setAttribute('class', 'errmsg')
       input.errmsg.style = input.errmsg.style || {}

package/src/themes/materialize.js

@@ -140,7 +140,7 @@ export class materializeTheme extends AbstractTheme {
    * @param {string|HTMLElement} text The header text or element.
    * @returns {HTMLElement} The header element.
    */
-  getHeader (text) {
+  getHeader (text, pathDepth) {
     const el = document.createElement('h5')
 
     if (typeof text === 'string') {

package/src/themes/spectre.js

@@ -98,7 +98,7 @@ export class spectreTheme extends AbstractTheme {
     return el
   }
 
-  getHeader (text) {
+  getHeader (text, pathDepth) {
     const el = document.createElement('h4')
     if (typeof text === 'string') {
       el.textContent = text
@@ -146,7 +146,6 @@ export class spectreTheme extends AbstractTheme {
   }
 
   getMultiCheckboxHolder (controls, label, description, infoText) {
-    console.log('mul')
     return super.getMultiCheckboxHolder(controls, label, description, infoText)
   }
 
@@ -218,13 +217,17 @@ export class spectreTheme extends AbstractTheme {
     const group = document.createElement('div')
     group.classList.add('form-group')
 
-    if (label) {
-      if (input.type === 'checkbox') {
-        label = this.getFormCheckboxControl(label, input, false)
-      }
-      label.classList.add('form-label')
+    if (label && (input.type === 'checkbox' || input.type === 'radio')) {
+      group.classList.add(input.type)
+      label.insertBefore(input, label.firstChild)
       group.appendChild(label)
-      if (infoText) group.insertBefore(infoText, group.firstChild)
+    } else {
+      if (label) {
+        label.classList.add('form-label')
+        group.appendChild(label)
+        if (infoText) label.appendChild(infoText)
+      }
+      group.appendChild(input)
     }
 
     if (this.options.input_size === 'small') input.classList.add('input-sm', 'select-sm')

package/src/themes/tailwind.js

@@ -60,7 +60,7 @@ export class tailwindTheme extends AbstractTheme {
   }
 
   getTitle () {
-    return this.schema.title
+    return this.translateProperty(this.schema.title)
   }
 
   getSelectInput (options, multiple) {

package/src/validator.js

@@ -7,9 +7,21 @@ export class Validator {
     this.schema = schema || this.jsoneditor.schema
     this.options = options || {}
     this.translate = this.jsoneditor.translate || defaults.translate
+    this.translateProperty = this.jsoneditor.translateProperty || defaults.translateProperty
     this.defaults = defaults
 
     this._validateSubSchema = {
+      const (schema, value, path) {
+        const valid = JSON.stringify(schema.const) === JSON.stringify(value) && !(Array.isArray(value) || typeof value === 'object')
+        if (!valid) {
+          return [{
+            path,
+            property: 'const',
+            message: this.translate('error_const')
+          }]
+        }
+        return []
+      },
       enum (schema, value, path) {
         const stringified = JSON.stringify(value)
         const valid = schema.enum.some(e => stringified === JSON.stringify(e))
@@ -244,7 +256,6 @@ export class Validator {
             /* If this item has a specific schema tied to it */
             /* Validate against it */
             if (schema.items[i]) {
-              console.log('-->')
               errors.push(...this._validateSchema(schema.items[i], value[i], `${path}.${i}`))
               /* If all additional items are allowed */
             } else if (schema.additionalItems === true) {
@@ -339,12 +350,13 @@ export class Validator {
           schema.required.forEach(e => {
             if (typeof value[e] !== 'undefined') return
             const editor = this.jsoneditor.getEditor(`${path}.${e}`)
+            if (editor && editor.dependenciesFulfilled === false) return
             /* Ignore required error if editor is of type "button" or "info" */
             if (editor && ['button', 'info'].includes(editor.schema.format || editor.schema.type)) return
             errors.push({
               path,
               property: 'required',
-              message: this.translate('error_required', [e])
+              message: this.translate('error_required', [schema && schema.properties && schema.properties[e] && schema.properties[e].title ? schema.properties[e].title : e])
             })
           })
         }
@@ -375,6 +387,91 @@ export class Validator {
     }
 
     this._validateObjectSubSchema2 = {
+      propertyNames (schema, value, path, validatedProperties) {
+        const errors = []
+        const keys = Object.keys(value)
+        let k = null
+        for (let i = 0; i < keys.length; i++) {
+          let msg = ''
+          let truthy = false
+          k = keys[i]
+          /* Check property names that don't match */
+          if (typeof schema.propertyNames === 'boolean') {
+            if (schema.propertyNames === true) {
+              continue
+            }
+            errors.push({
+              path,
+              property: 'propertyNames',
+              message: this.translate('error_property_names_false', [k])
+            })
+            break
+          }
+          truthy = Object.entries(schema.propertyNames).every(([j, prop]) => {
+            let match = false
+            let regex = null
+            switch (j) {
+              case 'maxLength':
+                if (typeof prop !== 'number') {
+                  msg = 'error_property_names_maxlength'
+                  break
+                }
+                if (k.length > prop) {
+                  msg = 'error_property_names_exceeds_maxlength'
+                  break
+                }
+                return true
+              case 'const':
+                if (prop !== k) {
+                  msg = 'error_property_names_const_mismatch'
+                  break
+                }
+                return true
+              case 'enum':
+                if (!Array.isArray(prop)) {
+                  msg = 'error_property_names_enum'
+                  break
+                }
+                prop.forEach(p => {
+                  if (p === k) {
+                    match = true
+                  }
+                })
+                if (!match) {
+                  msg = 'error_property_names_enum_mismatch'
+                  break
+                }
+                return true
+              case 'pattern':
+                if (typeof prop !== 'string') {
+                  msg = 'error_property_names_pattern'
+                  break
+                }
+                regex = new RegExp(prop)
+                if (!regex.test(k)) {
+                  msg = 'error_property_names_pattern_mismatch'
+                  break
+                }
+                return true
+              default:
+                errors.push({
+                  path,
+                  property: 'propertyNames',
+                  message: this.translate('error_property_names_unsupported', [j])
+                })
+                return false
+            }
+            errors.push({
+              path,
+              property: 'propertyNames',
+              message: this.translate(msg, [k])
+            })
+            return false
+          })
+          if (!truthy) break
+        }
+        return errors
+      },
       additionalProperties (schema, value, path, validatedProperties) {
         const errors = []
         const keys = Object.keys(value)
@@ -431,20 +528,35 @@ export class Validator {
     const fit = { match: 0, extra: 0 }
     if (typeof value === 'object' && value !== null) {
       /* Work on a copy of the schema */
-      const properties = this._getSchema(givenSchema).properties
-
-      for (const i in properties) {
-        if (!hasOwnProperty(properties, i)) {
-          fit.extra += weight
-          continue
-        }
-        if (typeof value[i] === 'object' && typeof properties[i] === 'object' && typeof properties[i].properties === 'object') {
-          const result = this.fitTest(value[i], properties[i], weight / 100)
-          fit.match += result.match
-          fit.extra += result.extra
+      const schema = this._getSchema(givenSchema)
+      /* If the schema is an anyOf declaration, do use the properties of the allowed sub schemata instead.
+      Of these sub schemata, the best fit is selected */
+      if (schema.anyOf) {
+        let bestFit = { ...fit }
+        for (const subSchema of schema.anyOf) {
+          const subFit = this.fitTest(value, subSchema, weight)
+          /* The best fit is the one with the best value for match. If there are multiple results
+          with the same match value, use the one with the least number of extra properties */
+          if ((subFit.match > bestFit.match) || (subFit.match === bestFit.match && subFit.extra < bestFit.extra)) {
+            bestFit = subFit
+          }
         }
-        if (typeof value[i] !== 'undefined') {
-          fit.match += weight
+        return bestFit
+      } else {
+        const properties = this._getSchema(givenSchema).properties
+        for (const i in properties) {
+          if (!hasOwnProperty(properties, i)) {
+            fit.extra += weight
+            continue
+          }
+          if (typeof value[i] === 'object' && typeof properties[i] === 'object' && typeof properties[i].properties === 'object') {
+            const result = this.fitTest(value[i], properties[i], weight / 100)
+            fit.match += result.match
+            fit.extra += result.extra
+          }
+          if (typeof value[i] !== 'undefined') {
+            fit.match += weight
+          }
         }
       }
     }
@@ -522,7 +634,7 @@ export class Validator {
   }
 
   _validateV3Required (schema, value, path) {
-    if ((typeof schema.required !== 'undefined' && schema.required === true) || (typeof schema.required === 'undefined' && this.jsoneditor.options.required_by_default === true)) {
+    if (((typeof schema.required !== 'undefined' && schema.required === true) || (typeof schema.required === 'undefined' && this.jsoneditor.options.required_by_default === true)) && (schema.type !== 'info')) {
       return [{
         path,
         property: 'required',

package/tests/codeceptjs/core_test.js

@@ -57,7 +57,7 @@ Scenario('should watch a specific field for changes', async (I) => {
   I.seeElement('.name-changed')
 })
 
-Scenario('should watch form for changes @optional', async (I) => {
+Scenario('should watch form for changes', async (I) => {
   I.amOnPage('core.html')
   I.dontSeeElement('.form-changed')
   I.click('.set-value')
@@ -91,3 +91,127 @@ Scenario('should change the form if form_name_root option is set @core', async (
   I.click('#get-value-form-2')
   assert.equal(await I.grabValueFrom('#value-form-2'), '"no"')
 })
+
+Scenario('should validate against oneOf schemas and display single oneOf and editors error messages @core @oneof', async (I) => {
+  I.amOnPage('oneof.html')
+  I.waitForText('Object is missing the required property \'p4\'', '.alert-danger')
+  I.waitForText('Value must validate against exactly one of the provided schemas. It currently validates against 0 of the schemas.', '.alert-danger')
+  I.waitForText('Object is missing the required property \'p1\'', '.alert-danger')
+  I.waitForText('Object is missing the required property \'p2\'', '.alert-danger')
+  I.waitForText('Property must be set.', '[data-schemapath="root.p4"] .invalid-feedback')
+  I.waitForText('Property must be set.', '[data-schemapath="root.p5.p1"] .invalid-feedback')
+  I.waitForText('Property must be set.', '[data-schemapath="root.p5.p2"] .invalid-feedback')
+  I.fillField('root[p4]', 'to')
+  I.fillField('root[p5][p1]', 'to')
+  I.fillField('root[p5][p2]', 'to')
+  I.click('Get Value')
+  I.wait(3)
+  I.dontSee('Object is missing the required property \'p4\'', '.alert-danger')
+  I.dontSee('Object is missing the required property \'p1\'', '.alert-danger')
+  I.dontSee('Object is missing the required property \'p2\'', '.alert-danger')
+  I.waitForText('Value must be at least 4 characters long.', '[data-schemapath="root.p4"] .invalid-feedback')
+  I.waitForText('Value must be at least 4 characters long.', '[data-schemapath="root.p5.p1"] .invalid-feedback')
+  I.waitForText('Value must be at least 4 characters long.', '[data-schemapath="root.p5.p2"] .invalid-feedback')
+  I.fillField('root[p4]', 'todo')
+  I.fillField('root[p5][p1]', 'todo')
+  I.fillField('root[p5][p2]', 'todo')
+  I.click('Get Value')
+  I.wait(3)
+  I.dontSee('Value must be at least 4 characters long.', '[data-schemapath="root.p4"] .invalid-feedback')
+  I.dontSee('Value must be at least 4 characters long.', '[data-schemapath="root.p5.p1"] .invalid-feedback')
+  I.dontSee('Value must be at least 4 characters long.', '[data-schemapath="root.p5.p2"] .invalid-feedback')
+})
+
+Scenario('should validate against anyOf schemas and display single anyOf and editors error messages @core @anyof', async (I) => {
+  I.amOnPage('anyof.html')
+  I.dontSeeElement('.alert-danger')
+  I.selectOption('.je-switcher', 'Value, number')
+  I.dontSeeElement('.alert-danger')
+  I.selectOption('.je-switcher', 'Value, null')
+  I.dontSeeElement('.alert-danger')
+  I.selectOption('.je-switcher', 'Value, string')
+  I.waitForText('Object is missing the required property \'age\'', '.alert-danger')
+  I.waitForText('Property must be set.', '[data-schemapath="root.age"] .invalid-feedback')
+  I.fillField('root[age]', 'to')
+  I.click('Get Value')
+  I.wait(3)
+  I.dontSee('Object is missing the required property \'age\'', '.alert-danger')
+  I.dontSee('Property must be set.', '[data-schemapath="root.age"] .invalid-feedback')
+})
+
+Scenario('should display anyOf and oneOf error messages in the correct places @848', async (I) => {
+  I.amOnPage('issues/issue-gh-848.html')
+  I.selectOption('.je-switcher', 'Value, string')
+  I.waitForElement('[data-schemapath="root.list"] .invalid-feedback', 5)
+  I.dontSeeElement('[data-schemapath="root.list_group"] .invalid-feedback', 5)
+})
+
+Scenario('should validate against oneOf schemas and display single oneOf and editors error messages @core @translate-property', async (I) => {
+  I.amOnPage('translate-property.html?lang=en')
+  I.waitForText('Object Title')
+  I.waitForText('Object Description')
+  I.waitForText('Boolean Title')
+  I.waitForText('Boolean Description')
+  I.seeInSource('Boolean Info Text')
+  I.waitForText('String Title')
+  I.waitForText('String Description')
+  I.seeInSource('String Info Text')
+  I.waitForText('String Radio Title')
+  I.waitForText('String Radio Description')
+  I.seeInSource('String Radio Info Text')
+  I.waitForText('Integer Title')
+  I.waitForText('Integer Description')
+  I.seeInSource('Integer Info Text')
+  I.waitForText('Number Title')
+  I.waitForText('Number Description')
+  I.seeInSource('Number Info Text')
+  I.waitForText('Array Title')
+  I.waitForText('Array Description')
+  I.seeInSource('Array Info Text')
+  I.waitForText('Array Tabs Title')
+  I.waitForText('Array Tabs Description')
+  I.seeInSource('Array Tabs Info Text')
+  I.waitForText('Array Table Title')
+  I.waitForText('Array Table Description')
+  I.seeInSource('Array Table Info Text')
+  I.waitForText('Signature Title')
+  I.waitForText('Signature Description')
+  I.seeInSource('Signature Info Text')
+  I.waitForText('Rating Title')
+  I.waitForText('Rating Description')
+  I.seeInSource('Rating Info Text')
+
+  I.amOnPage('translate-property.html?lang=de')
+  I.waitForText('Object Title (but in german)')
+  I.waitForText('Object Description (but in german)')
+  I.waitForText('Boolean Title (but in german)')
+  I.waitForText('Boolean Description (but in german)')
+  I.seeInSource('Boolean Info Text (but in german)')
+  I.waitForText('String Title (but in german)')
+  I.waitForText('String Description (but in german)')
+  I.seeInSource('String Info Text (but in german)')
+  I.waitForText('String Radio Title (but in german)')
+  I.waitForText('String Radio Description (but in german)')
+  I.seeInSource('String Radio Info Text (but in german)')
+  I.waitForText('Integer Title (but in german)')
+  I.waitForText('Integer Description (but in german)')
+  I.seeInSource('Integer Info Text (but in german)')
+  I.waitForText('Number Title (but in german)')
+  I.waitForText('Number Description (but in german)')
+  I.seeInSource('Number Info Text (but in german)')
+  I.waitForText('Array Title (but in german)')
+  I.waitForText('Array Description (but in german)')
+  I.seeInSource('Array Info Text (but in german)')
+  I.waitForText('Array Tabs Title (but in german)')
+  I.waitForText('Array Tabs Description (but in german)')
+  I.seeInSource('Array Tabs Info Text (but in german)')
+  I.waitForText('Array Table Title (but in german)')
+  I.waitForText('Array Table Description (but in german)')
+  I.seeInSource('Array Table Info Text (but in german)')
+  I.waitForText('Signature Title (but in german)')
+  I.waitForText('Signature Description (but in german)')
+  I.seeInSource('Signature Info Text (but in german)')
+  I.waitForText('Rating Title (but in german)')
+  I.waitForText('Rating Description (but in german)')
+  I.seeInSource('Rating Info Text (but in german)')
+})

package/tests/codeceptjs/editors/array_test.js

@@ -1,3 +1,5 @@
+/* global Feature Scenario */
+
 var assert = require('assert');
 
 Feature('array');
@@ -8,7 +10,7 @@ Scenario('should have correct initial value', async (I) => {
   assert.equal(await I.grabValueFrom('.debug'), '[]');
 });
 
-Scenario('should trigger array (table) editing triggers', async (I) => {
+Scenario('should trigger array (table) editing triggers @retry', async (I) => {
   I.amOnPage('table-move-events.html');
   I.seeElement('[data-schemapath="root.0"]');
   I.seeElement('[data-schemapath="root.1"]');
@@ -21,7 +23,7 @@ Scenario('should trigger array (table) editing triggers', async (I) => {
 
   I.amAcceptingPopups();
   I.click('//button[contains(@class, "json-editor-btn-moveup") and @data-i="1"]');
-  I.seeInPopup('moveRow');
+  I.retry({ retries: 5, minTimeout: 500 }).seeInPopup('moveRow');
   I.acceptPopup();
   I.click('.get-value');
   value = await I.grabValueFrom('.debug');
@@ -29,7 +31,7 @@ Scenario('should trigger array (table) editing triggers', async (I) => {
 
   I.amAcceptingPopups();
   I.click('//button[contains(@class, "json-editor-btn-movedown") and @data-i="1"]');
-  I.seeInPopup('moveRow');
+  I.retry({ retries: 5, minTimeout: 500 }).seeInPopup('moveRow');
   I.acceptPopup();
   I.click('.get-value');
   value = await I.grabValueFrom('.debug');
@@ -37,7 +39,7 @@ Scenario('should trigger array (table) editing triggers', async (I) => {
 
   I.amAcceptingPopups();
   I.click('//button[contains(@class, "json-editor-btn-copy") and @data-i="2"]');
-  I.seeInPopup('copyRow');
+  I.retry({ retries: 5, minTimeout: 500 }).seeInPopup('copyRow');
   I.acceptPopup();
   I.click('.get-value');
   value = await I.grabValueFrom('.debug');
@@ -45,7 +47,7 @@ Scenario('should trigger array (table) editing triggers', async (I) => {
 
   I.amAcceptingPopups();
   I.click('.json-editor-btntype-add');
-  I.seeInPopup('addRow');
+  I.retry({ retries: 5, minTimeout: 500 }).seeInPopup('addRow');
   I.acceptPopup();
   I.click('.get-value');
   value = await I.grabValueFrom('.debug');
@@ -58,10 +60,10 @@ Scenario('should trigger array (table) editing triggers', async (I) => {
   // form field. Similar to the '.debug' field.
   I.amAcceptingPopups();
   I.click('.json-editor-btntype-deletelast');
-  I.seeInPopup('Are you sure you want to remove this node?');
+  I.retry({ retries: 5, minTimeout: 500 }).seeInPopup('Are you sure you want to remove this node?');
   I.acceptPopup();
   I.amAcceptingPopups();
-  I.seeInPopup('deleteRow');
+  I.retry({ retries: 5, minTimeout: 500 }).seeInPopup('deleteRow');
   I.acceptPopup();
   I.click('.get-value');
   value = await I.grabValueFrom('.debug');
@@ -70,10 +72,10 @@ Scenario('should trigger array (table) editing triggers', async (I) => {
   // This test will fail when using Puppeteer due to the way Puppeteer handles popups.
   I.amAcceptingPopups();
   I.click('.json-editor-btntype-deleteall');
-  I.seeInPopup('Are you sure you want to remove this node?');
+  I.retry({ retries: 5, minTimeout: 500 }).seeInPopup('Are you sure you want to remove this node?');
   I.acceptPopup();
   I.amAcceptingPopups();
-  I.seeInPopup('deleteAllRows');
+  I.retry({ retries: 5, minTimeout: 500 }).seeInPopup('deleteAllRows');
   I.acceptPopup();
   I.click('.get-value');
   value = await I.grabValueFrom('.debug');
@@ -873,9 +875,9 @@ Scenario('should work well with nested array editors', async (I) => {
 Scenario('should work well with selectize multiselect editors', async (I) => {
   I.amOnPage('array-selectize.html');
   I.click('Add item');
+  await I.seeElement('[data-schemapath="root.0"]');
   I.click('Add item');
-  I.seeElement('[data-schemapath="root.0"]');
-  I.seeElement('[data-schemapath="root.1"]');
+  await I.seeElement('[data-schemapath="root.1"]');
   I.click('.get-value');
   value = await I.grabValueFrom('.debug');
   // ensure defaults

package/tests/codeceptjs/editors/button_test.js

@@ -12,7 +12,7 @@ Scenario('should work with button editor callbacks', async (I) => {
 Scenario('should work with option "validated"', async (I) => {
   I.amOnPage('button-callbacks.html');
   I.seeElement('[data-schemapath="root.button1"] button');
-  I.seeDisabledAttribute('[data-schemapath="root.button2"] button');
+  I.retry({ retries: 3, minTimeout: 500 }).seeDisabledAttribute('[data-schemapath="root.button2"] button');
 
   await I.fillField('[name="root[textinput]"]', 'Hello World');
 
@@ -28,3 +28,8 @@ Scenario('should not leave any footprints in result', async (I) => {
   assert.equal(await I.grabValueFrom('.value'), JSON.stringify({"textinput":""}));
 });
 
+Scenario('should be disabled if "readonly" is specified', async (I) => {
+  I.amOnPage('read-only.html');
+
+  I.seeDisabledAttribute('[data-schemapath="root.button"] button');
+});

package/tests/codeceptjs/editors/issues/issue-gh-812_test.js

@@ -0,0 +1,32 @@
+/* global Feature Scenario */
+
+const assert = require('assert')
+let value
+
+Feature('issues')
+
+Scenario('GitHub issue 812 should remain fixed @issue-812', async (I) => {
+  I.amOnPage('issues/issue-gh-812.html')
+
+  I.click('.get-value')
+  value = await I.grabValueFrom('.debug')
+  assert.equal(value, '{"students":[{"name":"AAA","sessions":[{"student_name":"AAA","minutes":15},{"student_name":"AAA","minutes":15}]},{"name":"BBB","sessions":[{"student_name":"BBB","minutes":20}]},{"name":"CCC","sessions":[{"student_name":"CCC","minutes":10}]}]}')
+
+  I.amAcceptingPopups()
+  I.click('//*[@id="root.students.0"]/span[2]/button[contains(@class, "json-editor-btn-delete") and @data-i="0"]')
+  I.retry({ retries: 5, minTimeout: 500 }).seeInPopup('Are you sure you want to remove this node?')
+  I.acceptPopup()
+
+  I.click('.get-value')
+  value = await I.grabValueFrom('.debug')
+  assert.equal(value, '{"students":[{"name":"BBB","sessions":[{"student_name":"BBB","minutes":20}]},{"name":"CCC","sessions":[{"student_name":"CCC","minutes":10}]}]}')
+
+  I.amAcceptingPopups()
+  I.click('//*[@id="root.students.0"]/span[2]/button[contains(@class, "json-editor-btn-delete") and @data-i="0"]')
+  I.retry({ retries: 5, minTimeout: 500 }).seeInPopup('Are you sure you want to remove this node?')
+  I.acceptPopup()
+
+  I.click('.get-value')
+  value = await I.grabValueFrom('.debug')
+  assert.equal(value, '{"students":[{"name":"CCC","sessions":[{"student_name":"CCC","minutes":10}]}]}')
+})

package/tests/codeceptjs/editors/number_test.js

@@ -12,7 +12,7 @@ Scenario('should validate value', async (I) => {
   I.amOnPage('number.html');
   await I.fillField('[name="root[number]"]', '12-12');
   I.click('.get-value');
-  I.see('Value must be of type number.', '[data-schemapath="root.number"] div');
+  I.waitForText('Value must be of type number.', 5, '[data-schemapath="root.number"] .invalid-feedback');
   assert.equal(await I.grabValueFrom('.value'), '{"number":"12-12","number_number":5.75,"number_range":5.75}');
 });