if @reservation.valid? вынести из транзакции и превратить в guard clause

добавить проверку на reservation_type

огромное спасибо

(i. e. когда он не now и не later)

@reservation.final_cost = @reservation.calculated_cost_of_ride ugh

это должно быть в модели, если ты дёргаешь метод модели и присваиваешь аттрибуту модели

вообще всё что связано с output можно в модель вынести

мне нравится пока что рефакторить код немного легче чем его переписывать, я думал надо будет почти все менять, а получается что более менее

reservation.try_drivers(@params[:drivers_ids]) как в params вообще :driver_ids появятся?

ReservationDetail.create bang it

в одном случае с клиента появляется, в другом получаю из метода

ReservationDetail.create bang it

не совсем понял

зачем от клиента получать айдишники водителей?

не совсем понял

create!

в одном случае он дергает апи и получает список тех кто ему подходит, потом он себе подбирает тех кому ему подходят

общие места из build_*_reservation тоже вынести в методы или ещё куда

я рад что я нашел этот паттерн

а то думал как все свои толстые контролеры переписать

в одном случае он дергает апи и получает список тех кто ему подходит, потом он себе подбирает тех кому ему подходят

тогда их надо валидировать, чтобы клиент не заказал водителя с другого конца света

to_formatted_s(:db) тебе вряд ли нужен

да, читал что надо везде использовать Time.current

огромное спасибо за ревью

нет, это другой вопрос. Time.now не нужно никак приводить, его AR сама приведёт к нужному для БД виду.

Всем привет. Еще такой вопрос по тестам. Есть класс А, класс Б и класс С, который юзает в себе А и Б. На класс С тесты нужно писать такие чтобы покрывали исключительно логику класса С (то есть проверять что вызывается то-то с такими-то параметрами) или и логику классов А,Б. С одной стороны хочется на 100 процентов проверить метод, с другой стороны дублирование тестов.

Надо покрывать только класс C

Классы A и B надо стабить

надо писать тесты, которые покрывают поведение, а не логику

логику вообще не надо тестировать

implementation details (i. e. логика) неважны совершенно

ну и тесты поведения обычно вполне дедуплицирются с помощью shared_examples и иже

Всем привет. Еще такой вопрос по тестам. Есть класс А, класс Б и класс С, который юзает в себе А и Б. На класс С тесты нужно писать такие чтобы покрывали исключительно логику класса С (то есть проверять что вызывается то-то с такими-то параметрами) или и логику классов А,Б. С одной стороны хочется на 100 процентов проверить метод, с другой стороны дублирование тестов.

Как мне показалось, суть в том, что речь идёт о юнит тестах и тестируя класс С начинает повторяться тестирование класса А или В? Тогда я не совсем понимаю как могут помочь shared_examples ? ojab поясни пожалуйста

будем стабить

Как мне показалось, суть в том, что речь идёт о юнит тестах и тестируя класс С начинает повторяться тестирование класса А или В? Тогда я не совсем понимаю как могут помочь shared_examples ? ojab поясни пожалуйста

если класс A что-то делает и класс C вызвает класс А, чтобы это сделать — очевидно что все тесты, покрывающие этот случай, можно вынести в shared_examples и использовать как в классе А, так и в классе С.

А зачем? Я всегда думал, что изолированные тесты это более выгодная практика для unit тестирования.

Вроде как нашёл подтверждение на http://www.betterspecs.org/#single

что такое "изолированные тесты"? Переписать логику, которая прописана в классе, в тестах?

А вообще важно, что класс С вызывапет класс А? Или важно что класс С создаёт какую-то условную запись в базе (которая создаётся классом А, при текущей реализации)?

что такое "изолированные тесты"? Переписать логику, которая прописана в классе, в тестах?

Нет. Иногда просто важно, чтобы произошёл вызов метода класса А, а иногда надо чтобы просто вернулось ожидаемое значение от вызова метода класса А.

тогда при изменении класса А меняется поведение класса C, так что прошлые вызовы класса С могут делать совсем не то, что планировалось при написании тестов.

это не очень ок

Т. е. если содержимое класса А вообще удалить, то тесты будут проходить успешно, что не очень хорошо.

Да но ведь, тестируя класс С, я хочу убедиться что работает именно он, а не класс А, потому что удаляя содержимое класса А, должен упасть юнит тест класса А. Разве не в этом смысл юнит теста?

PS: моя цель не поспорить, а узнать для себя сейчас что-то новое

Юнит тесты класса А тоже удалятся, да. Суть в том что сторонние изменения ломают _поведение_ класса С. Так что человек, которому надо что-то поменять в классе А, запросто сломает _поведение_ твоего класса С.

потому что класс А стал делать не то, что было при написании класса С

логика/какие методы вызываются в процессе — это деталь реализации, их вообще не нужно проверять

Если класс делает то, что от него хотят — неважно что там внутри написано

Я полностью согласен со всем, что ты говоришь, но мне кажется , что это больше к интеграционным тестам, когда тестируется именно то, как отдельные модули взаимодействуют между собой в группе http://softwaretestingfundamentals.com/integration-testing/

