Drop build process, move tests next to source files

Co-authored-by: fregante <1402241+fregante@users.noreply.github.com>

Author: Copilot

.gitignore

@@ -1,4 +1,3 @@
 node_modules
 npm-debug.log
-distribution
 !test/fixtures/*/.DS_Store

.npmignore

@@ -1,3 +1,2 @@
 test
-source
-build.js
+source/**/*.test.js

build.js

@@ -22,18 +22,14 @@
   ],
   "type": "module",
   "bin": {
-    "chrome-webstore-upload": "distribution/cli.js"
+    "chrome-webstore-upload": "source/cli.js"
   },
   "scripts": {
-    "build": "node build.js",
-    "pretest": "npm run build",
     "test": "xo && ava"
   },
   "xo": {
     "ignore": [
-      "test/fixtures",
-      "distribution",
-      "build.js"
+      "test/fixtures"
     ],
     "rules": {
       "object-curly-spacing": [
@@ -45,6 +41,7 @@
   },
   "ava": {
     "files": [
+      "source/**/*.test.js",
       "!test/fixtures/**",
       "!test/helpers/**"
     ]

package.json

@@ -1,6 +1,6 @@
 import test from 'ava';
 import { execa } from 'execa';
-import { mockFileSystem } from './helpers/stubs.js';
+import { mockFileSystem } from '../test/helpers/stubs.js';
 
 function env(vars = {}) {
     return {
@@ -16,7 +16,7 @@ mockFileSystem();
 
 test('Exits w/ message when param required by "Web Store Upload" is not provided', async t => {
     try {
-        await execa('./distribution/cli.js', ['upload', '--source', 'test/fixtures/without-junk']);
+        await execa('./source/cli.js', ['upload', '--source', 'test/fixtures/without-junk']);
         t.fail('Should have errored');
     } catch (error) {
         t.regex(error.message, /Option ".+" is required/);
@@ -25,7 +25,7 @@ test('Exits w/ message when param required by "Web Store Upload" is not provided
 
 test('It should attempt to read ./manifest.json when the --source param is not provided', async t => {
     try {
-        await execa('./distribution/cli.js', ['upload'], {
+        await execa('./source/cli.js', ['upload'], {
             env: env(),
         });
         t.pass();

test/cli.js source/cli.test.jstest/cli.js renamed to source/cli.test.js

@@ -1,7 +1,7 @@
 import process from 'node:process';
 import test from 'ava';
-import createConfig from '../source/config.js';
-import { mockFileSystem } from './helpers/stubs.js';
+import { mockFileSystem } from '../test/helpers/stubs.js';
+import createConfig from './config.js';
 
 mockFileSystem();
 

test/config.js source/config.test.jstest/config.js renamed to source/config.test.js

@@ -1,6 +1,6 @@
 import process from 'node:process';
 import test from 'ava';
-import { stubConsoleLog } from './helpers/stubs.js';
+import { stubConsoleLog } from '../test/helpers/stubs.js';
 
 test('CWSError displays message with ❌ prefix and helpful links', t => {
     const logs = [];

test/error-handler.js source/error-handler.test.jstest/error-handler.js renamed to source/error-handler.test.js

@@ -1,8 +1,8 @@
 import path from 'node:path';
 import test from 'ava';
 import mockFs from 'mock-fs';
-import findSource from '../source/find-source.js';
-import { mockFileSystem } from './helpers/stubs.js';
+import { mockFileSystem } from '../test/helpers/stubs.js';
+import findSource from './find-source.js';
 
 mockFileSystem();
 

test/find-source.js source/find-source.test.jstest/find-source.js renamed to source/find-source.test.js

@@ -1,6 +1,6 @@
 import test from 'ava';
 import { isArchive,
-    isUploadSuccess } from '../source/util.js';
+    isUploadSuccess } from './util.js';
 
 test('isArchive', t => {
     t.true(isArchive('extension.zip'));