tests #24

Merged
peri4 merged 12 commits from tests into master 2020-09-01 21:01:47 +03:00
Collaborator
No description provided.
Owner

image

![image](/attachments/04f0bc00-3efa-4492-9942-11b19182d1f4)
Author
Collaborator

option(TESTS "Build tests and perform their before install step" OFF)
38-ая строка у меня стояло ON

option(TESTS "Build tests and perform their before install step" OFF) 38-ая строка у меня стояло ON
maakshishov started working 2020-07-24 14:45:12 +03:00
Owner

option(TESTS "Build tests and perform their before install step" OFF)
38-ая строка у меня стояло ON

я вижу что у тебя ON, в том и вопрос зачем ты сделал ON?
было OFF а ты сделал ON

и лучше пиши ответ прямо где мой комментарий а не тут

> option(TESTS "Build tests and perform their before install step" OFF) > 38-ая строка у меня стояло ON я вижу что у тебя ON, в том и вопрос зачем ты сделал ON? было OFF а ты сделал ON и лучше пиши ответ прямо где мой комментарий а не тут
andrey added a new dependency 2020-07-31 12:47:16 +03:00
Owner

Поясню - в CMakeLists прописывается дефолтное значение, и трогать его не надо. А вот уже в конфиге сборки, как на скриншоте, надо отметить галочку и ткнуть кнопочку Применить. При этом в текущей конфигурации сборки эта опция установится в ON и он пойдет делать тесты

Поясню - в CMakeLists прописывается дефолтное значение, и трогать его не надо. А вот уже в конфиге сборки, как на скриншоте, надо отметить галочку и ткнуть кнопочку Применить. При этом в текущей конфигурации сборки эта опция установится в ON и он пойдет делать тесты
Owner

В связи с большим рефакторингом CMakeLists.txt и каталога pip нужно будет перенести тесты под новую версию, но теперь там стало проще тесты добавлять

В связи с большим рефакторингом CMakeLists.txt и каталога pip нужно будет перенести тесты под новую версию, но теперь там стало проще тесты добавлять
maakshishov was assigned by andrey 2020-08-01 23:56:49 +03:00
andrey self-assigned this 2020-08-01 23:56:51 +03:00
zzuummaa was assigned by andrey 2020-08-01 23:56:52 +03:00
zzuummaa approved these changes 2020-08-31 15:24:51 +03:00
@@ -0,0 +1,936 @@
#include "gtest/gtest.h"
#include "pimathmatrix.h"
using namespace std;
Contributor

Эта строчка скорее всего не нужна в данном файле.

Эта строчка скорее всего не нужна в данном файле.
Author
Collaborator

Да, не нужна

Да, не нужна
@@ -0,0 +3,4 @@
using namespace std;
PIMathMatrix<double> origMatr;
Contributor

см. comment-740

см. [comment-740](https://git.shs.tools/SHS/pip/pulls/24/files#issuecomment-740)
@@ -0,0 +12,4 @@
double d;
bool a, b;
matrix = origMatr.identity(3, 3);
d = matrix.determinant();
Contributor

Вычисление определителя не соответствует названию теста. Если проверка определителя созданной единичной матрицы необходима, нужно вынести ее в отдельный тест.

Вычисление определителя не соответствует названию теста. Если проверка определителя созданной единичной матрицы необходима, нужно вынести ее в отдельный тест.
Author
Collaborator

Вопрос решён

Вопрос решён
@@ -0,0 +257,4 @@
}
}
i++;
}
Contributor

Блок кода слишком длинный и сложный для понимания. Нужно убрать лишние переменные и упростить логику. К тому же, в коде обнаружено несколько дубликатов этого куска кода. Нужно объединить все в одну функцию и вызывать ее.

Блок кода слишком длинный и сложный для понимания. Нужно убрать лишние переменные и упростить логику. К тому же, в коде обнаружено несколько дубликатов этого куска кода. Нужно объединить все в одну функцию и вызывать ее.
@@ -0,0 +369,4 @@
}
}
i++;
}
Contributor

см. comment-738

см. [comment-738](https://git.shs.tools/SHS/pip/pulls/24/files#issuecomment-738)
@@ -0,0 +493,4 @@
}
}
i++;
}
Contributor

см. comment-738

см. [comment-738](https://git.shs.tools/SHS/pip/pulls/24/files#issuecomment-738)
@@ -0,0 +531,4 @@
}
}
i++;
}
Contributor

см. comment-738

см. [comment-738](https://git.shs.tools/SHS/pip/pulls/24/files#issuecomment-738)
@@ -0,0 +566,4 @@
}
}
i++;
}
Contributor

см. comment-738

см. [comment-738](https://git.shs.tools/SHS/pip/pulls/24/files#issuecomment-738)
@@ -0,0 +601,4 @@
}
}
i++;
}
Contributor

см. comment-738

см. [comment-738](https://git.shs.tools/SHS/pip/pulls/24/files#issuecomment-738)
@@ -0,0 +638,4 @@
}
}
i++;
}
Contributor

см. comment-738

см. [comment-738](https://git.shs.tools/SHS/pip/pulls/24/files#issuecomment-738)
@@ -0,0 +4,4 @@
const uint rows = 3;
const uint cols = 3;
PIMathMatrixT<rows, cols, double> matr;
PIMathVectorT<rows, double> vect;
Contributor

Нельзя использовать изменияемые глобальные данные из нескольких тестов т.к. действия внутри тестов могут влиять на результаты других тестов. Вместо этого можно использовать Фиксации(fixtures).

Нельзя использовать изменияемые глобальные данные из нескольких тестов т.к. действия внутри тестов могут влиять на результаты других тестов. Вместо этого можно использовать [Фиксации(fixtures)](https://habr.com/ru/post/119090/).
peri4 merged commit 5ef37dc7fd into master 2020-09-01 21:01:46 +03:00
Sign in to join this conversation.