Merge pull request #18 from nickel/more-about-pictos

Enabling/disabling pictos

app/packages/planning/models/picto.rb

@@ -4,6 +4,6 @@ class Picto < ApplicationRecord
   has_one_attached :image
 
   def to_struct
-    CustomStruct.new(attributes)
+    CustomStruct.new(attributes.merge(enabled?: enabled))
   end
 end

app/packages/planning/operations/picto/create.rb

@@ -7,7 +7,7 @@ class Form
     attribute :keyword, :string
     attribute :external_id, :integer
     attribute :external_source, :string
-    attribute :active, :boolean
+    attribute :enabled, :boolean, default: true
 
     attribute :image
     attribute :data
@@ -17,7 +17,7 @@ class Form
 
   def call
     Picto
-      .new(keyword:, external_id:, external_source:, active:, data:)
+      .new(keyword:, external_id:, external_source:, enabled:, data:)
       .save_with_response
       .and_then do |picto|
         picto.image.attach(io: File.open(image), filename: "image.png") if image

app/packages/planning/operations/picto/disable.rb

@@ -0,0 +1,24 @@
+# frozen_string_literal: true
+
+class Picto::Disable < CommandHandler::Command
+  class Form
+    include CommandHandler::Form
+
+    attribute :picto_id, :integer
+  end
+
+  delegate(*Form.new.attributes.keys, to: :form)
+
+  def call
+    Picto::Find
+      .call(picto_id:)
+      .and_then do
+        Picto
+          .find_by(id: picto_id)
+          .update_with_response(enabled: false)
+          .and_then do |picto|
+            Response.success(picto.to_struct)
+          end
+      end
+  end
+end

app/packages/planning/operations/picto/enable.rb

@@ -0,0 +1,24 @@
+# frozen_string_literal: true
+
+class Picto::Enable < CommandHandler::Command
+  class Form
+    include CommandHandler::Form
+
+    attribute :picto_id, :integer
+  end
+
+  delegate(*Form.new.attributes.keys, to: :form)
+
+  def call
+    Picto::Find
+      .call(picto_id:)
+      .and_then do
+        Picto
+          .find_by(id: picto_id)
+          .update_with_response(enabled: true)
+          .and_then do |picto|
+            Response.success(picto.to_struct)
+          end
+      end
+  end
+end

app/packages/planning/operations/picto/find.rb

@@ -0,0 +1,22 @@
+# frozen_string_literal: true
+
+class Picto::Find < CommandHandler::Command
+  class Form
+    include CommandHandler::Form
+
+    attribute :picto_id, :integer
+  end
+
+  delegate(*Form.new.attributes.keys, to: :form)
+
+  def call
+    if (picto = Picto.find_by(id: picto_id))
+      Response.success(picto.to_struct)
+    else
+      Response.failure(
+        Errors::RecordNotFoundError
+          .build(form:)
+      )
+    end
+  end
+end

db/migrate/20240326153956_create_pictos.rb

@@ -8,7 +8,7 @@ def change
 
       t.jsonb :data
 
-      t.boolean :active, default: true
+      t.boolean :enabled, default: true
 
       t.timestamps
     end

test/packages/planning/operations/picto/disable_test.rb

@@ -0,0 +1,16 @@
+# frozen_string_literal: true
+
+require "test_helper"
+
+class Picto::DisableTest < ActiveSupport::TestCase
+  test "picto should be disabled" do
+    picto = Factory.generate_picto(enabled: true)
+
+    assert picto.enabled?
+
+    response = Picto::Disable.call(picto_id: picto.id)
+
+    assert response.success?
+    refute response.value.enabled?
+  end
+end

test/packages/planning/operations/picto/enable_test.rb

@@ -0,0 +1,16 @@
+# frozen_string_literal: true
+
+require "test_helper"
+
+class Picto::EnableTest < ActiveSupport::TestCase
+  test "picto should be enabled" do
+    picto = Factory.generate_picto(enabled: false)
+
+    refute picto.enabled?
+
+    response = Picto::Enable.call(picto_id: picto.id)
+
+    assert response.success?
+    assert response.value.enabled?
+  end
+end

test/support/factory.rb

@@ -18,4 +18,12 @@ def generate_plan(**input)
         name: "This is a plan"
       ).value!
   end
+
+  def generate_picto(**input)
+    Picto::Create
+      .call(
+        keyword: "Picto #{Time.now.to_f}",
+        enabled: input.fetch(:enabled, true)
+      ).value!
+  end
 end