PoC for replacing mocha with tape

TODO.md

@@ -0,0 +1,55 @@
+# TODO: Migrate Dynalite Tests from Mocha to Tape
+
+## Rules - never delete this
+
+- Make sure to never change the signature of helpers without refactoring all the code that uses them. Use static analysis in that case. You mess up when faced with a lot of things to refactor. Let's NOT make this mistake again.
+
+- When converting to tape - test files individually, but when rewriting helpers ALWAYS run all the tests after to make sure we're cuasing regressions
+
+- ALSO before checkin - run ALL tape tests to make sure we haven't caused regressions 
+
+## TODO
+
+| File                                | LOC  | Status      | Notes                                     |
+|-------------------------------------|-----:|-------------|-------------------------------------------|
+| test-tape/mocha-source-split/bench.js | 46   | ✅ Converted | Kept skipped, uses helpers.batchBulkPut, helpers.request |
+| test-tape/mocha-source-split/getItem.part1.js | 52   | ✅ Converted | Type checks using helpers.assertType |
+| test-tape/mocha-source-split/describeTable.js | 56   | ✅ Converted | Type/validation checks using helpers |
+| test-tape/mocha-source-split/batchGetItem.part1.js | 61   | ✅ Converted | Type checks using helpers.assertType |
+| test-tape/mocha-source-split/batchWriteItem.part1.js | 62   | ✅ Converted |                                           |
+| test-tape/mocha-source-split/describeTimeToLive.js | 71   | ✅ Converted |                                           |
+| test-tape/mocha-source-split/deleteItem.part1.js | 77   | ✅ Converted |                                           |
+| test-tape/mocha-source-split/putItem.part1.js | 79   | ✅ Converted |                                           |
+| test-tape/mocha-source-split/untagResource.js | 87   | ✅ Converted | Fixed assertion discrepancy vs Mocha src. |
+| test-tape/mocha-source-split/tagResource.js | 95   | ✅ Converted | Fixed assertion discrepancy vs Mocha src. |
+| test-tape/mocha-source-split/updateItem.part1.js | 100  | ✅ Converted |                                           |
+| test-tape/mocha-source-split/deleteTable.js | 106  | ✅ Converted |                                           |
+| test-tape/mocha-source-split/scan.part1.js | 107  | ✅ Converted |                                           |
+| test-tape/mocha-source-split/updateTable.part1.js | 121  | ✅ Converted |                                           |
+| test-tape/mocha-source-split/listTagsOfResource.js | 125  | ✅ Converted | Fixed assertion/ARN issues.            |
+| test-tape/mocha-source-split/query.part1.js | 132  | ✅ Converted |                                           |
+| test-tape/mocha-source-split/createTable.part1.js | 166  | ✅ Converted |                                           |
+| test-tape/mocha-source-split/updateTable.part3.js | 195  | ✅ Converted | Skipped long-running PAY_PER_REQUEST test |
+| test-tape/mocha-source-split/updateTable.part2.js | 214  | ✅ Converted |                                           |
+| test-tape/mocha-source-split/getItem.part3.js | 225  | ✅ Converted |                                           |
+| test-tape/mocha-source-split/batchWriteItem.part3.js | 238  | ✅ Converted | Includes skipped throughput test.        |
+| test-tape/mocha-source-split/deleteItem.part3.js | 244  | ✅ Converted | Updated capacity expectation.            |
+| test-tape/mocha-source-split/listTables.js | 268  | ✅ Converted | Skipped assertions for Limit+ExclusiveStartTableName combo. See plans/discrepancies.md |
+| test-tape/mocha-source-split/createTable.part3.js | 322  | ✅ Converted | Fixed ARN regex, LSI comparison.        |
+| test-tape/mocha-source-split/batchGetItem.part3.js | 343  | ✅ Converted | Includes skipped throughput test.         |
+| test-tape/mocha-source-split/batchGetItem.part2.js | 352  | ✅ Converted |                                           |
+| test-tape/mocha-source-split/getItem.part2.js | 364  | ✅ Converted |                                           |
+| test-tape/mocha-source-split/batchWriteItem.part2.js | 370  | ✅ Converted |                                           |
+| test-tape/mocha-source-split/deleteItem.part2.js | 382  | ✅ Converted |                                           |
+| test-tape/mocha-source-split/connection.js | 387  | ✅ Converted | Skipped 413 test, see discrepancies.md     |
+| test-tape/mocha-source-split/putItem.part2.js | 486  | ⬜ Not started |                                           |
+| test-tape/mocha-source-split/updateItem.part3.js | 666  | ⬜ Not started |                                           |
+| test-tape/mocha-source-split/updateItem.part2.js | 902  | ⬜ Not started |                                           |
+| test-tape/mocha-source-split/putItem.part3.js | 980  | ⬜ Not started |                                           |
+| test-tape/mocha-source-split/createTable.part2.js | 984  | ⬜ Not started |                                           |
+| test-tape/mocha-source-split/scan.part2.js | 1068 | ⬜ Not started |                                           |
+| test-tape/mocha-source-split/query.part3.js | 1485 | ⬜ Not started |                                           |
+| test-tape/mocha-source-split/query.part2.js | 1780 | ⬜ Not started |                                           |
+| test-tape/mocha-source-split/scan.part3.js | 2719 | ⬜ Not started |                                           |
+
+

package.json

@@ -12,9 +12,9 @@
   "bin": "cli.js",
   "scripts": {
     "build": "for file in ./db/*.pegjs; do pegjs \"$file\"; done",
-    "test": "npm run lint && mocha --require should --reporter spec -t $([ $REMOTE ] && echo 30s || echo 4s)",
-    "coverage": "npx nyc@latest mocha --require should -t 4s",
-    "lint": "eslint . --fix"
+    "test": "npm run lint && node test-tape/run-tape-suite.js | tap-spec",
+    "coverage": "npx nyc@latest node test-tape/run-tape-suite.js | tap-spec",
+    "lint": "eslint index.js cli.js actions/ db/ validations/ test-tape/convert-to-tape/ test-tape/scripts/ --fix"
   },
   "engines": {
     "node": ">=16"
@@ -40,9 +40,13 @@
     "@architect/eslint-config": "^2.1.1",
     "aws4": "^1.12.0",
     "eslint": "^8.48.0",
-    "mocha": "^10.2.0",
+    "glob": "^11.0.1",
+    "mocha": "^11.1.0",
     "pegjs": "^0.10.0",
-    "should": "^13.2.3"
+    "recast": "^0.23.9",
+    "should": "^13.2.3",
+    "tap-spec": "^5.0.0",
+    "tape": "^5.8.1"
   },
   "keywords": [
     "aws",
@@ -58,11 +62,13 @@
   ],
   "eslintConfig": {
     "extends": "@architect/eslint-config",
-    "env": {
-      "mocha": true
-    },
+    "env": {},
     "rules": {
-      "filenames/match-regex": [ "error", "^[a-zA-Z0-9-_.]+$", true ]
+      "filenames/match-regex": [
+        "error",
+        "^[a-zA-Z0-9-_.]+$",
+        true
+      ]
     }
   }
 }