refactor: Squirrel away internal tools

Move `eslint` and `integration-test` directories into a `tools` directory.

.github/dependabot.yml

@@ -13,15 +13,15 @@ updates:
     schedule:
       interval: 'weekly'
       # somewhat arbitrary, this is to spread Dependabot PRs throughout the week to avoid Monday overload
-      day: "wednesday"      
+      day: "wednesday"
     commit-message:
       prefix: "chore(deps): "
   - package-ecosystem: 'npm'
-    directory: '/integration-test'
+    directory: '/tools/integration-test'
     schedule:
       interval: 'weekly'
       # somewhat arbitrary, this is to spread Dependabot PRs throughout the week to avoid Monday overload
-      day: "wednesday"      
+      day: "wednesday"
     commit-message:
       prefix: "chore(deps): "
     # The version of @aws-cdk/* libraries must match those from @guardian/cdk.

docs/006-testing.md

@@ -31,7 +31,7 @@ This stack isn't user facing. Accidentally causing destruction there is ok - bet
 
 <!-- only links below here -->
 [internal-testing-adr]: ./architecture-decision-records/004-testing.md
-[internal-integration-project]: ../integration-test
-[internal-integration-project-stack]: ../integration-test/src/integration-test-stack.ts
+[internal-integration-project]: ../tools/integration-test
+[internal-integration-project-stack]: ../tools/integration-test/src/integration-test-stack.ts
 
 [cdk-playground]: https://github.com/guardian/cdk-playground

jest.config.js

@@ -6,7 +6,7 @@ module.exports = {
   setupFilesAfterEnv: ["./jest.setup.js"],
 
   // ignore this file as it's used to demonstrate custom lint rule and Jest flags unused declarations
-  testPathIgnorePatterns: ["<rootDir>/eslint/rules/valid-constructors.test.ts"],
+  testPathIgnorePatterns: ["<rootDir>/tools/eslint/rules/valid-constructors.test.ts"],
 
   /*
   Ignore `lib` to prevent a 'duplicate manual mock found" warning

package.json

@@ -13,7 +13,7 @@
     "format": "prettier --write \"src/**/*.ts\"",
     "watch": "tsc -w",
     "test": "jest --detectOpenHandles --runInBand",
-    "test:custom-lint-rule": "eslint eslint/rules/valid-constructors.test.ts",
+    "test:custom-lint-rule": "eslint tools/eslint/rules/valid-constructors.test.ts",
     "test:dev": "jest --detectOpenHandles --runInBand --watch",
     "prepare": "tsc",
     "release": "semantic-release",
@@ -29,7 +29,7 @@
     "@types/node": "15.12.5",
     "cz-conventional-changelog": "^3.3.0",
     "eslint": "^7.29.0",
-    "eslint-plugin-custom-rules": "file:eslint",
+    "eslint-plugin-custom-rules": "file:tools/eslint",
     "gh-pages": "^3.2.3",
     "jest": "^26.4.2",
     "npm-check-updates": "^11.7.1",

script/ci

@@ -12,7 +12,7 @@ preamble() {
 
 runIntegrationTest() {
   (
-    cd "$ROOT_DIR/integration-test"
+    cd "$ROOT_DIR/tools/integration-test"
     ./script/ci
   )
 }

eslint/rules/valid-constructors.test.ts → tools/eslint/rules/valid-constructors.test.ts

@@ -1,6 +1,6 @@
 /* eslint-disable @typescript-eslint/no-unused-vars -- testing file */
 
-import type { GuStack } from "../../src/constructs/core";
+import type { GuStack } from "../../../src/constructs/core";
 
 interface MyProps {
   name: string;

tools/integration-test/.nvmrc

@@ -0,0 +1 @@
+../../.nvmrc

integration-test/package.json → tools/integration-test/package.json

@@ -29,7 +29,7 @@
     "typescript": "~4.3.4"
   },
   "dependencies": {
-    "@guardian/cdk": "file:.."
+    "@guardian/cdk": "file:../.."
   },
   "private": true
 }

tsconfig.eslint.json

@@ -2,7 +2,7 @@
   "extends": "./tsconfig.json",
   "include": [
     "src/**/*",
-    "eslint/rules/valid-constructors.test.ts"
+    "tools/eslint/rules/valid-constructors.test.ts"
   ],
   "exclude": [
     "node_modules"