следуете ли вы этому предложению?
# russiane
Eugen Martynov
01/28/2019, 10:02 AMследуете ли вы этому предложению?
g
ghedeon
01/28/2019, 10:06 AMнет, имхо чем меньше шума в тестах, тем лучше. Поля private тоже не делаю.
g
gildor
01/28/2019, 10:11 AMe
Eugen Martynov
01/28/2019, 10:15 AMя тоже, но если поменять тест в модуле, то возможно зависимые модули тоже будут перекомпилированы
g
gildor
01/28/2019, 10:15 AMэто почему? Тест же leaf модуль. Не думаю что это хоть как то влияет на компиляцию в плане скорости
e
Eugen Martynov
01/28/2019, 10:15 AMу меня есть модуль сеть
Eugen Martynov
01/28/2019, 10:16 AMна него зависимы фича модули
Eugen Martynov
01/28/2019, 10:16 AMя меняю только тест в сети
g
gildor
01/28/2019, 10:16 AMну так фича модули же не зависят на тесты только на сеть
e
Eugen Martynov
01/28/2019, 10:16 AMвсе модули будут инвалидированы
g
gildor
01/28/2019, 10:16 AMнет, тесты же отдельный модуль
e
Eugen Martynov
01/28/2019, 10:17 AMну в тестах фича можно достучать тест из сети
g
gildor
01/28/2019, 10:18 AMэто как?
gildor
01/28/2019, 10:18 AMне должно
gildor
01/28/2019, 10:18 AMесли явно нет зависимости
e
Eugen Martynov
01/28/2019, 10:18 AMоке буду пробовать
Eugen Martynov
01/28/2019, 10:19 AMне думаю что грейдл прямо помнит импорты из других модулей
g
gildor
01/28/2019, 10:19 AMну т.е. это не то как Gradle работает по умолчавнию точно
gildor
01/28/2019, 10:20 AMtest source set не становятся доступен как зависиомость, именно поэтому у него свой source set
e
Eugen Martynov
01/28/2019, 10:21 AMзабудем про тесты, есть класс в модуле, он публик, на него можно сослаться в зависимом модуле
g
gildor
01/28/2019, 10:21 AMкогда не теситы то да
e
Eugen Martynov
01/28/2019, 10:22 AMя попробовал и смог обратиться к тест классу из нижнего модуля
Eugen Martynov
01/28/2019, 10:30 AMoke
Eugen Martynov
01/28/2019, 10:30 AMпохоже баг Андроид Студии
Eugen Martynov
01/28/2019, 10:31 AMкод валидный в ней, но компиляция не проходит
m
miha-x64
01/28/2019, 12:42 PMкажется, тесты — это классы, доступные в определённой конфигурации модуля, изменились — можно инвалидировать
e
Eugen Martynov
01/28/2019, 4:45 PMНе понял 🙂
g
ghedeon
01/28/2019, 6:23 PMдавайте по существу. Есть какие-то доказательства, что с internal в тестах будет меньше пересборок gradle?
g
gildor
01/29/2019, 12:25 AMЭто учитывается при инкрементальной компиляции внутри модуля
Но пока что нет compile avoidance поддержки Kotlin в Gradle, поэтому это никак положительно не влияет на пересборки между модулями, просто потому что все задетые модули (из графа зависимостей изменённого модуля) перезапускают компиляцию, вне зависимости поменялось ABI или нет.
Но в 1.3.10 появилась первая версию тулы для того что бы из Котлин модуля экстрактить ABI, что первый шаг к тому что compile avoidance может появится в Котлин на Gradle и тогда это уже будет влиять.
Но, так как тесты это в 99% случаев leaf модуль, то на него никто не зависит и даже внутри модуля, связей мало между тестами, то думаю это никак не влияет ни сейчас ни в будущем
Но я могу ошибаться, это то как я понимаю текущую фичи компилятора, но не то что я это проверял
👍 1
m
miha-x64
01/29/2019, 9:06 AMкажется, тесты можно считать модулем только при выставленной галке create separate module per source set. Всегда её снимаю, сложилось впечатление, что с ней «ничо не работает».
g
gildor
01/29/2019, 9:39 AMэто идеевские модули просто, их во время компиляции на самом деле нет, у Gradle там свои заморочки, модуль на тесты и код один, но test source set не становится доступен в зависимостях и компилируется он отдельно, поэтому считай свой модуль
2 Views