616db33c0e
In order to split backend.proto into a more manageable size, the protobuf handling needed to be updated. This took more time than I would have liked, as each language handles protobuf differently: - The Python Protobuf code ignores "package" directives, and relies solely on how the files are laid out on disk. While it would have been nice to keep the generated files in a private subpackage, Protobuf gets confused if the files are located in a location that does not match their original .proto layout, so the old approach of storing them in _backend/ will not work. They now clutter up pylib/anki instead. I'm rather annoyed by that, but alternatives seem to be having to add an extra level to the Protobuf path, making the other languages suffer, or trying to hack around the issue by munging sys.modules. - Protobufjs fails to expose packages if they don't start with a capital letter, despite the fact that lowercase packages are the norm in most languages :-( This required a patch to fix. - Rust was the easiest, as Prost is relatively straightforward compared to Google's tools. The Protobuf files are now stored in /proto/anki, with a separate package for each file. I've split backend.proto into a few files as a test, but the majority of that work is still to come. The Python Protobuf building is a bit of a hack at the moment, hard-coding "proto" as the top level folder, but it seems to get the job done for now. Also changed the workspace name, as there seems to be a number of Bazel repos moving away from the more awkward reverse DNS naming style.
331 lines
9.3 KiB
TypeScript
331 lines
9.3 KiB
TypeScript
// Copyright: Ankitects Pty Ltd and contributors
|
|
// License: GNU AGPL, version 3 or later; http://www.gnu.org/licenses/agpl.html
|
|
|
|
/* eslint
|
|
@typescript-eslint/no-explicit-any: "off",
|
|
*/
|
|
|
|
import { Backend } from "lib/proto";
|
|
import { DeckOptionsState } from "./lib";
|
|
import { get } from "svelte/store";
|
|
|
|
const exampleData = {
|
|
allConfig: [
|
|
{
|
|
config: {
|
|
id: "1",
|
|
name: "Default",
|
|
mtimeSecs: "1618570764",
|
|
usn: -1,
|
|
config: {
|
|
learnSteps: [1, 10],
|
|
relearnSteps: [10],
|
|
newPerDay: 10,
|
|
reviewsPerDay: 200,
|
|
initialEase: 2.5,
|
|
easyMultiplier: 1.2999999523162842,
|
|
hardMultiplier: 1.2000000476837158,
|
|
intervalMultiplier: 1,
|
|
maximumReviewInterval: 36500,
|
|
minimumLapseInterval: 1,
|
|
graduatingIntervalGood: 1,
|
|
graduatingIntervalEasy: 4,
|
|
leechAction: "LEECH_ACTION_TAG_ONLY",
|
|
leechThreshold: 8,
|
|
capAnswerTimeToSecs: 60,
|
|
other: "eyJuZXciOnsic2VwYXJhdGUiOnRydWV9LCJyZXYiOnsiZnV6eiI6MC4wNSwibWluU3BhY2UiOjF9fQ==",
|
|
},
|
|
},
|
|
useCount: 1,
|
|
},
|
|
{
|
|
config: {
|
|
id: "1618570764780",
|
|
name: "another one",
|
|
mtimeSecs: "1618570781",
|
|
usn: -1,
|
|
config: {
|
|
learnSteps: [1, 10, 20, 30],
|
|
relearnSteps: [10],
|
|
newPerDay: 40,
|
|
reviewsPerDay: 200,
|
|
initialEase: 2.5,
|
|
easyMultiplier: 1.2999999523162842,
|
|
hardMultiplier: 1.2000000476837158,
|
|
intervalMultiplier: 1,
|
|
maximumReviewInterval: 36500,
|
|
minimumLapseInterval: 1,
|
|
graduatingIntervalGood: 1,
|
|
graduatingIntervalEasy: 4,
|
|
leechAction: "LEECH_ACTION_TAG_ONLY",
|
|
leechThreshold: 8,
|
|
capAnswerTimeToSecs: 60,
|
|
},
|
|
},
|
|
useCount: 1,
|
|
},
|
|
],
|
|
currentDeck: {
|
|
name: "Default::child",
|
|
configId: "1618570764780",
|
|
parentConfigIds: [1],
|
|
},
|
|
defaults: {
|
|
config: {
|
|
learnSteps: [1, 10],
|
|
relearnSteps: [10],
|
|
newPerDay: 20,
|
|
reviewsPerDay: 200,
|
|
initialEase: 2.5,
|
|
easyMultiplier: 1.2999999523162842,
|
|
hardMultiplier: 1.2000000476837158,
|
|
intervalMultiplier: 1,
|
|
maximumReviewInterval: 36500,
|
|
minimumLapseInterval: 1,
|
|
graduatingIntervalGood: 1,
|
|
graduatingIntervalEasy: 4,
|
|
leechAction: "LEECH_ACTION_TAG_ONLY",
|
|
leechThreshold: 8,
|
|
capAnswerTimeToSecs: 60,
|
|
},
|
|
},
|
|
};
|
|
|
|
function startingState(): DeckOptionsState {
|
|
return new DeckOptionsState(
|
|
123,
|
|
Backend.DeckConfigsForUpdate.fromObject(exampleData)
|
|
);
|
|
}
|
|
|
|
test("start", () => {
|
|
const state = startingState();
|
|
expect(state.currentDeck.name).toBe("Default::child");
|
|
});
|
|
|
|
test("deck list", () => {
|
|
const state = startingState();
|
|
expect(get(state.configList)).toStrictEqual([
|
|
{
|
|
current: true,
|
|
idx: 1,
|
|
name: "another one",
|
|
useCount: 1,
|
|
},
|
|
{
|
|
current: false,
|
|
idx: 0,
|
|
name: "Default",
|
|
useCount: 1,
|
|
},
|
|
]);
|
|
expect(get(state.currentConfig).newPerDay).toBe(40);
|
|
|
|
// rename
|
|
state.setCurrentName("zzz");
|
|
expect(get(state.configList)).toStrictEqual([
|
|
{
|
|
current: false,
|
|
idx: 0,
|
|
name: "Default",
|
|
useCount: 1,
|
|
},
|
|
{
|
|
current: true,
|
|
idx: 1,
|
|
name: "zzz",
|
|
useCount: 1,
|
|
},
|
|
]);
|
|
|
|
// add
|
|
state.addConfig("hello");
|
|
expect(get(state.configList)).toStrictEqual([
|
|
{
|
|
current: false,
|
|
idx: 0,
|
|
name: "Default",
|
|
useCount: 1,
|
|
},
|
|
{
|
|
current: true,
|
|
idx: 2,
|
|
name: "hello",
|
|
useCount: 1,
|
|
},
|
|
{
|
|
current: false,
|
|
idx: 1,
|
|
name: "zzz",
|
|
useCount: 0,
|
|
},
|
|
]);
|
|
expect(get(state.currentConfig).newPerDay).toBe(20);
|
|
|
|
// change current
|
|
state.setCurrentIndex(0);
|
|
expect(get(state.configList)).toStrictEqual([
|
|
{
|
|
current: true,
|
|
idx: 0,
|
|
name: "Default",
|
|
useCount: 2,
|
|
},
|
|
{
|
|
current: false,
|
|
idx: 2,
|
|
name: "hello",
|
|
useCount: 0,
|
|
},
|
|
{
|
|
current: false,
|
|
idx: 1,
|
|
name: "zzz",
|
|
useCount: 0,
|
|
},
|
|
]);
|
|
expect(get(state.currentConfig).newPerDay).toBe(10);
|
|
|
|
// can't delete default
|
|
expect(() => state.removeCurrentConfig()).toThrow();
|
|
|
|
// deleting old deck should work
|
|
state.setCurrentIndex(1);
|
|
state.removeCurrentConfig();
|
|
expect(get(state.currentConfig).newPerDay).toBe(10);
|
|
|
|
// as should newly added one
|
|
state.setCurrentIndex(1);
|
|
state.removeCurrentConfig();
|
|
expect(get(state.currentConfig).newPerDay).toBe(10);
|
|
|
|
// only the pre-existing deck should be listed for removal
|
|
const out = state.dataForSaving(false);
|
|
expect(out.removedConfigIds).toStrictEqual([1618570764780]);
|
|
});
|
|
|
|
test("duplicate name", () => {
|
|
const state = startingState();
|
|
|
|
// duplicate will get renamed
|
|
state.addConfig("another one");
|
|
expect(get(state.configList).find((e) => e.current)?.name).toMatch(/another.*\d+$/);
|
|
|
|
// should handle renames too
|
|
state.setCurrentName("Default");
|
|
expect(get(state.configList).find((e) => e.current)?.name).toMatch(/Default\d+$/);
|
|
});
|
|
|
|
test("parent counts", () => {
|
|
const state = startingState();
|
|
|
|
expect(get(state.parentLimits)).toStrictEqual({ newCards: 10, reviews: 200 });
|
|
|
|
// adjusting the current deck config won't alter parent
|
|
state.currentConfig.update((c) => {
|
|
c.newPerDay = 123;
|
|
return c;
|
|
});
|
|
expect(get(state.parentLimits)).toStrictEqual({ newCards: 10, reviews: 200 });
|
|
|
|
// but adjusting the default config will, since the parent deck uses it
|
|
state.setCurrentIndex(0);
|
|
state.currentConfig.update((c) => {
|
|
c.newPerDay = 123;
|
|
return c;
|
|
});
|
|
expect(get(state.parentLimits)).toStrictEqual({ newCards: 123, reviews: 200 });
|
|
});
|
|
|
|
test("saving", () => {
|
|
let state = startingState();
|
|
let out = state.dataForSaving(false);
|
|
expect(out.removedConfigIds).toStrictEqual([]);
|
|
expect(out.targetDeckId).toBe(123);
|
|
// in no-changes case, currently selected config should
|
|
// be returned
|
|
expect(out.configs.length).toBe(1);
|
|
expect(out.configs[0].name).toBe("another one");
|
|
expect(out.applyToChildren).toBe(false);
|
|
|
|
// rename, then change current deck
|
|
state.setCurrentName("zzz");
|
|
out = state.dataForSaving(true);
|
|
state.setCurrentIndex(0);
|
|
|
|
// renamed deck should be in changes, with current deck as last element
|
|
out = state.dataForSaving(true);
|
|
expect(out.configs.map((c) => c.name)).toStrictEqual(["zzz", "Default"]);
|
|
expect(out.applyToChildren).toBe(true);
|
|
|
|
// start again, adding new deck
|
|
state = startingState();
|
|
state.addConfig("hello");
|
|
|
|
// deleting it should not change removedConfigs
|
|
state.removeCurrentConfig();
|
|
out = state.dataForSaving(true);
|
|
expect(out.removedConfigIds).toStrictEqual([]);
|
|
|
|
// select the other non-default deck & remove
|
|
state.setCurrentIndex(1);
|
|
state.removeCurrentConfig();
|
|
|
|
// should be listed in removedConfigs, and modified should
|
|
// only contain Default, which is the new current deck
|
|
out = state.dataForSaving(true);
|
|
expect(out.removedConfigIds).toStrictEqual([1618570764780]);
|
|
expect(out.configs.map((c) => c.name)).toStrictEqual(["Default"]);
|
|
});
|
|
|
|
test("aux data", () => {
|
|
const state = startingState();
|
|
expect(get(state.currentAuxData)).toStrictEqual({});
|
|
state.currentAuxData.update((val) => {
|
|
return { ...val, hello: "world" };
|
|
});
|
|
|
|
// check default
|
|
state.setCurrentIndex(0);
|
|
expect(get(state.currentAuxData)).toStrictEqual({
|
|
new: {
|
|
separate: true,
|
|
},
|
|
rev: {
|
|
fuzz: 0.05,
|
|
minSpace: 1,
|
|
},
|
|
});
|
|
state.currentAuxData.update((val) => {
|
|
return { ...val, defaultAddition: true };
|
|
});
|
|
|
|
// ensure changes serialize
|
|
const out = state.dataForSaving(true);
|
|
expect(out.configs.length).toBe(2);
|
|
const json = out.configs.map(
|
|
(c) =>
|
|
JSON.parse(new TextDecoder().decode((c.config as any).other)) as Record<
|
|
string,
|
|
unknown
|
|
>
|
|
);
|
|
expect(json).toStrictEqual([
|
|
// other deck comes first
|
|
{
|
|
hello: "world",
|
|
},
|
|
// default is selected, so comes last
|
|
{
|
|
defaultAddition: true,
|
|
new: {
|
|
separate: true,
|
|
},
|
|
rev: {
|
|
fuzz: 0.05,
|
|
minSpace: 1,
|
|
},
|
|
},
|
|
]);
|
|
});
|