RSpec Rails — почему в features проверяются не все тесты?

Question

[Diesel-nick]

Пробую тестировать Rails приложение через RSpec с использованием синтаксиса, рекомендованного на этих ресурсах:
https://relishapp.com/rspec/rspec-rails/v/3-2/docs...
betterspecs.org/ru

Написал такой спек для тестирования main pages:
spec/features/main_pages_spec.rb

require 'rails_helper'

RSpec.describe "main pages", type: :features do

  scenario "root page" do
    visit root_path

    expect(page).to have_link("Login", href: login_path(locale: I18n.locale))
    expect(page).to have_link("Signup", href: signup_path(locale: I18n.locale))
  end
end

Но при запуске тестируется только первый expect

bundle exec rspec spec/features/main_pages_spec.rb
Run options: include {:focus=>true}

All examples were filtered out; ignoring {:focus=>true}

Randomized with seed 47521

main pages
  root page

Top 1 slowest examples (1.4 seconds, 99.9% of total time):
  main pages root page
    1.4 seconds ./spec/features/main_pages_spec.rb:8

Finished in 1.4 seconds (files took 4.58 seconds to load)
1 example, 0 failures

Это мои гемы приложения

gem 'rails', '~> 4.2', '>= 4.2.5'
gem 'rspec-rails', '~> 3.3'
gem 'capybara', '~> 2.5'
gem 'minitest'
gem 'selenium-webdriver', '~> 2.48'

Почему тестируется только первый expect?

Answer 1

[Андрей Сидоров]

Проходят оба expect. В feature тестах scenario является синонимом it из обычных тестов.
Если rspec выдал в логи, что тест прошёл, значит выполнилось всё, что в нём написано.
Чтобы убедиться в этом, попробуйте выполнить следующий код

scenario "root page" do
  visit root_path

  puts 'A'
  expect(page).to have_link("Login", href: login_path(locale: I18n.locale))
  puts 'B'
  expect(page).to have_link("Signup", href: signup_path(locale: I18n.locale))
  puts 'C'
end

Comments

[Diesel-nick]

Да, отображается ABC. Но в итоге это правильный синтаксис - писать 2-а expect внутри одного scenario? Или каждый expect нужно оборачивать в отдельный scenario?

[Андрей Сидоров]

Diesel-nick: один expect в одном it/scenario лично мне кажутся гораздо эстетичнее и читабельнее. стараюсь использовать такой синтаксис в случаях, когда в тестах не делается запросов к базе/к серверу(как в feature спеках).
в остальных случаях группирую expect'ы в одном it/scenario. причина - при большом числе спеков группировка в разы ускоряет скорость прохождения тестов.

[Diesel-nick]

morr: А как связянвключение нескольких expect it/scenario с обращением к БД? Там особый синтаксис?

[Андрей Сидоров]

Diesel-nick: не особый синтаксис. Как правило чтобы выполнить тест, вы сначала создаёте какие-то объекты через before блок или через let
let(:site) { create :site }
соответственно, когда создаваемые объекты пишутся в базу, то работает это гораздо медленнее.
представьте, у вас для создания нужный условий к тесту создаётся десяток объектов.
а потом вы пишите тесты
it { ... }
it { ... }
it { ... }
it { ... }
it { ... }

на каждый it блок будет создаваться по десятку объекту. 10 * 5 = 50 штук. соответственно, если пять it заменить на один, то объектов создатся в пять раз меньше. один it в такой ситуации отработает гораздо быстрее, чем пять.
когда у вас наберётся несколько тысяч подобных тестов, то разница между один "it - один expect" и "один it - много expect" станет очень заметной. сильно зависит от тестов, но скорее всего "один it - много expect" будут выполняться в 2-4 раза быстрее
одна-две минуту на все спеки, или 4-5 минут - разница очень существенна

Answer 2

[Nazar Jeveling]

Один scenario блок это и есть один тест