florian.wilde
7920b6a20f
CI / pruefen (pull_request) Canceled after 0s
Die Angaben stammten aus der Zeit, als Nebula nur gelesen hat. Seit v1.7.0 legt es Issues an und schliesst sie - wer sich an README oder den Hinweis im Verbindungsdialog hielt, bekam dabei einen 403. Erhoben wurde es nicht aus dem Gedaechtnis, sondern aus dem Code: jeder ausgehende Aufruf in lib/git/gitea.ts, github.ts und gitlab.ts, abgebildet auf das noetige Recht. Aufgeschluesselt nach Funktion statt nach System, damit erkennbar ist, was sich weglassen laesst: der Abgleich allein kommt rein lesend aus, nur die Issue-Anbindung schreibt. **GitLab ist der unangenehme Fall.** Es kennt keinen schmalen Schreib-Scope fuer Issues - dort geht nur "api", also alles, was das Konto darf. Das steht jetzt ausdruecklich da, samt der Alternative: bei read_api bleiben und auf die Issue-Anbindung verzichten. Der Abgleich laeuft dann vollstaendig, und geschlossene Issues werden weiterhin erkannt, nur nicht zurueckgeschrieben. Ebenfalls festgehalten, was Nebula *nicht* braucht: Schreibrechte auf Code, Adminrechte, Rechte zum Anlegen von Webhooks. Letzteres ist der Grund, warum der Haken von Hand eingetragen wird - so bleibt der Token davon frei. Der Hinweis in lib/git/meta.ts steht direkt am Eingabefeld und ist die Stelle, die tatsaechlich gelesen wird; er ist deshalb mitgezogen. Dazu ein Waechter in meta.test.ts: jeder Provider muss in GitClient createIssue und setIssueState koennen, folglich muss auch jeder Hinweis sagen, was das an Rechten kostet. Genau diese Kopplung war hier gerissen. Geprueft: 267 Unit-Tests (6 neue), 36 E2E-Tests, Hinweistext je System gegen die laufende Anwendung gelesen, axe-core auf 375px und 1280px ohne Beanstandung. Refs #10
61 lines
2.3 KiB
TypeScript
61 lines
2.3 KiB
TypeScript
import { describe, expect, it } from "vitest";
|
||
import { PROVIDER_META, guessProvider } from "./meta";
|
||
|
||
const PROVIDER = ["GITHUB", "GITLAB", "GITEA"] as const;
|
||
|
||
describe("PROVIDER_META", () => {
|
||
it("kennt jedes unterstuetzte System", () => {
|
||
for (const provider of PROVIDER) {
|
||
expect(PROVIDER_META[provider], provider).toBeDefined();
|
||
}
|
||
// Und nichts darueber hinaus – ein Eintrag ohne Gegenstueck im Schema
|
||
// waere ein Ueberbleibsel.
|
||
expect(Object.keys(PROVIDER_META).sort()).toEqual([...PROVIDER].sort());
|
||
});
|
||
|
||
it("hat zu jedem System Beschriftung, Basis-URL und Dokuverweis", () => {
|
||
for (const provider of PROVIDER) {
|
||
const meta = PROVIDER_META[provider];
|
||
expect(meta.label.length, provider).toBeGreaterThan(0);
|
||
expect(meta.defaultBaseUrl, provider).toMatch(/^https:\/\//);
|
||
expect(meta.docsUrl, provider).toMatch(/^https:\/\//);
|
||
}
|
||
});
|
||
|
||
/**
|
||
* Der eigentliche Waechter.
|
||
*
|
||
* Der Hinweis am Eingabefeld war eine Zeit lang rein lesend formuliert,
|
||
* obwohl Nebula seit v1.7.0 Issues anlegt und schliesst – wer sich daran
|
||
* hielt, bekam einen 403. Jeder Provider in GitClient muss createIssue und
|
||
* setIssueState koennen; folglich muss auch jeder Hinweis sagen, was das
|
||
* an Rechten kostet.
|
||
*/
|
||
it("sagt bei jedem System etwas ueber die Rechte fuer Issues", () => {
|
||
for (const provider of PROVIDER) {
|
||
expect(PROVIDER_META[provider].tokenHelp, provider).toMatch(/issue/i);
|
||
}
|
||
});
|
||
|
||
it("nennt einen konkreten Scope statt nur 'Leserechte'", () => {
|
||
// GitLab nennt Scopes in Anfuehrungszeichen, GitHub und Gitea
|
||
// Berechtigungsnamen – gemeinsam ist, dass etwas Nachschlagbares dasteht.
|
||
expect(PROVIDER_META.GITLAB.tokenHelp).toMatch(/read_api/);
|
||
expect(PROVIDER_META.GITEA.tokenHelp).toMatch(/read:repository/);
|
||
expect(PROVIDER_META.GITHUB.tokenHelp).toMatch(/Metadata/);
|
||
});
|
||
});
|
||
|
||
describe("guessProvider", () => {
|
||
it("erkennt die oeffentlichen Instanzen am Hostnamen", () => {
|
||
expect(guessProvider("github.com")).toBe("GITHUB");
|
||
expect(guessProvider("gitlab.com")).toBe("GITLAB");
|
||
});
|
||
|
||
it("faellt bei allem anderen auf Gitea zurueck", () => {
|
||
// Selbst gehostet ist hier der haeufigste Fall, und Gitea die
|
||
// wahrscheinlichste Wahl dafuer.
|
||
expect(guessProvider("git.wilde-server.de")).toBe("GITEA");
|
||
});
|
||
});
|