@reventlessdev/reventless-seed 1.0.0-alpha.20 → 1.0.0-alpha.21

package/lib/ocaml/Seed_RandomTest.res

@@ -16,36 +16,45 @@ describe("Seed_Random:", () => {
   }
 
   describe("reproducibility — the reason this module exists:", () => {
-    testSync("the same seed replays the same sequence", () =>
-      expect(take(~seed=42, ~n=25))->toEqual(take(~seed=42, ~n=25))
+    testSync(
+      "the same seed replays the same sequence",
+      () => expect(take(~seed=42, ~n=25))->toEqual(take(~seed=42, ~n=25)),
     )
 
     // Without this, a generator that returned a constant would satisfy the
     // case above and nothing else in the suite would catch it.
-    testSync("different seeds diverge", () =>
-      expect(take(~seed=42, ~n=25) == take(~seed=43, ~n=25))->toBe(false)
+    testSync(
+      "different seeds diverge",
+      () => expect(take(~seed=42, ~n=25) == take(~seed=43, ~n=25))->toBe(false),
     )
 
-    testSync("a fresh generator does not continue the previous one", () => {
-      let r = Seed_Random.make(~seed=7)
-      let first = Seed_Random.float(r)
-      let _ = Seed_Random.float(r)
-      expect(Seed_Random.float(Seed_Random.make(~seed=7)))->toEqual(first)
-    })
+    testSync(
+      "a fresh generator does not continue the previous one",
+      () => {
+        let r = Seed_Random.make(~seed=7)
+        let first = Seed_Random.float(r)
+        let _ = Seed_Random.float(r)
+        expect(Seed_Random.float(Seed_Random.make(~seed=7)))->toEqual(first)
+      },
+    )
   })
 
   describe("float stays a unit interval:", () => {
     // mulberry32's last step is a signed 32-bit value folded back to unsigned.
     // Get that fold wrong and roughly half the draws come out negative, which
     // every consumer below (int, pick, sampleWeighted) silently mis-scales.
-    testSync("every draw is in [0, 1)", () =>
-      expect(take(~seed=99, ~n=500)->Array.every(f => f >= 0.0 && f < 1.0))->toBe(true)
+    testSync(
+      "every draw is in [0, 1)",
+      () => expect(take(~seed=99, ~n=500)->Array.every(f => f >= 0.0 && f < 1.0))->toBe(true),
     )
 
-    testSync("and the draws are not all the same value", () => {
-      let draws = take(~seed=99, ~n=100)
-      expect(draws->Array.some(f => f != draws->Array.getUnsafe(0)))->toBe(true)
-    })
+    testSync(
+      "and the draws are not all the same value",
+      () => {
+        let draws = take(~seed=99, ~n=100)
+        expect(draws->Array.some(f => f != draws->Array.getUnsafe(0)))->toBe(true)
+      },
+    )
   })
 
   describe("int is inclusive on both ends, as documented:", () => {
@@ -54,40 +63,50 @@ describe("Seed_Random:", () => {
       Array.make(~length=n, 0)->Array.map(_ => Seed_Random.int(r, ~min, ~max))
     }
 
-    testSync("never escapes the range", () =>
-      expect(draws(~min=3, ~max=7, ~n=500)->Array.every(i => i >= 3 && i <= 7))->toBe(true)
+    testSync(
+      "never escapes the range",
+      () => expect(draws(~min=3, ~max=7, ~n=500)->Array.every(i => i >= 3 && i <= 7))->toBe(true),
     )
 
     // The documented contract is "inclusive on both ends"; an off-by-one in the
     // `max - min + 1` scaling shows up as a bound that is never drawn.
-    testSync("reaches both bounds", () => {
-      let d = draws(~min=3, ~max=7, ~n=500)
-      expect((d->Array.includes(3), d->Array.includes(7)))->toEqual((true, true))
-    })
+    testSync(
+      "reaches both bounds",
+      () => {
+        let d = draws(~min=3, ~max=7, ~n=500)
+        expect((d->Array.includes(3), d->Array.includes(7)))->toEqual((true, true))
+      },
+    )
 
-    testSync("a single-value range is that value", () =>
-      expect(draws(~min=4, ~max=4, ~n=10)->Array.every(i => i == 4))->toBe(true)
+    testSync(
+      "a single-value range is that value",
+      () => expect(draws(~min=4, ~max=4, ~n=10)->Array.every(i => i == 4))->toBe(true),
     )
   })
 
   describe("pick:", () => {
-    testSync("returns None for an empty array", () =>
-      expect(Seed_Random.make(~seed=1)->Seed_Random.pick([]))->toEqual(None)
+    testSync(
+      "returns None for an empty array",
+      () => expect(Seed_Random.make(~seed=1)->Seed_Random.pick([]))->toEqual(None),
     )
 
-    testSync("always lands on a member", () => {
-      let r = Seed_Random.make(~seed=2)
-      let xs = ["a", "b", "c", "d"]
-      let hits = Array.make(~length=200, "")->Array.map(_ =>
-        r->Seed_Random.pickOr(~fallback="MISS", xs)
-      )
-      expect(hits->Array.every(h => xs->Array.includes(h)))->toBe(true)
-    })
+    testSync(
+      "always lands on a member",
+      () => {
+        let r = Seed_Random.make(~seed=2)
+        let xs = ["a", "b", "c", "d"]
+        let hits =
+          Array.make(~length=200, "")->Array.map(_ => r->Seed_Random.pickOr(~fallback="MISS", xs))
+        expect(hits->Array.every(h => xs->Array.includes(h)))->toBe(true)
+      },
+    )
 
-    testSync("pickOr falls back only when there is nothing to pick", () =>
-      expect(Seed_Random.make(~seed=3)->Seed_Random.pickOr(~fallback="fallback", []))->toBe(
-        "fallback",
-      )
+    testSync(
+      "pickOr falls back only when there is nothing to pick",
+      () =>
+        expect(Seed_Random.make(~seed=3)->Seed_Random.pickOr(~fallback="fallback", []))->toBe(
+          "fallback",
+        ),
     )
   })
 
@@ -96,65 +115,86 @@ describe("Seed_Random:", () => {
 
     // The documented shuffle use: equal weights, count = length. If sampling
     // ever replaced instead of removing, duplicates would appear here.
-    testSync("count = length yields a permutation, not a multiset", () => {
-      let xs = ["a", "b", "c", "d", "e"]
-      let out = Seed_Random.make(~seed=11)->Seed_Random.sampleWeighted(equal(xs), ~count=5)
-      let sorted = out->Array.toSorted(String.compare)
-      expect((out->Array.length, sorted))->toEqual((5, ["a", "b", "c", "d", "e"]))
-    })
-
-    testSync("count < length returns exactly count distinct items", () => {
-      let xs = ["a", "b", "c", "d", "e"]
-      let out = Seed_Random.make(~seed=12)->Seed_Random.sampleWeighted(equal(xs), ~count=3)
-      let distinct = Set.fromArray(out)->Set.size
-      expect((out->Array.length, distinct))->toEqual((3, 3))
-    })
-
-    testSync("count > length clamps to the pool", () =>
-      expect(
-        Seed_Random.make(~seed=13)
-        ->Seed_Random.sampleWeighted(equal(["a", "b"]), ~count=99)
-        ->Array.length,
-      )->toBe(2)
-    )
-
-    testSync("a zero-weight entry is never chosen while others remain", () => {
-      let out =
-        Seed_Random.make(~seed=14)->Seed_Random.sampleWeighted(
-          [("never", 0.0), ("a", 1.0), ("b", 1.0)],
-          ~count=2,
-        )
-      expect(out->Array.includes("never"))->toBe(false)
-    })
-
-    testSync("is reproducible for a given seed", () => {
-      let run = () =>
-        Seed_Random.make(~seed=15)->Seed_Random.sampleWeighted(
-          equal(["a", "b", "c", "d", "e", "f"]),
-          ~count=4,
-        )
-      expect(run())->toEqual(run())
-    })
+    testSync(
+      "count = length yields a permutation, not a multiset",
+      () => {
+        let xs = ["a", "b", "c", "d", "e"]
+        let out = Seed_Random.make(~seed=11)->Seed_Random.sampleWeighted(equal(xs), ~count=5)
+        let sorted = out->Array.toSorted(String.compare)
+        expect((out->Array.length, sorted))->toEqual((5, ["a", "b", "c", "d", "e"]))
+      },
+    )
+
+    testSync(
+      "count < length returns exactly count distinct items",
+      () => {
+        let xs = ["a", "b", "c", "d", "e"]
+        let out = Seed_Random.make(~seed=12)->Seed_Random.sampleWeighted(equal(xs), ~count=3)
+        let distinct = Set.fromArray(out)->Set.size
+        expect((out->Array.length, distinct))->toEqual((3, 3))
+      },
+    )
+
+    testSync(
+      "count > length clamps to the pool",
+      () =>
+        expect(
+          Seed_Random.make(~seed=13)
+          ->Seed_Random.sampleWeighted(equal(["a", "b"]), ~count=99)
+          ->Array.length,
+        )->toBe(2),
+    )
+
+    testSync(
+      "a zero-weight entry is never chosen while others remain",
+      () => {
+        let out =
+          Seed_Random.make(~seed=14)->Seed_Random.sampleWeighted(
+            [("never", 0.0), ("a", 1.0), ("b", 1.0)],
+            ~count=2,
+          )
+        expect(out->Array.includes("never"))->toBe(false)
+      },
+    )
+
+    testSync(
+      "is reproducible for a given seed",
+      () => {
+        let run = () =>
+          Seed_Random.make(~seed=15)->Seed_Random.sampleWeighted(
+            equal(["a", "b", "c", "d", "e", "f"]),
+            ~count=4,
+          )
+        expect(run())->toEqual(run())
+      },
+    )
   })
 
   describe("zipfWeights skews toward the head:", () => {
     let weights = Seed_Random.zipfWeights(["a", "b", "c", "d", "e"], ~exponent=1.1)
 
-    testSync("keeps every item, in order", () =>
-      expect(weights->Array.map(((item, _)) => item))->toEqual(["a", "b", "c", "d", "e"])
+    testSync(
+      "keeps every item, in order",
+      () => expect(weights->Array.map(((item, _)) => item))->toEqual(["a", "b", "c", "d", "e"]),
     )
 
-    testSync("weight decreases monotonically with rank", () => {
-      let ws = weights->Array.map(((_, w)) => w)
-      let decreasing =
-        ws->Array.everyWithIndex((w, i) => i == 0 || w < ws->Array.getUnsafe(i - 1))
-      expect(decreasing)->toBe(true)
-    })
+    testSync(
+      "weight decreases monotonically with rank",
+      () => {
+        let ws = weights->Array.map(((_, w)) => w)
+        let decreasing =
+          ws->Array.everyWithIndex((w, i) => i == 0 || w < ws->Array.getUnsafe(i - 1))
+        expect(decreasing)->toBe(true)
+      },
+    )
 
-    testSync("the leader is clear — first outweighs last several times over", () => {
-      let first = weights->Array.get(0)->Option.mapOr(0.0, ((_, w)) => w)
-      let last = weights->Array.get(4)->Option.mapOr(0.0, ((_, w)) => w)
-      expect(first > last *. 3.0)->toBe(true)
-    })
+    testSync(
+      "the leader is clear — first outweighs last several times over",
+      () => {
+        let first = weights->Array.get(0)->Option.mapOr(0.0, ((_, w)) => w)
+        let last = weights->Array.get(4)->Option.mapOr(0.0, ((_, w)) => w)
+        expect(first > last *. 3.0)->toBe(true)
+      },
+    )
   })
 })

package/lib/ocaml/Seed_Runner.res

@@ -2,7 +2,6 @@
 
 open Seed_Types
 
-
 let envOr = (key: string, fallback: string): string =>
   NodeProcess.env->Dict.get(key)->Option.getOr(fallback)
 
@@ -58,14 +57,12 @@ let verifyViews = async (client: Seed_Client.t, ~views: array<view>): dict<int>
     Console.log(`  ${count->Int.toString->String.padStart(4, " ")}  ${name}${note}`)
   })
 
-  let empty =
-    views
-    ->Array.filterMap(v =>
-      switch v {
-      | Seeded(name) if counts->Dict.get(name)->Option.getOr(0) == 0 => Some(name)
-      | _ => None
-      }
-    )
+  let empty = views->Array.filterMap(v =>
+    switch v {
+    | Seeded(name) if counts->Dict.get(name)->Option.getOr(0) == 0 => Some(name)
+    | _ => None
+    }
+  )
   if empty->Array.length > 0 {
     throw(Failed(`these views are still empty after seeding: ${empty->Array.join(", ")}`))
   }
@@ -138,9 +135,7 @@ let run = async (main: unit => promise<unit>): unit =>
     Console.error("")
     Console.error("Seeding aborted with an unexpected error:")
     Console.error("")
-    Console.error(
-      exn->JsExn.fromException->Option.flatMap(JsExn.message)->Option.getOr("unknown"),
-    )
+    Console.error(exn->JsExn.fromException->Option.flatMap(JsExn.message)->Option.getOr("unknown"))
     NodeProcess.exit(1)
   }
 

package/lib/ocaml/Seed_Upload.res

@@ -64,7 +64,9 @@ let uploadAsset = async (
   // GraphQL string literals: JSON-quote the argument values (JSON string escaping is a
   // superset-safe subset of GraphQL's for these ASCII-ish inputs).
   let q = s => JSON.stringify(JSON.Encode.string(s))
-  let query = `mutation { r: Upload_Presign(store: ${q(store)}, fileName: ${q(fileName)}, contentType: ${q(contentType)}) { uploadUrl storageRef } }`
+  let query = `mutation { r: Upload_Presign(store: ${q(store)}, fileName: ${q(
+      fileName,
+    )}, contentType: ${q(contentType)}) { uploadUrl storageRef } }`
 
   let presign = try Ok(await Seed_Client.gql(client, ~query, ~label="Upload_Presign")) catch {
   | Seed_Types.Failed(m) => Error(m)
@@ -89,7 +91,7 @@ let uploadAsset = async (
       | Ok(putRes) if putRes->responseOk => Ok(storageRef)
       | Ok(putRes) =>
         let detail = await putRes->responseText
-        Error(`upload PUT failed with HTTP ${(putRes->responseStatus)->Int.toString}: ${detail}`)
+        Error(`upload PUT failed with HTTP ${putRes->responseStatus->Int.toString}: ${detail}`)
       }
     | _ => Error("Upload_Presign response missing uploadUrl/storageRef")
     }

package/lib/ocaml/Seed_Users.res

@@ -11,8 +11,20 @@
 
 @module("yaml") external parseYaml: string => JSON.t = "parse"
 
-/** One account: what to log in as, and what to send with it. */
-type user = {username: string, password: string, groups: array<string>}
+/**
+ * One account: what to log in as, what to send with it, and — when the file
+ * records one — the id the platform stamps on rows this account writes.
+ *
+ * `userId` is optional because absence is normal on both platforms: a local file
+ * may omit it and let the auth adapter default it to the username, and a
+ * hand-maintained AWS file records subs the pool minted, so it can lag.
+ */
+type user = {
+  username: string,
+  password: string,
+  groups: array<string>,
+  userId: option<string>,
+}
 
 let asString = (json: JSON.t): option<string> =>
   switch json {
@@ -32,7 +44,7 @@ let userOf = (json: JSON.t): option<user> =>
       | Some(JSON.Array(items)) => items->Array.filterMap(asString)
       | _ => []
       }
-      Some({username, password, groups})
+      Some({username, password, groups, userId: obj->Dict.get("userId")->Option.flatMap(asString)})
     | _ => None
     }
   | _ => None

package/lib/ocaml/Seed_UsersTest.res

@@ -50,6 +50,22 @@ describe("Seed_Users.parseString:", () => {
     ])
   })
 
+  // The half that was dropped, and the reason owner-scoped demo rows were keyed
+  // to ids nobody on a Cognito deployment holds: the file records the id the
+  // platform stamps, and reading it only for a password threw that away.
+  //
+  // Absent stays absent rather than defaulting to the username — a local file
+  // may omit it and let the auth adapter default it, and an AWS file records a
+  // sub no default could invent. Guessing here would look like an answer.
+  testSync("keeps the userId an entry declares, and reads a missing one as absent", () => {
+    let users = Seed_Users.parseString(document)
+    expect(users->Array.map(u => u.Seed_Users.userId))->Expect.toEqual([
+      Some("local-admin"),
+      None,
+      None,
+    ])
+  })
+
   // An entry half-written by hand is the likely flaw in this file. Dropping it
   // keeps the other accounts selectable; failing the parse would not.
   testSync("skips an entry that names no password", () => {
@@ -82,11 +98,32 @@ describe("Seed_Users.load:", () => {
 describe("Seed_Users.label:", () => {
   testSync("names the groups an account carries", () => {
     expect(
-      Seed_Users.label({username: "admin", password: "x", groups: ["Admin", "Shopper"]}),
+      Seed_Users.label({
+        username: "admin",
+        password: "x",
+        groups: ["Admin", "Shopper"],
+        userId: None,
+      }),
     )->Expect.toBe("admin  [Admin, Shopper]")
   })
 
   testSync("is the bare username when the file records no groups", () => {
-    expect(Seed_Users.label({username: "admin", password: "x", groups: []}))->Expect.toBe("admin")
+    expect(
+      Seed_Users.label({username: "admin", password: "x", groups: [], userId: None}),
+    )->Expect.toBe("admin")
+  })
+
+  // The menu is unchanged by the id: an operator picks an account by name and
+  // membership, and a Cognito sub in the list would be noise at the one moment
+  // they are choosing a login.
+  testSync("says nothing about the userId", () => {
+    expect(
+      Seed_Users.label({
+        username: "admin",
+        password: "x",
+        groups: ["Admin"],
+        userId: Some("4275e4a4-00c1-70e1-1cb5-363b06b992b5"),
+      }),
+    )->Expect.toBe("admin  [Admin]")
   })
 })

package/package.json

@@ -1,11 +1,11 @@
 {
   "name": "@reventlessdev/reventless-seed",
-  "version": "1.0.0-alpha.20",
+  "version": "1.0.0-alpha.21",
   "description": "Reusable harness for seeding a Reventless platform through its public GraphQL command API",
   "license": "Apache-2.0",
   "dependencies": {
     "yaml": "^2.8.3",
-    "@reventlessdev/rescript-node": "2.0.0-alpha.10"
+    "@reventlessdev/rescript-node": "2.0.0-alpha.11"
   },
   "devDependencies": {
     "rescript": "12.3.0",

package/src/Seed.res

@@ -16,12 +16,18 @@ module Connect = Seed_Connect
 exception Failed = Seed_Types.Failed
 
 // A live target a data set seeds against: an authenticated client, whether uploads
-// are skipped this run, and a label. Uploads mint through the domain API's
-// `Upload_Presign` mutation on the client (route B), so no upload endpoint is carried.
+// are skipped this run, a label, and who the run is acting as. Uploads mint through
+// the domain API's `Upload_Presign` mutation on the client (route B), so no upload
+// endpoint is carried.
 type connection = Seed_Connect.connection = {
   client: Seed_Client.t,
   uploadsSkipped: bool,
   label: string,
+  accounts: array<Seed_Users.user>,
+  caller: Seed_Users.user,
+  callerId: option<string>,
+  login: (~username: string, ~password: string) => promise<string>,
+  roleSwitch: option<Seed_Connect.roleSwitch>,
 }
 
 // A named, seedable data set. `seed` owns everything domain-specific.