А, увы

Всем кофе в этом чате! Помогите разобраться плз. Есть 3 модели: search, item, price и сервис, который создает или обновляет items и создает prices. За пределами тестов все норм и работает как надо. Но простая спека не хочет работать, причем как-то странно. При запуске rspec ./mysterious_spec.rb тест проходит, а если все спеки запускаю, то не проходит Кто-то с подобным сталкивался? Куда копать? Текст спеки: it 'update old items and create prices' do expect(items_data).to be expect(old_item).to be_valid expect do described_class.call(search_id: search.id, data: items_data) end.to change { Item.first.updated_at } .and change { Price.count }.by(1) end первые два expect проходят, т.е. old_item созданный factory_bot виден, а когда я через дебаггер оказываюсь внутри described_class, то Item.count = 0 и сервис не увидев старый item, создает новый, а спека выдает expected `Item.first.updated_at` to have changed, but is still 2018-07-28 11:26:56.262143000 +0000 wtf?

Всем кофе в этом чате! Помогите разобраться плз. Есть 3 модели: search, item, price и сервис, который создает или обновляет items и создает prices. За пределами тестов все норм и работает как надо. Но простая спека не хочет работать, причем как-то странно. При запуске rspec ./mysterious_spec.rb тест проходит, а если все спеки запускаю, то не проходит Кто-то с подобным сталкивался? Куда копать? Текст спеки: it 'update old items and create prices' do expect(items_data).to be expect(old_item).to be_valid expect do described_class.call(search_id: search.id, data: items_data) end.to change { Item.first.updated_at } .and change { Price.count }.by(1) end первые два expect проходят, т.е. old_item созданный factory_bot виден, а когда я через дебаггер оказываюсь внутри described_class, то Item.count = 0 и сервис не увидев старый item, создает новый, а спека выдает expected `Item.first.updated_at` to have changed, but is still 2018-07-28 11:26:56.262143000 +0000 wtf?

этот old_item точно через FactoryBot.create создается? может он просто билдится и не записывается в бд?

let(:old_item) { create(:item, search: search) }

У тебя он лениво создаётся

то есть если binding.pry поставить между 2 и 3 expect то там будет Item.coun == 1 а внутритовего сервиса 0?

Я хз как у тебя там логика устроена, но попробуй let!

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

config.before(:suite) do DatabaseCleaner.clean_with(:truncation) end config.before(:each) do DatabaseCleaner.strategy = :transaction end config.before(:each, js: true) do DatabaseCleaner.strategy = :truncation end config.before(:each) do DatabaseCleaner.start end config.after(:each) do DatabaseCleaner.clean end

Или вызов сделай old_item в дебагере перед тем как юзать

при таких настройках перед каждым it база обнуляется?

Датабейс клинер тут не причем по мойму

Или вызов сделай old_item в дебагере перед тем как юзать

да 2 экспект и так его дергает, по идее должен создасться

Всем кофе в этом чате! Помогите разобраться плз. Есть 3 модели: search, item, price и сервис, который создает или обновляет items и создает prices. За пределами тестов все норм и работает как надо. Но простая спека не хочет работать, причем как-то странно. При запуске rspec ./mysterious_spec.rb тест проходит, а если все спеки запускаю, то не проходит Кто-то с подобным сталкивался? Куда копать? Текст спеки: it 'update old items and create prices' do expect(items_data).to be expect(old_item).to be_valid expect do described_class.call(search_id: search.id, data: items_data) end.to change { Item.first.updated_at } .and change { Price.count }.by(1) end первые два expect проходят, т.е. old_item созданный factory_bot виден, а когда я через дебаггер оказываюсь внутри described_class, то Item.count = 0 и сервис не увидев старый item, создает новый, а спека выдает expected `Item.first.updated_at` to have changed, but is still 2018-07-28 11:26:56.262143000 +0000 wtf?

если ты говоришь что отдельно этот спек проходит а вместе с остальными спеками падает то восползуйся bisect

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

