-
Notifications
You must be signed in to change notification settings - Fork 6
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.
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
fluent в советДругогоМастера #106
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,180 @@ | ||
#Использовать asserts | ||
#Использовать ".." | ||
|
||
Перем Рефлектор; // Рефлектор | ||
|
||
&Тест | ||
Процедура ЗначенияДеталек_Получить() Экспорт | ||
|
||
// Дано | ||
|
||
ЗначенияДеталек = Новый Соответствие; | ||
ЗначенияДеталек.Вставить("Ключ", "Значение"); | ||
|
||
СоветДругогоМастера = Новый СоветДругогоМастера(); | ||
|
||
Comment on lines
+14
to
+15
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🛠️ Refactor suggestion Устранение дублирования кода в тестовых процедурах В нескольких тестовых процедурах повторяется код создания объекта Например, можно создать общую процедуру инициализации:
И использовать её в тестах:
Also applies to: 36-37, 56-57, 77-78, 97-98, 118-119, 139-140, 164-165 |
||
Рефлектор.УстановитьСвойство(СоветДругогоМастера, "ЗначенияДеталек", ЗначенияДеталек); | ||
|
||
// Когда | ||
|
||
Результат = СоветДругогоМастера.ЗначенияДеталек(); | ||
|
||
// Тогда | ||
|
||
Ожидаем.Что(Результат).Равно(ЗначенияДеталек); | ||
|
||
КонецПроцедуры | ||
|
||
&Тест | ||
Процедура ЗначенияДеталек_Установить() Экспорт | ||
|
||
// Дано | ||
|
||
ЗначенияДеталек = Новый Соответствие; | ||
ЗначенияДеталек.Вставить("Ключ", "Значение"); | ||
|
||
СоветДругогоМастера = Новый СоветДругогоМастера(); | ||
|
||
// Когда | ||
|
||
Результат = СоветДругогоМастера.ЗначенияДеталек(ЗначенияДеталек); | ||
|
||
// Тогда | ||
|
||
Ожидаем.Что(Результат).Равно(СоветДругогоМастера); | ||
Ожидаем.Что(Рефлектор.ПолучитьСвойство(СоветДругогоМастера, "ЗначенияДеталек")).Равно(ЗначенияДеталек); | ||
|
||
КонецПроцедуры | ||
|
||
&Тест | ||
Процедура ПрефиксПеременныхСреды_Получить() Экспорт | ||
|
||
// Дано | ||
|
||
ПрефиксПеременныхСреды = "Тест"; | ||
|
||
СоветДругогоМастера = Новый СоветДругогоМастера(); | ||
|
||
Рефлектор.УстановитьСвойство(СоветДругогоМастера, "ПрефиксПеременныхСреды", ПрефиксПеременныхСреды); | ||
|
||
// Когда | ||
|
||
Результат = СоветДругогоМастера.ПрефиксПеременныхСреды(); | ||
|
||
// Тогда | ||
|
||
Ожидаем.Что(Результат).Равно(ПрефиксПеременныхСреды); | ||
|
||
КонецПроцедуры | ||
|
||
&Тест | ||
Процедура ПрефиксПеременныхСреды_Установить() Экспорт | ||
|
||
// Дано | ||
|
||
ПрефиксПеременныхСреды = "Тест"; | ||
|
||
СоветДругогоМастера = Новый СоветДругогоМастера(); | ||
|
||
// Когда | ||
|
||
Результат = СоветДругогоМастера.ПрефиксПеременныхСреды(ПрефиксПеременныхСреды); | ||
|
||
// Тогда | ||
|
||
Ожидаем.Что(Результат).Равно(СоветДругогоМастера); | ||
Ожидаем.Что(Рефлектор.ПолучитьСвойство(СоветДругогоМастера, "ПрефиксПеременныхСреды")).Равно(ПрефиксПеременныхСреды); | ||
|
||
КонецПроцедуры | ||
|
||
&Тест | ||
Процедура ИмяФайлаСоЗначениямиДеталек_Получить() Экспорт | ||
|
||
// Дано | ||
|
||
ИмяФайлаСоЗначениямиДеталек = "/path/to/file"; | ||
|
||
СоветДругогоМастера = Новый СоветДругогоМастера(); | ||
|
||
Рефлектор.УстановитьСвойство(СоветДругогоМастера, "ИмяФайлаСоЗначениямиДеталек", ИмяФайлаСоЗначениямиДеталек); | ||
|
||
// Когда | ||
|
||
Результат = СоветДругогоМастера.ИмяФайлаСоЗначениямиДеталек(); | ||
|
||
// Тогда | ||
|
||
Ожидаем.Что(Результат).Равно(ИмяФайлаСоЗначениямиДеталек); | ||
|
||
КонецПроцедуры | ||
|
||
&Тест | ||
Процедура ИмяФайлаСоЗначениямиДеталек_Установить() Экспорт | ||
|
||
// Дано | ||
|
||
ИмяФайлаСоЗначениямиДеталек = "/path/to/file"; | ||
|
||
СоветДругогоМастера = Новый СоветДругогоМастера(); | ||
|
||
// Когда | ||
|
||
Результат = СоветДругогоМастера.ИмяФайлаСоЗначениямиДеталек(ИмяФайлаСоЗначениямиДеталек); | ||
|
||
// Тогда | ||
|
||
Ожидаем.Что(Результат).Равно(СоветДругогоМастера); | ||
Ожидаем.Что(Рефлектор.ПолучитьСвойство(СоветДругогоМастера, "ИмяФайлаСоЗначениямиДеталек")) | ||
.Равно(ИмяФайлаСоЗначениямиДеталек); | ||
|
||
КонецПроцедуры | ||
|
||
&Тест | ||
Процедура ДополнительныйКаталогПоискаФайлаСоЗначениямиДеталек_Получить() Экспорт | ||
|
||
// Дано | ||
|
||
ДополнительныйКаталогПоискаФайлаСоЗначениямиДеталек = "/path/to/dir"; | ||
|
||
СоветДругогоМастера = Новый СоветДругогоМастера(); | ||
|
||
Рефлектор.УстановитьСвойство( | ||
СоветДругогоМастера, | ||
"ДополнительныйКаталогПоискаФайлаСоЗначениямиДеталек", | ||
ДополнительныйКаталогПоискаФайлаСоЗначениямиДеталек | ||
); | ||
|
||
// Когда | ||
|
||
Результат = СоветДругогоМастера.ДополнительныйКаталогПоискаФайлаСоЗначениямиДеталек(); | ||
|
||
// Тогда | ||
|
||
Ожидаем.Что(Результат).Равно(ДополнительныйКаталогПоискаФайлаСоЗначениямиДеталек); | ||
|
||
КонецПроцедуры | ||
|
||
&Тест | ||
Процедура ДополнительныйКаталогПоискаФайлаСоЗначениямиДеталек_Установить() Экспорт | ||
|
||
// Дано | ||
|
||
ДополнительныйКаталогПоискаФайлаСоЗначениямиДеталек = "/path/to/dir"; | ||
|
||
СоветДругогоМастера = Новый СоветДругогоМастера(); | ||
|
||
// Когда | ||
|
||
Результат = СоветДругогоМастера.ДополнительныйКаталогПоискаФайлаСоЗначениямиДеталек( | ||
ДополнительныйКаталогПоискаФайлаСоЗначениямиДеталек | ||
); | ||
|
||
// Тогда | ||
|
||
Ожидаем.Что(Результат).Равно(СоветДругогоМастера); | ||
Ожидаем.Что(Рефлектор.ПолучитьСвойство(СоветДругогоМастера, "ДополнительныйКаталогПоискаФайлаСоЗначениямиДеталек")) | ||
.Равно(ДополнительныйКаталогПоискаФайлаСоЗначениямиДеталек); | ||
|
||
КонецПроцедуры | ||
|
||
Рефлектор = Новый Рефлектор(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Инициализация переменной "Рефлектор" происходит после её использования
Переменная
Рефлектор
объявлена на строке 4, но инициализируется только на строке 180, после того как она используется в тестовых процедурах. Это может привести к ошибке выполнения из-за обращения к неинициализированной переменной. Рекомендуется инициализироватьРефлектор
перед его использованием.Примените следующий дифф для исправления:
И удалите инициализацию на строке 180:
-Рефлектор = Новый Рефлектор();
Also applies to: 180-180