add main_test.go

.github/workflows/test.yaml

@@ -30,6 +30,11 @@ jobs:
         go-version-file: go.mod
     - name: test
       run: go test -coverprofile coverage.out -covermode atomic ./...
+      env:
+        BLOGSYNC_TEST_BLOG: ${{ secrets.BLOGSYNC_BLOG }}
+        BLOGSYNC_USERNAME: ${{ secrets.BLOGSYNC_USERNAME }}
+        BLOGSYNC_PASSWORD: ${{ secrets.BLOGSYNC_PASSWORD }}
+        BLOGSYNC_OWNER: ${{ secrets.BLOGSYNC_OWNER }}
     - name: Send coverage
       uses: shogo82148/actions-goveralls@v1
       with:

.gitignore

@@ -1,3 +1,4 @@
+.env*
 /blogsync
 /dist
 coverage.out

config_test.go

@@ -32,7 +32,7 @@ func TestLoadConfigration(t *testing.T) {
 					if tmpVal != "" {
 						return os.Setenv(envKey, tmpVal)
 					}
-					return nil
+					return os.Unsetenv(envKey)
 				}, func() error {
 					if ok {
 						return os.Setenv(envKey, env)

main.go

@@ -325,7 +325,7 @@ var commandPost = &cli.Command{
 			return fmt.Errorf("blog not found: %s", blog)
 		}
 
-		entry, err := entryFromReader(os.Stdin)
+		entry, err := entryFromReader(c.App.Reader)
 		if err != nil {
 			return err
 		}

main_test.go

@@ -0,0 +1,202 @@
+//go:build darwin || integration
+
+package main
+
+import (
+	"bytes"
+	"fmt"
+	"os"
+	"path/filepath"
+	"regexp"
+	"strings"
+	"testing"
+	"time"
+
+	"github.com/urfave/cli/v2"
+)
+
+func blogsyncApp(app *cli.App) func(...string) (string, error) {
+	buf := &bytes.Buffer{}
+	app.Writer = buf
+	return func(args ...string) (string, error) {
+		buf.Reset()
+		err := app.Run(append([]string{""}, args...))
+		return strings.TrimSpace(buf.String()), err
+	}
+}
+
+func exists(path string) bool {
+	_, err := os.Stat(path)
+	return err == nil
+}
+
+var draftFileReg = regexp.MustCompile(`entry/_draft/\d+\.md$`)
+
+func TestBlogsync(t *testing.T) {
+	blogID := os.Getenv("BLOGSYNC_TEST_BLOG")
+	if blogID == "" {
+		t.Skip("BLOGSYNC_TEST_BLOG not set")
+	}
+
+	pwd, err := os.Getwd()
+	if err != nil {
+		t.Fatal(err)
+	}
+	dir := t.TempDir()
+	if err := os.Chdir(dir); err != nil {
+		t.Fatal(err)
+	}
+	defer func() {
+		if err := os.Chdir(pwd); err != nil {
+			t.Fatal(err)
+		}
+	}()
+
+	dir, err = filepath.EvalSymlinks(dir)
+	if err != nil {
+		t.Fatal(err)
+	}
+
+	confYAML := fmt.Sprintf(`%s:
+  local_root: .
+  omit_domain: true
+`, blogID)
+	if owner := os.Getenv("BLOGSYNC_OWNER"); owner != "" {
+		confYAML += fmt.Sprintf("  owner: %s\n", owner)
+	}
+	conf := filepath.Join(dir, "blogsync.yaml")
+	if err := os.WriteFile(conf, []byte(confYAML), 0644); err != nil {
+		t.Fatal(err)
+	}
+
+	app := newApp()
+	blogsync := blogsyncApp(app)
+
+	t.Run("pull", func(t *testing.T) {
+		if _, err := blogsync("pull"); err != nil {
+			t.Fatal(err)
+		}
+	})
+
+	t.Run("post draft and publish", func(t *testing.T) {
+		t.Log("Post a draft without a custom path and check if the file is saved in the proper location")
+		app.Reader = strings.NewReader("draft\n")
+		entryFile, err := blogsync("post", "--draft", blogID)
+		app.Reader = os.Stdin
+		if err != nil {
+			t.Fatal(err)
+		}
+		defer func() {
+			t.Log("remove the published entry")
+			if _, err := blogsync("remove", entryFile); err != nil {
+				t.Fatal(err)
+			}
+		}()
+
+		if !draftFileReg.MatchString(entryFile) {
+			t.Fatalf("unexpected draft file: %s", entryFile)
+		}
+
+		t.Log("Draft files under `_draft/` will revert to the original file name if the file is renamed and pushed again")
+		d, f := filepath.Split(entryFile)
+		movedPath := filepath.Join(d, "_"+f)
+		if err := os.Rename(entryFile, movedPath); err != nil {
+			t.Fatal(err)
+		}
+		originalEntryFile := entryFile
+		entryFile = movedPath
+		if err := appendFile(movedPath, "updated\n"); err != nil {
+			t.Fatal(err)
+		}
+		draftFile, err := blogsync("push", entryFile)
+		if err != nil {
+			t.Fatal(err)
+		}
+		if draftFile != originalEntryFile {
+			entryFile = draftFile
+			t.Fatalf("unexpected draft file: %s", draftFile)
+		}
+		if exists(entryFile) {
+			t.Errorf("renamed draft file is not deleted: %s", movedPath)
+		}
+		entryFile = draftFile
+
+		t.Log("When a draft is published, a URL is issued and the file is saved in the corresponding location")
+		publishedFile, err := blogsync("push", "--publish", entryFile)
+		if err != nil {
+			t.Fatal(err)
+		}
+		if exists(entryFile) {
+			t.Errorf("draft file not deleted: %s", entryFile)
+		}
+		entryFile = publishedFile
+
+		_, entryPath := extractEntryPath(entryFile)
+		if !isLikelyGivenPath(entryPath) {
+			t.Errorf("unexpected published file: %s", entryFile)
+		}
+	})
+
+	t.Run("post draft and publish with custom path", func(t *testing.T) {
+		t.Log("Creating a draft with a custom path saves the file in the specified location, not under `_draft/`")
+		localFile := filepath.Join(dir, "entry", time.Now().Format("custom-20060102150405")+".md")
+		if err := os.WriteFile(localFile, []byte(`---
+Draft: true
+---
+test`), 0644); err != nil {
+			t.Fatal(err)
+		}
+		entryFile, err := blogsync("push", localFile)
+		if err != nil {
+			t.Fatal(err)
+		}
+		defer func() {
+			t.Log("remove the published entry")
+			if _, err := blogsync("remove", entryFile); err != nil {
+				t.Fatal(err)
+			}
+		}()
+		if entryFile != localFile {
+			t.Errorf("unexpected published file: %s", entryFile)
+		}
+
+		t.Log("When publishing a draft with a custom path, the file location is unchanged")
+		publishedFile, err := blogsync("push", "--publish", entryFile)
+		if err != nil {
+			t.Fatal(err)
+		}
+		if publishedFile != entryFile {
+			t.Errorf("unexpected published file: %s", publishedFile)
+		}
+
+		t.Log("If the file name of an entry is changed, the custom path will follow suit")
+		d, f := filepath.Split(entryFile)
+		movedPath := filepath.Join(d, "custom-"+f)
+		if err := os.Rename(entryFile, movedPath); err != nil {
+			t.Fatal(err)
+		}
+		entryFile = movedPath
+		if err := appendFile(entryFile, "updated\n"); err != nil {
+			t.Fatal(err)
+		}
+		publishedFile, err = blogsync("push", entryFile)
+		if err != nil {
+			t.Fatal(err)
+		}
+		if publishedFile != entryFile {
+			entryFile = publishedFile
+			t.Errorf("unexpected published file: %s", publishedFile)
+		}
+	})
+}
+
+func appendFile(path string, content string) error {
+	fh, err := os.OpenFile(path, os.O_APPEND|os.O_WRONLY, 0644)
+	if err != nil {
+		return err
+	}
+	if _, err := fh.WriteString(content); err != nil {
+		return err
+	}
+	return fh.Close()
+}