surveyor 0.6.1

data/spec/models/dependency_spec.rb

@@ -0,0 +1,82 @@
+require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
+
+describe Dependency do
+  before(:each) do
+    @dependency = Dependency.new(:rule => "1 and 2 or 3", :question_id => 23)
+  end
+
+  it "should be valid" do
+    @dependency.should be_valid
+  end
+
+  it "should be invalid without a rule" do
+    @dependency.rule = nil
+    @dependency.should have(2).errors_on(:rule)
+    @dependency.rule = " "
+    @dependency.should have(1).errors_on(:rule)
+  end
+
+  it "should be invalid without a question_id" do
+    @dependency.question_id = nil
+    @dependency.should have(1).error_on(:question_id)
+  end
+
+  it "should alias question_id as dependent_question_id" do
+    @dependency.question_id = 19
+    @dependency.dependent_question_id.should == 19
+    @dependency.dependent_question_id = 14
+    @dependency.question_id.should == 14
+  end
+
+  it "should be invalid unless rule composed of only references and operators" do
+    @dependency.rule = "foo"
+    @dependency.should have(1).error_on(:rule)
+    @dependency.rule = "1 to 2"
+    @dependency.should have(1).error_on(:rule)
+    @dependency.rule = "a and b"
+    @dependency.should have(1).error_on(:rule)
+  end
+  
+end
+
+describe Dependency, "when evaluating dependency conditions of a question in a response set" do
+
+  before(:each) do
+    @dep = Dependency.new(:rule => "A", :question_id => 1)
+    @dep2 = Dependency.new(:rule => "A and B", :question_id => 1)
+    @dep3 = Dependency.new(:rule => "A or B", :question_id => 1)
+    @dep4 = Dependency.new(:rule => "!(A and B) and C", :question_id => 1)
+    
+    @dep_c = mock_model(DependencyCondition, :id => 1, :rule_key => "A", :to_evaluation_hash => {:A => true})
+    @dep_c2 = mock_model(DependencyCondition, :id => 2, :rule_key => "B", :to_evaluation_hash => {:B => false})
+    @dep_c3 = mock_model(DependencyCondition, :id => 3, :rule_key => "C", :to_evaluation_hash => {:C => true})
+
+    @dep.stub!(:dependency_conditions).and_return([@dep_c])
+    @dep2.stub!(:dependency_conditions).and_return([@dep_c, @dep_c2])
+    @dep3.stub!(:dependency_conditions).and_return([@dep_c, @dep_c2])
+    @dep4.stub!(:dependency_conditions).and_return([@dep_c, @dep_c2, @dep_c3])
+  end
+
+  it "knows if the dependencies are met" do
+    @dep.met?(@response_set).should be_true
+    @dep2.met?(@response_set).should be_false
+    @dep3.met?(@response_set).should be_true
+    @dep4.met?(@response_set).should be_true
+  end
+
+  it "returns the proper keyed pairs from the dependency conditions" do
+    @dep.keyed_conditions(@response_set).should == {:A => true}
+    @dep2.keyed_conditions(@response_set).should == {:A => true, :B => false}
+    @dep3.keyed_conditions(@response_set).should == {:A => true, :B => false}
+    @dep4.keyed_conditions(@response_set).should == {:A => true, :B => false, :C => true}
+  end
+  
+  it "evaluates the rule from the keyed pairs and return a boolean value" do
+    @dep.rule_evaluation(@dep.keyed_conditions(@response_set)).should be_true
+    @dep2.rule_evaluation(@dep2.keyed_conditions(@response_set)).should be_false
+    @dep3.rule_evaluation(@dep3.keyed_conditions(@response_set)).should be_true
+    @dep4.rule_evaluation(@dep4.keyed_conditions(@response_set)).should be_true
+  end
+  
+  
+end

data/spec/models/question_group_spec.rb

@@ -0,0 +1,11 @@
+require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
+
+describe QuestionGroup do
+  before(:each) do
+    @question_group = QuestionGroup.new
+  end
+
+  it "should be valid" do
+    @question_group.should be_valid
+  end
+end

data/spec/models/question_spec.rb

