WIP fix tests

Author: matthew-dean

packages/less/Gruntfile.js

@@ -214,7 +214,7 @@ module.exports = function(grunt) {
                 command: "node build/rollup.js --browser --out=./tmp/browser/less.min.js"
             },
             test: {
-                command: 'ts-node test/test-es6.ts && node test/index.js'
+                command: 'npx ts-node test/test-es6.ts && node test/index.js'
             },
             generatebrowser: {
                 command: 'node test/browser/generator/generate.js'

packages/less/package.json

@@ -44,7 +44,8 @@
 		"clean": "shx rm -rf ./lib tsconfig.tsbuildinfo",
 		"compile": "tsc -p tsconfig.build.json",
 		"dev": "tsc -p tsconfig.build.json -w",
-		"prepublishOnly": "grunt dist"
+		"prepublishOnly": "grunt dist",
+		"postinstall": "node scripts/postinstall.js"
 	},
 	"optionalDependencies": {
 		"errno": "^0.1.1",
@@ -89,8 +90,6 @@
 		"minimist": "^1.2.0",
 		"mocha": "^6.2.1",
 		"mocha-teamcity-reporter": "^3.0.0",
-		"msw": "~2.11.1",
-		"nock": "^14.0.10",
 		"npm-run-all": "^4.1.5",
 		"performance-now": "^0.2.0",
 		"phin": "^2.2.3",

packages/less/scripts/postinstall.js

@@ -0,0 +1,61 @@
+#!/usr/bin/env node
+
+/**
+ * Post-install script for Less.js package
+ * 
+ * This script installs Playwright browsers only when:
+ * 1. This is a development environment (not when installed as a dependency)
+ * 2. We're in a monorepo context (parent package.json exists)
+ * 3. Not running in CI or other automated environments
+ */
+
+const fs = require('fs');
+const path = require('path');
+const { execSync } = require('child_process');
+
+// Check if we're in a development environment
+function isDevelopmentEnvironment() {
+    // Skip if this is a global install or user config
+    if (process.env.npm_config_user_config || process.env.npm_config_global) {
+        return false;
+    }
+    
+    // Skip in CI environments
+    if (process.env.CI || process.env.GITHUB_ACTIONS || process.env.TRAVIS) {
+        return false;
+    }
+    
+    // Check if we're in a monorepo (parent package.json exists)
+    const parentPackageJson = path.join(__dirname, '../../../package.json');
+    if (!fs.existsSync(parentPackageJson)) {
+        return false;
+    }
+    
+    // Check if this is the root of the monorepo
+    const currentPackageJson = path.join(__dirname, '../package.json');
+    if (!fs.existsSync(currentPackageJson)) {
+        return false;
+    }
+    
+    return true;
+}
+
+// Install Playwright browsers
+function installPlaywrightBrowsers() {
+    try {
+        console.log('🎭 Installing Playwright browsers for development...');
+        execSync('pnpm exec playwright install', { 
+            stdio: 'inherit',
+            cwd: path.join(__dirname, '..')
+        });
+        console.log('✅ Playwright browsers installed successfully');
+    } catch (error) {
+        console.warn('⚠️  Failed to install Playwright browsers:', error.message);
+        console.warn('   You can install them manually with: pnpm exec playwright install');
+    }
+}
+
+// Main execution
+if (isDevelopmentEnvironment()) {
+    installPlaywrightBrowsers();
+}

packages/less/test/index.js

@@ -245,7 +245,7 @@ var testMap = [
     }
 ];
 
-// Note: nock mocks are set up globally at the top of the file
+// Note: needle mocking is set up globally at the top of the file
 
 testMap.forEach(function(testConfig) {
     // For glob patterns, pass lessOptions as the first parameter and patterns as the second

packages/less/test/less-test.js

@@ -23,8 +23,7 @@ logger.addListener({
 module.exports = function(testFilter) {
     var path = require('path'),
         fs = require('fs'),
-        clone = require('copy-anything').copy,
-        nock = require('nock');
+        clone = require('copy-anything').copy;
 
     var less = require('../');
 
@@ -377,7 +376,6 @@ module.exports = function(testFilter) {
             var includePatterns = [];
             var excludePatterns = [];
 
-            console.log('DEBUG: Processing glob patterns:', patterns);
 
             patterns.forEach(function(pattern) {
                 if (pattern.startsWith('!')) {
@@ -399,7 +397,7 @@ module.exports = function(testFilter) {
                 allFiles = allFiles.concat(files);
             });
 
-            // Note: nock mocks are set up globally in index.js
+            // Note: needle mocking is set up globally in index.js
 
             // Process each .less file found
             allFiles.forEach(function(filePath) {
@@ -464,11 +462,6 @@ module.exports = function(testFilter) {
 
             var name = getBasename(file, relativePath);
 
-            // Debug logging for import-redirect test
-            if (fullPath.includes('import-redirect')) {
-                console.log('DEBUG: import-redirect test running, fullPath:', fullPath);
-                console.log('DEBUG: nock.pendingMocks():', require('nock').pendingMocks());
-            }
 
             if (oneTestOnly && typeof oneTestOnly === 'string' && !name.includes(oneTestOnly)) {
                 return;