Conversation
| .WillOnce(testing::Return(true)); | ||
| } | ||
|
|
||
| void BeforeCopyFileTest(const std::string& srcFolder, |
| EXPECT_NO_THROW(fileCopier.Copy(s_srcFolder, s_dstFolder)); | ||
| } | ||
|
|
||
| TEST(FileCopierTests, Copy_FolderWithoutFiles) |
There was a problem hiding this comment.
я не увидел, чтобы тест был красным.
Здесь или последовательность тестов неправильная или тест явно лишний
| EXPECT_NO_THROW(fileCopier.Copy(s_srcFolder, s_dstFolder)); | ||
| } | ||
|
|
||
| TEST(FileCopierTests, Copy_FileAndFolderWithFile) |
There was a problem hiding this comment.
Посути, этот тест делает то, для чего уже были красные и зеленые тесты. Только здесь идет проверка, что мы скопируем как файлы в текущей папке так и из другой папки, находящейся в этой.
С одной стороны то что он желает уже покрыто, но нам также нужно знать что он выполняет и эти функции.
| const std::string dstPath(ConcatPath(dst, relativePath)); | ||
| if (m_fileSystem->IsDirectory(srcFilePath)) | ||
| { | ||
| Copy(srcFilePath, dstPath); |
There was a problem hiding this comment.
меня смущает то, что если хотя бы один файл из иерархии будет отсутствовать, то мы не скопируем дальше ничего
There was a problem hiding this comment.
Идет проверка не существования файла, а существования папки. Ну и кроме проверки при первом вызове копирования, она больше не нужна. Следовало бы вынести функционал копирования в отдельную функцию.
5 participants
No description provided.