@@ -0,0 +1,75 @@
+require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
+
+describe Question, "when creating a new question" do
+  before(:each) do
+    @ss = mock_model(SurveySection)
+    @question = Question.new(:text => "What is your favorite color?", :survey_section => @ss, :is_mandatory => true, :display_order => 1)
+  end
+
+  it "should be valid" do
+    @question.should be_valid
+  end
+
+  it "should be invalid without text" do
+    @question.text = nil
+    @question.should have(1).error_on(:text)
+  end
+    
+  it "should have a parent survey section" do
+    @question.survey_section = nil
+    @question.should have(1).error_on(:survey_section_id)
+  end
+    
+  it "should be mandatory by default" do
+    @question.mandatory?.should be_true
+  end
+  
+end
+
+describe Question, "that has answers" do
+  before(:each) do
+    @question = Factory(:question, :text => "What is your favorite color?")
+    Factory(:answer, :question => @question, :display_order => 3, :text => "blue")
+    Factory(:answer, :question => @question, :display_order => 1, :text => "red")
+    Factory(:answer, :question => @question, :display_order => 2, :text => "green")
+  end
+  
+  it "should have answers" do
+    @question.answers.should have(3).answers
+  end
+  
+  it "should retrieve those answers in display_order" do
+    @question.answers.map(&:display_order).should == [1,2,3]
+  end
+  
+end
+
+describe Question, "when interacting with an instance" do
+  
+  before(:each) do
+    @ss = mock_model(SurveySection)
+    @question = Question.new(:text => "What is your favorite color?", :survey_section => @ss)
+  end
+
+  it "should return 'default' for nil display type" do
+    @question.display_type = nil
+    @question.display_type.should == "default"
+  end
+  
+end
+
+describe Question, "with dependencies" do
+  before(:each) do
+    @ss = mock_model(SurveySection)
+    @rs = mock_model(ResponseSet)
+    @question = Question.new(:text => "Which island?", :survey_section => @ss)
+  end
+
+  it "should check its dependency" do
+    @dependency = mock_model(Dependency)
+    @dependency.stub!(:met?).with(@rs).and_return(true)
+    @question.stub!(:dependency).and_return(@dependency)
+    @question.dependency_is_satisfied?(@rs).should == true
+  end
+  
+end

data/spec/models/response_set_spec.rb

