1e04b7bdcb
Sind beide Mega-Debrid-Modi (API + Web) aktiv, entscheidet megaDebridPreferApi, welcher Modus zuerst versucht wird. Beim Speichern des Account-Dialogs hat applyAccountDialogToSettings diese Einstellung jedoch fest überschrieben: ein Bearbeiten der API-Zeile erzwang megaDebridPreferApi=true, ein Bearbeiten der Web-Zeile erzwang false. Wer also bewusst Web bevorzugt (preferApi=false) hatte und nur einen zweiten Account hinzufügen oder das Tageslimit ändern wollte und dafür die API-Zeile bearbeitete, dem wurde die Reihenfolge ungefragt auf "API zuerst" umgestellt — was die Provider-Auflösung für alle künftigen Mega-Links ändert. Fix: Die festen megaDebridPreferApi-Werte aus beiden Dialog-Zweigen entfernt. Der bestehende Wert wird über den ...settings-Spread durchgereicht. Erst-Anlage bleibt korrekt, weil der Default ohnehin true ist (constants). Bei nur EINEM aktiven Modus ist preferApi ohnehin irrelevant (die single-mode-Zweige kurzschließen davor). Test: bei beiden Modi aktiv + preferApi=false bleibt nach API-Edit false; bei preferApi=true bleibt nach Web-Edit true. Mit der alten Festverdrahtung kippt der Wert (rot bewiesen).
36 lines
1.4 KiB
TypeScript
36 lines
1.4 KiB
TypeScript
import { describe, expect, it } from "vitest";
|
|
import { applyAccountDialogToSettings, AccountDialogState } from "../src/renderer/App";
|
|
import { defaultSettings } from "../src/main/constants";
|
|
|
|
function megaDialog(kind: "megadebrid-api" | "megadebrid-web"): AccountDialogState {
|
|
return {
|
|
mode: "edit",
|
|
kind,
|
|
token: "",
|
|
login: "",
|
|
password: "",
|
|
dailyLimitGb: "",
|
|
keyDailyLimitGbById: {},
|
|
megaAccounts: [{ login: "user@x", password: "pw" }],
|
|
megaNewLogin: "",
|
|
megaNewPassword: "",
|
|
megaDisabledIds: []
|
|
};
|
|
}
|
|
|
|
describe("applyAccountDialogToSettings — keeps the user's Mega preferApi choice", () => {
|
|
it("does not flip megaDebridPreferApi to true when editing the API account", () => {
|
|
const settings = { ...defaultSettings(), megaDebridApiEnabled: true, megaDebridWebEnabled: true, megaDebridPreferApi: false };
|
|
const next = applyAccountDialogToSettings(settings, megaDialog("megadebrid-api"));
|
|
expect(next.megaDebridApiEnabled).toBe(true);
|
|
expect(next.megaDebridPreferApi).toBe(false);
|
|
});
|
|
|
|
it("does not flip megaDebridPreferApi to false when editing the Web account", () => {
|
|
const settings = { ...defaultSettings(), megaDebridApiEnabled: true, megaDebridWebEnabled: true, megaDebridPreferApi: true };
|
|
const next = applyAccountDialogToSettings(settings, megaDialog("megadebrid-web"));
|
|
expect(next.megaDebridWebEnabled).toBe(true);
|
|
expect(next.megaDebridPreferApi).toBe(true);
|
|
});
|
|
});
|