Yeah, I think that's better for sure.


On Thu, Oct 31, 2013 at 11:45 AM, Elliot Winkler
<notifications@github.com>wrote:

> In
> spec/shoulda/matchers/active_record/association_matchers/model_reflection_spec.rb:
>
> > +
> > +    it "returns false if not" do
> > +      define_model(:person, country_id: :integer)
> > +      country_model = define_model(:country) do
> > +        has_many :people
> > +      end
> > +      delegate_reflection = country_model.reflect_on_association(:people)
> > +      reflection = described_class.new(delegate_reflection)
> > +
> > +      expect(reflection).not_to be_through
> > +    end
> > +  end
> > +
> > +  describe '#join_table' do
> > +    context 'when the association was defined with a :join_table option' do
> > +      it "returns the value" do
>
> Yeah -- what I mean is "returns the value of the option", does that sound
> better?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/thoughtbot/shoulda-matchers/pull/373/files#r7345608>
> .
>