refactor(db, config, repos): migrate from functions to struct #9

Maks1mS · 2025-01-14T08:56:13Z

Maks1mS commented

2025-01-14 08:56:13 +00:00

Плюсы:

избавление от глобальных переменных;
инверсия зависимостей;
модульность;
упрощение модульного тестирования.

Для постепенной миграции были оставлены старые функции, которые используют глобальный instance config и db.
Под постепенной миграцией подразумевается переписывание конкретной команды ALR, чтобы не было вызовов deprecated методов или методов, использующих deprecated методы.
Для примера в данном PR я отрефакторил команду list.

Так же в данном PR были исправлены все тесты (closes #6)

Плюсы: - избавление от глобальных переменных; - инверсия зависимостей; - модульность; - упрощение модульного тестирования. Для постепенной миграции были оставлены старые функции, которые используют глобальный instance config и db. Под постепенной миграцией подразумевается переписывание конкретной команды ALR, чтобы не было вызовов deprecated методов или методов, использующих deprecated методы. Для примера в данном PR я отрефакторил команду list. Так же в данном PR были исправлены все тесты (closes #6)

Maks1mS added 5 commits 2025-01-14 08:56:14 +00:00

refactor: migrate db and config packages to use struct-based API 52d3ab7791

Removed global variables in favor of instance variables. This makes the code more maintainable and making it easier to write unit tests without relying on global state.

Marked the old functions with global state as obsolete, redirecting them to use a new API based on struct in order to rewrite the code using these functions gradually.

refactor: migrate list command to struct API a13acc5ed0

refactor: use context logger e827fb8049

refactor: migrate repo to struct 91937a1fc5

refactor: move defaultConfig from config_legacy to config eeb25c239b

Maks1mS added 2 commits 2025-01-14 10:04:17 +00:00

refactor: migrate dlcache to struct 6bc6bfdcd9

test: fix decoder and handlers tests 12d83f2015

Maks1mS added 1 commit 2025-01-14 10:19:57 +00:00

refactor: migrate repos find to struct e1829c4824

xpamych merged commit 1fb9c6b574 into master

2025-01-18 15:27:11 +00:00

xpamych referenced this issue from a commit