Editor #3
Editor #3
Conversation
There was a problem hiding this comment.
По большей части вы тестите то, что корректно работает Qt, тестить нужно вашу логику работы приложения. Если у вас есть компонент, который должен выводить файлы из директории - отличным тестом будет проверка при передаче директории, действительно выводятся файлы из нее. Либо, например, сделать стаб объект от с интерфейсом как у QFileSystemModel и проверять, что ваш компонент выводит переданное. Оцениваю на 7
|
|
||
| void testAddThreeObjects() | ||
| { | ||
| DFESceneInspectorTree *scene_tree = new DFESceneInspectorTree(); |
DFEditor/include/DFESceneObject.hpp
Outdated
|
|
||
| QString GetName() { return m_name; } | ||
|
|
||
| virtual IDFESO_ERR Rename(QString new_name) |
There was a problem hiding this comment.
у интерфейсов не может быть реализации и членов данных
| public: | ||
| explicit DFESceneInspector(QWidget *parent = nullptr, Qt::WindowFlags flags = Qt::WindowFlags()); | ||
|
|
||
| DFESI_ERR AddSceneObject(std::shared_ptr<IDFESceneObject> &new_object); |
There was a problem hiding this comment.
Смотрите лекции, shared_ptr передается по значению
|
|
||
| QCOMPARE(obj->windowTitle(), "Content Manager"); | ||
| QVERIFY2(obj->widget() != 0, "Internal widget was not initialized properly"); | ||
| delete obj; |
There was a problem hiding this comment.
при выбросе исключения объект не будет удален
There was a problem hiding this comment.
По большей части вы тестите то, что корректно работает Qt, тестить нужно вашу логику работы приложения. Если у вас есть компонент, который должен выводить файлы из директории - отличным тестом будет проверка при передаче директории, действительно выводятся файлы из нее. Либо, например, сделать стаб объект от с интерфейсом как у QFileSystemModel и проверять, что ваш компонент выводит переданное. Оцениваю на 7
Tests