Increased test coverage

Author: rsmithlal

spec/requests/better_together/geography/region_settlements_controller_spec.rb

@@ -0,0 +1,28 @@
+# frozen_string_literal: true
+
+require 'rails_helper'
+
+RSpec.describe 'BetterTogether::Geography::RegionSettlementsController', type: :request do
+  let(:locale) { I18n.default_locale }
+
+  before do
+    configure_host_platform
+    login('[email protected]', 'password12345')
+  end
+
+  describe 'GET /:locale/.../host/geography/region_settlements' do
+    it 'renders index' do
+      get better_together.geography_region_settlements_path(locale:)
+      expect(response).to have_http_status(:ok)
+    end
+  end
+
+  describe 'GET /:locale/.../host/geography/region_settlements/:id' do
+    let!(:region_settlement) { create(:region_settlement) }
+
+    it 'renders show' do
+      get better_together.geography_region_settlement_path(locale:, id: region_settlement.id)
+      expect(response).to have_http_status(:ok)
+    end
+  end
+end

spec/requests/better_together/metrics/link_click_reports_download_spec.rb

@@ -0,0 +1,25 @@
+# frozen_string_literal: true
+
+require 'rails_helper'
+
+RSpec.describe 'BetterTogether::Metrics::LinkClickReportsController download', type: :request do
+  let(:locale) { I18n.default_locale }
+
+  before do
+    configure_host_platform
+    login('[email protected]', 'password12345')
+  end
+
+  it 'downloads an attached report file' do
+    report = BetterTogether::Metrics::LinkClickReport.create!(file_format: 'csv')
+    report.report_file.attach(
+      io: StringIO.new('a,b\n1,2\n'),
+      filename: 'report.csv',
+      content_type: 'text/csv'
+    )
+
+    get better_together.download_metrics_link_click_report_path(locale:, id: report.id)
+    expect(response).to have_http_status(:ok)
+    expect(response.header['Content-Type']).to include('text/csv')
+  end
+end

spec/requests/better_together/people_controller_spec.rb

@@ -0,0 +1,39 @@
+# frozen_string_literal: true
+
+require 'rails_helper'
+
+RSpec.describe 'BetterTogether::PeopleController', type: :request do # rubocop:todo Metrics/BlockLength
+  let(:locale) { I18n.default_locale }
+
+  before do
+    configure_host_platform
+    login('[email protected]', 'password12345')
+  end
+
+  describe 'GET /:locale/.../host/p/:id' do
+    let!(:person) { create(:better_together_person) }
+
+    it 'renders show' do
+      get better_together.person_path(locale:, id: person.slug)
+      expect(response).to have_http_status(:ok)
+    end
+
+    it 'renders edit' do
+      get better_together.edit_person_path(locale:, id: person.slug)
+      expect(response).to have_http_status(:ok)
+    end
+  end
+
+  describe 'PATCH /:locale/.../host/p/:id' do
+    let!(:person) { create(:better_together_person) }
+
+    it 'updates name and redirects' do
+      patch better_together.person_path(locale:, id: person.slug), params: {
+        person: { name: 'Updated Name' }
+      }
+      expect(response).to have_http_status(:see_other)
+      follow_redirect!
+      expect(response).to have_http_status(:ok)
+    end
+  end
+end

spec/requests/better_together/platforms_controller_spec.rb

@@ -0,0 +1,37 @@
+# frozen_string_literal: true
+
+require 'rails_helper'
+
+RSpec.describe 'BetterTogether::PlatformsController', type: :request do # rubocop:todo Metrics/BlockLength
+  let(:locale) { I18n.default_locale }
+
+  before do
+    configure_host_platform
+    login('[email protected]', 'password12345')
+  end
+
+  describe 'GET /:locale/.../host/platforms' do
+    it 'renders index' do
+      get better_together.platforms_path(locale:)
+      expect(response).to have_http_status(:ok)
+    end
+
+    it 'renders show for host platform' do
+      host_platform = BetterTogether::Platform.find_by(host: true)
+      get better_together.platform_path(locale:, id: host_platform.slug)
+      expect(response).to have_http_status(:ok)
+    end
+  end
+
+  describe 'PATCH /:locale/.../host/platforms/:id' do
+    it 'updates settings and redirects' do
+      host_platform = BetterTogether::Platform.find_by(host: true)
+      patch better_together.platform_path(locale:, id: host_platform.slug), params: {
+        platform: { url: host_platform.url, time_zone: host_platform.time_zone, requires_invitation: true }
+      }
+      expect(response).to have_http_status(:see_other)
+      follow_redirect!
+      expect(response).to have_http_status(:ok)
+    end
+  end
+end