Open ViktorNosenko opened 2 years ago
@thxnk Враховуючи зміни в рамках #25 тести також потребуватимуть внесення змін.
1) Питання з StaticTest.setDoNotRunTrigger(); все ще відкрито.
Подумай чи потрібна нам ця конструкція взагалі.
Як я розумію основною причиною створеня цього класу стали проблеми з витягуванням записів для тестування.
Можливо ми можемо витягнути записи з БД іншим методом?
2) Замість System.assertEquals(true, result > 0); можна використовувати System.assert(result > 0);
3) Чи потрібно нам приводити число до числа
(Integer)amountOfTours.AmountOfToursToDisplay__c
Тести виглядають гарно, але є речі які можна покращити.
Питання до роздумів: 1) Чи потрібен нам клас StaticTest? Чи можемо ми створити статичну змінну в самому хендлері щоб не ускладнювати логіку? 2) Методи startTest() і stopTest() використовуються для того щоб прослідкувати за СФ лімітами. Тобто будь-якому коду, який виконується після виклику startTest і до stopTest, призначається новий набір СФ лімітів. Тому бажано:
Більше про це тут - Using Limits, startTest, and stopTest