Move tests to be co-located with source files (#94)

Copilot · web-flow · commit c517d38e6283 · 2025-11-27T23:46:02.000Z
Fixes #93
diff --git a/lib/express-middleware/authentication.test.js b/lib/express-middleware/authentication.test.js
@@ -5,7 +5,7 @@ const cookieParser = require('cookie-parser')
 const express = require('express')
 const request = require('supertest')
 
-const authentication = require('../../lib/express-middleware/authentication')
+const authentication = require('./authentication')
 
 describe('authentication', () => {
   let app, originalPassword
diff --git a/lib/express-middleware/auto-routing.test.js b/lib/express-middleware/auto-routing.test.js
@@ -7,7 +7,7 @@ const express = require('express')
 const nunjucks = require('nunjucks')
 const request = require('supertest')
 
-const { matchRoutes } = require('../../lib/express-middleware/auto-routing')
+const { matchRoutes } = require('./auto-routing')
 
 describe('matchRoutes', () => {
   // Create test templates directory
diff --git a/lib/express-middleware/auto-store-data.test.js b/lib/express-middleware/auto-store-data.test.js
@@ -1,7 +1,7 @@
 const assert = require('node:assert')
 const { describe, it, beforeEach } = require('node:test')
 
-const autoStoreData = require('../../lib/express-middleware/auto-store-data')
+const autoStoreData = require('./auto-store-data')
 
 describe('autoStoreData middleware', () => {
   let req, res, nextCalled
diff --git a/lib/express-middleware/production-headers.test.js b/lib/express-middleware/production-headers.test.js
@@ -5,7 +5,7 @@ const { describe, it } = require('node:test')
 const express = require('express')
 const request = require('supertest')
 
-const productionHeaders = require('../../lib/express-middleware/production-headers')
+const productionHeaders = require('./production-headers')
 
 const app = express()
 app.use(productionHeaders)
diff --git a/lib/express-middleware/render-error-page.test.js b/lib/express-middleware/render-error-page.test.js
@@ -10,11 +10,11 @@ const request = require('supertest')
 const originalError = console.error
 let errorCalls = []
 
-const renderErrorPage = require('../../lib/express-middleware/render-error-page')
+const renderErrorPage = require('./render-error-page')
 
 const app = express()
 app.set('view engine', 'html')
-nunjucks.configure([join(__dirname, '.')], {
+nunjucks.configure([join(__dirname, 'test-templates')], {
   express: app,
   noCache: true
 })
diff --git a/lib/express-middleware/render-page-not-found.test.js b/lib/express-middleware/render-page-not-found.test.js
@@ -7,11 +7,11 @@ const express = require('express')
 const nunjucks = require('nunjucks')
 const request = require('supertest')
 
-const renderPageNotFound = require('../../lib/express-middleware/render-page-not-found')
+const renderPageNotFound = require('./render-page-not-found')
 
 const app = express()
 app.set('view engine', 'html')
-nunjucks.configure([join(__dirname, '.')], {
+nunjucks.configure([join(__dirname, 'test-templates')], {
   express: app,
   noCache: true
 })
diff --git a/lib/express-middleware/reset-session-data.test.js b/lib/express-middleware/reset-session-data.test.js
@@ -1,7 +1,7 @@
 const assert = require('node:assert')
 const { describe, it, beforeEach } = require('node:test')
 
-const resetSessionData = require('../../lib/express-middleware/reset-session-data')
+const resetSessionData = require('./reset-session-data')
 
 describe('resetSessionData middleware', () => {
   let req, res, nextCalled
diff --git a/lib/express-middleware/set-current-page-in-locals.test.js b/lib/express-middleware/set-current-page-in-locals.test.js
@@ -1,7 +1,7 @@
 const assert = require('node:assert')
 const { describe, it, beforeEach } = require('node:test')
 
-const setCurrentPageInLocals = require('../../lib/express-middleware/set-current-page-in-locals')
+const setCurrentPageInLocals = require('./set-current-page-in-locals')
 
 describe('setCurrentPageInLocals middleware', () => {
   let req, res, nextCalled
diff --git a/lib/express-middleware/set-session-data-defaults.test.js b/lib/express-middleware/set-session-data-defaults.test.js
@@ -1,7 +1,7 @@
 const assert = require('node:assert')
 const { describe, it, beforeEach } = require('node:test')
 
-const setSessionDataDefaults = require('../../lib/express-middleware/set-session-data-defaults')
+const setSessionDataDefaults = require('./set-session-data-defaults')
 
 describe('setSessionDataDefaults middleware', () => {
   let req, res, nextCalled
diff --git a/lib/express-middleware/test-templates/404.html b/lib/express-middleware/test-templates/404.html
diff --git a/lib/express-middleware/test-templates/500.html b/lib/express-middleware/test-templates/500.html
diff --git a/lib/express-middleware/test-templates/error.html b/lib/express-middleware/test-templates/error.html
diff --git a/lib/express-middleware/test-templates/index.html b/lib/express-middleware/test-templates/index.html
diff --git a/lib/express-middleware/test-templates/nested/index.html b/lib/express-middleware/test-templates/nested/index.html
diff --git a/lib/express-middleware/test-templates/simple.html b/lib/express-middleware/test-templates/simple.html
diff --git a/lib/express-settings/query-parser.test.js b/lib/express-settings/query-parser.test.js
@@ -3,7 +3,7 @@ const { describe, it } = require('node:test')
 
 const express = require('express')
 
-const setQueryParser = require('../../lib/express-settings/query-parser')
+const setQueryParser = require('./query-parser')
 
 const app = express()
 
diff --git a/lib/express-settings/trust-proxy.test.js b/lib/express-settings/trust-proxy.test.js
@@ -3,7 +3,7 @@ const { describe, it } = require('node:test')
 
 const express = require('express')
 
-const setTrustProxy = require('../../lib/express-settings/trust-proxy')
+const setTrustProxy = require('./trust-proxy')
 
 const app = express()
 
diff --git a/lib/express-settings/view-engine.test.js b/lib/express-settings/view-engine.test.js
@@ -3,7 +3,7 @@ const { describe, it } = require('node:test')
 
 const express = require('express')
 
-const setViewEngine = require('../../lib/express-settings/view-engine')
+const setViewEngine = require('./view-engine')
 
 const app = express()
 
diff --git a/lib/nunjucks-filters/format-nhs-number.test.js b/lib/nunjucks-filters/format-nhs-number.test.js
@@ -1,7 +1,7 @@
 const assert = require('node:assert')
 const { describe, it } = require('node:test')
 
-const formatNhsNumber = require('../../lib/nunjucks-filters/format-nhs-number')
+const formatNhsNumber = require('./format-nhs-number')
 
 describe('formatNhsNumber', () => {
   it('should format a string of 10 digits', () => {
diff --git a/lib/nunjucks-filters/starts-with.test.js b/lib/nunjucks-filters/starts-with.test.js
@@ -1,7 +1,7 @@
 const assert = require('node:assert')
 const { describe, it } = require('node:test')
 
-const startsWith = require('../../lib/nunjucks-filters/starts-with')
+const startsWith = require('./starts-with')
 
 describe('startsWith', () => {
   it('matches string', () => {
diff --git a/package.json b/package.json
@@ -59,7 +59,9 @@
     "supertest": "7.1.4"
   },
   "files": [
-    "lib"
+    "lib",
+    "!lib/**/*.test.js",
+    "!lib/**/test-templates"
   ],
   "exports": {
     ".": "./lib/index.js",