Нашел где проблема, добавил к спеку expect(old_item.product_id).to eq(items_data.keys.first) Получил Failure/Error: expect(old_item.product_id).to eq(items_data.keys.first) expected: "product_1" got: "product_2" код фабрики: factory :item do sequence(:title) { |n| "title_#{n}" } sequence(:product_id) { |n| "product_#{n}" } url 'url' search end Что за магия? Почему factory_bot выдает product_2? И как после того, как был создан old_item, может быть такое, что оказываясь внутри сервиса, Item.count = 0

если ты говоришь что отдельно этот спек проходит а вместе с остальными спеками падает то восползуйся bisect

thx

Если тебе важно соответствие product_id то я бы не стал доверять это sequence

явсепонял, как часто и бывает, идиотская ошибка. фабрика создает объект с id=2, видимо надо получше разобраться с db cleaner А Item.count = 0 внутри сервиса получилось из-за того, что у меня в спеке 2 раза дергается сервис, и в первом случае действительно Item.count должно быть равно нулю. Там я потерянный old_item и искал Почувствуй себя дебилом, называется :D

Ловите балтийский берег

Народ

@gambala @nikmelnikov @viis_w @tralalatralala в сидрерии на моховой около 22?

Кто хочет ещё приходите тоже, обзнакомимся

Норм, я подтяну ногу

Я могу немного задержаться, дождитесь уж)

Разберёмся

Я может тоже задержусь

У меня так то связка порвана

Ок

@gambala @nikmelnikov @viis_w @tralalatralala в сидрерии на моховой около 22?

Постараюсь

А из Самары есть кто интересно?

А из Самары есть кто интересно?

Я мож в четверг буду

Может тоже тогда собраться всееми и на следующей неделе пойти в бар какой-нибудь?

@gambala @nikmelnikov @viis_w @tralalatralala в сидрерии на моховой около 22?

Буду

Раз Илона будет, точно приду ?

А есть hr блеклисты?

Иногда в компаниях бывают, но большого общего нет

Началось

Никит

Как перекат?

@gambala @nikmelnikov @viis_w @tralalatralala в сидрерии на моховой около 22?

Жень, мож пораньше?

а то я уже спать хочу)))

Как перекат?

Хату снял уже

Щас в ашан поеду всякую поеботу покупать

Завтра утром в офис почалю

Жень, мож пораньше?

Я не смогу раньше

Иди щас поспи)

В кино разве что

Подскажите пожалуйста, как сюда сортировку по price_date добавить? # Return all items for search with actual item.price scope :with_actual_price, ( lambda do |search_id| where(search_id: search_id) .select('DISTINCT ON (items.product_id) items.id, items.product_id, items.source, items.title, items.url, prices.value AS price, prices.created_at AS price_date') .joins(:prices) .having('prices.created_at = max(prices.created_at)') .group('items.id, items.product_id, items.source, items.title, items.url, price, price_date') end )

э а order(prices.created_at) не помогает?

кстати наличие distinct on и отсутвие order by намекает что он тебе не очень то и нужен

Мы в пути

Кто прибудет первым, пишите куда сели

Кто прибудет первым, пишите куда сели

Ок

скоре буду, паци

Опаздываю

Eta 22:40

Ок

Примерно так же

Сук я двух Никит сегодня спутал по телефону

Это провал

Сук я двух Никит сегодня спутал по телефону

Думал, что я с тобой разговаривал? ?

Da

Ну номер незнакомый

Никита

ERROR: S client not available

Я не заподозрил подвоха

А по голосу в диком шуме было анрил узнать

Я барную стойку уже подпираю

В 15 минутах

Я тут

Если вы стол не бронили, то в дальнем зале вот

Илону ждем

а че димасик бороду сбрил

И кепку отрастил

Весело у вас!

Эх душевно

а че димасик бороду сбрил

Жарко

э а order(prices.created_at) не помогает?

