Upstream sync

spec/controllers/medical_releases_controller_spec.rb

@@ -0,0 +1,91 @@
+require "rails_helper"
+
+RSpec.describe MedicalReleasesController, type: :controller do
+  render_views
+
+  let(:user) { create(:user) }
+  let(:account) { user.primary_account } 
+  let(:project) { create(:project, account: user.primary_account) }
+
+  before do
+    sign_in user
+  end
+
+  describe "#index" do
+    it "responds successfully" do
+      get :index, params: { project_id: project }
+
+      expect(response).to be_successful
+    end
+
+    it "renders content" do
+      release = create(:medical_release, project: project,
+                                        person_first_name: "My",
+                                        person_last_name: "Release", 
+                                        person_phone: "5551234567", 
+                                        person_email: "jane.doe@test.com")
+      create(:note, notable: release, content: "Some notes here")
+
+      get :index, params: { project_id: project }
+
+      expect(response.body).to have_content "My Release"
+      expect(response.body).to have_content "Some notes here"
+      expect(response.body).to have_content "Manage"
+    end
+
+    context "when there are no medical releases" do
+      it "renders an empty message" do
+        get :index, params: { project_id: project }
+
+        expect(response.body).to have_content("Medical releases will appear here")
+      end
+    end
+
+    context "when there are many records" do
+      it "paginates the table" do
+        create_list(:medical_release, 20, project: project)
+
+        get :index, params: { project_id: project }
+
+        expect(response.body).to have_link("2", href: project_medical_releases_path(project, page: 2))
+      end
+    end
+
+    context "for xhr request" do
+      it "filters the releases by a query param" do
+        medical_releases = [
+          create(:medical_release, person_name: "Adam Sandler", project: project),
+          create(:medical_release, person_name: "Zoe Perry", project: project),
+        ]
+
+        get :index, params: { project_id: project, query: "Zoe" }, xhr: true
+
+        expect(response.body).not_to have_content("Adam Sandler")
+        expect(response.body).to have_content("Zoe Perry")
+      end
+    end
+  end
+
+  describe "#destroy" do
+    let!(:medical_release) { create(:medical_release, project: project) }
+
+    it "responds with redirect" do
+      delete :destroy, params: { project_id: project, id: medical_release }
+
+      expect(response).to be_redirect
+      expect(response).to redirect_to [project, :medical_releases]
+    end
+
+    it "sets the flash" do
+      delete :destroy, params: { project_id: project, id: medical_release }
+
+      expect(flash.alert).not_to be_nil
+    end
+
+    it "destroys the record" do
+      expect {
+        delete :destroy, params: { project_id: project, id: medical_release }
+      }.to change(MedicalRelease, :count).by(-1)
+    end
+  end
+end

spec/controllers/public/location_releases_controller_spec.rb

@@ -7,6 +7,15 @@ describe Public::LocationReleasesController do
   render_views
 
   describe "#create" do
+    it "allows photos param" do
+      contract_template = create(:contract_template, project: project)
+
+      post :create, params: { account_id: user.primary_account.to_param, project_id: project, contract_template_id: contract_template, location_release: location_release_params_with_photos }
+
+      expect(response).to be_successful
+      expect(LocationRelease.last.photos.attached?).to eq true
+    end
+
     it "logs analytics" do
       contract_template = create(:contract_template, project: project)
 
@@ -67,6 +76,10 @@ describe Public::LocationReleasesController do
     attributes_for(:location_release, :native).except(:signature).merge(signature_param).merge(person_address_params)
   end
 
