package e2e

import (
	"net/http"
	"os/exec"
	"strings"
	"testing"
)

// Chapter 14: a malformed config falls back to the last version that parsed, not to the defaults.
func TestABrokenConfigKeepsTheSettingsThatWorked(t *testing.T) {
	if _, err := exec.LookPath("git"); err != nil {
		t.Skip("git is not installed")
	}
	in := newInstance(t)
	john := in.account("john")
	lisa := in.account("lisa")
	work := seed(t, in, john, "john", "johnbot")

	write(t, work, ".barerepo/config", "[repo]\nvisibility = \"public\"\n\n[access]\npush = [\"john\", \"lisa\"]\n")
	run(t, work, "git", "add", "-A")
	run(t, work, "git", "commit", "-qm", "open it up and let lisa push")
	run(t, work, "git", "push", "-q", in.url(john, "/john/johnbot"), "master")

	// A stranger can read it and lisa can push, which is what the good file bought.
	if code, _, _ := get(t, in.http.URL+"/john/johnbot"); code != http.StatusOK {
		t.Fatalf("the repository is not public yet, so the test proves nothing later")
	}

	// One unclosed bracket, which is the shape of every real config typo.
	write(t, work, ".barerepo/config", "[repo\nvisibility = \"public\"\n")
	run(t, work, "git", "add", "-A")
	run(t, work, "git", "commit", "-qm", "a typo in the config")
	out := run(t, work, "git", "push", in.url(john, "/john/johnbot"), "master")

	// The push that makes the typo is the one that reports it, not whoever pushes next.
	if !strings.Contains(out, "does not parse") {
		t.Errorf("the push that broke the file said nothing about it:\n%s", out)
	}

	// And every later push says which older version is doing the deciding.
	write(t, work, "conn.go", "package irc\n")
	run(t, work, "git", "add", "-A")
	run(t, work, "git", "commit", "-qm", "an unrelated change")
	later := run(t, work, "git", "push", in.url(john, "/john/johnbot"), "master")
	if !strings.Contains(later, "does not parse") {
		t.Errorf("a later push said nothing about the broken file:\n%s", later)
	}
	if !strings.Contains(later, "still in force") {
		t.Errorf("a later push did not name the settings that are deciding:\n%s", later)
	}

	// The defaults are private, so a public repository is proof the old file is still being read.
	if code, _, _ := get(t, in.http.URL+"/john/johnbot"); code != http.StatusOK {
		t.Errorf("a typo in the config took the repository private, and chapter 14 forbids that")
	}

	// And the access list survived, which is the clause about not locking anyone out.
	mineDir := clone(t, in, lisa, "/john/johnbot")
	commit(t, mineDir, "package main\n\nfunc main() {}\n", "lisa can still push")
	if _, err := try(t, mineDir, "git", "push", in.url(lisa, "/john/johnbot"), "master"); err != nil {
		t.Errorf("lisa lost her push access to a typo: %v", err)
	}

	// A reader of the config page is reading the broken file and must be told it is not in force.
	code, _, body := get(t, in.http.URL+"/john/johnbot/config")
	if code != http.StatusOK {
		t.Fatalf("the config page answered %d", code)
	}
	if !strings.Contains(body, "does not parse") {
		t.Errorf("the config page shows the broken file as though it were in force:\n%s", body)
	}
}