@atproto/bsky 0.0.251 → 0.0.253

package/tests/views/account-deactivation.test.ts

@@ -1,81 +0,0 @@
-import { afterAll, beforeAll, beforeEach, describe, expect, it } from 'vitest'
-import { AtpAgent, ids } from '@atproto/api'
-import { SeedClient, TestNetwork, basicSeed } from '@atproto/dev-env'
-import type { DidString } from '@atproto/syntax'
-
-describe('bsky account deactivation', () => {
-  let network: TestNetwork
-  let agent: AtpAgent
-  let sc: SeedClient
-
-  let alice: DidString
-
-  beforeAll(async () => {
-    network = await TestNetwork.create({
-      dbPostgresSchema: 'bsky_views_account_deactivation',
-    })
-    agent = network.bsky.getAgent()
-    sc = network.getSeedClient()
-    await basicSeed(sc)
-    alice = sc.dids.alice
-    const pdsAgent = network.pds.getAgent()
-    await pdsAgent.com.atproto.server.deactivateAccount(
-      {},
-      { encoding: 'application/json', headers: sc.getHeaders(alice) },
-    )
-  })
-
-  beforeEach(async () => network.processAll())
-  afterAll(async () => network?.close())
-
-  it('does not return deactivated profiles', async () => {
-    const attempt = agent.api.app.bsky.actor.getProfile({
-      actor: alice,
-    })
-    await expect(attempt).rejects.toThrow('Account is deactivated')
-    const res = await agent.api.app.bsky.actor.getProfiles({
-      actors: [sc.dids.alice, sc.dids.bob, sc.dids.carol],
-    })
-    expect(res.data.profiles.length).toBe(2)
-    expect(res.data.profiles.some((p) => p.did === alice)).toBe(false)
-  })
-
-  it('does not return deactivated accounts in follows', async () => {
-    const follows = await agent.api.app.bsky.graph.getFollows({
-      actor: sc.dids.bob,
-    })
-    expect(follows.data.follows.some((f) => f.did === alice)).toBe(false)
-    const followers = await agent.api.app.bsky.graph.getFollowers({
-      actor: sc.dids.bob,
-    })
-    expect(followers.data.followers.some((f) => f.did === alice)).toBe(false)
-  })
-
-  it('does not return posts from deactivated accounts', async () => {
-    const uris = [
-      sc.posts[sc.dids.alice][0].ref.uriStr,
-      sc.posts[sc.dids.alice][1].ref.uriStr,
-      sc.posts[sc.dids.bob][0].ref.uriStr,
-      sc.posts[sc.dids.carol][0].ref.uriStr,
-      sc.posts[sc.dids.dan][1].ref.uriStr,
-      sc.replies[sc.dids.alice][0].ref.uriStr,
-    ]
-    const res = await agent.api.app.bsky.feed.getPosts({ uris })
-
-    expect(res.data.posts.length).toBe(3)
-    expect(res.data.posts.some((p) => p.author.did === alice)).toBe(false)
-  })
-
-  it('does not return posts from deactivated in timelines', async () => {
-    const res = await agent.api.app.bsky.feed.getTimeline(
-      {},
-      {
-        headers: await network.serviceHeaders(
-          sc.dids.bob,
-          ids.AppBskyFeedGetTimeline,
-        ),
-      },
-    )
-    expect(res.data.feed.some((p) => p.post.author.did === alice)).toBe(false)
-  })
-})

package/tests/views/actor-likes.test.ts

