special unknown operator

app/admin/operator.rb

@@ -24,7 +24,7 @@
 
   config.clear_action_items!
 
-  action_item :toggle_active, only: [:show] do
+  action_item :toggle_active, only: [:show], if: -> { !resource.special_unknown? } do
     if resource.is_active
       link_to I18n.t("shared.deactivate"), deactivate_admin_producer_path(resource),
         method: :put, data: {confirm: I18n.t("active_admin.operator_page.confirm_deactivate", name: resource.name)}
@@ -34,7 +34,7 @@
     end
   end
 
-  action_item :edit, only: [:show] do
+  action_item :edit, only: [:show], if: -> { !resource.special_unknown? } do
     link_to I18n.t("active_admin.operator_page.edit"), edit_admin_producer_path(resource)
   end
 
@@ -97,6 +97,8 @@
     end
     column(:created_at) { |o| o.created_at.to_date }
     column(I18n.t("active_admin.shared.actions")) do |operator|
+      next if operator.special_unknown?
+
       if operator.is_active
         link_to I18n.t("shared.deactivate"), deactivate_admin_producer_path(operator),
           method: :put, data: {confirm: I18n.t("active_admin.operator_page.confirm_deactivate", name: operator.name)}
@@ -108,7 +110,7 @@
 
     actions defaults: false do |operator|
       item I18n.t("active_admin.view"), admin_producer_path(operator)
-      item I18n.t("active_admin.edit"), edit_admin_producer_path(operator)
+      item I18n.t("active_admin.edit"), edit_admin_producer_path(operator) unless operator.special_unknown?
       if operator.can_hard_delete?
         item I18n.t("active_admin.delete"), admin_producer_path(operator), method: :delete, data: {confirm: I18n.t("active_admin.operator_page.confirm_delete")}
       end

app/models/observation.rb

@@ -118,6 +118,7 @@ class Observation < ApplicationRecord
 
   with_options if: :operator? do
     validate :validate_governments_absences
+    validate :validate_known_operator, if: -> { operator? && (validation_status != "Created") }
   end
 
   with_options if: :government? do
@@ -288,6 +289,10 @@ def validate_governments_absences
     errors.add(:governments, "Should have no governments with 'operator' type")
   end
 
+  def validate_known_operator
+    errors.add(:operator, "can't be blank or unknown") if operator.blank? || operator.special_unknown?
+  end
+
   # Validates the statuses transitions.
   # This method makes sure the state machine of observations' statuses is enforced.
   def status_changes

app/models/operator.rb

@@ -86,7 +86,7 @@ class Operator < ApplicationRecord
   validates :name, uniqueness: {case_sensitive: false}
   validates :website, url: true, if: lambda { |x| x.website.present? }
   validates :operator_type, inclusion: {in: TYPES, message: "can't be %{value}. Valid values are: #{TYPES.join(", ")} "}
-  validates :country, presence: true, on: :create
+  validates :country, presence: true, on: :create, unless: :special_unknown?
 
   scope :by_name_asc, -> { order(name: :asc) }
 
@@ -135,13 +135,24 @@ def can_hard_delete?
       all_observations.with_deleted.none? &&
       relevant_observations.with_deleted.none? &&
       users.none? &&
-      operator_documents.with_deleted.none?
+      operator_documents.with_deleted.none? &&
+      !special_unknown?
   end
 
   def set_slug
     self.slug = name.parameterize
   end
 
+  def name
+    return I18n.t("filters.unknown") if special_unknown?
+
+    super
+  end
+
+  def special_unknown?
+    slug == "unknown"
+  end
+
   private
 
   # Saves the fmus of the operator to update the operator's name.

app/resources/v1/operator_resource.rb

@@ -38,10 +38,6 @@ def set_active
       @model.is_active = false
     end
 
-    def name
-      I18n.with_locale(:en) { @model.name }
-    end
-
     filter :certification, apply: ->(records, value, _options) {
       values = value.select { |c| %w[fsc pefc olb pafc fsc_cw tlv ls].include? c }
       return records unless values.any?

db/fixtures/06_operators.rb

@@ -1 +1,2 @@
 Rake::Task["import:operators"].invoke unless Operator.any?
+Operator.find_or_create_by!(name: "Unknown", operator_type: "Unknown", slug: "unknown")

db/migrate/20231115113432_add_unknown_operator.rb

@@ -0,0 +1,9 @@
+class AddUnknownOperator < ActiveRecord::Migration[7.0]
+  def up
+    Operator.find_or_create_by!(name: "Unknown", operator_type: "Unknown", slug: "unknown")
+  end
+
+  def down
+    Operator.where(slug: "unknown").destroy_all
+  end
+end

spec/factories/operators.rb

@@ -30,6 +30,13 @@
     operator_type { "Logging company" }
     is_active { true }
 
+    factory :unknown_operator do
+      country { nil }
+      name { "Unknown" }
+      slug { "unknown" }
+      operator_type { "Unknown" }
+    end
+
     trait :with_sawmills do
       after(:create) do |op|
         create_list(:sawmill, 2, operator: op)

spec/models/observation_spec.rb

@@ -94,9 +94,11 @@
     describe "#status_changes" do
       let(:country) { create(:country) }
       let(:status) { "Created" }
+      let(:operator) { create(:operator, country: country) }
       let(:observation) {
         build :observation,
           country: country,
+          operator: operator,
           validation_status: status
       }
       subject {
@@ -165,6 +167,28 @@
           it { is_expected.to_not be_valid }
         end
       end
+
+      describe "for all" do
+        context "with unknown operator" do
+          let(:operator) { create(:unknown_operator) }
+          before { observation.save }
+          subject { observation }
+
+          context "when creating an observation with status `Created`" do
+            it { is_expected.to be_valid }
+            it { is_expected.to be_persisted }
+          end
+
+          context "when moving to 'Ready for QC'" do
+            it "should not be valid" do
+              observation.validation_status = "Ready for QC"
+              observation.operator = Operator.find_by(slug: "unknown")
+              expect(observation.save).to be false
+              expect(observation.errors[:operator]).to eql ["can't be blank or unknown"]
+            end
+          end
+        end
+      end
     end
 
     describe "#active_government" do