Голый magnet без dn/контекста заставлял namer звать LLM на пустом входе, и модель галлюцинировала мусорное имя («Unknown»), которое писалось и в display_name, и в rename qBittorrent, а заодно ломало UI-фолбэк на распознанное название. Верное каноническое имя, вычисляемое позже при распознавании, никуда не переливалось. - naming: гард пустого входа в DeriveName (нет контекста и подсказки → "" без вызова LLM) + детерминированный форматтер FormatTitleYear. - qbt: операция RenameTorrent (переименование существующей раздачи). - store: SetDisplayName — обновление имени постфактум без гарда состояния. - worker: refreshDisplayNameLocked/RefreshDisplayName — перелив канонического имени (эффективный план) в display_name + best-effort rename раздачи по реальному t.Hash; авто-триггер при подтверждении матча (choose/manual add). - web-ui: кнопка «Обновить имя» на странице загрузки (htmx-своп заголовка, деградация без JS), видимая при наличии распознавания (вкл. done/orphaned). Спека: дельты ingest/review/web-ui влиты в openspec/specs; change refresh-display-name заархивирован. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
241 lines
7.3 KiB
Go
241 lines
7.3 KiB
Go
package qbt
|
|
|
|
import (
|
|
"context"
|
|
"net/http"
|
|
"net/http/httptest"
|
|
"strings"
|
|
"testing"
|
|
)
|
|
|
|
// fakeQBittorrent — минимальный стенд WebUI API: требует cookie SID, выдаёт
|
|
// его на /auth/login. Так проверяется и ленивый логин по 403.
|
|
func fakeQBittorrent(t *testing.T, info string) *httptest.Server {
|
|
t.Helper()
|
|
mux := http.NewServeMux()
|
|
|
|
mux.HandleFunc("/api/v2/auth/login", func(w http.ResponseWriter, r *http.Request) {
|
|
_ = r.ParseForm()
|
|
if r.PostForm.Get("username") != "admin" || r.PostForm.Get("password") != "secret" {
|
|
w.WriteHeader(http.StatusForbidden)
|
|
return
|
|
}
|
|
http.SetCookie(w, &http.Cookie{Name: "SID", Value: "token", Path: "/"})
|
|
_, _ = w.Write([]byte("Ok."))
|
|
})
|
|
|
|
authed := func(next http.HandlerFunc) http.HandlerFunc {
|
|
return func(w http.ResponseWriter, r *http.Request) {
|
|
if c, err := r.Cookie("SID"); err != nil || c.Value != "token" {
|
|
w.WriteHeader(http.StatusForbidden)
|
|
return
|
|
}
|
|
next(w, r)
|
|
}
|
|
}
|
|
|
|
mux.HandleFunc("/api/v2/torrents/add", authed(func(w http.ResponseWriter, r *http.Request) {
|
|
if err := r.ParseMultipartForm(1 << 20); err != nil {
|
|
http.Error(w, err.Error(), http.StatusBadRequest)
|
|
return
|
|
}
|
|
if r.FormValue("category") != "jellybit" || !strings.Contains(r.FormValue("urls"), "magnet:") {
|
|
w.WriteHeader(http.StatusBadRequest)
|
|
return
|
|
}
|
|
_, _ = w.Write([]byte("Ok."))
|
|
}))
|
|
|
|
mux.HandleFunc("/api/v2/torrents/info", authed(func(w http.ResponseWriter, r *http.Request) {
|
|
if r.URL.Query().Get("category") != "jellybit" {
|
|
w.WriteHeader(http.StatusBadRequest)
|
|
return
|
|
}
|
|
w.Header().Set("Content-Type", "application/json")
|
|
_, _ = w.Write([]byte(info))
|
|
}))
|
|
|
|
srv := httptest.NewServer(mux)
|
|
t.Cleanup(srv.Close)
|
|
return srv
|
|
}
|
|
|
|
func newClient(t *testing.T, url string) *Client {
|
|
t.Helper()
|
|
c, err := New(Config{URL: url, Username: "admin", Password: "secret"}, nil)
|
|
if err != nil {
|
|
t.Fatalf("New: %v", err)
|
|
}
|
|
return c
|
|
}
|
|
|
|
func TestAddSendsRename(t *testing.T) {
|
|
tests := []struct {
|
|
name string
|
|
rename string
|
|
wantField string
|
|
wantHasKey bool
|
|
}{
|
|
{name: "with rename", rename: "Дюна: Часть вторая (2024)", wantField: "Дюна: Часть вторая (2024)", wantHasKey: true},
|
|
{name: "empty rename omits field", rename: "", wantHasKey: false},
|
|
}
|
|
for _, tc := range tests {
|
|
t.Run(tc.name, func(t *testing.T) {
|
|
var gotField string
|
|
var hasKey bool
|
|
mux := http.NewServeMux()
|
|
mux.HandleFunc("/api/v2/torrents/add", func(w http.ResponseWriter, r *http.Request) {
|
|
if err := r.ParseMultipartForm(1 << 20); err != nil {
|
|
http.Error(w, err.Error(), http.StatusBadRequest)
|
|
return
|
|
}
|
|
_, hasKey = r.MultipartForm.Value["rename"]
|
|
gotField = r.FormValue("rename")
|
|
_, _ = w.Write([]byte("Ok."))
|
|
})
|
|
srv := httptest.NewServer(mux)
|
|
t.Cleanup(srv.Close)
|
|
|
|
c := newClient(t, srv.URL)
|
|
err := c.Add(context.Background(), AddRequest{
|
|
URLs: []string{"magnet:?xt=urn:btih:541adcff3b6dd5dba7088ea83317d9d6fac331d6"},
|
|
Rename: tc.rename,
|
|
})
|
|
if err != nil {
|
|
t.Fatalf("Add: %v", err)
|
|
}
|
|
if hasKey != tc.wantHasKey {
|
|
t.Errorf("наличие поля rename = %v, want %v", hasKey, tc.wantHasKey)
|
|
}
|
|
if gotField != tc.wantField {
|
|
t.Errorf("rename = %q, want %q", gotField, tc.wantField)
|
|
}
|
|
})
|
|
}
|
|
}
|
|
|
|
func TestDeleteSendsHashesAndDeleteFiles(t *testing.T) {
|
|
var gotHashes, gotDeleteFiles string
|
|
mux := http.NewServeMux()
|
|
mux.HandleFunc("/api/v2/torrents/delete", func(w http.ResponseWriter, r *http.Request) {
|
|
_ = r.ParseForm()
|
|
gotHashes = r.PostForm.Get("hashes")
|
|
gotDeleteFiles = r.PostForm.Get("deleteFiles")
|
|
_, _ = w.Write([]byte("Ok."))
|
|
})
|
|
srv := httptest.NewServer(mux)
|
|
t.Cleanup(srv.Close)
|
|
|
|
c := newClient(t, srv.URL)
|
|
err := c.Delete(context.Background(), []string{"aaa", "", "bbb"}, true)
|
|
if err != nil {
|
|
t.Fatalf("Delete: %v", err)
|
|
}
|
|
if gotHashes != "aaa|bbb" { // пустые хеши отфильтрованы, склейка через |
|
|
t.Errorf("hashes = %q, want %q", gotHashes, "aaa|bbb")
|
|
}
|
|
if gotDeleteFiles != "true" {
|
|
t.Errorf("deleteFiles = %q, want true", gotDeleteFiles)
|
|
}
|
|
}
|
|
|
|
func TestRenameTorrentSendsHashAndName(t *testing.T) {
|
|
var gotHash, gotName string
|
|
mux := http.NewServeMux()
|
|
mux.HandleFunc("/api/v2/torrents/rename", func(w http.ResponseWriter, r *http.Request) {
|
|
_ = r.ParseForm()
|
|
gotHash = r.PostForm.Get("hash")
|
|
gotName = r.PostForm.Get("name")
|
|
_, _ = w.Write([]byte("Ok."))
|
|
})
|
|
srv := httptest.NewServer(mux)
|
|
t.Cleanup(srv.Close)
|
|
|
|
c := newClient(t, srv.URL)
|
|
if err := c.RenameTorrent(context.Background(), "aaa", "Harold and the Purple Crayon (2024)"); err != nil {
|
|
t.Fatalf("RenameTorrent: %v", err)
|
|
}
|
|
if gotHash != "aaa" {
|
|
t.Errorf("hash = %q, want aaa", gotHash)
|
|
}
|
|
if gotName != "Harold and the Purple Crayon (2024)" {
|
|
t.Errorf("name = %q", gotName)
|
|
}
|
|
}
|
|
|
|
func TestRenameTorrentEmptyHashIsError(t *testing.T) {
|
|
c := newClient(t, "http://unused")
|
|
if err := c.RenameTorrent(context.Background(), " ", "x"); err == nil {
|
|
t.Fatal("RenameTorrent with empty hash must error before any request")
|
|
}
|
|
}
|
|
|
|
func TestRenameTorrentNon200IsError(t *testing.T) {
|
|
mux := http.NewServeMux()
|
|
mux.HandleFunc("/api/v2/torrents/rename", func(w http.ResponseWriter, r *http.Request) {
|
|
http.Error(w, "Not Found", http.StatusNotFound)
|
|
})
|
|
srv := httptest.NewServer(mux)
|
|
t.Cleanup(srv.Close)
|
|
|
|
c := newClient(t, srv.URL)
|
|
if err := c.RenameTorrent(context.Background(), "aaa", "x"); err == nil {
|
|
t.Fatal("RenameTorrent must error on non-200")
|
|
}
|
|
}
|
|
|
|
func TestDeleteNoHashesIsError(t *testing.T) {
|
|
c := newClient(t, "http://unused")
|
|
if err := c.Delete(context.Background(), []string{"", " "}, true); err == nil {
|
|
t.Fatal("Delete with no hashes must error before any request")
|
|
}
|
|
}
|
|
|
|
func TestAddPerformsLazyLogin(t *testing.T) {
|
|
srv := fakeQBittorrent(t, "[]")
|
|
c := newClient(t, srv.URL)
|
|
|
|
// Первый вызов без cookie → сервер вернёт 403 → клиент логинится и повторяет.
|
|
err := c.Add(context.Background(), AddRequest{
|
|
URLs: []string{"magnet:?xt=urn:btih:541adcff3b6dd5dba7088ea83317d9d6fac331d6"},
|
|
Category: "jellybit",
|
|
SavePath: "/srv/media/downloads",
|
|
})
|
|
if err != nil {
|
|
t.Fatalf("Add: %v", err)
|
|
}
|
|
}
|
|
|
|
func TestTorrents(t *testing.T) {
|
|
const body = `[{"hash":"541adcff3b6dd5dba7088ea83317d9d6fac331d6","name":"Dune","state":"uploading","save_path":"/srv/media/downloads","content_path":"/srv/media/downloads/Dune","progress":1.0,"amount_left":0}]`
|
|
srv := fakeQBittorrent(t, body)
|
|
c := newClient(t, srv.URL)
|
|
|
|
ts, err := c.Torrents(context.Background(), "jellybit")
|
|
if err != nil {
|
|
t.Fatalf("Torrents: %v", err)
|
|
}
|
|
if len(ts) != 1 {
|
|
t.Fatalf("torrents = %d, want 1", len(ts))
|
|
}
|
|
got := ts[0]
|
|
if got.Hash != "541adcff3b6dd5dba7088ea83317d9d6fac331d6" || got.State != "uploading" {
|
|
t.Errorf("torrent = %+v", got)
|
|
}
|
|
if got.ContentPath != "/srv/media/downloads/Dune" {
|
|
t.Errorf("content_path = %q", got.ContentPath)
|
|
}
|
|
}
|
|
|
|
func TestLoginFailure(t *testing.T) {
|
|
srv := fakeQBittorrent(t, "[]")
|
|
c, err := New(Config{URL: srv.URL, Username: "admin", Password: "wrong"}, nil)
|
|
if err != nil {
|
|
t.Fatal(err)
|
|
}
|
|
// 403 → попытка логина с неверным паролем → снова 403 → ошибка.
|
|
if _, err := c.Torrents(context.Background(), "jellybit"); err == nil {
|
|
t.Error("ожидалась ошибка логина")
|
|
}
|
|
}
|