@@ -1,138 +0,0 @@
-import { afterAll, beforeAll, beforeEach, describe, expect, it } from 'vitest'
-import { AtUri, AtpAgent, ids } from '@atproto/api'
-import { SeedClient, TestNetwork, basicSeed } from '@atproto/dev-env'
-import type { DidString } from '@atproto/syntax'
-
-describe('bsky actor likes feed views', () => {
-  let network: TestNetwork
-  let agent: AtpAgent
-  let pdsAgent: AtpAgent
-  let sc: SeedClient
-
-  // account dids, for convenience
-  let alice: DidString
-  let bob: DidString
-  let carol: DidString
-
-  beforeAll(async () => {
-    network = await TestNetwork.create({
-      dbPostgresSchema: 'bsky_views_actor_likes',
-    })
-    agent = network.bsky.getAgent()
-    pdsAgent = network.pds.getAgent()
-    sc = network.getSeedClient()
-    await basicSeed(sc)
-    alice = sc.dids.alice
-    bob = sc.dids.bob
-    carol = sc.dids.carol
-  })
-
-  beforeEach(async () => network.processAll())
-  afterAll(async () => network?.close())
-
-  it('returns posts liked by actor', async () => {
-    const {
-      data: { feed: bobLikes },
-    } = await agent.api.app.bsky.feed.getActorLikes(
-      { actor: sc.accounts[bob].handle },
-      {
-        headers: await network.serviceHeaders(
-          bob,
-          ids.AppBskyFeedGetActorLikes,
-        ),
-      },
-    )
-
-    expect(bobLikes).toHaveLength(3)
-
-    await expect(
-      agent.api.app.bsky.feed.getActorLikes(
-        { actor: sc.accounts[bob].handle },
-        {
-          headers: await network.serviceHeaders(
-            carol,
-            ids.AppBskyFeedGetActorLikes,
-          ),
-        },
-      ),
-    ).rejects.toThrow('Profile not found')
-  })
-
-  it('viewer has blocked author of liked post(s)', async () => {
-    const bobBlocksAlice = await pdsAgent.api.app.bsky.graph.block.create(
-      {
-        repo: bob, // bob blocks alice
-      },
-      {
-        subject: alice,
-        createdAt: new Date().toISOString(),
-      },
-      sc.getHeaders(bob),
-    )
-
-    await network.processAll()
-
-    const {
-      data: { feed },
-    } = await agent.api.app.bsky.feed.getActorLikes(
-      { actor: sc.accounts[bob].handle },
-      {
-        headers: await network.serviceHeaders(
-          bob,
-          ids.AppBskyFeedGetActorLikes,
-        ),
-      },
-    )
-
-    expect(
-      feed.every((item) => {
-        return item.post.author.did !== alice
-      }),
-    ).toBe(true)
-
-    // unblock
-    await pdsAgent.api.app.bsky.graph.block.delete(
-      { repo: bob, rkey: new AtUri(bobBlocksAlice.uri).rkey },
-      sc.getHeaders(bob),
-    )
-  })
-
-  it('liked post author has blocked viewer', async () => {
-    const aliceBlockBob = await pdsAgent.api.app.bsky.graph.block.create(
-      {
-        repo: alice, // alice blocks bob
-      },
-      {
-        subject: bob,
-        createdAt: new Date().toISOString(),
-      },
-      sc.getHeaders(alice),
-    )
-
-    await network.processAll()
-
-    const {
-      data: { feed },
-    } = await agent.api.app.bsky.feed.getActorLikes(
-      { actor: sc.accounts[bob].handle },
-      {
-        headers: await network.serviceHeaders(
-          bob,
-          ids.AppBskyFeedGetActorLikes,
-        ),
-      },
-    )
-
-    expect(
-      feed.every((item) => {
-        return item.post.author.did !== alice
-      }),
-    ).toBe(true)
-
-    // unblock
-    await pdsAgent.api.app.bsky.graph.block.delete(
-      { repo: alice, rkey: new AtUri(aliceBlockBob.uri).rkey },
-      sc.getHeaders(alice),
-    )
-  })
-})

package/tests/views/actor-search.test.ts

