tests #24
4 Participants
Notifications
Due Date
No due date set.
Blocks
Reference: SHS/pip#24
Reference in New Issue
Block a user
Delete Branch "tests"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
option(TESTS "Build tests and perform their before install step" OFF)
38-ая строка у меня стояло ON
я вижу что у тебя ON, в том и вопрос зачем ты сделал ON?
было OFF а ты сделал ON
и лучше пиши ответ прямо где мой комментарий а не тут
Поясню - в CMakeLists прописывается дефолтное значение, и трогать его не надо. А вот уже в конфиге сборки, как на скриншоте, надо отметить галочку и ткнуть кнопочку Применить. При этом в текущей конфигурации сборки эта опция установится в ON и он пойдет делать тесты
В связи с большим рефакторингом CMakeLists.txt и каталога pip нужно будет перенести тесты под новую версию, но теперь там стало проще тесты добавлять
@@ -0,0 +1,936 @@#include "gtest/gtest.h"#include "pimathmatrix.h"using namespace std;Эта строчка скорее всего не нужна в данном файле.
Да, не нужна
@@ -0,0 +3,4 @@using namespace std;PIMathMatrix<double> origMatr;см. comment-740
@@ -0,0 +12,4 @@double d;bool a, b;matrix = origMatr.identity(3, 3);d = matrix.determinant();Вычисление определителя не соответствует названию теста. Если проверка определителя созданной единичной матрицы необходима, нужно вынести ее в отдельный тест.
Вопрос решён
@@ -0,0 +257,4 @@}}i++;}Блок кода слишком длинный и сложный для понимания. Нужно убрать лишние переменные и упростить логику. К тому же, в коде обнаружено несколько дубликатов этого куска кода. Нужно объединить все в одну функцию и вызывать ее.
@@ -0,0 +369,4 @@}}i++;}см. comment-738
@@ -0,0 +493,4 @@}}i++;}см. comment-738
@@ -0,0 +531,4 @@}}i++;}см. comment-738
@@ -0,0 +566,4 @@}}i++;}см. comment-738
@@ -0,0 +601,4 @@}}i++;}см. comment-738
@@ -0,0 +638,4 @@}}i++;}см. comment-738
@@ -0,0 +4,4 @@const uint rows = 3;const uint cols = 3;PIMathMatrixT<rows, cols, double> matr;PIMathVectorT<rows, double> vect;Нельзя использовать изменияемые глобальные данные из нескольких тестов т.к. действия внутри тестов могут влиять на результаты других тестов. Вместо этого можно использовать Фиксации(fixtures).