Wiki Deploy Code

2026-04-30 19:51:50 +02:00
parent d893785c0c
commit f480815e5d
7 changed files with 801 additions and 407 deletions
@@ -17,49 +17,45 @@ func TestDeployWikiDryRunDoesNotWriteRemoteOrManifest(t *testing.T) {
 	if err := os.MkdirAll(filepath.Join(sourceDir, "skills"), 0755); err != nil {
 		t.Fatalf("create source dir: %v", err)
 	}
-	if err := os.WriteFile(filepath.Join(sourceDir, "skills", "athletics.txt"), []byte("Generated athletics page\n"), 0644); err != nil {
+	generated := "<!-- sow-topdata-wiki:managed:start -->\n<h1>Athletics</h1>\n<p>Generated athletics page</p>\n<!-- sow-topdata-wiki:managed:end -->\n"
+	if err := os.WriteFile(filepath.Join(sourceDir, "skills", "athletics.html"), []byte(generated), 0644); err != nil {
 		t.Fatalf("write source page: %v", err)
 	}
+	old := "<!-- sow-topdata-wiki:managed:start -->\n<h1>Athletics</h1>\n<p>Existing athletics page</p>\n<!-- sow-topdata-wiki:managed:end -->\n"
+	oldHash := computeManagedHash(old)
+	manifestPath := filepath.Join(root, "deploy-manifest.json")
+	if err := saveDeployManifest(manifestPath, wikiDeployManifest{
+		Version: "nodebb-v1",
+		Pages: map[string]wikiDeployManifestPage{
+			"skills:athletics": {Hash: oldHash, TID: 7, PID: 42, CID: 3},
+		},
+	}); err != nil {
+		t.Fatalf("write deploy manifest: %v", err)
+	}

-	putPageCalls := 0
+	updateCalls := 0
 	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
-		var req struct {
-			Method string `json:"method"`
+		if got := r.Header.Get("Authorization"); got != "Bearer nodebb-token" {
+			t.Fatalf("unexpected authorization header %q", got)
 		}
-		if err := json.NewDecoder(r.Body).Decode(&req); err != nil {
-			t.Fatalf("decode rpc request: %v", err)
-		}
-
-		var result any
-		switch req.Method {
-		case "wiki.getPage":
-			result = "Existing athletics page\n"
-		case "dokuwiki.getPagelist":
-			result = []any{}
-		case "wiki.putPage":
-			putPageCalls++
-			result = true
+		switch {
+		case r.Method == http.MethodGet && r.URL.Path == "/api/v3/posts/42":
+			w.Header().Set("Content-Type", "application/json")
+			_ = json.NewEncoder(w).Encode(map[string]any{"response": map[string]any{"pid": 42, "tid": 7, "content": old}})
+		case r.Method == http.MethodPut && r.URL.Path == "/api/v3/posts/42":
+			updateCalls++
+			w.Header().Set("Content-Type", "application/json")
+			_ = json.NewEncoder(w).Encode(map[string]any{"response": map[string]any{"pid": 42, "tid": 7}})
 		default:
-			t.Fatalf("unexpected rpc method %q", req.Method)
-		}
-
-		w.Header().Set("Content-Type", "application/json")
-		if err := json.NewEncoder(w).Encode(map[string]any{
-			"jsonrpc": "2.0",
-			"id":      1,
-			"result":  result,
-		}); err != nil {
-			t.Fatalf("encode rpc response: %v", err)
+			t.Fatalf("unexpected request %s %s", r.Method, r.URL.Path)
 		}
 	}))
 	defer server.Close()

-	manifestPath := filepath.Join(root, "deploy-manifest.json")
 	result, err := DeployWikiWithOptions(&project.Project{}, DeployWikiOptions{
 		SourceDir:    sourceDir,
 		Endpoint:     server.URL,
-		Username:     "wiki-user",
-		Password:     "wiki-pass",
+		Token:        "nodebb-token",
 		ManifestPath: manifestPath,
 		DryRun:       true,
 	}, nil)
@@ -70,10 +66,77 @@ func TestDeployWikiDryRunDoesNotWriteRemoteOrManifest(t *testing.T) {
 	if result.Updated != 1 {
 		t.Fatalf("expected one planned update, got %d", result.Updated)
 	}
-	if putPageCalls != 0 {
-		t.Fatalf("expected dry run not to call wiki.putPage, got %d calls", putPageCalls)
+	if updateCalls != 0 {
+		t.Fatalf("expected dry run not to update NodeBB posts, got %d calls", updateCalls)
 	}
-	if _, err := os.Stat(manifestPath); !os.IsNotExist(err) {
-		t.Fatalf("expected dry run not to write manifest, got %v", err)
+	after := loadDeployManifest(manifestPath)
+	if after.Pages["skills:athletics"].Hash != oldHash {
+		t.Fatalf("expected dry run not to rewrite manifest")
+	}
+}
+
+func TestDeployWikiCreatesNodeBBTopicAndWritesManifest(t *testing.T) {
+	root := t.TempDir()
+	sourceDir := filepath.Join(root, "pages")
+	if err := os.MkdirAll(filepath.Join(sourceDir, "skills"), 0755); err != nil {
+		t.Fatalf("create source dir: %v", err)
+	}
+	generated := "<!-- sow-topdata-wiki:managed:start -->\n<h1>Athletics</h1>\n<p>Generated athletics page</p>\n<!-- sow-topdata-wiki:managed:end -->\n"
+	if err := os.WriteFile(filepath.Join(sourceDir, "skills", "athletics.html"), []byte(generated), 0644); err != nil {
+		t.Fatalf("write source page: %v", err)
+	}
+
+	createCalls := 0
+	server := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+		if got := r.Header.Get("Authorization"); got != "Bearer nodebb-token" {
+			t.Fatalf("unexpected authorization header %q", got)
+		}
+		if r.Method != http.MethodPost || r.URL.Path != "/api/v3/topics" {
+			t.Fatalf("unexpected request %s %s", r.Method, r.URL.Path)
+		}
+		createCalls++
+		var req struct {
+			CID     int    `json:"cid"`
+			Title   string `json:"title"`
+			Content string `json:"content"`
+		}
+		if err := json.NewDecoder(r.Body).Decode(&req); err != nil {
+			t.Fatalf("decode create request: %v", err)
+		}
+		if req.CID != 5 || req.Title != "Athletics" || req.Content != generated {
+			t.Fatalf("unexpected create request: %#v", req)
+		}
+		w.Header().Set("Content-Type", "application/json")
+		_ = json.NewEncoder(w).Encode(map[string]any{
+			"response": map[string]any{
+				"tid": 11,
+				"mainPost": map[string]any{
+					"pid": 42,
+					"tid": 11,
+				},
+			},
+		})
+	}))
+	defer server.Close()
+
+	manifestPath := filepath.Join(root, "deploy-manifest.json")
+	result, err := DeployWikiWithOptions(&project.Project{}, DeployWikiOptions{
+		SourceDir:    sourceDir,
+		Endpoint:     server.URL,
+		Token:        "nodebb-token",
+		ManifestPath: manifestPath,
+		CategoryIDs:  map[string]int{"skills": 5},
+		AllowCreates: true,
+	}, nil)
+	if err != nil {
+		t.Fatalf("DeployWikiWithOptions create failed: %v", err)
+	}
+	if result.Created != 1 || createCalls != 1 {
+		t.Fatalf("expected one created topic, result=%d calls=%d", result.Created, createCalls)
+	}
+	manifest := loadDeployManifest(manifestPath)
+	entry := manifest.Pages["skills:athletics"]
+	if entry.TID != 11 || entry.PID != 42 || entry.CID != 5 || entry.Hash != computeManagedHash(generated) {
+		t.Fatalf("unexpected manifest entry: %#v", entry)
 	}
 }