@@ -0,0 +1,153 @@
+require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
+
+describe ResponseSet, "class methods" do
+  before(:each) do
+    @response_set = Factory(:response_set, :access_code => "PDQ")
+  end
+  it "should find a response set by response_set.access_code or return false" do
+    ResponseSet.find_by_access_code("PDQ").should == @response_set
+    ResponseSet.find_by_access_code("Different").should be_nil
+  end
+end
+describe ResponseSet, "with responses" do
+  before(:each) do
+    @response_set = Factory(:response_set)
+  end
+
+  it "can tell you its responses" do
+    3.times{ |x| @response_set.responses.build }  
+    @response_set.responses.should have(3).items
+  end
+
+  it "is completable" do
+    @response_set.completed_at.should be_nil
+    @response_set.complete!
+    @response_set.completed_at.should_not be_nil
+    @response_set.completed_at.is_a?(Time).should be_true
+  end
+
+  it "does not allow completion through mass assignment" do
+    @response_set.completed_at.should be_nil
+    @response_set.update_attributes(:completed_at => Time.now)
+    @response_set.completed_at.should be_nil
+  end
+
+end
+describe ResponseSet, "Creating new set for user" do
+  before(:each) do
+    @response_set = Factory(:response_set)
+  end
+  it "should have a unique code with length 10 that identifies the survey" do
+    @response_set.access_code.should_not be_nil
+    @response_set.access_code.length.should == 10
+  end
+end
+describe ResponseSet, "Updating the response set" do
+  before(:each) do
+    @response_set = Factory(:response_set)
+    # {"responses"=>{
+    #   "6"=>{"question_id"=>"6", "20"=>{"string_value"=>""}}, 
+    #   "7"=>{"question_id"=>"7", "21"=>{"text_value"=>"Brian is tired"}}, 
+    #   "1"=>{"question_id"=>"1", "answer_id"=>"1", "4"=>{"string_value"=>"XXL"}}, 
+    #   "2"=>{"question_id"=>"2", "answer_id"=>"6"}, 
+    #   "3"=>{"question_id"=>"3"}, 
+    #   "4"=>{"question_id"=>"4"}, 
+    #   "5"=>{"question_id"=>"5", "19"=>{"string_value"=>""}}}, 
+    # "survey_code"=>"test_survey", 
+    # "commit"=>"Next Section (Utensiles and you!) >>", 
+    # "authenticity_token"=>"d03bc1b52fa9669e1ed87c313b939836e7b93e34", 
+    # "_method"=>"put", 
+    # "action"=>"update", 
+    # "controller"=>"app", 
+    # "response_set_code"=>"cIFn0DnxlU", 
+    # "section"=>"2"}
+      
+    #TODO test views to produce these params. e.g., blank responses should still have a hash with question_id
+    @radio_response_attributes = HashWithIndifferentAccess.new({
+      "1"=>{"question_id"=>"1", "answer_id"=>"1", "4"=>{"string_value"=>"XXL"}}, 
+      "2"=>{"question_id"=>"2", "answer_id"=>"6"}, 
+      "3"=>{"question_id"=>"3"}
+    })
+    @other_response_attributes = HashWithIndifferentAccess.new({
+      "6"=>{"question_id"=>"6", "20"=>{"string_value"=>""}}, 
+      "7"=>{"question_id"=>"7", "21"=>{"text_value"=>"Brian is tired"}}, 
+      "5"=>{"question_id"=>"5", "19"=>{"string_value"=>""}}   
+    })
+  end
+
+  it "should delete existing responses corresponding to every question key that comes in" do
+    Response.should_receive(:delete_all).exactly(3).times
+    @response_set.update_attributes(:response_attributes => @radio_response_attributes)
+  end
+  it "should save new responses from radio buttons, ignoring blanks" do
+    @response_set.update_attributes(:response_attributes => @radio_response_attributes)
+    @response_set.responses.should have(2).items
+    @response_set.responses.detect{|r| r.question_id == 2}.answer_id.should == 6
+  end
+  it "should save new responses from other types, ignoring blanks" do
+    @response_set.update_attributes(:response_attributes => @other_response_attributes)
+    @response_set.responses.should have(1).items
+    @response_set.responses.detect{|r| r.question_id == 7}.text_value.should == "Brian is tired"
+  end
+  it "should ignore data if corresponding radio button is not selected" do
+    @response_set.update_attributes(:response_attributes => @radio_response_attributes)
+    @response_set.responses.select{|r| r.question_id == 2}.should have(1).item
+    @response_set.responses.detect{|r| r.question_id == 2}.string_value.should == nil
+  end
+  it "should preserve data in checkboxes regardless of selection" do
+    pending
+  end
+
+  it "should give convenient access to responses through response_for" do
+    @response_set.save #need to save for the associated models to build/save
+    @response_set.attributes = {:response_attributes => @radio_response_attributes}
+    @response_set.save.should be_true
+    @response_set.responses.should have(2).items
+    
+    pending
+  end
+end
+
+describe ResponseSet, "knowing internal status" do
+  before(:each) do
+    @response_set = Factory(:response_set)
+  end
+  it "knows when it is empty"  do
+    @response_set.empty?.should be_true
+  end
+
+  it "knows when it is not empty" do
+    @response_set.responses.build({:question_id => 1, :answer_id => 8})
+    @response_set.empty?.should be_false
+  end
+end
+describe ResponseSet, "with dependencies" do
+  before(:each) do
+    # Questions
+    @do_you_like_pie = Factory(:question, :text => "Do you like pie?")
+    @what_flavor = Factory(:question, :text => "What flavor?")
+    @what_bakery = Factory(:question, :text => "What bakery?")
+    # Answers
+    @do_you_like_pie.answers << Factory(:answer, :text => "yes", :question_id => @do_you_like_pie.id)
+    @do_you_like_pie.answers << Factory(:answer, :text => "no", :question_id => @do_you_like_pie.id)
+    @what_flavor.answers << Factory(:answer, :response_class => :string, :question_id => @what_flavor.id)
+    @what_bakery.answers << Factory(:answer, :response_class => :string, :question_id => @what_bakery.id)
+    # Dependency
+    @what_flavor_dep = Factory(:dependency, :rule => "1", :question_id => @what_flavor.id)
+    @what_flavor_dep.dependency_conditions << Factory(:dependency_condition, :rule_key => "1", :question_id => @do_you_like_pie.id, :operator => "==", :answer_id => @do_you_like_pie.answers.first.id, :dependency_id => @what_flavor_dep.id)
+    @what_bakery_dep = Factory(:dependency, :rule => "2", :question_id => @what_bakery.id)
+    @what_bakery_dep.dependency_conditions << Factory(:dependency_condition, :rule_key => "2", :question_id => @do_you_like_pie.id, :operator => "==", :answer_id => @do_you_like_pie.answers.first.id, :dependency_id => @what_bakery_dep.id)
+    # Responses
+    @response_set = Factory(:response_set)
+    @response_set.responses << Factory(:response, :question_id => @do_you_like_pie.id, :answer_id => @do_you_like_pie.answers.first.id, :response_set_id => @response_set.id)
+    @response_set.responses << Factory(:response, :string_value => "pecan pie", :question_id => @what_flavor.id, :answer_id => @what_flavor.answers.first.id, :response_set_id => @response_set.id)
+  end
+  
+  it "should list unanswered dependencies to show at the top of the next page (javascript turned off)" do
+    @response_set.unanswered_dependencies.should == [@what_bakery]
+  end
+  it "should list answered and unanswered dependencies to show inline (javascript turned on)" do
+    @response_set.all_dependencies[:show].should == [@what_flavor, @what_bakery]
+  end
+  
+end

