Merge pull request #46 from JoinTheAlliance/0.0.28

fix tests and expand scope of tests

Author: lalalune

Diff for: .github/workflows/test.yaml

@@ -20,4 +20,4 @@ jobs:
         - uses: actions/setup-node@v2
 
         - run: npm ci
-        - run: npm run test -- ./src/lib/ --runInBand
+        - run: npm run test

Diff for: src/lib/__tests__/messages.test.ts

@@ -35,8 +35,11 @@ describe("Messages Library", () => {
   });
 
   test("formatActors should format actors into a readable string", () => {
+    console.log("*** actors", actors);
     const formattedActors = formatActors({ actors });
+    console.log("*** formattedActors", formattedActors);
     actors.forEach((actor) => {
+      console.log("*** actor.name", actor.name);
       expect(formattedActors).toContain(actor.name);
     });
   });

Diff for: src/lib/actions/__tests__/continue.test.ts

@@ -10,6 +10,7 @@ import { Content, type Message } from "../../types";
 import action from "../continue";
 import ignore from "../ignore";
 import { zeroUuid } from "../../constants";
+import wait from "../wait";
 
 dotenv.config({ path: ".dev.vars" });
 
@@ -52,7 +53,7 @@ describe("User Profile", () => {
   beforeAll(async () => {
     const setup = await createRuntime({
       env: process.env as Record<string, string>,
-      actions: [action, ignore],
+      actions: [action, ignore, wait],
     });
     user = setup.session.user;
     runtime = setup.runtime;

Diff for: src/lib/actions/__tests__/ignore.test.ts

@@ -190,7 +190,7 @@ describe("Ignore action tests", () => {
         senderId: user?.id as UUID,
         agentId: zeroUuid,
         userIds: [user?.id as UUID, zeroUuid],
-        content: { content: "Never talk to me again" },
+        content: { content: "Never talk to me again", action: "WAIT" },
         room_id: room_id as UUID,
       };
 
@@ -212,7 +212,7 @@ describe("Ignore action tests", () => {
           senderId: user.id as UUID,
           agentId: zeroUuid,
           userIds: [user?.id as UUID, zeroUuid],
-          content: { content: "" },
+          content: { content: "", action: "IGNORE" },
           room_id: room_id as UUID,
         };
 
@@ -239,7 +239,7 @@ describe("Ignore action tests", () => {
           senderId: user.id as UUID,
           agentId: zeroUuid,
           userIds: [user?.id as UUID, zeroUuid],
-          content: { content: "" },
+          content: { content: "", action: "IGNORE" },
           room_id: room_id as UUID,
         };
 
@@ -264,7 +264,7 @@ describe("Ignore action tests", () => {
         senderId: user.id as UUID,
         agentId: zeroUuid,
         userIds: [user?.id as UUID, zeroUuid],
-        content: { content: "Bye" },
+        content: { content: "Bye", action: "WAIT" },
         room_id: room_id as UUID,
       };
 

Diff for: src/lib/messages.ts

@@ -42,7 +42,7 @@ export async function getActorDetails({
  */
 export function formatActors({ actors }: { actors: Actor[] }) {
   const actorStrings = actors.map((actor: Actor) => {
-    const header = `${actor.name}${actor.details.tagline ? ": " + actor.details.tagline : ""}\n${actor.details.summary || "No information available"}`;
+    const header = `${actor.name}${actor.details.tagline ? ": " + actor.details.tagline : ""}${actor.details?.summary ? "\n" + actor.details?.summary : ""}`;
     return header;
   });
   const finalActorStrings = actorStrings.join("\n");

Diff for: src/test/constants.ts

@@ -4,5 +4,5 @@ export const SUPABASE_ANON_KEY =
   "eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJzdXBhYmFzZSIsInJlZiI6InByb252enJ6ZndzcHRrb2p2dWRkIiwicm9sZSI6ImFub24iLCJpYXQiOjE3MDY4NTYwNDcsImV4cCI6MjAyMjQzMjA0N30.I6_-XrqssUb2SWYg5DjsUqSodNS3_RPoET3-aPdqywM";
 export const TEST_EMAIL = "[email protected]";
 export const TEST_PASSWORD = "[email protected]";
-export const TEST_EMAIL_2 = "testuser123@gmail.com";
-export const TEST_PASSWORD_2 = "testuser123@gmail.com";
+export const TEST_EMAIL_2 = "testuser234@gmail.com";
+export const TEST_PASSWORD_2 = "testuser234@gmail.com";