Skip to content
This repository

I could have fixed the bug on rspec I reported. #13

wants to merge 1 commit into from

3 participants

Noriaki Hamamoto David Chelimsky Julio Santos
Noriaki Hamamoto

Hi David,

I could have fixed the bug on rspec I reported here:
I'm not sure if it's proper way to fix it or not, but it works for those explained in my report except the ControllerSpec one.
The ControllerSpec one might have a different reason because it's still indicating the same error on the test.

Anyway, please take a look at this patch, and if you agree with my update, please pull it.

Thank you,

David Chelimsky

Clicking on the link to the report gets me a 404. What was the issue?

Julio Santos
julio commented

The link Nori posted has some extra characters. This is the correct one:

Noriaki Hamamoto
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Showing 1 unique commit by 1 author.

Jan 14, 2011
Noriaki Hamamoto norisuke3 fixed a bug reported at 1a5714c
This page is out of date. Refresh to see the latest.

Showing 1 changed file with 1 addition and 1 deletion. Show diff stats Hide diff stats

  1. +1 1  lib/spec/example/subject.rb
2  lib/spec/example/subject.rb
@@ -21,7 +21,7 @@ def subject(&block)
21 21 end
22 22
23 23 def its(attribute, &block)
24   - describe(attribute) do
  24 + describe(instance_eval(&subject)) do
25 25 example do
26 26 self.class.class_eval do
27 27 define_method(:subject) do

Tip: You can add notes to lines in a file. Hover to the left of a line to make a note

Something went wrong with that request. Please try again.