
Alex
19.09.2017
13:52:46
ну я пытаюсь понять как вы без тестов получаете _прирост_ скорости.

v
19.09.2017
13:52:57

Alex
19.09.2017
13:53:35
померяй
ты буквально перед этим процитировал сообщение где я как раз это делал.

Google

v
19.09.2017
13:53:56

Nikolay
19.09.2017
13:55:18

Vasiliy
19.09.2017
13:57:37
ну как
ты такой пишешь фичу
тестишь там все 100500 вариантов, юнит тест значит, интеграционный тест, руками там туда сюда прогнал
замокал сторонние объекты такой, сайдэффекты все проверил
неделю например писал всё это гавно
зашёл в админку а твоей фичей за год два человека воспользовались и вот резонно было столько времени тратить на это?

Gleb
19.09.2017
14:00:06
тесты на импорт это прям даже смешнее, чем без тестов ) не в обиду конечно

Vasiliy
19.09.2017
14:00:22
ну или там стата для админов

Alex
19.09.2017
14:01:37
кроме того знать какая фича будет успешна а какая нет - не всегда возможно

Vasiliy
19.09.2017
14:01:51
а если всю семью?

Google

Gleb
19.09.2017
14:02:08
сжигая свое время ты типа деньги не тратишь )

Vasiliy
19.09.2017
14:02:51
лан хорошо что фича в прод уйдёт

Alex
19.09.2017
14:02:54
а так приходилось прям в коде костыли ставить чтобы врубить нужный файл и т.д.

Vasiliy
19.09.2017
14:03:07
а по agile может оказаться что это не нужно уже

Alex
19.09.2017
14:03:14
и базу резетить и все эти дела.
а там еще долгий сервис запускался который можно было бы замокать.

Vasiliy
19.09.2017
14:04:13
так я же не говорю всё не тестить

Gleb
19.09.2017
14:04:27

Nikolay
19.09.2017
14:04:35
а как отличить что тестить что нет?

Alex
19.09.2017
14:04:49

Oleg
19.09.2017
14:04:55

Alex
19.09.2017
14:05:00
импорты как на подбор все ненормализированные.

Vasiliy
19.09.2017
14:05:02
мой посыл что с умом надо подходить к этому

Alex
19.09.2017
14:05:19
Было так что в одном месте поправил а другое место сломалось, потому что не ожидало этого нового значения и т.д.
А тесты могут ограниченные кейсы быстро пронать к тому же замокав все лишнее.

Gleb
19.09.2017
14:05:41

Alex
19.09.2017
14:06:03

Vlad
19.09.2017
14:06:44
Рефакторинг зло, тесты ещё большее
Всем спасибо

Google

Gleb
19.09.2017
14:06:58
хуяк хуяк

Dm
19.09.2017
14:07:04
тесты для тех, кто пишет баги

Alex
19.09.2017
14:07:05
драйвен

Nikolay
19.09.2017
14:07:20

Dm
19.09.2017
14:07:22
настоящие пацаны не пишут тесты

Gleb
19.09.2017
14:07:25
и правда. просто сразу без багов пишешь и делов то

Vasiliy
19.09.2017
14:07:31
наоборот баги пишут для тестов

Vlad
19.09.2017
14:07:44

Alex
19.09.2017
14:07:46

No
19.09.2017
14:08:07
Руби для хипстеров

Alex
19.09.2017
14:08:18

Nikolay
19.09.2017
14:08:19
еще из любимых - "нехер писать тесты, ручное тестирование всё равно больше багов отлавливает"

Vasiliy
19.09.2017
14:08:20
Роба для зеков

Alex
19.09.2017
14:08:21
руби для ретроградов
нормальные посоны пишут на js/go

No
19.09.2017
14:08:37
нормальные пацаны лузгают семки

Alex
19.09.2017
14:09:03

Gleb
19.09.2017
14:09:23
нормальные пацаны руководят блокчейн стартапами и ничего не пишут

No
19.09.2017
14:09:41
Я помазал и увеличил свой чл... скилл. Это реально работает, без СМС

Vlad
19.09.2017
14:09:47
КВН разминка)

Google

No
19.09.2017
14:10:05
Скорее рэп баттл
Раунд

Dm
19.09.2017
14:10:08
tdd удобно использовать, когда фича хорошо описана. а когда mvp собираешь, то всегда удобнее писать тесты после.

Alex
19.09.2017
14:10:26
у тебя есть перезапускаемое точное окружение
грохнул не то? плевать, перезапустил

Vlad
19.09.2017
14:10:53
Тесты как религия

Dm
19.09.2017
14:10:59
tdd и просто тесты разные вещи

Alex
19.09.2017
14:11:20
tdd это написание тестов перед написанием кода.

Dm
19.09.2017
14:11:29
угу

Admin
ERROR: S client not available

Alex
19.09.2017
14:11:39
это неудобно
и после тоже неудобно

Dm
19.09.2017
14:11:58
вот если фича не описана супер точно, то tdd неудобно юзать

Gleb
19.09.2017
14:12:00
да ебать. тесты это тоже инструмент, нифига это не религия. инструмент закрывает мои (компании) надобности или не используется

Alex
19.09.2017
14:12:34
не понимаю почему нет термина для кейса где разрабатываешь код и тест рука об руку.
Это не TDD но выглядит даже эффективнее.

Nikolay
19.09.2017
14:13:21
TLD

Vasiliy
19.09.2017
14:13:22

Dm
19.09.2017
14:14:21
на мой взгляд, не писать тесты в устоявшемся ruby проекте это нехорошо. тк это усложняет жизнь других членам команды втч

Google

Dm
19.09.2017
14:15:16
ну и на сложные фичи не писать тесты тоже странно. руками чтоли каждый раз проверять?

Lupsick
19.09.2017
15:04:47
посоны

Oleg
19.09.2017
15:05:06
м?

Lupsick
19.09.2017
15:05:06
как можно упростить?
сhart_config[:options][:axis] ||= {}
chart_config[:options][:axis][:y] ||= {}
chart_config[:options][:axis][:y][:tick] ||= {}
chart_config[:options][:axis][:y][:tick][:format] = value_format_alias

Nikita
19.09.2017
15:06:06
https://github.com/solnic/transproc/blob/master/lib/transproc/hash.rb#L397

Oleg
19.09.2017
15:09:22
Что-то где-то было про дефолтные значения
И там не просто дефолтные, а на лету под несуществующий ключ создавался объект

Lupsick
19.09.2017
15:09:46
надо нативно

Oleg
19.09.2017
15:09:55
И можно было вот такое написать и оно бы не упало

Lupsick
19.09.2017
15:10:01
ну короче вот я тоже не знаю нативного способа

Oleg
19.09.2017
15:10:18
Было что-то, может быть даже по поиску в чате найдется

Nikita
19.09.2017
15:11:12
надо нативно
ну там же код написан, скопируй его себе, любой, кто увидит функцию deep_merge, поймет что она делает

Lupsick
19.09.2017
15:11:19
да ну фу
то что выше норм

Oleg
19.09.2017
15:11:30
hash default_proc
Мб

Nikita
19.09.2017
15:12:02

Oleg
19.09.2017
15:14:04
def super_hash
Hash.new {|h,k| h[k] = super_hash}
end
Ноэтонеточно
Да, падать оно не будет, но записывать значения тоже не будет ?
В этом примере
Но, возможно, в этом будет другое решение