Skip to content

test: Тест проверки выделяемой памяти пустого бенчмарка #20

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
Stivo182 merged 2 commits into from
May 31, 2025
Merged

test: Тест проверки выделяемой памяти пустого бенчмарка #20

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
121e81f
test: Тест проверки выделяемой памяти пустого бенчмарка
Stivo182 May 31, 2025
aaa0f7e
test: Увеличен диапазон веделенной памяти
Stivo182 May 31, 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
24 changes: 21 additions & 3 deletions tests/ТестыЗамеров.os
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
#Использовать "helpers"

&Тест
Процедура Тест_ВыделяемаяПамять() Экспорт
Процедура Тест_ДолженПроверитьВыделяемуюПамять() Экспорт

РазмерВБайтах = 1024 * 1024;

Expand All @@ -16,14 +16,32 @@
ДескрипторБенчмарка.ДобавитьПараметры(РазмерВБайтах);

Результат = Бенчмаркинг.Запустить(ДескрипторыБенчмарков);
ВыделеноБайт = Результат.Отчет.Таблица[0][КолонкиОтчетаБенчмарков.ВыделяемаяПамять];


ВыделеноБайт = Результат.РезультатыЗапусков[0].Статистика.ВыделяемаяПамять;
Copy link

@coderabbitai coderabbitai bot May 31, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Verification agent

🧩 Analysis chain

API update looks good - verify consistency across codebase.

The new way of accessing allocated memory statistics through РезультатыЗапусков[0].Статистика.ВыделяемаяПамять is more direct and clear than the previous table-based access.

Ensure that all other occurrences of the old API pattern are updated consistently:

🏁 Script executed:

#!/bin/bash
# Description: Search for any remaining usage of the old memory access pattern
# Expected: No occurrences of the old pattern should remain

# Search for old pattern usage
rg "Отчет\.Таблица.*ВыделяемаяПамять" --type os
rg "КолонкиОтчетаБенчмарков\.ВыделяемаяПамять" --type os

Length of output: 162

🏁 Script executed:

#!/usr/bin/env bash
# Re-run search for old memory access pattern without file-type restriction
# Expected: no matches if all old API calls have been replaced

rg "Отчет\.Таблица.*ВыделяемаяПамять"
rg "КолонкиОтчетаБенчмарков\.ВыделяемаяПамять"

Length of output: 354

Fix remaining old memory API references

The search uncovered two places still using the old Отчет.Таблица…ВыделяемаяПамять pattern. Please update these to use the new РезультатыЗапусков[0].Статистика.ВыделяемаяПамять API (or remove them if no longer needed):

• tests/ТестыКонфигурации.os

// before  
Ожидаем.Что(Результат.Отчет.Таблица[0]["ВыделяемаяПамять"]).Заполнено();  
// after  
Ожидаем.Что(Результат.РезультатыЗапусков[0].Статистика.ВыделяемаяПамять).Заполнено();

• src/BenchmarkOneScript/columns/Модули/КолонкаОтчетаБенчмарковВыделяемаяПамять.os

// before  
Колонка = Новый КолонкаОтчетаБенчмарков(КолонкиОтчетаБенчмарков.ВыделяемаяПамять);  
// after (example)  
Колонка = Новый КолонкаСтатистики(Статистика.ВыделяемаяПамять);

(Or remove this column if the table-based report is deprecated.)

🤖 Prompt for AI Agents 
In tests/ТестыЗамеров.os at line 20 and related files tests/ТестыКонфигурации.os
and
src/BenchmarkOneScript/columns/Модули/КолонкаОтчетаБенчмарковВыделяемаяПамять.os,
update all remaining old API references accessing allocated memory statistics
via Отчет.Таблица or КолонкиОтчетаБенчмарков.ВыделяемаяПамять to use the new
direct API РезультатыЗапусков[0].Статистика.ВыделяемаяПамять. Replace
expressions like Результат.Отчет.Таблица[0]["ВыделяемаяПамять"] with
Результат.РезультатыЗапусков[0].Статистика.ВыделяемаяПамять, and refactor or
remove column definitions relying on the old table-based pattern accordingly.

Коэффициент = РазмерВБайтах / ВыделеноБайт;

Ожидаем.Что(Коэффициент).Между(0.99, 1);

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

&Тест
Процедура Тест_ДолженПроверитьВыделяемуюПамятьПустогоБенчмарка() Экспорт

Тип = Тип("ПустойБенчмарк");

Конфигурация = Новый КонфигурацияБенчмарков(Тип)
.ДобавитьМониторингПамяти()
.УстановитьКоличествоИтераций(5)
.УстановитьКоличествоПрогревочныхИтераций(0);

Результат = Бенчмаркинг.Запустить(Тип, Конфигурация);

ВыделеноБайт = Результат.РезультатыЗапусков[0].Статистика.ВыделяемаяПамять;

Ожидаем.Что(ВыделеноБайт).Между(1, 2000);

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

&Тест
Процедура Тест_Производительность() Экспорт

Expand Down
3 changes: 1 addition & 2 deletions tests/ТестыКонфигурации.os
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -313,8 +313,7 @@

Результат = Бенчмаркинг.Запустить(Тип("ПустойБенчмарк"), Конфигурация);

Ожидаем.Что(Результат.Отчет.Таблица).Заполнено();
Ожидаем.Что(Результат.Отчет.Таблица[0]["ВыделяемаяПамять"]).Заполнено();
Ожидаем.Что(Результат.РезультатыЗапусков[0].Статистика.ВыделяемаяПамять).Заполнено();
Ожидаем.Что(Результат.Конфигурация.ТребуетсяМониторингПамяти()).ЭтоИстина();

КонецПроцедуры
Expand Down
Loading