polendina 1.0.0 → 2.0.0-alpha

package/.github/dependabot.yml

@@ -0,0 +1,154 @@
+version: 2
+updates:
+  - package-ecosystem: 'github-actions'
+    directory: '/'
+    schedule:
+      interval: 'daily'
+    commit-message:
+      prefix: 'chore'
+      include: 'scope'
+
+  - package-ecosystem: 'npm'
+    directory: '/'
+    schedule:
+      interval: 'daily'
+    commit-message:
+      prefix: 'chore'
+      include: 'scope'
+
+  - package-ecosystem: 'npm'
+    directory: '/test/fixtures/bare-async'
+    schedule:
+      interval: 'daily'
+    commit-message:
+      prefix: 'chore'
+      include: 'scope'
+
+  - package-ecosystem: 'npm'
+    directory: '/test/fixtures/bare-async-esm'
+    schedule:
+      interval: 'daily'
+    commit-message:
+      prefix: 'chore'
+      include: 'scope'
+
+  - package-ecosystem: 'npm'
+    directory: '/test/fixtures/bare-async-failure'
+    schedule:
+      interval: 'daily'
+    commit-message:
+      prefix: 'chore'
+      include: 'scope'
+
+  - package-ecosystem: 'npm'
+    directory: '/test/fixtures/bare-async-failure-esm'
+    schedule:
+      interval: 'daily'
+    commit-message:
+      prefix: 'chore'
+      include: 'scope'
+
+  - package-ecosystem: 'npm'
+    directory: '/test/fixtures/bare-sync'
+    schedule:
+      interval: 'daily'
+    commit-message:
+      prefix: 'chore'
+      include: 'scope'
+
+  - package-ecosystem: 'npm'
+    directory: '/test/fixtures/bare-sync-esm'
+    schedule:
+      interval: 'daily'
+    commit-message:
+      prefix: 'chore'
+      include: 'scope'
+
+  - package-ecosystem: 'npm'
+    directory: '/test/fixtures/bare-sync-failure'
+    schedule:
+      interval: 'daily'
+    commit-message:
+      prefix: 'chore'
+      include: 'scope'
+
+  - package-ecosystem: 'npm'
+    directory: '/test/fixtures/bare-sync-failure-esm'
+    schedule:
+      interval: 'daily'
+    commit-message:
+      prefix: 'chore'
+      include: 'scope'
+
+  - package-ecosystem: 'npm'
+    directory: '/test/fixtures/mocha'
+    schedule:
+      interval: 'daily'
+    commit-message:
+      prefix: 'chore'
+      include: 'scope'
+
+  - package-ecosystem: 'npm'
+    directory: '/test/fixtures/mocha-esm'
+    schedule:
+      interval: 'daily'
+    commit-message:
+      prefix: 'chore'
+      include: 'scope'
+
+  - package-ecosystem: 'npm'
+    directory: '/test/fixtures/mocha-failure'
+    schedule:
+      interval: 'daily'
+    commit-message:
+      prefix: 'chore'
+      include: 'scope'
+
+  - package-ecosystem: 'npm'
+    directory: '/test/fixtures/mocha-failure-esm'
+    schedule:
+      interval: 'daily'
+    commit-message:
+      prefix: 'chore'
+      include: 'scope'
+
+  - package-ecosystem: 'npm'
+    directory: '/test/fixtures/tape'
+    schedule:
+      interval: 'daily'
+    commit-message:
+      prefix: 'chore'
+      include: 'scope'
+
+  - package-ecosystem: 'npm'
+    directory: '/test/fixtures/tape-esm'
+    schedule:
+      interval: 'daily'
+    commit-message:
+      prefix: 'chore'
+      include: 'scope'
+
+  - package-ecosystem: 'npm'
+    directory: '/test/fixtures/tape-failure'
+    schedule:
+      interval: 'daily'
+    commit-message:
+      prefix: 'chore'
+      include: 'scope'
+
+  - package-ecosystem: 'npm'
+    directory: '/test/fixtures/tape-failure-esm'
+    schedule:
+      interval: 'daily'
+    commit-message:
+      prefix: 'chore'
+      include: 'scope'
+
+  - package-ecosystem: 'npm'
+    directory: '/test/fixtures/webpack-merge'
+    schedule:
+      interval: 'daily'
+    commit-message:
+      prefix: 'chore'
+      include: 'scope'
+

package/.github/workflows/test-and-release.yml