data/spec/models/response_spec.rb

@@ -0,0 +1,94 @@
+require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
+
+describe Response, "when saving a response" do
+  before(:each) do
+    @response = Response.new(:question_id => 314, :response_set_id => 159, :answer_id => 1)
+  end
+  
+  it "should be valid" do
+    @response.should be_valid
+  end
+  
+  it "should be invalid without a parent response set and question" do
+    @response.response_set_id = nil
+    @response.should have(1).error_on(:response_set_id)
+    
+    @response.question_id = nil
+    @response.should have(1).error_on(:question_id)
+  end
+
+  describe "returns the response as the type requested" do
+    
+    it "returns 'string'" do
+      @response.string_value = "blah"
+      @response.as("string").should == "blah"
+      @response.as(:string).should == "blah"
+    end
+    
+    it "returns 'integer'" do
+      @response.integer_value = 1001
+      @response.as(:integer).should == 1001
+    end
+    
+    it "returns 'float'"do
+      @response.float_value = 3.14
+      @response.as(:float).should == 3.14
+    end
+    
+    it "returns 'answer'" do
+      @response.answer_id = 14
+      @response.as(:answer).should == 14
+    end
+    
+    it "default returns answer type if not specified" do
+      @response.answer_id =18
+      @response.as(:stuff).should == 18
+    end
+    
+    it "returns empty elements if the response is cast as a type that is not present" do
+      resp = Response.new(:question_id => 314, :response_set_id => 156)
+      resp.as(:string).should == nil
+      resp.as(:integer).should == nil
+      resp.as(:float).should == nil
+      resp.as(:answer).should == nil
+      resp.as(:stuff).should == nil
+    end
+        
+  end
+  
+  describe "model validation defined in the answer class attributes" do
+    
+    before(:each) do
+      # faking out all the basic validations on the model
+      @response = Response.new(:question_id => 123, :response_set_id => 421, :answer_id => 213)
+    end
+    
+    describe "Integer response_type validations" do
+      
+      before(:each) do
+        @response.stub!(:answer).and_return(mock(Answer, {:response_class => :integer}))
+      end
+      
+      it "validates an upper limit on responses" do
+        @response.answer.stub!(:max_value).and_return(10)
+        @response.integer_value = 100
+        @response.save.should be_false
+        @response.should have(1).error_on(:integer_value)
+      end
+    
+    end
+    
+    describe "Float response_type validations" do
+
+        before(:each) do
+          @response.stub!(:answer).and_return(mock(Answer, {:response_class => :float}))
+        end
+
+        it "validates an upper limit on responses" do
+          pending
+        end
+
+      end
+    
+  end
+end

data/spec/models/survey_section_spec.rb

@@ -0,0 +1,34 @@
+require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
+
+describe SurveySection, "when saving a survey_section" do
+  before(:each) do
+    @valid_attributes={:title => "foo", :survey_id => 2, :display_order => 4}
+    @survey_section = SurveySection.new(@valid_attributes)
+  end
+
+  it "should be invalid without title" do
+    @survey_section.title = nil
+    @survey_section.should have(1).error_on(:title)
+  end
+  
+  it "should have a parent survey" do
+    @survey_section.survey_id = nil
+    @survey_section.should have(1).error_on(:survey)
+  end
+end
+
+describe SurveySection, "with questions" do
+  before(:each) do
+    @survey = mock_model(Survey)
+    @valid_attributes={:title => "Rhymes", :survey => @survey, :display_order => 4}
+    @survey_section = SurveySection.create(@valid_attributes)
+    @q1 = @survey_section.questions.create(:text => "Peep", :display_order => 3)
+    @q2 = @survey_section.questions.create(:text => "Little", :display_order => 1)
+    @q3 = @survey_section.questions.create(:text => "Bo", :display_order => 2)
+  end
+  
+  it "should return questions sorted in display order" do
+    @survey_section.questions.should have(3).questions
+    @survey_section.questions.should == [@q2,@q3,@q1]
+  end
+end