Add automated tests (#39)

.github/workflows/ci.yml

@@ -1,27 +1,34 @@
 env: {}
 
-# FILE GENERATED WITH: npx ghat fregante/ghatemplates/node
-# SOURCE: https://github.com/fregante/ghatemplates
-# OPTIONS: {"exclude":["jobs.Test"]}
+# FILE GENERATED WITH: npx ghat fregante/content-scripts-register-polyfill/.github/workflows/ci.yml
+# SOURCE: https://github.com/fregante/content-scripts-register-polyfill
+
+name: Test
 
-name: CI
 on:
   - pull_request
   - push
+
 jobs:
   Lint:
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v2
-      - name: install
-        run: npm ci || npm install
-      - name: XO
-        run: npx xo
+      - uses: actions/checkout@v3
+      - run: npm ci
+      - run: npx xo
+
   Build:
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v2
-      - name: install
-        run: npm ci || npm install
-      - name: build
-        run: npm run build
+      - uses: actions/checkout@v3
+      - run: npm ci
+      - run: npm run build
+      - run: npm run demo:build
+
+  Test:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v3
+      - run: sudo apt-get install xvfb
+      - run: npm ci
+      - run: xvfb-run --auto-servernum npm run jest

.gitignore

@@ -10,3 +10,5 @@ logs
 *.map
 index.js
 index.d.ts
+.parcel-cache
+dist

.parcelrc

@@ -0,0 +1,3 @@
+{
+	"extends": "@parcel/config-webextension"
+}

how-to-add-github-enterprise-support-to-web-extensions.md

@@ -25,10 +25,11 @@ import addDomainPermissionToggle from 'webext-domain-permission-toggle';
 addDomainPermissionToggle();
 ```
 
-## manifest.json example
+## manifest.json v2 example
 
 ```js
 {
+	"version": 2,
 	"permissions": [
 		"https://github.com/*",
 		"contextMenus",
@@ -49,15 +50,9 @@ addDomainPermissionToggle();
 	},
 	"content_scripts": [
 		{
-			"matches": [
-				"https://github.com/*"
-			],
-			"css": [
-				"content.css"
-			],
-			"js": [
-				"content.js"
-			]
+			"matches": ["https://github.com/*"],
+			"css": ["content.css"],
+			"js": ["content.js"]
 		}
 	]
 }

index.ts

@@ -19,7 +19,7 @@ function convertPath(file: string): browser.extensionTypes.ExtensionFileOrCode {
 	return {file: url.pathname};
 }
 
-function injectOnExistingTabs(origins: string[], scripts: ContentScripts) {
+function injectToExistingTabs(origins: string[], scripts: ContentScripts) {
 	if (origins.length === 0) {
 		return;
 	}
@@ -42,7 +42,7 @@ async function registerOnOrigins({
 	const manifest = chrome.runtime.getManifest().content_scripts;
 
 	if (!manifest) {
-		throw new Error('webext-dynamic-content-scripts tried to register scripts on th new host permissions, but no content scripts were found in the manifest.');
+		throw new Error('webext-dynamic-content-scripts tried to register scripts on the new host permissions, but no content scripts were found in the manifest.');
 	}
 
 	// Register one at a time to allow removing one at a time as well
@@ -60,7 +60,9 @@ async function registerOnOrigins({
 		}
 	}
 
-	injectOnExistingTabs(newOrigins || [], manifest);
+	// May not be needed in the future in Firefox
+	// https://bugzilla.mozilla.org/show_bug.cgi?id=1458947
+	injectToExistingTabs(newOrigins || [], manifest);
 }
 
 (async () => {

jest-puppeteer.config.cjs

@@ -0,0 +1,11 @@
+const path = require('path');
+
+module.exports = {
+	launch: {
+		headless: false,
+		args: [
+			'--disable-extensions-except=' + path.resolve(__dirname, 'test/dist'),
+			'--window-size=400,800',
+		],
+	},
+};