135

Given that I have a Personable concern in my Rails 4 application which has a full_name method, how would I go about testing this using RSpec?

concerns/personable.rb

module Personable
  extend ActiveSupport::Concern

  def full_name
    "#{first_name} #{last_name}"
  end
end
10

5 Answers 5

220

The method you found will certainly work to test a little bit of functionality but seems pretty fragile—your dummy class (actually just a Struct in your solution) may or may not behave like a real class that includes your concern. Additionally if you're trying to test model concerns, you won't be able to do things like test the validity of objects or invoke ActiveRecord callbacks unless you set up the database accordingly (because your dummy class won't have a database table backing it). Moreover, you'll want to not only test the concern but also test the concern's behavior inside your model specs.

So why not kill two birds with one stone? By using RSpec's shared example groups, you can test your concerns against the actual classes that use them (e.g., models) and you'll be able to test them everywhere they're used. And you only have to write the tests once and then just include them in any model spec that uses your concern. In your case, this might look something like this:

# app/models/concerns/personable.rb
module Personable
  extend ActiveSupport::Concern

  def full_name
    "#{first_name} #{last_name}"
  end
end

# spec/concerns/personable_spec.rb
require 'spec_helper'

RSpec.shared_examples_for "personable" do
  let(:model) { described_class } # the class that includes the concern

  it "has a full name" do
    person = FactoryBot.build(model.to_s.underscore.to_sym, first_name: "Stewart", last_name: "Home")
    expect(person.full_name).to eq("Stewart Home")
  end
end

# spec/models/master_spec.rb
require 'spec_helper'
require Rails.root.join "spec/concerns/personable_spec.rb"

describe Master do
  it_behaves_like "personable"
end

# spec/models/apprentice_spec.rb
require 'spec_helper'

describe Apprentice do
  it_behaves_like "personable"
end

The advantages of this approach become even more obvious when you start doing things in your concern like invoking AR callbacks, where anything less than an AR object just won't do.

10
  • 2
    One disadvantage of this is that it will slow down parallel_tests. I think it will be better to have separate tests instead of using shared_examples_for and it_behaves_like. Commented Jan 21, 2014 at 12:23
  • 9
    @ArtemKalinchuk I'm not sure that's true, per github.com/grosser/parallel_tests/issues/168 parallel_tests are based per file, so shared examples should not slow it down. I would also argue that properly grouped shared behaviors, trumps testing speed.
    – Aaron K
    Commented Apr 9, 2014 at 22:16
  • 8
    Make sure to include the concerns directory in your spec_helper.rb github.com/rspec/rspec-core/issues/407#issuecomment-1409871
    – Ziggy
    Commented Jun 6, 2014 at 0:20
  • 2
    I couldn't find anything about including the concerns directory in that link. Could you please clarify how this is done? I can't get my RSpec test to recognize the module in one of my concerns.
    – Jake Smith
    Commented Dec 28, 2014 at 2:08
  • 5
    Do not add _spec to the filename which contains shared_examples_for (personable_spec.rb in this case), otherwise you will get a misleading warning message - github.com/rspec/rspec-core/issues/828.
    – Lalu
    Commented Mar 11, 2016 at 16:18
77

In response to the comments I've received, here's what I've ended up doing (if anyone has improvements please feel free to post them):

spec/concerns/personable_spec.rb

require 'spec_helper'

describe Personable do
  let(:test_class) { Struct.new(:first_name, :last_name) { include Personable } }
  let(:personable) { test_class.new("Stewart", "Home") }

  it "has a full_name" do
    expect(personable.full_name).to eq("#{personable.first_name} #{personable.last_name}")
  end
end
5
  • 2
    Yes, this will break other tests if they happen to test a real class called Person. I'll edit to fix.
    – Russell
    Commented May 13, 2013 at 15:25
  • This doesn't work. It gives me the error: undefined method 'full_name' for #<struct first_name="Stewart", last_name="Home">
    – Kyle Decot
    Commented May 13, 2013 at 18:29
  • 1
    Try including Personable rather than extending it. I'll update the answer.
    – Russell
    Commented May 13, 2013 at 21:45
  • 1
    Works great now. Thanks for pointing me in the right direction and helping me refactor @Russell
    – Kyle Decot
    Commented May 14, 2013 at 13:14
  • Works great and looks nice
    – Edward
    Commented Aug 4, 2016 at 17:36
8

Another thought is to use the with_model gem to test things like this. I was looking to test a concern myself and had seen the pg_search gem doing this. It seems a lot better than testing on individual models, since those might change, and it's nice to define the things you're going to need in your spec.

0
2

The following worked for me. In my case my concern was calling generated *_path methods and the others approaches didn't seem to work. This approach will give you access to some of the methods only available in the context of a controller.

Concern:

module MyConcern
  extend ActiveSupport::Concern

  def foo
    ...
  end
end

Spec:

require 'rails_helper'

class MyConcernFakeController < ApplicationController
  include MyConcernFakeController
end

RSpec.describe MyConcernFakeController, type: :controller do    
  context 'foo' do
    it '' do
      expect(subject.foo).to eq(...)
    end
  end
end
2
  • if the concern is defined in controllers/concerns/my_concern.rb, where is your concern spec? spec/controllers/concerns/my_concern_spec.rb, or spec/controllers/my_concern_fake_controller_spec.rb? Commented Feb 24, 2022 at 22:37
  • 1
    I think that's a matter of preference, but in my case I wanted to test the concern, not the controller (but yes, they're connected) so your first suggestion: spec/controllers/concerns/my_concern_spec.rb
    – Nathan
    Commented Mar 1, 2022 at 7:33
-3

just include your concern in spec and test it if it returns the right value.

RSpec.describe Personable do
  include Personable

  context 'test' do
    let!(:person) { create(:person) }

    it 'should match' do
       expect(person.full_name).to eql 'David King'
    end
  end
end

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service and acknowledge you have read our privacy policy.

Not the answer you're looking for? Browse other questions tagged or ask your own question.