@@ -0,0 +1,61 @@
+name: Test & Maybe Release
+on: [push, pull_request]
+jobs:
+  test:
+    strategy:
+      fail-fast: false
+      matrix:
+        node: [14.x, 16.x]
+        os: [macos-latest, ubuntu-latest, windows-latest]
+    runs-on: ${{ matrix.os }}
+    steps:
+      - name: Checkout Repository
+        uses: actions/checkout@v2.3.4
+      - name: Use Node.js ${{ matrix.node }}
+        uses: actions/setup-node@v2.4.0
+        with:
+          node-version: ${{ matrix.node }}
+      - name: Install Dependencies
+        run: |
+          npm install --no-progress
+      - name: Run tests
+        run: |
+          npm config set script-shell bash
+          npm test
+  release:
+    name: Release
+    needs: test
+    runs-on: ubuntu-latest
+    if: github.event_name == 'push' && github.ref == 'refs/heads/master'
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v2.3.4
+        with:
+          fetch-depth: 0
+      - name: Setup Node.js
+        uses: actions/setup-node@v2.4.0
+        with:
+          node-version: 14
+      - name: Install dependencies
+        run: |
+          npm install --no-progress --no-package-lock --no-save
+      - name: Build
+        run: |
+          npm run build
+      - name: Install plugins
+        run: |
+          npm install \
+            @semantic-release/commit-analyzer \
+            conventional-changelog-conventionalcommits \
+            @semantic-release/release-notes-generator \
+            @semantic-release/npm \
+            @semantic-release/github \
+            @semantic-release/git \
+            @semantic-release/changelog \
+            --no-progress --no-package-lock --no-save
+      - name: Release
+        env:
+          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
+          NPM_TOKEN: ${{ secrets.NPM_TOKEN }}
+        run: npx semantic-release
+

package/lib/log.js

