cleaner way to access dead and alive status + use that for plants

app/models/plant.rb

@@ -17,11 +17,11 @@ def trefle_data
 
   class << self
     def dead
-      Plant.where(status_id: Status.find_by(name: 'Dead'))
+      Plant.where(status_id: Status.dead)
     end
 
     def alive
-      Plant.where(status_id: Status.find_by(name: 'Alive'))
+      Plant.where(status_id: Status.alive)
     end
   end
 end

app/models/status.rb

@@ -1,3 +1,19 @@
 class Status < ApplicationRecord
   has_many :plants
+
+  DEAD = 'Dead'.freeze
+
+  ALIVE = 'Alive'.freeze
+
+  class << self
+    # @return [Status]
+    def dead
+      find_by(name: DEAD)
+    end
+
+    # @return [Status]
+    def alive
+      find_by(name: ALIVE)
+    end
+  end
 end

spec/models/status_spec.rb

@@ -6,4 +6,22 @@
       expect(subject).to all(be_instance_of(Plant))
     end
   end
+
+  describe '.dead' do
+    subject { Status.dead }
+
+    it do
+      expect(subject).to be_instance_of(Status)
+      expect(subject.name).to eql('Dead')
+    end
+  end
+
+  describe '.alive' do
+    subject { Status.alive }
+
+    it do
+      expect(subject).to be_instance_of(Status)
+      expect(subject.name).to eql('Alive')
+    end
+  end
 end