Do not clear tests group if test_group_path is the same as project_group_path #193
Conversation
lib/generamba/module_generator.rb
Outdated
| # It's possible that current project doesn't test targets configured, so it doesn't need to generate tests. | ||
| # The same is for files property - a template can have only test or project files | ||
| if targets.count == 0 || files == nil || files.count == 0 || dir_path == nil || group_path == nil | ||
| return | ||
| end | ||
|
|
||
| XcodeprojHelper.clear_group(project, targets, group_path) | ||
| XcodeprojHelper.clear_group(project, targets, group_path) if clear_group |
There was a problem hiding this comment.
Думаю, лучше сразу на уровне метода process_files_if_needed формировать этот флаг. Действие по очистке группы внутри проектного файла смотрится как дополнительное и не является необходимым для выполнения метода. Поэтому предлагаю флаг clear_group убрать из интерфейса метода
There was a problem hiding this comment.
Внутри process_files_if_needed у нас недостаточно информации для решения стоит ли чистить группы. А именно нам нужно проверить: чистили ли мы уже эту группу или нет (или, по-другому, совпадает ли группа для тестов с группой для кода).
Поэтому, правильно ли я понимаю, что очистку групп надо будет убрать из process_files_if_needed и делать в самом вызывающем коде, то есть в generate_module?
There was a problem hiding this comment.
Давай передадим в метод process_files_if_needed не флаг, а вот этот параметр code_module.project_group_path. И у нас будет достаточно информации. Тогда, если я правильно понимаю, сам метод сможет принимать решение, как ему управлять группой внутри проекта.
There is no need to clear the same group twice when tests are placed alongside with feature implementation.