фак ркн, только сейчас заметил, что телега отвалилась( если добавить .order('prices.created_at'), то всплывает SELECT DISTINCT ON expressions must match initial ORDER BY expressions я решил пока через order_by(&:price_date), но понимаю, что можно красивее

кстати наличие distinct on и отсутвие order by намекает что он тебе не очень то и нужен

запрос с DISTINCT reload!;Item.with_actual_price(1).to_a.size => 29 без reload!;Item.with_actual_price(1).to_a.size => 31 это запрос для получения списка items с актуальными ценами(последнаяя созданная цена для item) схема простая совсем, если что create_table "items", force: :cascade do |t| t.integer "search_id", null: false t.string "title" t.string "url" t.string "product_id" t.datetime "created_at", null: false t.datetime "updated_at", null: false t.string "source" t.index ["product_id"], name: "index_items_on_product_id", unique: true end create_table "prices", force: :cascade do |t| t.integer "item_id", null: false t.integer "value" t.datetime "created_at", null: false t.datetime "updated_at", null: false t.index ["item_id"], name: "index_prices_on_item_id" end

запрос с DISTINCT reload!;Item.with_actual_price(1).to_a.size => 29 без reload!;Item.with_actual_price(1).to_a.size => 31 это запрос для получения списка items с актуальными ценами(последнаяя созданная цена для item) схема простая совсем, если что create_table "items", force: :cascade do |t| t.integer "search_id", null: false t.string "title" t.string "url" t.string "product_id" t.datetime "created_at", null: false t.datetime "updated_at", null: false t.string "source" t.index ["product_id"], name: "index_items_on_product_id", unique: true end create_table "prices", force: :cascade do |t| t.integer "item_id", null: false t.integer "value" t.datetime "created_at", null: false t.datetime "updated_at", null: false t.index ["item_id"], name: "index_prices_on_item_id" end

я к тому что там скорее всего обычный distinct можно заюзать, а не distinct on

я к тому что там скорее всего обычный distinct можно заюзать, а не distinct on

спасибо, разберусь с ним получше

спасибо, разберусь с ним получше

попробуй убрать ON и добавть ордер

я к тому что там скорее всего обычный distinct можно заюзать, а не distinct on

а разве в случае использования простого distinct, он не работает по итоговой таблице и удаляет полностью идентичные строки? у меня строки с одним item, но разными price, будут различаться и он не должен сработать, верно?

попробуй убрать ON и добавть ордер

не сработало, distinct из-за различия в ценах позиций не подходит(

не сработало, distinct из-за различия в ценах позиций не подходит(

а ты просто destinct сделал или destinct(items.product_id) ?

а ты просто destinct сделал или destinct(items.product_id) ?

первый вариант, секунду

кажется в синтаксисе ошибка (PG::SyntaxError: ERROR: syntax error at or near "." LINE 2: items.id, что-то не так? scope :with_actual_price, ( lambda do |search_id| where(search_id: search_id) .select('DISTINCT (items.product_id) items.id, items.product_id, items.source, items.title, items.url, prices.value AS price, prices.created_at AS price_date') .joins(:prices) .having('prices.created_at = max(prices.created_at)') .group('items.id, items.product_id, items.source, items.title, items.url, price, price_date') .order('prices.created_at') end )

запятая после distinct

в итоге неверно работает, не фильтрует дублирующиеся items.product_id

попробуй убрать ON и добавть ордер

по идее должно работать, если .having('prices.created_at = max(prices.created_at)') срабатывал бы

хм а если сделать distinct on (items.product_id) и добавть order("items.product_id, prices.created_at desc")

и убрать having с group

вообще

как то так - scope :with_actual_price, ( lambda do |search_id| where(search_id: search_id) .select('DISTINCT ON (items.product_id) items.id, items.product_id, items.source, items.title, items.url, prices.value AS price, prices.created_at AS price_date') .joins(:prices) .order('items.product_id, prices.created_at DESC') end )

так по идее у тебя к каждому product_id должна применится последняя цена