From e7da23d865c788099f8769f21b5c7c06a8525d4c Mon Sep 17 00:00:00 2001
From: Michael Barrett <mike@ghost.org>
Date: Wed, 14 Aug 2024 10:39:01 +0100
Subject: [PATCH] wip

---
 src/user.ts           | 12 ++++++
 src/user.unit.test.ts | 87 +++++++++++++++++++++++++++++++++++++++++++
 2 files changed, 99 insertions(+)
 create mode 100644 src/user.unit.test.ts

diff --git a/src/user.ts b/src/user.ts
index 88516a5f..c03c7932 100644
--- a/src/user.ts
+++ b/src/user.ts
@@ -23,6 +23,7 @@ export type PersonData = {
     outbox: string;
     following: string;
     followers: string;
+    url: string;
 };
 
 export async function getUserData(ctx: RequestContext<ContextData>, handle: string) {
@@ -36,6 +37,14 @@ export async function getUserData(ctx: RequestContext<ContextData>, handle: stri
             console.log('Could not create Image from Icon value', existing.icon);
             console.log(err);
         }
+
+        let url = null;
+        try {
+            url = new URL(existing.url);
+        }  catch (err) {
+            console.log('Could not create url from value', existing.url);
+            console.log(err);
+        }
         return {
             id: new URL(existing.id),
             name: existing.name,
@@ -49,6 +58,7 @@ export async function getUserData(ctx: RequestContext<ContextData>, handle: stri
             publicKeys: (await ctx.getActorKeyPairs(handle)).map(
                 (key) => key.cryptographicKey,
             ),
+            url,
         };
     }
 
@@ -65,6 +75,7 @@ export async function getUserData(ctx: RequestContext<ContextData>, handle: stri
         publicKeys: (await ctx.getActorKeyPairs(handle)).map(
             (key) => key.cryptographicKey,
         ),
+        url: new URL(`https://${ctx.host}`),
     };
 
     const dataToStore: PersonData = {
@@ -77,6 +88,7 @@ export async function getUserData(ctx: RequestContext<ContextData>, handle: stri
         outbox: data.outbox.href,
         following: data.following.href,
         followers: data.followers.href,
+        url: data.url.href,
     };
 
     await ctx.data.db.set(['handle', handle], dataToStore);
diff --git a/src/user.unit.test.ts b/src/user.unit.test.ts
new file mode 100644
index 00000000..eb93cc38
--- /dev/null
+++ b/src/user.unit.test.ts
@@ -0,0 +1,87 @@
+import assert from 'assert';
+import {
+    Image,
+} from '@fedify/fedify';
+import sinon from 'sinon';
+
+import {
+    ACTOR_DEFAULT_ICON,
+    ACTOR_DEFAULT_NAME,
+    ACTOR_DEFAULT_SUMMARY
+} from './constants';
+import { getUserData } from './user';
+
+
+const HANDLE = 'foo';
+
+function getCtx() {
+    const host = 'example.com';
+
+    const ctx = {
+        data: {
+            db: {
+                get: sinon.stub(),
+                set: sinon.stub(),
+            },
+        },
+        getActorKeyPairs: sinon.stub(),
+        getActorUri: sinon.stub(),
+        getInboxUri: sinon.stub(),
+        getOutboxUri: sinon.stub(),
+        getFollowingUri: sinon.stub(),
+        getFollowersUri: sinon.stub(),
+        host,
+    };
+
+    ctx.getActorKeyPairs.withArgs(HANDLE).resolves([
+        { cryptographicKey: 'abc123' }
+    ]);
+
+    ctx.getActorUri.withArgs(HANDLE).returns(new URL(`https://${host}/${HANDLE}`));
+    ctx.getInboxUri.withArgs(HANDLE).returns(new URL(`https://${host}/${HANDLE}/inbox`));
+    ctx.getOutboxUri.withArgs(HANDLE).returns(new URL(`https://${host}/${HANDLE}/outbox`));
+    ctx.getFollowingUri.withArgs(HANDLE).returns(new URL(`https://${host}/${HANDLE}/following`));
+    ctx.getFollowersUri.withArgs(HANDLE).returns(new URL(`https://${host}/${HANDLE}/followers`));
+
+    return ctx;
+}
+
+describe('getUserData', function () {
+    it('persists a user to the database if it does not exist', async function () {
+        const ctx = getCtx();
+
+        ctx.data.db.get.resolves(null);
+
+        const result = await getUserData(ctx, HANDLE);
+
+        const expectedUserData = {
+            id: new URL(`https://${ctx.host}/${HANDLE}`),
+            name: ACTOR_DEFAULT_NAME,
+            summary: ACTOR_DEFAULT_SUMMARY,
+            preferredUsername: HANDLE,
+            icon: new Image({ url: new URL(ACTOR_DEFAULT_ICON) }),
+            inbox: new URL(`https://${ctx.host}/${HANDLE}/inbox`),
+            outbox: new URL(`https://${ctx.host}/${HANDLE}/outbox`),
+            following: new URL(`https://${ctx.host}/${HANDLE}/following`),
+            followers: new URL(`https://${ctx.host}/${HANDLE}/followers`),
+            publicKeys: ['abc123'],
+            url: new URL(`https://${ctx.host}`),
+        }
+
+        assert.ok(
+            ctx.data.db.set.calledWith(['handle', HANDLE], {
+                id: expectedUserData.id.href,
+                name: expectedUserData.name,
+                summary: expectedUserData.summary,
+                preferredUsername: expectedUserData.preferredUsername,
+                icon: ACTOR_DEFAULT_ICON,
+                inbox: expectedUserData.inbox.href,
+                outbox: expectedUserData.outbox.href,
+                following: expectedUserData.following.href,
+                followers: expectedUserData.followers.href,
+                url: expectedUserData.url.href,
+            })
+        );
+        assert.deepStrictEqual(result, expectedUserData);
+    });
+});