From fc9911abf4b1c297b4ae84d7c17c4f13618dd156 Mon Sep 17 00:00:00 2001 From: Manuel Bustillo Date: Sun, 8 Jun 2025 23:09:14 +0200 Subject: [PATCH] Fix rubocop ofenses --- app/controllers/websites_controller.rb | 4 +++- app/models/website.rb | 2 ++ spec/factories/websites.rb | 4 +++- spec/models/website_spec.rb | 6 ++++-- 4 files changed, 12 insertions(+), 4 deletions(-) diff --git a/app/controllers/websites_controller.rb b/app/controllers/websites_controller.rb index e801106..46e0d01 100644 --- a/app/controllers/websites_controller.rb +++ b/app/controllers/websites_controller.rb @@ -1,5 +1,7 @@ # Copyright (C) 2024-2025 LibreWeddingPlanner contributors +# frozen_string_literal: true + class WebsitesController < ApplicationController def show render json: current_tenant.website.as_json(only: %i[content]) || {}, status: :ok @@ -16,6 +18,6 @@ class WebsitesController < ApplicationController private def website_params - params.require(:website).permit(:content) + params.expect(website: [:content]) end end diff --git a/app/models/website.rb b/app/models/website.rb index 42c719e..401b101 100644 --- a/app/models/website.rb +++ b/app/models/website.rb @@ -1,5 +1,7 @@ # Copyright (C) 2024-2025 LibreWeddingPlanner contributors +# frozen_string_literal: true + # == Schema Information # # Table name: websites diff --git a/spec/factories/websites.rb b/spec/factories/websites.rb index 76e2d8b..b8c4048 100644 --- a/spec/factories/websites.rb +++ b/spec/factories/websites.rb @@ -1,8 +1,10 @@ # Copyright (C) 2024-2025 LibreWeddingPlanner contributors +# frozen_string_literal: true + FactoryBot.define do factory :website do - content { "MyText" } + content { 'MyText' } wedding { nil } end end diff --git a/spec/models/website_spec.rb b/spec/models/website_spec.rb index c22f339..a9533cd 100644 --- a/spec/models/website_spec.rb +++ b/spec/models/website_spec.rb @@ -1,7 +1,9 @@ # Copyright (C) 2024-2025 LibreWeddingPlanner contributors +# frozen_string_literal: true + require 'rails_helper' -RSpec.describe Website, type: :model do - it { should belong_to(:wedding) } +RSpec.describe Website do + it { is_expected.to belong_to(:wedding) } end