@@ -1,261 +0,0 @@
-import { afterAll, beforeAll, beforeEach, describe, expect, it } from 'vitest'
-import { AppBskyActorSearchActors, AtpAgent, ids } from '@atproto/api'
-import { wait } from '@atproto/common'
-import { SeedClient, TestNetwork, usersBulkSeed } from '@atproto/dev-env'
-import { forSnapshot, paginateAll, stripViewer } from '../_util.js'
-
-// @NOTE skipped to help with CI failures
-// The search code is not used in production & we should switch it out for tests on the search proxy interface
-describe.skip('pds actor search views', () => {
-  let network: TestNetwork
-  let agent: AtpAgent
-  let sc: SeedClient
-  let headers: { [s: string]: string }
-  let headersTypeahead: { [s: string]: string }
-
-  beforeAll(async () => {
-    network = await TestNetwork.create({
-      dbPostgresSchema: 'bsky_views_actor_search',
-    })
-    agent = network.bsky.getAgent()
-    sc = network.getSeedClient()
-
-    await wait(50) // allow pending sub to be established
-    await network.processAll()
-    await network.bsky.sub.stop()
-    await usersBulkSeed(sc)
-
-    // Skip did/handle resolution for expediency
-    const { db } = network.bsky
-    const now = new Date().toISOString()
-    await db.db
-      .insertInto('actor')
-      .values(
-        Object.entries(sc.dids).map(([handle, did]) => ({
-          did,
-          handle,
-          indexedAt: now,
-        })),
-      )
-      .onConflict((oc) => oc.doNothing())
-      .execute()
-
-    // Process remaining profiles
-    await network.bsky.sub.restart()
-    headers = await network.serviceHeaders(
-      Object.values(sc.dids)[0],
-      ids.AppBskyActorSearchActors,
-    )
-    headersTypeahead = await network.serviceHeaders(
-      Object.values(sc.dids)[0],
-      ids.AppBskyActorSearchActorsTypeahead,
-    )
-  })
-
-  beforeEach(async () => network.processAll(50_000), 50_100)
-  afterAll(async () => network?.close())
-
-  it('typeahead gives relevant results', async () => {
-    const result = await agent.api.app.bsky.actor.searchActorsTypeahead(
-      { term: 'car' },
-      { headers: headersTypeahead },
-    )
-
-    const handles = result.data.actors.map((u) => u.handle)
-
-    const shouldContain = [
-      'cara-wiegand69.test',
-      'eudora-dietrich4.test', // Carol Littel
-      'shane-torphy52.test', // Sadie Carter
-      'aliya-hodkiewicz.test', // Carlton Abernathy IV
-      'carlos6.test',
-      'carolina-mcderm77.test',
-    ]
-
-    shouldContain.forEach((handle) => expect(handles).toContain(handle))
-    expect(handles).toContain('cayla-marquardt39.test') // Fuzzy match
-
-    const shouldNotContain = [
-      'sven70.test',
-      'hilario84.test',
-      'santa-hermann78.test',
-      'dylan61.test',
-      'preston-harris.test',
-      'loyce95.test',
-      'melyna-zboncak.test',
-    ]
-
-    shouldNotContain.forEach((handle) => expect(handles).not.toContain(handle))
-
-    const sorted = result.data.actors.sort((a, b) =>
-      a.handle > b.handle ? 1 : -1,
-    )
-    expect(forSnapshot(sorted)).toMatchSnapshot()
-  })
-
-  it('typeahead gives empty result set when provided empty term', async () => {
-    const result = await agent.api.app.bsky.actor.searchActorsTypeahead(
-      { term: '' },
-      { headers: headersTypeahead },
-    )
-
-    expect(result.data.actors).toEqual([])
-  })
-
-  it('typeahead applies limit', async () => {
-    const full = await agent.api.app.bsky.actor.searchActorsTypeahead(
-      { term: 'p' },
-      { headers: headersTypeahead },
-    )
-
-    expect(full.data.actors.length).toBeGreaterThan(5)
-
-    const limited = await agent.api.app.bsky.actor.searchActorsTypeahead(
-      { term: 'p', limit: 5 },
-      { headers: headersTypeahead },
-    )
-
-    // @NOTE it's expected that searchActorsTypeahead doesn't have stable pagination
-
-    const limitedIndexInFull = limited.data.actors.map((needle) => {
-      return full.data.actors.findIndex(
-        (haystack) => needle.did === haystack.did,
-      )
-    })
-
-    // subset exists in full and is monotonic
-    expect(limitedIndexInFull.every((idx) => idx !== -1)).toEqual(true)
-    expect(limitedIndexInFull).toEqual(
-      [...limitedIndexInFull].sort((a, b) => a - b),
-    )
-  })
-
-  it('typeahead gives results unauthed', async () => {
-    const { data: authed } =
-      await agent.api.app.bsky.actor.searchActorsTypeahead(
-        { term: 'car' },
-        { headers: headersTypeahead },
-      )
-    const { data: unauthed } =
-      await agent.api.app.bsky.actor.searchActorsTypeahead({
-        term: 'car',
-      })
-    expect(unauthed.actors.length).toBeGreaterThan(0)
-    expect(unauthed.actors).toEqual(authed.actors.map(stripViewer))
-  })
-
-  it('search gives relevant results', async () => {
-    const result = await agent.api.app.bsky.actor.searchActors(
-      { term: 'car' },
-      { headers },
-    )
-
-    const handles = result.data.actors.map((u) => u.handle)
-
-    const shouldContain = [
-      'cara-wiegand69.test',
-      'eudora-dietrich4.test', // Carol Littel
-      'shane-torphy52.test', // Sadie Carter
-      'aliya-hodkiewicz.test', // Carlton Abernathy IV
-      'carlos6.test',
-      'carolina-mcderm77.test',
-    ]
-
-    shouldContain.forEach((handle) => expect(handles).toContain(handle))
-    expect(handles).toContain('cayla-marquardt39.test') // Fuzzy match
-
-    const shouldNotContain = [
-      'sven70.test',
-      'hilario84.test',
-      'santa-hermann78.test',
-      'dylan61.test',
-      'preston-harris.test',
-      'loyce95.test',
-      'melyna-zboncak.test',
-    ]
-
-    shouldNotContain.forEach((handle) => expect(handles).not.toContain(handle))
-
-    const sorted = result.data.actors.sort((a, b) =>
-      a.handle > b.handle ? 1 : -1,
-    )
-    expect(forSnapshot(sorted)).toMatchSnapshot()
-  })
-
-  it('search gives empty result set when provided empty term', async () => {
-    const result = await agent.api.app.bsky.actor.searchActors(
-      { term: '' },
-      { headers },
-    )
-
-    expect(result.data.actors).toEqual([])
-  })
-
-  it('paginates', async () => {
-    const results = (results: AppBskyActorSearchActors.OutputSchema[]) =>
-      results.flatMap((res) => res.actors)
-    const paginator = async (cursor?: string) => {
-      const res = await agent.api.app.bsky.actor.searchActors(
-        { term: 'p', cursor, limit: 3 },
-        { headers },
-      )
-      return res.data
-    }
-
-    const paginatedAll = await paginateAll(paginator)
-    paginatedAll.forEach((res) =>
-      expect(res.actors.length).toBeLessThanOrEqual(3),
-    )
-
-    const full = await agent.api.app.bsky.actor.searchActors(
-      { term: 'p' },
-      { headers },
-    )
-
-    expect(full.data.actors.length).toBeGreaterThan(5)
-    const sortedFull = results([full.data]).sort((a, b) =>
-      a.handle > b.handle ? 1 : -1,
-    )
-    const sortedPaginated = results(paginatedAll).sort((a, b) =>
-      a.handle > b.handle ? 1 : -1,
-    )
-    expect(sortedPaginated).toEqual(sortedFull)
-  })
-
-  it('search handles bad input', async () => {
-    // Mostly for sqlite's benefit, since it uses LIKE and these are special characters that will
-    // get stripped. This input triggers a special case where there are no "safe" words for sqlite to search on.
-    const result = await agent.api.app.bsky.actor.searchActors(
-      { term: ' % _ ' },
-      { headers },
-    )
-
-    expect(result.data.actors).toEqual([])
-  })
-
-  it('search gives results unauthed', async () => {
-    const { data: authed } = await agent.api.app.bsky.actor.searchActors(
-      { term: 'car' },
-      { headers },
-    )
-    const { data: unauthed } = await agent.api.app.bsky.actor.searchActors({
-      term: 'car',
-    })
-    expect(unauthed.actors.length).toBeGreaterThan(0)
-    expect(unauthed.actors).toEqual(authed.actors.map(stripViewer))
-  })
-
-  it('search blocks by actor takedown', async () => {
-    await network.bsky.server.ctx.dataplane.takedownActor({
-      did: sc.dids['cara-wiegand69.test'],
-    })
-    const result = await agent.api.app.bsky.actor.searchActorsTypeahead(
-      { term: 'car' },
-      { headers: headersTypeahead },
-    )
-    const handles = result.data.actors.map((u) => u.handle)
-    expect(handles).toContain('carlos6.test')
-    expect(handles).toContain('carolina-mcderm77.test')
-    expect(handles).not.toContain('cara-wiegand69.test')
-  })
-})