@@ -1,5 +1,7 @@
-const colors = require('ansi-colors')
-const stripAnsi = require('strip-ansi')
+import process from 'process'
+import colors from 'ansi-colors'
+import stripAnsi from 'strip-ansi'
+
 const tty = process.stdout.isTTY && process.stderr.isTTY
 
 function brand (error) {
@@ -7,27 +9,25 @@ function brand (error) {
   return tty ? `${colors[error ? 'red' : 'green'](brand)}` : brand
 }
 
-function log (msg) {
+export function log (msg) {
   if (msg === undefined) {
     return console.log()
   }
   console.log(`  ${brand()}   ${tty ? colors.gray.italic(msg) : msg}`)
 }
 
-function error (msg) {
+export function error (msg) {
   console.log(`  ${brand(true)}   ${tty ? colors.gray.italic(msg) : msg}`)
 }
 
-function logRaw (type, args) {
+export function logRaw (type, args) {
   if (!tty) {
     args = args.map((a) => typeof a === 'string' ? stripAnsi(a) : a)
   }
   console[type === 'info' ? 'log' : type].apply(null, args)
 }
 
-function logWrite (args) {
+export function logWrite (args) {
   args = Array.isArray(args) ? args : [args]
   process.stdout.write.apply(process.stdout, args)
 }
-
-module.exports = { log, error, logRaw, logWrite }

package/lib/puppeteer.js

@@ -1,13 +1,13 @@
-const puppeteer = require('puppeteer')
+import puppeteer from 'puppeteer'
 // TODO
 // const pti = require('puppeteer-to-istanbul')
 // const { copyFile } = require('fs').promises
-const { log, error, logRaw, logWrite } = require('./log')
+import { log, error, logRaw, logWrite } from './log.js'
 
 // wrap our convoluted _run() function in a pure Promise that can handle
 // both standard throws and the callback that ends it. _run() needs to handle
 // ends in a few different ways, hence the hack.
-function run (outputDir, port, timeout, mode, runner, coverage) {
+export function run (outputDir, port, timeout, mode, runner, coverage) {
   return new Promise((resolve, reject) => {
     _run(outputDir, port, timeout, mode, runner, coverage, (err, errors) => {
       if (err) {
@@ -79,6 +79,9 @@ async function _run (outputDir, port, timeout, mode, runner, coverage, callback)
     maybeEnd()
   })
 
+  page.on('error', (error) => console.error(error))
+  page.on('pageerror', (error) => console.error(error))
+
   await page.exposeFunction('polendinaEnd', (errors) => {
     end(errors)
   })
@@ -109,5 +112,3 @@ async function _run (outputDir, port, timeout, mode, runner, coverage, callback)
     browser.close().then(cb).catch(cb)
   }, timeout * 1000).unref()
 }
-
-module.exports = run

package/lib/webpack.config.js

@@ -1,11 +1,14 @@
-const path = require('path')
-const glob = require('glob')
+import path from 'path'
+import glob from 'glob'
+import { fileURLToPath } from 'url'
 
-module.exports = function (env, options, runner) {
-  const bundleRun = path.resolve(options.outputDir, runner)
+const __dirname = fileURLToPath(path.dirname(import.meta.url))
+
+export function webpackConfig (env, options, runner) {
+  const bundleRun = path.normalize(path.resolve(options.outputDir, runner))
   const testFiles = options._.reduce((p, c) => {
     return p.concat(glob.sync(c, { absolute: true }))
-  }, [])
+  }, []).map(path.normalize)
 
   // our globs may be fruitless
   if (!testFiles.length) {
@@ -16,6 +19,7 @@ module.exports = function (env, options, runner) {
 
   return {
     mode: 'development',
+    context: process.cwd(),
     entry: testFiles,
     output: {
       path: path.resolve(process.cwd(), options.outputDir),
@@ -27,31 +31,24 @@ module.exports = function (env, options, runner) {
     },
     resolve: {
       modules: [
-        path.join(__dirname, '../node_modules'),
-        path.join(process.cwd(), 'node_modules')
+        path.join(process.cwd(), 'node_modules'),
+        path.join(__dirname, '../node_modules')
       ]
     },
     resolveLoader: {
       modules: [
-        path.join(__dirname, '../node_modules'),
-        path.join(process.cwd(), 'node_modules')
+        path.join(process.cwd(), 'node_modules'),
+        path.join(__dirname, '../node_modules')
       ]
     },
-    node: {
-      dgram: 'empty',
-      fs: 'empty',
-      net: 'empty',
-      tls: 'empty',
-      dns: 'empty',
-      child_process: 'empty'
-    },
     module: {
       rules: [
         {
           test: testFiles,
           exclude: [/node_modules/, bundleRun],
+          type: 'javascript/auto', // needed so the wrap-loader can get a sync require() in
           use: [{
-            loader: require.resolve('./wrap-loader.js'),
+            loader: path.resolve(__dirname, 'wrap-loader.cjs'),
             options: options
           }]
         }

package/lib/{wrap-loader.js → wrap-loader.cjs}

@@ -5,11 +5,11 @@
 
 const path = require('path')
 
-function loader (src) {
+function wrapLoader (src) {
   return src
 }
 
-loader.pitch = function pitch (remainingRequest) {
+wrapLoader.pitch = function pitch (remainingRequest) {
   const argv = this.query
 
   // probably redundant
@@ -17,15 +17,18 @@ loader.pitch = function pitch (remainingRequest) {
     return
   }
 
-  // wrap the test in a function that is only called when we want it called
+  // wrap the test in a function that is only called when we want it called, using CJS here because
+  // we get synchronous require() out of it, this needs the tests to be loaded with javascript/auto
+  // so webpack can auto-detect
   return `
-    const registry = require(${JSON.stringify(`!!${path.resolve(argv.outputDir, 'test-registry.js')}`)})
-    registry.argv = JSON.parse('${JSON.stringify(argv)}')
+    const { registry } = require(${JSON.stringify(`!!${path.normalize(path.resolve(argv.outputDir, 'test-registry.js'))}`)})
+    registry.argv = JSON.parse(${JSON.stringify(JSON.stringify(argv))})
     registry.tests.push({
-      name: JSON.parse('${JSON.stringify(path.relative(process.cwd(), remainingRequest))}'),
+      name: JSON.parse('${JSON.stringify(path.normalize(path.relative(process.cwd(), remainingRequest)))}'),
       load: () => { return require(${JSON.stringify(`!!${remainingRequest}`)}) }
     })
+    module.exports = {}
   `
 }
 
-module.exports = loader
+module.exports = wrapLoader

package/package.json

@@ -1,33 +1,33 @@
 {
   "name": "polendina",
-  "version": "1.0.0",
+  "version": "2.0.0-alpha",
   "description": "Non-UI browser testing for JavaScript libraries from the command-line",
   "main": "polendina.js",
+  "type": "module",
   "dependencies": {
     "ansi-colors": "^4.1.1",
-    "glob": "^7.1.6",
-    "puppeteer": "^2.0.0",
-    "raw-loader": "^4.0.0",
-    "rimraf": "^3.0.0",
-    "st": "^2.0.0",
-    "strip-ansi": "^6.0.0",
-    "webpack": "^4.41.5",
-    "webpack-merge": "^4.2.2",
-    "yargs": "^15.1.0"
+    "glob": "^7.2.0",
+    "path-browserify": "^1.0.1",
+    "process": "^0.11.10",
+    "puppeteer": "~12.0.1",
+    "raw-loader": "~4.0.2",
+    "readable-stream": "^3.6.0",
+    "rimraf": "^3.0.2",
+    "st": "^3.0.0",
+    "stream-browserify": "^3.0.0",
+    "strip-ansi": "^6.0.1",
+    "webpack": "~5.65.0",
+    "webpack-merge": "~5.8.0",
+    "yargs": "^17.3.0"
   },
   "bin": {
     "polendina": "./polendina-cli.js",
     "polendina-node": "./polendina-node-cli.js"
   },
-  "devDependencies": {
-    "mocha": "^7.0.0",
-    "standard": "^14.3.1",
-    "tape": "^4.12.1"
-  },
   "scripts": {
-    "lint": "standard",
+    "lint": "npm_config_yes=true npx standard",
     "test:install": "for f in $(cd test/fixtures/; ls); do (cd test/fixtures/$f && grep devDependencies package.json > /dev/null && npm i --no-audit --no-fund --no-package-lock) || true; done",
-    "test:run": "mocha test/test-*.js",
+    "test:run": "npm_config_yes=true npx mocha test/test-*.js",
     "test": "npm run lint && npm run test:install && npm run test:run"
   },
   "repository": {
@@ -45,5 +45,86 @@
     "test"
   ],
   "author": "Rod <rod@vagg.org> (http://r.va.gg/)",
-  "license": "Apache-2.0"
+  "license": "Apache-2.0",
+  "release": {
+    "branches": [
+      "master"
+    ],
+    "plugins": [
+      [
+        "@semantic-release/commit-analyzer",
+        {
+          "preset": "conventionalcommits",
+          "releaseRules": [
+            {
+              "breaking": true,
+              "release": "major"
+            },
+            {
+              "revert": true,
+              "release": "patch"
+            },
+            {
+              "type": "feat",
+              "release": "minor"
+            },
+            {
+              "type": "fix",
+              "release": "patch"
+            },
+            {
+              "type": "chore",
+              "release": "patch"
+            },
+            {
+              "type": "docs",
+              "release": "patch"
+            },
+            {
+              "type": "test",
+              "release": "patch"
+            },
+            {
+              "scope": "no-release",
+              "release": false
+            }
+          ]
+        }
+      ],
+      [
+        "@semantic-release/release-notes-generator",
+        {
+          "preset": "conventionalcommits",
+          "presetConfig": {
+            "types": [
+              {
+                "type": "feat",
+                "section": "Features"
+              },
+              {
+                "type": "fix",
+                "section": "Bug Fixes"
+              },
+              {
+                "type": "chore",
+                "section": "Trivial Changes"
+              },
+              {
+                "type": "docs",
+                "section": "Trivial Changes"
+              },
+              {
+                "type": "test",
+                "section": "Tests"
+              }
+            ]
+          }
+        }
+      ],
+      "@semantic-release/changelog",
+      "@semantic-release/npm",
+      "@semantic-release/github",
+      "@semantic-release/git"
+    ]
+  }
 }

package/polendina-cli.js

@@ -1,12 +1,13 @@
 #!/usr/bin/env node
 
-const start = Date.now()
-
-const path = require('path')
-const { log } = require('./lib/log')
-const Polendina = require('./polendina')
+import path from 'path'
+import yargs from 'yargs'
+import { hideBin } from 'yargs/helpers'
+import { log } from './lib/log.js'
+import { Polendina } from './polendina.js'
 
-const argv = require('yargs')
+const start = Date.now()
+const argv = yargs(hideBin(process.argv))
   .usage('$0 testfile.js [testfile2.js [tests/**/test*.js ...] ]')
   .option('runner', {
     alias: 'r',

package/polendina-node-cli.js

@@ -1,12 +1,16 @@
 #!/usr/bin/env node
 
+import process from 'process'
+import path from 'path'
+import glob from 'glob'
+import stripAnsi from 'strip-ansi'
+import yargs from 'yargs'
+import { hideBin } from 'yargs/helpers'
+import { BareRunner } from './resources/bare.js'
+
 const start = Date.now()
-const glob = require('glob')
-const path = require('path')
-const stripAnsi = require('strip-ansi')
 const tty = process.stdout.isTTY && process.stderr.isTTY
-const BareRunner = require('./resources/bare')
-const argv = require('yargs')
+const argv = yargs(hideBin(process.argv))
   .command('bare-sync <test files..>', 'Run synchronous tests using a plain require(file)')
   .command('bare-async <test files..>', 'Run tests by executing exported functions from files as async')
   .demandCommand()
@@ -42,7 +46,7 @@ async function run () {
   const tests = testFiles.map((f) => {
     return {
       name: path.relative(process.cwd(), f),
-      load: () => require(path.resolve(process.cwd(), f))
+      load: () => import(new URL(`file://${path.resolve(process.cwd(), f)}`))
     }
   })
   const log = {

package/polendina.js

@@ -1,14 +1,21 @@
-const { promisify } = require('util')
-const path = require('path')
-const fs = require('fs').promises
-const http = require('http')
-const st = require('st')
-const rimraf = promisify(require('rimraf'))
-const webpack = promisify(require('webpack'))
-const merge = require('webpack-merge')
-const puppeteer = require('./lib/puppeteer')
-
-class Polendina {
+import { promisify } from 'util'
+import path from 'path'
+import fs from 'fs/promises'
+import http from 'http'
+import { fileURLToPath } from 'url'
+import st from 'st'
+import _rimraf from 'rimraf'
+import _webpack from 'webpack'
+import { merge } from 'webpack-merge'
+import { run as puppeteer } from './lib/puppeteer.js'
+import { webpackConfig } from './lib/webpack.config.js'
+
+const rimraf = promisify(_rimraf)
+const webpack = promisify(_webpack)
+
+const __dirname = fileURLToPath(path.dirname(import.meta.url))
+
+export class Polendina {
   constructor (options) {
     this._options = options
 
@@ -32,11 +39,27 @@ class Polendina {
   }
 
   async build () {
-    let webpackConfig = require('./lib/webpack.config')(process.env, this._options, this._runnerModule)
+    let config = webpackConfig(process.env, this._options, this._runnerModule)
+
+    if (this._options.runner === 'tape') {
+      config = merge(config, {
+        resolve: {
+          fallback: {
+            stream: path.join(__dirname, 'node_modules', 'stream-browserify'),
+            path: path.join(__dirname, 'node_modules', 'path-browserify')
+          }
+        },
+        plugins: [
+          new webpack.ProvidePlugin({
+            process: 'process/browser'
+          })
+        ]
+      })
+    }
 
     if (this._options.webpackConfig) {
-      const userConfig = require(path.join(process.cwd(), this._options.webpackConfig))
-      webpackConfig = merge.smart(webpackConfig, userConfig)
+      const userConfig = (await import(`file://${path.join(process.cwd(), this._options.webpackConfig)}`)).default
+      config = merge(config, userConfig)
     }
 
     await fs.mkdir(this.outputDir, { recursive: true })
@@ -48,7 +71,7 @@ class Polendina {
       return fs.copyFile(path.join(__dirname, 'resources', file), path.join(this.outputDir, file))
     }))
 
-    const stats = await webpack(webpackConfig)
+    const stats = await webpack(config)
     const info = stats.toJson()
 
     if (stats.hasErrors()) {
@@ -65,7 +88,7 @@ class Polendina {
     this.bundleFile = info.assetsByChunkName.main[0]
 
     if (this._options.stats) {
-      this.statsFile = path.join(webpackConfig.output.path, 'webpack-stats.json')
+      this.statsFile = path.join(config.output.path, 'webpack-stats.json')
       await fs.writeFile(this.statsFile, JSON.stringify(info), 'utf8')
     }
   }
@@ -89,7 +112,12 @@ class Polendina {
   }
 
   _executeMode (mode) {
-    const mount = st({ path: this.outputDir, index: 'index.html', url: '/' })
+    const mount = st({
+      path: this.outputDir,
+      index: 'index.html',
+      url: '/',
+      cache: false
+    })
     return new Promise((resolve, reject) => {
       const server = http.createServer((req, res) => {
         mount(req, res, () => {
@@ -114,5 +142,3 @@ class Polendina {
     })
   }
 }
-
-module.exports = Polendina