@openhi/constructs 0.0.83 → 0.0.85

package/package.json

@@ -31,10 +31,10 @@
     "constructs": "10.6.0"
   },
   "dependencies": {
-    "@aws-sdk/client-dynamodb": "^3.1032.0",
-    "@aws-sdk/client-eventbridge": "^3.1032.0",
-    "@aws-sdk/client-s3": "^3.1032.0",
-    "@aws-sdk/client-ssm": "^3.1032.0",
+    "@aws-sdk/client-dynamodb": "^3.1033.0",
+    "@aws-sdk/client-eventbridge": "^3.1033.0",
+    "@aws-sdk/client-s3": "^3.1033.0",
+    "@aws-sdk/client-ssm": "^3.1033.0",
     "@codedrifters/utils": "^0.0.20",
     "@codegenie/serverless-express": "^4.17.1",
     "@types/aws-lambda": "^8.10.161",
@@ -60,7 +60,7 @@
   "publishConfig": {
     "access": "public"
   },
-  "version": "0.0.83",
+  "version": "0.0.85",
   "types": "lib/index.d.ts",
   "//": "~~ Generated by projen. To modify, edit .projenrc.js and run \"pnpm exec projen\".",
   "scripts": {
@@ -78,6 +78,7 @@
     "reset": "projen reset",
     "run-lambda-local": "projen run-lambda-local",
     "test": "projen test",
+    "test:coverage": "projen test:coverage",
     "test:watch": "projen test:watch",
     "unbump": "projen unbump",
     "upgrade": "projen upgrade",

package/scripts/generate-route-tests.mjs

@@ -1,227 +0,0 @@
-#!/usr/bin/env node
-/**
- * Generate one route test file per FHIR type that has rest-api routes.
- * Run from constructs package: node scripts/generate-route-tests.mjs
- * Ensures routes/data/<folder>/<folder>.test.ts exists for each type.
- */
-import fs from "node:fs";
-import path from "node:path";
-import { fileURLToPath } from "node:url";
-
-const __dirname = path.dirname(fileURLToPath(import.meta.url));
-const ROOT = path.resolve(__dirname, "..");
-const ROUTES_DATA = path.join(ROOT, "src/data/rest-api/routes/data");
-const OPERATIONS = path.join(ROOT, "src/data/operations/data");
-
-const dirs = fs.readdirSync(OPERATIONS).filter((d) => {
-  const p = path.join(OPERATIONS, d);
-  return fs.statSync(p).isDirectory() && fs.existsSync(path.join(p, `${d}-create-operation.ts`));
-});
-
-const types = [];
-for (const dir of dirs) {
-  const createPath = path.join(OPERATIONS, dir, `${dir}-create-operation.ts`);
-  const line = fs.readFileSync(createPath, "utf8").split("\n")[0];
-  const match = line.match(/import type \{ ([^}]+) \}/);
-  if (!match) continue;
-  const imports = match[1].split(",").map((s) => s.trim());
-  const type = imports.find((t) => t !== "Meta");
-  if (type) types.push({ type, folder: dir });
-}
-types.sort((a, b) => a.type.localeCompare(b.type));
-
-function basePathKey(type) {
-  return type.toUpperCase().replace(/-/g, "");
-}
-
-function pluralRoute(type) {
-  return type + "s";
-}
-
-function testContent(type, folder) {
-  const key = basePathKey(type);
-  const plural = pluralRoute(type);
-  const createOp = `create${type}Operation`;
-  const updateOp = `update${type}Operation`;
-  const listOp = `list${plural}Operation`;
-  const getByIdOp = `get${type}ByIdOperation`;
-  const deleteOp = `delete${type}Operation`;
-  const createRoute = `create${type}Route`;
-  const updateRoute = `update${type}Route`;
-  const listRoute = `list${plural}Route`;
-  const getByIdRoute = `get${type}ByIdRoute`;
-  const deleteRoute = `delete${type}Route`;
-
-  return `import type { Request, Response } from "express";
-import { NotFoundError } from "../../../../errors";
-import { ${createRoute} } from "./${folder}-create-route";
-import { ${deleteRoute} } from "./${folder}-delete-route";
-import { ${getByIdRoute} } from "./${folder}-get-by-id-route";
-import { ${listRoute} } from "./${folder}-list-route";
-import { ${updateRoute} } from "./${folder}-update-route";
-import { create${type}Operation } from "../../../../operations/data/${folder}/${folder}-create-operation";
-import { delete${type}Operation } from "../../../../operations/data/${folder}/${folder}-delete-operation";
-import { get${type}ByIdOperation } from "../../../../operations/data/${folder}/${folder}-get-by-id-operation";
-import { list${plural}Operation } from "../../../../operations/data/${folder}/${folder}-list-operation";
-import { update${type}Operation } from "../../../../operations/data/${folder}/${folder}-update-operation";
-
-jest.mock("../../../../operations/data/${folder}/${folder}-create-operation");
-jest.mock("../../../../operations/data/${folder}/${folder}-update-operation");
-jest.mock("../../../../operations/data/${folder}/${folder}-list-operation");
-jest.mock("../../../../operations/data/${folder}/${folder}-get-by-id-operation");
-jest.mock("../../../../operations/data/${folder}/${folder}-delete-operation");
-
-const defaultContext = {
-  tenantId: "tenant-1",
-  workspaceId: "ws-1",
-  date: "2026-02-26T12:00:00.000Z",
-  actorId: "actor-1",
-  actorName: "Test Actor",
-};
-
-function mockRes(): Response {
-  const res = {} as Response;
-  res.status = jest.fn().mockReturnThis();
-  res.json = jest.fn().mockReturnThis();
-  res.location = jest.fn().mockReturnThis();
-  res.send = jest.fn().mockReturnThis();
-  return res;
-}
-
-function mockReq(overrides: Partial<{ params: Record<string, string>; body: unknown }> = {}): Request {
-  const req = {
-    openhiContext: defaultContext,
-    params: {},
-    body: {},
-    ...overrides,
-  } as unknown as Request;
-  return req;
-}
-
-describe("${type} routes", () => {
-  beforeEach(() => {
-    jest.clearAllMocks();
-  });
-
-  describe("${createRoute}", () => {
-    it("returns 201 and location when create succeeds", async () => {
-      const id = "01HXYZ";
-      (create${type}Operation as jest.Mock).mockResolvedValue({
-        id,
-        resource: { resourceType: "${type}", id },
-      });
-      const req = mockReq({
-        body: { resourceType: "${type}" },
-      });
-      const res = mockRes();
-
-      await ${createRoute}(req, res);
-
-      expect(create${type}Operation).toHaveBeenCalledWith({
-        context: defaultContext,
-        body: expect.objectContaining({ resourceType: "${type}" }),
-      });
-      expect(res.status).toHaveBeenCalledWith(201);
-      expect(res.location).toHaveBeenCalled();
-      expect(res.json).toHaveBeenCalledWith(expect.objectContaining({ id }));
-    });
-  });
-
-  describe("${listRoute}", () => {
-    it("returns bundle when list succeeds", async () => {
-      (list${plural}Operation as jest.Mock).mockResolvedValue({
-        entries: [{ id: "01HXYZ", resource: { resourceType: "${type}", id: "01HXYZ" } }],
-      });
-      const req = mockReq();
-      const res = mockRes();
-
-      await ${listRoute}(req, res);
-
-      expect(list${plural}Operation).toHaveBeenCalledWith({ context: defaultContext });
-      expect(res.json).toHaveBeenCalledWith(
-        expect.objectContaining({
-          resourceType: "Bundle",
-          type: "searchset",
-          entry: expect.any(Array),
-        }),
-      );
-    });
-  });
-
-  describe("${getByIdRoute}", () => {
-    it("returns 404 when resource not found", async () => {
-      (get${type}ByIdOperation as jest.Mock).mockRejectedValue(new NotFoundError("not found"));
-      const req = mockReq({ params: { id: "01HXYZ" } });
-      const res = mockRes();
-
-      await ${getByIdRoute}(req, res);
-
-      expect(get${type}ByIdOperation).toHaveBeenCalledWith({
-        context: defaultContext,
-        id: "01HXYZ",
-      });
-      expect(res.status).toHaveBeenCalledWith(404);
-    });
-
-    it("returns resource when found", async () => {
-      const resource = { resourceType: "${type}", id: "01HXYZ" };
-      (get${type}ByIdOperation as jest.Mock).mockResolvedValue({ resource });
-      const req = mockReq({ params: { id: "01HXYZ" } });
-      const res = mockRes();
-
-      await ${getByIdRoute}(req, res);
-
-      expect(res.json).toHaveBeenCalledWith(resource);
-    });
-  });
-
-  describe("${updateRoute}", () => {
-    it("returns 200 and resource when update succeeds", async () => {
-      const resource = { resourceType: "${type}", id: "01HXYZ" };
-      (update${type}Operation as jest.Mock).mockResolvedValue({ resource });
-      const req = mockReq({ params: { id: "01HXYZ" }, body: { resourceType: "${type}" } });
-      const res = mockRes();
-
-      await ${updateRoute}(req, res);
-
-      expect(update${type}Operation).toHaveBeenCalledWith({
-        context: defaultContext,
-        id: "01HXYZ",
-        body: expect.objectContaining({ resourceType: "${type}", id: "01HXYZ" }),
-      });
-      expect(res.json).toHaveBeenCalledWith(resource);
-    });
-  });
-
-  describe("${deleteRoute}", () => {
-    it("returns 204 when delete succeeds", async () => {
-      (delete${type}Operation as jest.Mock).mockResolvedValue(undefined);
-      const req = mockReq({ params: { id: "01HXYZ" } });
-      const res = mockRes();
-
-      await ${deleteRoute}(req, res);
-
-      expect(delete${type}Operation).toHaveBeenCalledWith({
-        context: defaultContext,
-        id: "01HXYZ",
-      });
-      expect(res.status).toHaveBeenCalledWith(204);
-      expect(res.send).toHaveBeenCalled();
-    });
-  });
-});
-`;
-}
-
-for (const { type, folder } of types) {
-  const routeDir = path.join(ROUTES_DATA, folder);
-  const testFile = path.join(routeDir, `${folder}.test.ts`);
-  if (fs.existsSync(testFile)) {
-    console.log(`Skip ${type} (test exists)`);
-    continue;
-  }
-  fs.writeFileSync(testFile, testContent(type, folder), "utf8");
-  console.log(`Generated ${folder}.test.ts for ${type}`);
-}
-
-console.log(`Total: ${types.length} route types; ensured tests for all.`);