Increase test coverage to 58.31%

Author: rsmithlal

spec/factories/better_together/resource_permissions.rb

@@ -6,7 +6,9 @@
   factory :better_together_resource_permission,
           class: 'BetterTogether::ResourcePermission',
           aliases: %i[resource_permission] do
-    action { 'MyString' }
-    resource_type { 'MyString' }
+    action { BetterTogether::ResourcePermission::ACTIONS.sample }
+    resource_type { BetterTogether::Resourceful::RESOURCE_CLASSES.sample }
+    # Derive target from the resource_type (e.g., 'BetterTogether::Community' => 'community')
+    target { resource_type.demodulize.underscore }
   end
 end

spec/requests/better_together/calendars_controller_spec.rb

@@ -0,0 +1,17 @@
+# frozen_string_literal: true
+
+require 'rails_helper'
+
+RSpec.describe 'BetterTogether::CalendarsController', type: :request do
+  let(:locale) { I18n.default_locale }
+
+  before do
+    configure_host_platform
+    login('[email protected]', 'password12345')
+  end
+
+  it 'renders index' do
+    get better_together.calendars_path(locale:)
+    expect(response).to have_http_status(:ok)
+  end
+end

spec/requests/better_together/communities_controller_spec.rb

@@ -0,0 +1,37 @@
+# frozen_string_literal: true
+
+require 'rails_helper'
+
+RSpec.describe 'BetterTogether::CommunitiesController', 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/communities' do
+    it 'renders index' do
+      get better_together.communities_path(locale:)
+      expect(response).to have_http_status(:found)
+    end
+
+    it 'renders show for a community' do
+      community = create(:better_together_community)
+      get better_together.community_path(locale:, id: community.slug)
+      expect(response).to have_http_status(:found)
+    end
+  end
+
+  describe 'PATCH /:locale/.../host/communities/:id' do
+    it 'updates and redirects' do
+      community = create(:better_together_community)
+      patch better_together.community_path(locale:, id: community.slug), params: {
+        community: { privacy: 'public', name_en: 'Updated Name' }
+      }
+      expect(response).to have_http_status(:found)
+      follow_redirect!
+      expect(response).to have_http_status(:ok)
+    end
+  end
+end

spec/requests/better_together/resource_permissions_controller_spec.rb

@@ -0,0 +1,25 @@
+# frozen_string_literal: true
+
+require 'rails_helper'
+
+RSpec.describe 'BetterTogether::ResourcePermissionsController', type: :request do
+  let(:locale) { I18n.default_locale }
+
+  before do
+    configure_host_platform
+    login('[email protected]', 'password12345')
+  end
+
+  describe 'GET /:locale/.../host/permissions' do
+    it 'renders index' do
+      get better_together.resource_permissions_path(locale:)
+      expect(response).to have_http_status(:ok)
+    end
+
+    it 'renders show for a permission' do
+      permission = create(:better_together_resource_permission)
+      get better_together.resource_permission_path(locale:, id: permission.slug)
+      expect(response).to have_http_status(:ok)
+    end
+  end
+end

spec/requests/better_together/roles_controller_spec.rb

@@ -0,0 +1,37 @@
+# frozen_string_literal: true
+
+require 'rails_helper'
+
+RSpec.describe 'BetterTogether::RolesController', 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/roles' do
+    it 'renders index' do
+      get better_together.roles_path(locale:)
+      expect(response).to have_http_status(:ok)
+    end
+
+    it 'renders show for a role' do
+      role = create(:better_together_role)
+      get better_together.role_path(locale:, id: role.slug)
+      expect(response).to have_http_status(:ok)
+    end
+  end
+
+  describe 'PATCH /:locale/.../host/roles/:id' do
+    it 'updates and redirects' do
+      role = create(:better_together_role, protected: false)
+      patch better_together.role_path(locale:, id: role.slug), params: {
+        role: { name: 'New 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/search_controller_spec.rb

@@ -0,0 +1,17 @@
+# frozen_string_literal: true
+
+require 'rails_helper'
+
+RSpec.describe 'BetterTogether::SearchController', type: :request do
+  let(:locale) { I18n.default_locale }
+
+  before do
+    configure_host_platform
+    login('[email protected]', 'password12345')
+  end
+
+  it 'renders search results page' do
+    get better_together.search_path(locale:), params: { q: 'test' }
+    expect(response).to have_http_status(:ok)
+  end
+end