git-workspace-bare 0.1.1 → 0.3.0

package/test/manifest.js

@@ -0,0 +1,170 @@
+import test from 'brittle'
+import fs from 'bare-fs'
+import path from 'bare-path'
+import setup from './helpers.js'
+
+function read(registry) {
+  return JSON.parse(fs.readFileSync(registry, 'utf8'))
+}
+
+test('stable IDs survive repeated adds, branch renames, and root owner changes', (t) => {
+  const { ok, git, registry } = setup(t)
+  t.alike(read(registry), { version: 1, repositories: [] })
+  ok('add', 'owner/repo@feature/jobs')
+  ok('add', 'owner/repo@other')
+  const initial = read(registry)
+  const root = initial.repositories[0]
+  t.ok(/^[0-9a-f-]{36}$/.test(root.id), 'repository UUID')
+  t.ok(
+    root.worktrees.every(({ id }) => /^[0-9a-f-]{36}$/.test(id)),
+    'worktree UUIDs'
+  )
+  t.is(new Set([root.id, ...root.worktrees.map(({ id }) => id)]).size, 3)
+  const head = git('-C', path.dirname(registry), 'rev-parse', 'HEAD')
+  ok('add', 'owner/repo@feature/jobs')
+  t.alike(read(registry), initial, 'idempotent add keeps IDs')
+  t.is(git('-C', path.dirname(registry), 'rev-parse', 'HEAD'), head, 'no ID churn commit')
+  ok('rename', 'owner/repo@feature/jobs', 'owner/repo@fixes')
+  t.alike(read(registry).repositories[0], {
+    id: root.id,
+    name: root.name,
+    worktrees: [{ id: root.worktrees[0].id, branch: 'fixes' }, root.worktrees[1]]
+  })
+  ok('rename', 'owner/repo', 'other/new')
+  const renamed = read(registry).repositories[0]
+  t.is(renamed.id, root.id, 'root identity retained across owners')
+  t.is(renamed.name, 'other/new')
+  t.alike(
+    renamed.worktrees.map(({ id }) => id),
+    root.worktrees.map(({ id }) => id)
+  )
+  t.is(ok('list'), 'other/new\nother/new@fixes\nother/new@other\n')
+})
+
+test('old names can be reused without reusing repository or worktree identities', (t) => {
+  const { ok, registry } = setup(t)
+  ok('add', 'owner/repo@jobs')
+  const original = read(registry).repositories[0]
+  ok('rename', 'owner/repo@jobs', 'owner/repo@fixes')
+  ok('add', 'owner/repo@jobs')
+  const reused = read(registry).repositories[0]
+  t.not(reused.worktrees.find(({ branch }) => branch === 'jobs').id, original.worktrees[0].id)
+  const removedId = reused.worktrees.find(({ branch }) => branch === 'jobs').id
+  ok('remove', 'owner/repo@jobs')
+  ok('add', 'owner/repo@jobs')
+  t.not(
+    read(registry).repositories[0].worktrees.find(({ branch }) => branch === 'jobs').id,
+    removedId
+  )
+  ok('rename', 'owner/repo', 'owner/new')
+  ok('add', 'owner/repo')
+  const replacement = read(registry).repositories.find(({ name }) => name === 'owner/repo')
+  t.not(replacement.id, original.id, 'reusing a renamed root name gets a fresh ID')
+  ok('remove', 'owner/repo')
+  ok('add', 'owner/repo')
+  t.not(read(registry).repositories.find(({ name }) => name === 'owner/repo').id, replacement.id)
+})
+
+test('version-1 identities are shared unchanged with another machine', (t) => {
+  const { ok, cliAt, git, home, registry, upstream } = setup(t)
+  ok('add', 'owner/repo@feature/jobs')
+  const contents = fs.readFileSync(registry, 'utf8')
+  git('clone', '--bare', path.dirname(registry), path.join(upstream, 'owner/workspace.git'))
+  const second = path.join(home, 'second')
+  fs.mkdirSync(second)
+  t.is(cliAt(second, 'init', 'owner/workspace').status, 0)
+  t.is(
+    fs.readFileSync(path.join(second, 'workspace/owner/workspace/workspace.json'), 'utf8'),
+    contents
+  )
+  t.absent(
+    fs.existsSync(path.join(second, 'workspace/owner/repo')),
+    'init does not apply the manifest'
+  )
+})
+
+test('IDs survive a failed commit and retry', (t) => {
+  const { cli, ok, git, registry } = setup(t)
+  const directory = path.dirname(registry)
+  const hooks = path.join(directory, '.git/hooks')
+  git('-C', directory, 'config', 'core.hooksPath', hooks)
+  const hook = path.join(hooks, 'pre-commit')
+  fs.writeFileSync(hook, '#!/bin/sh\nexit 1\n')
+  fs.chmodSync(hook, 0o755)
+  t.is(cli('add', 'owner/repo@jobs').status, 1)
+  const pending = read(registry)
+  t.is(pending.repositories.length, 1)
+  fs.unlinkSync(hook)
+  ok('add', 'owner/repo@jobs')
+  t.alike(read(registry), pending)
+  t.alike(JSON.parse(git('-C', directory, 'show', 'HEAD:workspace.json')), pending)
+})
+
+test('invalid manifests fail before filesystem or identity changes', (t) => {
+  const { cli, ok, git, registry, root } = setup(t)
+  ok('add', 'owner/repo@jobs')
+  const directory = path.dirname(registry)
+  const head = git('-C', directory, 'rev-parse', 'HEAD')
+  const good = read(registry)
+  const invalid = [
+    { ...good, version: 99 },
+    { version: 1, repositories: [{ ...good.repositories[0], id: '' }] },
+    {
+      version: 1,
+      repositories: [
+        { ...good.repositories[0], worktrees: [{ id: good.repositories[0].id, branch: 'jobs' }] }
+      ]
+    },
+    {
+      version: 1,
+      repositories: [good.repositories[0], { ...good.repositories[0], id: 'another' }]
+    },
+    {
+      version: 1,
+      repositories: [
+        {
+          ...good.repositories[0],
+          worktrees: [
+            { id: 'first', branch: 'jobs' },
+            { id: 'second', branch: 'jobs' }
+          ]
+        }
+      ]
+    },
+    { version: 1, repositories: [{ ...good.repositories[0], name: '../escape' }] },
+    {
+      version: 1,
+      repositories: [{ ...good.repositories[0], worktrees: [{ id: 'first', branch: '../escape' }] }]
+    },
+    [],
+    ['owner/repo'],
+    ['owner/repo', 'owner/repo@jobs'],
+    ['owner/repo', 'owner/repo'],
+    ['owner/repo@jobs']
+  ]
+  for (const manifest of invalid) {
+    const contents = JSON.stringify(manifest)
+    fs.writeFileSync(registry, contents)
+    for (const args of [
+      ['list'],
+      ['init', 'owner/workspace'],
+      ['add', 'owner/repo'],
+      ['remove', 'owner/repo@jobs'],
+      ['rename', 'owner/repo', 'owner/new']
+    ]) {
+      t.is(cli(...args).status, 1, args[0] + ' rejects invalid manifest')
+    }
+    t.is(fs.readFileSync(registry, 'utf8'), contents)
+    t.ok(fs.existsSync(root + '@jobs'))
+  }
+  t.is(git('-C', directory, 'rev-parse', 'HEAD'), head)
+})
+
+test('a missing root with registered worktrees cannot be removed from the new schema', (t) => {
+  const { cli, ok, registry, root } = setup(t)
+  ok('add', 'owner/repo@jobs')
+  const before = read(registry)
+  fs.renameSync(root, root + '-moved')
+  t.is(cli('remove', 'owner/repo').status, 1)
+  t.alike(read(registry), before)
+})