e2e/forkweb_test.go
101 lines · 4407 bytes
1package e2e
2
3import (
4 "net/url"
5 "os"
6 "path/filepath"
7 "strings"
8 "testing"
9)
10
11// Forking from the browser completes the contributor path: fork, edit a
12// file in the fork, open the merge request against the parent, without a
13// terminal at any step (#174).
14func TestForkWeb(t *testing.T) {
15 inst := startInstanceWith(t, "[web]\nmode = \"accounts\"\n")
16 aliceKey := inst.newKey(t, "alice")
17 bobKey := inst.newKey(t, "bob")
18 inst.admin(t, "admin", "user", "create", "alice", "--key", aliceKey+".pub",
19 "--email", "alice@example.test", "--verified")
20 inst.admin(t, "admin", "user", "create", "bob", "--key", bobKey+".pub",
21 "--email", "bob@example.test", "--verified")
22
23 if _, errOut, code := inst.ssh(t, aliceKey, "", "repo", "create", "alice/app"); code != 0 {
24 t.Fatalf("repo create: %s", errOut)
25 }
26 env := inst.gitEnv(aliceKey)
27 work := t.TempDir()
28 mustGit(t, work, env, "clone", inst.sshURL("alice/app"), "w")
29 dir := filepath.Join(work, "w")
30 os.WriteFile(filepath.Join(dir, "f.txt"), []byte("x\n"), 0o644)
31 mustGit(t, dir, env, "checkout", "-q", "-b", "main")
32 mustGit(t, dir, env, "add", ".")
33 mustGit(t, dir, env, "commit", "-q", "-m", "base")
34 mustGit(t, dir, env, "push", "-q", "origin", "main")
35
36 bob := inst.login(t, bobKey)
37 base := inst.base() + "/alice/app"
38
39 // The control is on the repository header for a signed-in visitor.
40 if _, page := browserGet(t, bob, base); !strings.Contains(page, `/alice/app/fork"`) {
41 t.Fatalf("no fork control on the repository header:\n%s", page)
42 }
43
44 // Forking lands on the fork, which carries the parent's history.
45 if status, body := browserPost(t, bob, base+"/fork", url.Values{}); status != 200 ||
46 !strings.Contains(body, "bob/app") {
47 t.Fatalf("fork did not land on the new repository: %d\n%s", status, body)
48 }
49 out, _, _ := inst.ssh(t, bobKey, "", "repo", "list", "--json")
50 if !strings.Contains(out, `"path":"bob/app"`) {
51 t.Fatalf("fork not created:\n%s", out)
52 }
53
54 // Editing a file in the fork from the browser — the editor commits to
55 // the ref in the URL — then proposing it to the parent through the
56 // source picker.
57 if status, _ := browserPost(t, bob, inst.base()+"/bob/app/edit/main/f.txt", url.Values{
58 "content": {"y\n"}, "message": {"change f"}}); status != 200 {
59 t.Fatal("web edit in the fork failed")
60 }
61 if _, page := browserGet(t, bob, base+"/mrs/new"); !strings.Contains(page, `value="bob/app:main"`) {
62 t.Fatalf("the fork's branch is not offered:\n%s", page)
63 }
64 if status, _ := browserPost(t, bob, base+"/mrs/new", url.Values{
65 "source": {"bob/app:main"}, "target": {"main"}, "title": {"change"}}); status != 200 {
66 t.Fatal("mr create from the fork failed")
67 }
68 if out, _, _ := inst.ssh(t, aliceKey, "", "mr", "show", "alice/app", "1", "--json"); !strings.Contains(out, `"source":"bob/app:main"`) {
69 t.Fatalf("merge request not opened from the fork:\n%s", out)
70 }
71
72 // Forking twice collides on the name, and says so on the form rather
73 // than failing silently.
74 _, body := browserPost(t, bob, base+"/fork", url.Values{})
75 if !strings.Contains(body, `class="error"`) {
76 t.Errorf("a second fork reported nothing:\n%s", body)
77 }
78
79 // A fork can land in an organization the forker administers, under a
80 // name of its own — which is also how you fork the same repository
81 // twice.
82 inst.ssh(t, bobKey, "", "org", "create", "bobco")
83 if _, page := browserGet(t, bob, base+"/fork"); !strings.Contains(page, `<option value="bobco"`) {
84 t.Fatalf("the fork form does not offer the organization:\n%s", page)
85 }
86 if status, body := browserPost(t, bob, base+"/fork", url.Values{
87 "owner": {"bobco"}, "name": {"app2"}}); status != 200 || !strings.Contains(body, "bobco/app2") {
88 t.Fatalf("fork into the organization did not land: %d\n%s", status, body)
89 }
90 if out, _, _ := inst.ssh(t, bobKey, "", "repo", "show", "bobco/app2", "--json"); !strings.Contains(out, `"fork_of":"alice/app"`) {
91 t.Fatalf("the organization's fork does not name its parent:\n%s", out)
92 }
93
94 // An organization you are only a member of is not somewhere you can
95 // put one: the owner is checked as it is on repo create.
96 inst.ssh(t, aliceKey, "", "org", "create", "aliceco")
97 inst.ssh(t, aliceKey, "", "org", "members", "add", "aliceco", "bob")
98 if _, _, code := inst.ssh(t, bobKey, "", "repo", "fork", "alice/app", "--owner", "aliceco"); code != 4 {
99 t.Errorf("fork into an org bob does not administer exited %d, want 4", code)
100 }
101}