+  def location_release_params_with_photos
+    attributes_for(:location_release, :native, :with_photo).except(:signature).merge(signature_param)
+  end
+
   def person_address_params
     {
       person_address_street1: "123 Broadway",

spec/controllers/public/material_releases_controller_spec.rb

@@ -7,6 +7,15 @@ describe Public::MaterialReleasesController do
   render_views
 
   describe "#create" do
+    it "allows photos param" do
+      contract_template = create(:contract_template, project: project)
+
+      post :create, params: { account_id: user.primary_account.to_param, project_id: project, contract_template_id: contract_template, material_release: material_release_params_with_photos }
+
+      expect(response).to be_successful
+      expect(MaterialRelease.last.photos.attached?).to eq true
+    end
+
     it "logs analytics" do
       contract_template = create(:contract_template, project: project)
 
@@ -65,6 +74,11 @@ describe Public::MaterialReleasesController do
     attributes_for(:material_release, :native).except(:signature).merge(signature_param)
   end
 
+  def material_release_params_with_photos
+    attributes_for(:material_release, :native, :with_photo).except(:signature).merge(signature_param)
+  end
+
+
   def signature_param
     file = file_fixture("signature.png")
     data_uri = Base64Image.from_image(file).data_uri

spec/controllers/public/medical_releases_controller_spec.rb

@@ -0,0 +1,71 @@
+require "rails_helper"
+
+RSpec.describe Public::MedicalReleasesController, type: :controller do
+  let(:user)    { create(:user) }
+  let(:project) { create(:project, account: user.primary_account) }
+
+  render_views
+
+  describe "#create" do
+    it "logs analytics" do
+      contract_template = create(:contract_template, project: project)
+
+      expect {
+        post :create, params: { account_id: project.account.to_param, project_id: project, contract_template_id: contract_template, medical_release: medical_release_params }
+      }.to(
+        have_enqueued_job(TrackAnalyticsJob)
+        .with(nil, nil, :track_create_native_release, release_type: "MedicalRelease", account: project.account, user_agent: "Rails Testing", user_ip: "0.0.0.0")
+      )
+    end
+
+    it "displays validation errors" do
+      contract_template = create(:contract_template, project: project)
+      sign_in(user)
+
+      post :create, params: { account_id: user.primary_account.to_param, project_id: project, contract_template_id: contract_template, medical_release: { person_address_city: "Albuquerque" } }
+      body = CGI.unescape_html(response.body)
+      expect(body).to match /Person first name can't be blank/
+      expect(body).to match /Person last name can't be blank/
+      expect(body).to match />can't be blank</
+    end
+
+    it "responds with success " do
+      contract_template = create(:contract_template, project: project)
+
+      post :create, params: { account_id: user.primary_account.to_param, project_id: project, contract_template_id: contract_template, medical_release: medical_release_params }
+
+      expect(response).to be_successful
+    end
+
+    it "runs attach contract to releasable job" do
+      contract_template = create(:contract_template, project: project)
+
+      expect {
+        post :create, params: { account_id: project.account.to_param, project_id: project, contract_template_id: contract_template, medical_release: medical_release_params }
+      }.to(
+        have_enqueued_job(AttachContractToReleasableJob)
+        .with(MedicalRelease.last)
+      )
+    end
+  end
+
+  private
+
+  def medical_release_params
+    attributes_for(:medical_release, :native).except(:signature).merge(signature_param, photos_param)
+  end
+
+  def photos_param
+    path = Rails.root.join("spec", "fixtures", "files", "person_photo.png")
+    photo = Rack::Test::UploadedFile.new(path, "image/png")
+
+    { photos: [photo] }
+  end
+
+  def signature_param
+    file = file_fixture("signature.png")
+    data_uri = Base64Image.from_image(file).data_uri
+
+    { signature_base64: data_uri }
+  end
+end

spec/controllers/public/zoom_meetings_controller_spec.rb

@@ -1,5 +1,4 @@
 require 'rails_helper'
-require 'zoom_gateway'
 
 RSpec.describe Public::ZoomMeetingsController, type: :controller do
   let(:user) { create(:user) }

spec/controllers/zoom_meetings_controller_spec.rb

@@ -1,5 +1,4 @@
 require 'rails_helper'
-require 'zoom_gateway'
 
 RSpec.describe ZoomMeetingsController, type: :controller do
   let(:user) { create(:user) }

spec/controllers/zoom_notifications_controller_spec..rb

@@ -1,82 +0,0 @@
-require "rails_helper"
-
-RSpec.describe ZoomNotificationsController, type: :controller do
-  render_views
-
-  let!(:zoom_meeting) { create(:zoom_meeting, api_meeting_id: 'meeting_id') }
-
-  let(:started_status) { {event: 'meeting.started', payload: {object: {id: 'meeting_id' }}} }
-  let(:ended_status) { {event: 'meeting.ended', payload: {object: {id: 'meeting_id' }}} }
-  let(:wrong_meeting_id) { {payload: {object: {id: 'wrong_id' }}} }
-
-  let(:authorization_header) { {'Authorization' => 'xxx-xxx-xxx'} }
-  let(:wrong_authorization_header) { {'Authorization' => 'yyy-yyy-yyy'} }
-
-  before do
-    allow(ENV).to receive(:[]).with('ZOOM_VERIFICATION_TOKEN').and_return('xxx-xxx-xxx')
-  end
-
-  describe '#create' do
-    context 'with no authorization key' do
-      it 'raises 403 response' do
-        post :create, params: started_status
-        expect(response).to have_http_status(403)
-      end
-    end
-
-    context 'with wrong authorization key' do
-      it 'raises 403 response' do
-        request.headers.merge!(wrong_authorization_header)
-        post :create, params: started_status
-        expect(response).to have_http_status(403)
-      end
-    end
-
-    context 'authorized' do
-      before do
-        request.headers.merge!(authorization_header)
-      end
-
-      context 'with wrong meeting id' do
-        it 'raises RecordNotFound' do
-          expect {
-            post :create, params: wrong_meeting_id
-          }.to raise_error(ActiveRecord::RecordNotFound)
-        end
-      end
-
-      context 'with right meeting id' do
-        it 'responds with 200' do
-          post :create, params: started_status
-
-          expect(response).to have_http_status(200)
-        end
-
-        it 'assigns the zoom meeting' do
-          post :create, params: started_status
-
-          expect(assigns(:zoom_meeting)).to eq(zoom_meeting)
-        end
-
-        it 'updates the zoom_meeting when started status is received in notification' do
-          post :create, params: started_status
-
-          expect(zoom_meeting.reload).to be_started
-        end
-
-        it 'updates the zoom_meeting when ended status is received in notification' do
-          post :create, params: ended_status
-
-          expect(zoom_meeting.reload).to be_ended
-        end
-
-        it 'updates the recording when recording complete notification is received' do
-          expect {
-            post :create, params: recording_complete
-          }.to change { zoom_meeting.recording }
-        end
-
-      end
-    end
-  end
-end

spec/controllers/zoom_notifications_controller_spec.rb

@@ -0,0 +1,101 @@
+require "rails_helper"
+
+RSpec.describe ZoomNotificationsController, type: :controller do
+  render_views
+
+  let!(:zoom_meeting) { create(:zoom_meeting, api_meeting_id: 'meeting_id') }
+
+  let(:started_status) { {event: 'meeting.started', payload: {object: {id: 'meeting_id' }}} }
+  let(:ended_status) {  {event: 'meeting.ended', payload: {object: {id: 'meeting_id' }}} }
+  let(:wrong_meeting_id) { {event: 'meeting.started', payload: {object: {id: 'wrong_id' }}} }
+
+  let(:recording_complete) { {event: 'recording.completed', payload: {object: {id: 'meeting_id', recording_files: [Object.new]}}} }
+
+  let(:authorization_header) { {'Authorization' => 'xxx-xxx-xxx'} }
+  let(:wrong_authorization_header) { {'Authorization' => 'yyy-yyy-yyy'} }
+
+  before do
+    allow(ENV).to receive(:[]).with('ZOOM_VERIFICATION_TOKEN').and_return('xxx-xxx-xxx')
+  end
+
+  describe '#create' do
+    context 'with no authorization key' do
+      it 'raises 403 response' do
+        post :create, params: started_status
+        expect(response).to have_http_status(403)
+      end
+    end
+
+    context 'with wrong authorization key' do
+      it 'raises 403 response' do
+        request.headers.merge!(wrong_authorization_header)
+        post :create, params: started_status
+        expect(response).to have_http_status(403)
+      end
+    end
+
+    context 'authorized' do
+      before do
+        request.headers.merge!(authorization_header)
+      end
+
+      context 'user hooks' do
+        before(:each) { ZoomUser.create api_id: 'zoom_user_id' }
+
+        it 'deletes the user from db if user.deleted is passed with existing user id' do
+          expect {
+            post :create, params: {event: 'user.deleted', payload: {object: {id: 'zoom_user_id'}}}
+          }.to change { ZoomUser.count }.by(-1)
+        end
+
+        it 'does not do anything if user.deleted is passed with non-existing user' do
+          expect {
+            post :create, params: {event: 'user.deleted', payload: {object: {id: 'wrong-user-id'}}}
+          }.not_to change { ZoomUser.count }
+        end
+      end
+
+      context 'meeting hooks' do
+        context 'with wrong meeting id' do
+          it 'raises RecordNotFound' do
+            expect {
+              post :create, params: wrong_meeting_id
+            }.to raise_error(ActiveRecord::RecordNotFound)
+          end
+        end
+
+        context 'with right meeting id' do
+          it 'responds with 200' do
+            post :create, params: started_status
+
+            expect(response).to have_http_status(200)
+          end
+
+          it 'assigns the zoom meeting' do
+            post :create, params: started_status
+
+            expect(assigns(:zoom_meeting)).to eq(zoom_meeting)
+          end
+
+          it 'updates the zoom_meeting when started status is received in notification' do
+            post :create, params: started_status
+
+            expect(zoom_meeting.reload).to be_started
+          end
+
+          it 'updates the zoom_meeting when ended status is received in notification' do
+            post :create, params: ended_status
+
+            expect(zoom_meeting.reload).to be_ended
+          end
+
+          it 'updates the recording when recording complete notification is received' do
+            expect(AttachRecordingToZoomMeetingJob).to receive(:perform_later)
+
+            post :create, params: recording_complete
+          end
+        end
+      end
+    end
+  end
+end