unit-тесты — это тестирование методов

тестируя поведение — ты тестируешь поведение метода

(если считать unit-тестированием тестирование логики метода — то unit-тесты не нужны, да)

https://github.com/trailblazer/trailblazer походу крутая штука

ибо тестирование поведения методов (пусть будет интеграционное тестирование) в любом случае нужно, но при наличии их — unit-тестирование нафиг не нужно, ибо ничего хорошего не привносит

https://github.com/trailblazer/trailblazer походу крутая штука

традиционно призывается @davydovanton

традиционно призывается @davydovanton

Да я уже устал

https://github.com/trailblazer/trailblazer походу крутая штука

короче это лютый комбайн, где всё слишком завязано друг на друга и dry.rb более лучше решает эти проблемы

ибо тестирование поведения методов (пусть будет интеграционное тестирование) в любом случае нужно, но при наличии их — unit-тестирование нафиг не нужно, ибо ничего хорошего не привносит

Я понял идею, аргументы кончились, спасибо)

(вкратце и неточно)

(вкратце и неточно)

Скорее это попытка сделать ддд Фреймворк для руби. Из-за того, что покрыть все и сразу не реально - получился комбайн, который плохо работает в специфичных случаях

Но идея и мотив - 10 из 10

короче это лютый комбайн, где всё слишком завязано друг на друга и dry.rb более лучше решает эти проблемы

то есть слишком enterprise?

то есть слишком enterprise?

Не думаю

я вот пытаюсь понять чего лучше придерживаться, опыта и знаний не хватает, ну и полной картины не вижу, вот сейчас изучаю всякие видео из конференций, здорово помогает. Там услышал об этом

trailblazer диктует единственно верную архитеутуру приложения, в то время как кустки dry.rb (i. e. dry-types/dry-transaction/etc) можно заюзать где удобно.

то есть более либерально, понял, спасибо

trailblazer диктует единственно верную архитеутуру приложения, в то время как кустки dry.rb (i. e. dry-types/dry-transaction/etc) можно заюзать где удобно.

Когда опыта мало, с драем отстрелишь ногу

я слишком тупой пока еще для dry-ruby

Когда опыта мало, с драем отстрелишь ногу

ну хз, скорее будешь применять там, где на самом деле не следует. Но ничето не мешает потом переписать какой-нибудь Dry::Container в голый класс.

мне кажется или много поляков в Rails комьюнити?

мне кажется или много поляков в Rails комьюнити?

Мне кажется, что как и везде

значит мне так везет, а то часто попадаю на статьи или видео с ними

вижу большой минус видео с конференций, они как рекламный ролик, ты повелся, начал заниматься, а на деле оказалось что то что рекламируют вовсе тебе не нужно или в что похуже, вовсе плохое

вижу большой минус видео с конференций, они как рекламный ролик, ты повелся, начал заниматься, а на деле оказалось что то что рекламируют вовсе тебе не нужно или в что похуже, вовсе плохое

https://t.me/pepegramming/24

спасибо большое за интересный канал

Последний пост в сентябре 17, чет перерыв затянулся)

Антоха загулял

мне кажется или много поляков в Rails комьюнити?

Ага? но пишут на инглише, как раз хотел перевести пару штук на русский) ну так, для комюнити)

Антоха загулял

Немного

Это же хорошо

Но я думаю скоро возобновлять

Да, было бы интересно

я не думал что во Львове есть комьюнити настолько серьезное, даже выхватили девелопера trailblazer

Зачем разделять по городам?

Всем привет. Кто-нибудь сталкивался с ошибкой Subscription class not found при настройке ActionCable? Я вроде все сделал по инструкции и 10 раз всё перепроверил, но рельсы как будто-бы не видят класс, описывающий вебсокет канал. Подробно написал тут: https://stackoverflow.com/questions/48965958/subscription-class-not-found-issue-with-actioncable

Комьюнити есть комьюнити

Всем привет. Кто-нибудь сталкивался с ошибкой Subscription class not found при настройке ActionCable? Я вроде все сделал по инструкции и 10 раз всё перепроверил, но рельсы как будто-бы не видят класс, описывающий вебсокет канал. Подробно написал тут: https://stackoverflow.com/questions/48965958/subscription-class-not-found-issue-with-actioncable

Дай полный трейс

spring перезапускал?

новые директории в /app подхватываются только после перезапуска, так что если после создания app/channels spring не перезапускался — рельсы не найдут NotificationsCountChannel

в гемах spring вообще нет

сразу скажу, что я фронт и я не так шарю во всей этой кухне

Зачем разделять по городам?

у меня нету возможности далеко ездить

у меня нету возможности далеко ездить

У тебя есть ютуб

да, пользуюсь во всю

я же правильно понимаю, что если в списке гемов нет spring, то он не установлен и не используется?

Посмотри по депенси лока

Ничего его не тянет?

Он у тебя в как зависимость идёт, ему не обязательно быть в гемфайле

в gemfile.lock поиск по spring ничего не дает