diff options
| author | mo khan <mo@mokhan.ca> | 2015-11-14 21:19:58 -0700 |
|---|---|---|
| committer | mo khan <mo@mokhan.ca> | 2015-11-14 21:19:58 -0700 |
| commit | d311e696b9689c8c8eaf8eb13dd6773243eba2c8 (patch) | |
| tree | f812771f88d02e4d9fcf4073f3026dc4c5b2ff3e /spec | |
| parent | f28ddb69d7cca6bce073d82391bca5a323db0d82 (diff) | |
remove redundant code from profiles controller.
Diffstat (limited to 'spec')
| -rw-r--r-- | spec/controllers/profiles_controller_spec.rb | 69 |
1 files changed, 21 insertions, 48 deletions
diff --git a/spec/controllers/profiles_controller_spec.rb b/spec/controllers/profiles_controller_spec.rb index 03fbe7d..e82b1de 100644 --- a/spec/controllers/profiles_controller_spec.rb +++ b/spec/controllers/profiles_controller_spec.rb @@ -1,24 +1,23 @@ require "rails_helper" describe ProfilesController do - describe "authenticated" do - - describe "#show" do - include_context "stronglifts_program" + include_context "stronglifts_program" + let(:user) { create(:user) } - let(:user) { create(:user) } - let(:other_user) { create(:user) } + before :each do + http_login(user) + end - before :each do - http_login(user) - end + describe "#show" do + let(:other_user) { create(:user) } it "loads the user's profile" do get :show, id: user.to_param expect(assigns(:user)).to eql(user) expect(assigns(:program)).to eql(Program.stronglifts) end + it "loads the other user's profile" do get :show, id: other_user.to_param expect(assigns(:user)).to eql(other_user) @@ -27,75 +26,49 @@ describe ProfilesController do end describe "#edit" do - include_context "stronglifts_program" - - let(:user) { create(:user) } let(:other_user) { create(:user) } - - before :each do - http_login(user) - end it "loads the user's profile into an edit view" do get :edit, id: user.to_param - expect(assigns(:user)).to eql(user) + expect(assigns(:current_user)).to eql(user) expect(assigns(:program)).to eql(Program.stronglifts) end - + it "will not load the other user's profile into an edit view" do get :edit, id: other_user.to_param - expect(assigns(:user)).to eql(user) + expect(assigns(:current_user)).to eql(user) expect(assigns(:program)).to eql(Program.stronglifts) end - end - - describe "#update" do - include_context "stronglifts_program" - let(:user) { create(:user) } - - before :each do - http_login(user) - end - + describe "#update" do it "updates the user profile" do - patch :update, id: user.to_param, profile: {gender: "male"} + patch :update, id: user.to_param, profile: { gender: "male" } user.reload expect(user.profile.male?).to be_truthy expect(response).to redirect_to(profile_path(user.profile)) end - end - end - - describe "unauthenticated" do - - describe "#show" do - include_context "stronglifts_program" - let(:user) { create(:user) } + describe "unauthenticated" do + include_context "stronglifts_program" + let(:user) { create(:user) } + describe "#show" do it "loads the user's profile" do get :show, id: user.to_param - expect(assigns(:user)).to eql(nil) - expect(assigns(:program)).to eql(nil) + expect(assigns(:user)).to be_nil + expect(assigns(:program)).to be_nil end end describe "#edit" do - include_context "stronglifts_program" - - let(:user) { create(:user) } - it "loads the user's profile into an edit view" do get :edit, id: user.to_param - expect(assigns(:user)).to eql(nil) - expect(assigns(:program)).to eql(nil) + expect(assigns(:user)).to be_nil + expect(assigns(:program)).to be_nil end end - end - end |
