Skip to content

Фоллбэк при сравнении версий пакета, не удовлетворяющих semver #242

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 5 commits into from
Jul 3, 2025
Merged

Фоллбэк при сравнении версий пакета, не удовлетворяющих semver #242

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
32d84a3
Initial plan
Copilot Jul 3, 2025
ef44da6
Add fallback mechanism for version comparison with non-semver versions
Copilot Jul 3, 2025
6599b64
Replace РаботаСВерсиями with local version comparison function for fa…
Copilot Jul 3, 2025
78c2da4
Use РаботаСВерсиями module instead of local version comparison function
Copilot Jul 3, 2025
0e9ec9d
Update version from 1.4.1 to 1.4.2
Copilot Jul 3, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion packagedef
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@
Константы_ЛокальнаяВерсия = ЗагрузитьСценарий(ПутьКСценариюКонстант);
ВерсияПродукта = Константы_ЛокальнаяВерсия.ВерсияПродукта;
Иначе
ВерсияПродукта = "1.4.1";
ВерсияПродукта = "1.4.2";
КонецЕсли;

Описание.Имя("opm")
Expand Down
23 changes: 18 additions & 5 deletions src/core/Классы/КэшУстановленныхПакетов.os
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -39,9 +39,20 @@
УстановленаКорректнаяВерсия = Истина;
ИначеЕсли ОписаниеЗависимости.МинимальнаяВерсия = ВерсияSNAPSHOT И УстановленнаяВерсия <> ВерсияSNAPSHOT Тогда
УстановленаКорректнаяВерсия = Ложь;
Иначе
УстановленаКорректнаяВерсия = ОписаниеЗависимости.МинимальнаяВерсия = Неопределено
ИЛИ Версии.СравнитьВерсии(УстановленнаяВерсия, ОписаниеЗависимости.МинимальнаяВерсия) >= 0;
Иначе
Если ОписаниеЗависимости.МинимальнаяВерсия = Неопределено Тогда
УстановленаКорректнаяВерсия = Истина;
Иначе
Попытка
// Сначала пытаемся использовать семвер
РезультатСравнения = Версии.СравнитьВерсии(УстановленнаяВерсия, ОписаниеЗависимости.МинимальнаяВерсия);
Исключение
// Если семвер не может обработать версию (например, 4-значную), используем модуль РаботаСВерсиями
РезультатСравнения = РаботаСВерсиями.СравнитьВерсии(УстановленнаяВерсия, ОписаниеЗависимости.МинимальнаяВерсия);
КонецПопытки;

УстановленаКорректнаяВерсия = РезультатСравнения >= 0;
КонецЕсли;
КонецЕсли;

Лог.Отладка("Пакет установлен: " + ПакетУстановлен);
Expand Down Expand Up @@ -160,8 +171,10 @@

Возврат ПутьКФайлуМетаданных;

КонецФункции

КонецФункции



Процедура ПриСозданииОбъекта(КаталогПоискаБиблиотек = "")

Лог = Логирование.ПолучитьЛог("oscript.app.opm");
Expand Down
2 changes: 1 addition & 1 deletion src/core/Модули/КонстантыOpm.os
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -24,5 +24,5 @@
ПутьВЗапасномХранилище = "/download/";
ЛокальныйКаталогУстановкиПакетов = "oscript_modules";
ИмяЛога = "oscript.app.opm";
ВерсияПродукта = "1.4.1";
ВерсияПродукта = "1.4.2";
ИмяФайлаНастроек = "opm.cfg";
21 changes: 21 additions & 0 deletions tests/versions-test.os
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,7 @@
СписокТестов.Добавить("ТестДолжен_ПроверитьЧтоВерсияАБольшеВерсииБ");
СписокТестов.Добавить("ТестДолжен_ПолучитьИмяИВерсиюПоИмениПакетаБезВерсии");
СписокТестов.Добавить("ТестДолжен_ПолучитьИмяИВерсиюПоИмениПакетаСВерсией");
СписокТестов.Добавить("ТестДолжен_СравнитьЧетырехзначныеВерсии");

Возврат СписокТестов;

Expand Down Expand Up @@ -89,3 +90,23 @@
Ожидаем.Что(Пакет.Версия).Равно("1.1.1");

КонецПроцедуры

Процедура ТестДолжен_СравнитьЧетырехзначныеВерсии() Экспорт

// Тестируем сравнение четырехзначных версий (не семвер)
ВерсияА = "1.2.3.4";
ВерсияБ = "1.2.3.3";

// Проверяем что старый механизм может сравнивать 4-значные версии
РезультатСравнения = РаботаСВерсиями.СравнитьВерсии(ВерсияА, ВерсияБ);
Ожидаем.Что(РезультатСравнения).Больше(0);

// Проверяем равенство
РезультатСравнения = РаботаСВерсиями.СравнитьВерсии(ВерсияА, ВерсияА);
Ожидаем.Что(РезультатСравнения).Равно(0);

// Проверяем меньше
РезультатСравнения = РаботаСВерсиями.СравнитьВерсии(ВерсияБ, ВерсияА);
Ожидаем.Что(РезультатСравнения).Меньше(0);

КонецПроцедуры