Merge pull request #16 from yiisoft/master

Updated JS test packages (yiisoft#20180)
honsa · Jun 5, 2024 · 7f5a377 · 7f5a377
2 parents 96ba123 + 855522c
commit 7f5a377
Show file tree

Hide file tree

Showing 8 changed files with 23 additions and 12 deletions.
diff --git a/.github/workflows/ci-node.yml b/.github/workflows/ci-node.yml
@@ -11,7 +11,7 @@ concurrency:
 
 jobs:
   test:
-    name: NPM 6 on ubuntu-latest
+    name: NPM 10 on ubuntu-latest
 
     runs-on: ubuntu-latest
 
@@ -26,9 +26,9 @@ jobs:
         run: composer require "bower-asset/jquery:3.6.*@stable"
 
       - name: Install node.js.
-        uses: actions/setup-node@v1
+        uses: actions/setup-node@v4
         with:
-          node-version: 6
+          node-version: 20
 
       - name: Tests.
         run: |

diff --git a/framework/base/package.json b/framework/base/package.json
@@ -0,0 +1,6 @@
+{
+    "name": "base",
+    "version": "1.0.0",
+    "dependencies": {
+    }
+}
diff --git a/package.json b/package.json
@@ -9,9 +9,9 @@
   "dependencies": {},
   "devDependencies": {
     "chai": "^3.5.0",
-    "jsdom": "11.11.0",
-    "leche": "^2.2.3",
-    "mocha": "^5.2.0",
+    "jsdom": "24.1.0",
+    "leche": "^2.3.0",
+    "mocha": "^6.2.3",
     "mocha-jsdom": "^2.0.0",
     "sinon": "^1.17.6"
   },

diff --git a/tests/js/tests/yii.activeForm.test.js b/tests/js/tests/yii.activeForm.test.js
@@ -49,7 +49,8 @@ describe('yii.activeForm', function () {
 
     jsdom({
         html: html,
-        src: fs.readFileSync(jQueryPath, 'utf-8')
+        src: fs.readFileSync(jQueryPath, 'utf-8'),
+        url: "http://foo.bar"
     });
 
     before(function () {

diff --git a/tests/js/tests/yii.captcha.test.js b/tests/js/tests/yii.captcha.test.js
@@ -30,7 +30,8 @@ describe('yii.captcha', function () {
 
     jsdom({
         html: html,
-        src: fs.readFileSync(jQueryPath, 'utf-8')
+        src: fs.readFileSync(jQueryPath, 'utf-8'),
+        url: "http://foo.bar"
     });
 
     before(function () {

diff --git a/tests/js/tests/yii.gridView.test.js b/tests/js/tests/yii.gridView.test.js
@@ -51,7 +51,8 @@ describe('yii.gridView', function () {
 
     jsdom({
         html: html,
-        src: fs.readFileSync(jQueryPath, 'utf-8')
+        src: fs.readFileSync(jQueryPath, 'utf-8'),
+        url: "http://foo.bar"
     });
 
     before(function () {
@@ -787,7 +788,7 @@ describe('yii.gridView', function () {
 
             assert.throws(function () {
                 $gridView1.yiiGridView('applyFilter');
-            }, "Cannot read property 'settings' of undefined");
+            }, "Cannot read properties of undefined (reading \'settings\')");
             $gridView1.yiiGridView(settings); // Reinitialize without "beforeFilter" and "afterFilter" event handlers
 
             $gridView1.yiiGridView('applyFilter');

diff --git a/tests/js/tests/yii.test.js b/tests/js/tests/yii.test.js
@@ -74,7 +74,8 @@ describe('yii', function () {
 
     jsdom({
         html: fs.readFileSync('tests/js/data/yii.html', 'utf-8'),
-        src: fs.readFileSync(jQueryPath, 'utf-8')
+        src: fs.readFileSync(jQueryPath, 'utf-8'),
+        url: "http://foo.bar"
     });
 
     before(function () {

diff --git a/tests/js/tests/yii.validation.test.js b/tests/js/tests/yii.validation.test.js
@@ -76,7 +76,8 @@ describe('yii.validation', function () {
     }
 
     jsdom({
-        src: fs.readFileSync('vendor/bower-asset/jquery/dist/jquery.js', 'utf-8')
+        src: fs.readFileSync('vendor/bower-asset/jquery/dist/jquery.js', 'utf-8'),
+        url: "http://foo.bar"
     });
 
     before(function () {