Changeset 46388

Timestamp:

10/04/2019 09:19:17 PM (4 years ago)

Author:

ocean90

Message:

QUnit: Include script dependencies for wp-a11y script to fix a TypeError.

Also, add missing message argument to Customizer notifications to ensure tests are passing.

Fixes #45066.

Location:

trunk/tests/qunit

Files:

• index.html (modified) (1 diff)
• wp-admin/js/customize-controls.js (modified) (3 diffs)

trunk/tests/qunit/index.html

@@ -25 +25 @@
         </script>
         <script src="../../build/wp-includes/js/wp-util.js"></script>
+        <script src="../../build/wp-includes/js/dist/vendor/wp-polyfill.js"></script>
+        <script src="../../build/wp-includes/js/dist/dom-ready.js"></script>
         <script src="../../build/wp-includes/js/dist/a11y.js"></script>
         <script>

trunk/tests/qunit/wp-admin/js/customize-controls.js

@@ -95 +95 @@
             container: container
         });
-        collection.add( 'mycode-1', new wp.customize.Notification( 'mycode-1' ) );
+        collection.add( 'mycode-1', new wp.customize.Notification( 'mycode-1', { message: 'My message 1' } ) );
         collection.render();
         items = collection.container.find( 'li' );
@@ -102 +102 @@
 
         collection.add( 'mycode-2', new wp.customize.Notification( 'mycode-2', {
+            message: 'My message 2',
             dismissible: true
         } ) );
@@ -188 +189 @@
             assert.equal( 0, notificationContainerElement.height() );
 
-            settingNotification = new wp.customize.Notification( 'setting_invalidity', 'Invalid setting' );
-            controlOnlyNotification = new wp.customize.Notification( 'control_invalidity', 'Invalid control' );
+            settingNotification = new wp.customize.Notification( 'setting_invalidity', { message: 'Invalid setting' } );
+            controlOnlyNotification = new wp.customize.Notification( 'control_invalidity', { message: 'Invalid control' } );
             control.settings['default'].notifications.add( settingNotification.code, settingNotification );
             control.notifications.add( controlOnlyNotification.code, controlOnlyNotification );