From ad1b4d6af85eb62ff32da8f51feaf0b1e92ac2e0 Mon Sep 17 00:00:00 2001
From: binarygit <inscription4@outlook.com>
Date: Wed, 17 Jul 2024 15:49:51 +0545
Subject: [PATCH 1/2] Write a comment and lint

---
 spec/dummy/app/models/user.rb | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/spec/dummy/app/models/user.rb b/spec/dummy/app/models/user.rb
index 436faf2ee..ea3fd0877 100644
--- a/spec/dummy/app/models/user.rb
+++ b/spec/dummy/app/models/user.rb
@@ -48,6 +48,8 @@ class User < ApplicationRecord
   scope :admins, -> { where "(roles->>'admin')::boolean is true" }
   scope :non_admins, -> { where "(roles->>'admin')::boolean != true" }
 
+  # Setter for permissions because it's a
+  # non-db backed boolean group
   attr_writer :permissions
 
   def is_admin?
@@ -96,7 +98,7 @@ def permissions
       create: true,
       update: false,
       read: true,
-      delete: true,
+      delete: true
     }
   end
 end

From 05253ce6fdac0e63bb4fb70b046c85b08c7f2274 Mon Sep 17 00:00:00 2001
From: Adrian Marin <adrian@adrianthedev.com>
Date: Fri, 19 Jul 2024 13:19:35 +0300
Subject: [PATCH 2/2] Update spec/dummy/app/models/user.rb

---
 spec/dummy/app/models/user.rb | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/spec/dummy/app/models/user.rb b/spec/dummy/app/models/user.rb
index ea3fd0877..5ddac35f9 100644
--- a/spec/dummy/app/models/user.rb
+++ b/spec/dummy/app/models/user.rb
@@ -48,8 +48,7 @@ class User < ApplicationRecord
   scope :admins, -> { where "(roles->>'admin')::boolean is true" }
   scope :non_admins, -> { where "(roles->>'admin')::boolean != true" }
 
-  # Setter for permissions because it's a
-  # non-db backed boolean group
+  # We're using a setter here because we want to test that the field is working properly with a non-db backed field.
   attr_writer :permissions
 
   def is_admin?