spree_cm_commissioner 2.5.0.pre.pre7 → 2.5.0.pre.pre8

data/app/views/spree/admin/integration_sessions/_integration_sync_sessions.html.erb

@@ -1,116 +0,0 @@
-<div class="table-responsive">
-  <table
-    class="table table-sm"
-    id="listing_integration_sync_sessions"
-    data-hook
-  >
-    <thead class="text-muted">
-      <tr>
-        <th><%= Spree.t(:id) %></th>
-        <th><%= Spree.t(:sync_type) %></th>
-        <th><%= Spree.t(:status) %></th>
-        <th><%= Spree.t(:started_at) %></th>
-        <th><%= Spree.t(:ended_at) %></th>
-        <th><%= Spree.t(:duration) %></th>
-        <th></th>
-      </tr>
-    </thead>
-
-    <tbody>
-      <% sync_sessions = defined?(@sync_sessions) ? @sync_sessions : integration.integration_sync_sessions.order(created_at: :desc).limit(25) %>
-      <% sync_sessions.each do |sess| %>
-        <tr id="integration_sync_session_<%= sess.id %>">
-          <td><%= sess.id %></td>
-          <td><%= sess.sync_type %></td>
-          <td><%= sess.status %></td>
-          <td><%= sess.started_at %></td>
-          <td><%= sess.ended_at %></td>
-          <td>
-            <% if sess.started_at && sess.ended_at %>
-              <% duration = (sess.ended_at - sess.started_at).to_i %>
-
-              <% if duration < 60 %>
-                <%= "#{duration} sec" %>
-              <% else %>
-                <%= "#{duration / 60} min" %>
-              <% end %>
-            <% else %>
-              N/A
-            <% end %>
-          </td>
-          <td class="actions">
-            <% if sess.sync_result.present? %>
-              <%= link_to_with_icon 'activity.svg', Spree.t("admin.external_integrations.view_result"), "#session_result_modal_#{sess.id}",
-                class: 'btn btn-light btn-sm',
-                no_text: true,
-                data: { toggle: 'modal', target: "#session_result_modal_#{sess.id}" },
-                title: Spree.t("admin.external_integrations.sync_result") %>
-            <% end %>
-          </td>
-        </tr>
-      <% end %>
-    </tbody>
-  </table>
-
-  <% if sync_sessions.empty? %>
-    <small class="form-text text-muted"><%= Spree.t("admin.external_integrations.no_results") %></small>
-  <% end %>
-
-  <% if defined?(@sync_sessions) && @sync_sessions.respond_to?(:total_pages) %>
-    <div class="mt-3">
-      <%= paginate @sync_sessions, theme: 'admin-twitter-bootstrap-4' %>
-    </div>
-  <% end %>
-</div>
-
-<% sync_sessions.each do |sess| %>
-  <% if sess.sync_result.present? %>
-    <div
-      class="modal fade"
-      id="session_result_modal_<%= sess.id %>"
-      tabindex="-1"
-      role="dialog"
-      aria-labelledby="session_result_modal_label_<%= sess.id %>"
-      aria-hidden="true"
-    >
-      <div class="modal-dialog modal-lg" role="document">
-        <div class="modal-content">
-          <div class="modal-header">
-            <h5
-              class="modal-title"
-              id="session_result_modal_label_<%= sess.id %>"
-            >
-              <%= Spree.t("admin.external_integrations.sync_result") %> - Session #<%= sess.id %>
-            </h5>
-
-            <button
-              type="button"
-              class="close"
-              data-dismiss="modal"
-              aria-label="Close"
-            >
-              <span aria-hidden="true">&times;</span>
-            </button>
-          </div>
-
-          <div class="modal-body">
-            <pre
-              class="small mb-0 p-2 bg-light border rounded"
-              style="max-height: 500px; overflow-y: auto;"
-            ><code><%= JSON.pretty_generate(sess.sync_result) %></code></pre>
-          </div>
-
-          <div class="modal-footer">
-            <button
-              type="button"
-              class="btn btn-secondary"
-              data-dismiss="modal"
-            >
-              <%= Spree.t(:close) %>
-            </button>
-          </div>
-        </div>
-      </div>
-    </div>
-  <% end %>
-<% end %>

data/app/views/spree/admin/integration_sessions/index.html.erb

@@ -1,42 +0,0 @@
-<% content_for :page_title do %>
-  <%= link_to Spree.t('external_integrations'), spree.admin_integrations_url %> / <%= @integration.name %>
-<% end %>
-
-<% content_for :page_tabs do %>
-  <li class="nav-item">
-    <%= link_to Spree.t(:edit),
-      edit_admin_integration_path(@integration),
-      class: "nav-link" %>
-  </li>
-
-  <li class="nav-item">
-    <%= link_to Spree.t(:integration_sync_sessions),
-      admin_integration_sessions_path(@integration),
-      class: "nav-link active" %>
-  </li>
-
-  <li class="nav-item">
-    <%= link_to Spree.t(:integration_mappings),
-      admin_integration_mappings_path(@integration),
-      class: "nav-link" %>
-  </li>
-<% end %>
-
-<div data-hook="admin_integrations_sessions" class="card mb-3">
-  <div class="card-header d-flex justify-content-between align-items-center">
-    <h5 class="card-title mb-0 h6">
-      <%= Spree.t(:integration_sync_sessions) %>
-    </h5>
-
-    <div class="card-header-actions">
-      <%= button_to Spree.t(:enqueue_full_sync),
-          enqueue_polling_admin_integration_path(@integration, sync_type: :full),
-          method: :post,
-          class: 'btn btn-primary btn-sm' %>
-    </div>
-  </div>
-
-  <div class="card-body">
-    <%= render partial: 'integration_sync_sessions', locals: { integration: @integration } %>
-  </div>
-</div>

data/app/views/spree/admin/integrations/_form.html.erb

@@ -1,104 +0,0 @@
-<div data-hook="admin_external_integrations_form_fields" class="row">
-  <div class="col-6">
-    <%= f.field_container :type do %>
-      <%= f.label :type, raw(I18n.t('spree.admin.external_integrations.integration_type', default: 'Integration type') + required_span_tag) %>
-
-      <% if f.object.persisted? %>
-        <%= f.text_field :type, value: f.object.type, class: 'form-control', disabled: true, readonly: true %>
-        <%= f.hidden_field :type, value: f.object.type %>
-      <% else %>
-        <%= f.select :type,
-          options_for_select(@integration_type_options, f.object.type),
-          { include_blank: Spree.t(:choose_an_option) },
-          class: 'form-control select2',
-          required: true %>
-      <% end %>
-
-      <%= f.error_message_on :type %>
-    <% end %>
-  </div>
-
-  <div class="col-6">
-    <%= f.field_container :name do %>
-      <%= f.label :name, raw(Spree.t(:name) + required_span_tag) %>
-      <%= f.text_field :name, required: true, class: 'form-control', placeholder: Spree.t(:name) %>
-      <%= f.error_message_on :name %>
-    <% end %>
-  </div>
-
-  <div class="col-6">
-    <%= f.field_container :vendor_id do %>
-      <%= f.label :vendor_id, raw(I18n.t('spree.admin.external_integrations.vendor', default: 'Vendor') + required_span_tag) %>
-
-      <%= f.select :vendor_id,
-        options_from_collection_for_select(@vendors, :id, :name, f.object.vendor_id),
-        { include_blank: Spree.t(:choose_an_option) },
-        class: 'form-control select2',
-        required: true %>
-
-      <%= f.error_message_on :vendor_id %>
-    <% end %>
-  </div>
-
-  <div class="col-6">
-    <%= f.field_container :tenant_id do %>
-      <%= f.label :tenant_id, I18n.t('spree.admin.external_integrations.tenant', default: 'Tenant') %>
-
-      <%= f.select :tenant_id,
-          options_from_collection_for_select(@tenants, :id, :name, f.object.tenant_id),
-          { include_blank: Spree.t(:choose_an_option), prompt: Spree.t(:optional) },
-          class: 'form-control select2',
-          data: { allow_clear: true, placeholder: Spree.t(:optional) } %>
-
-      <%= f.error_message_on :tenant_id %>
-    <% end %>
-  </div>
-
-  <div class="col-6">
-    <%= f.field_container :status do %>
-      <%= f.label :status, raw(Spree.t(:status) + required_span_tag) %>
-
-      <%= f.select :status,
-        options_for_select(@status_options, f.object.status),
-        {},
-        class: 'form-control select2',
-        required: true %>
-
-      <%= f.error_message_on :status %>
-    <% end %>
-  </div>
-
-  <div class="col-6">
-    <%= f.field_container :conflict_strategy do %>
-      <%= f.label :conflict_strategy, raw(I18n.t('spree.admin.external_integrations.conflict_strategy', default: 'Conflict strategy') + required_span_tag) %>
-
-      <%= f.select :conflict_strategy,
-        options_for_select(@conflict_strategy_options, f.object.conflict_strategy),
-        {},
-        class: 'form-control select2',
-        required: true %>
-
-      <%= f.error_message_on :conflict_strategy %>
-    <% end %>
-  </div>
-
-  <div class="col-6">
-    <%= f.field_container :incremental_sync_interval_seconds do %>
-      <%= f.label :incremental_sync_interval_seconds, raw(I18n.t('spree.admin.external_integrations.incremental_sync_interval_seconds', default: 'Incremental sync interval (seconds)') + required_span_tag) %>
-      <%= f.number_field :incremental_sync_interval_seconds, class: 'form-control', min: 1, max: 3600, required: true %>
-      <%= f.error_message_on :incremental_sync_interval_seconds %>
-    <% end %>
-  </div>
-
-  <div class="col-6">
-    <%= f.field_container :full_sync_interval_hours do %>
-      <%= f.label :full_sync_interval_hours, raw(I18n.t('spree.admin.external_integrations.full_sync_interval_hours', default: 'Full sync interval (hours)') + required_span_tag) %>
-      <%= f.number_field :full_sync_interval_hours, class: 'form-control', min: 1, max: 168, required: true %>
-      <%= f.error_message_on :full_sync_interval_hours %>
-    <% end %>
-  </div>
-
-  <% if f.object.type == 'SpreeCmCommissioner::Integrations::StadiumXV1' || (f.object.new_record? && params[:type] == 'SpreeCmCommissioner::Integrations::StadiumXV1') %>
-    <%= render partial: 'stadium_x_v1_fields', locals: { f: f } %>
-  <% end %>
-</div>

data/app/views/spree/admin/integrations/_stadium_x_v1_fields.html.erb

@@ -1,29 +0,0 @@
-<!-- Stadium X V1 API Credentials -->
-<div class="col-12">
-  <hr class="my-4">
-
-  <h6 class="text-muted">
-    <%= I18n.t('spree.admin.external_integrations.stadium_x_credentials', default: 'Stadium X API Credentials') %>
-  </h6>
-</div>
-
-<div class="col-6">
-  <%= f.field_container :public_key do %>
-    <%= f.label :public_key, raw(I18n.t('spree.admin.external_integrations.public_key', default: 'Public Key') + required_span_tag) %>
-    <%= text_field_tag 'spree_cm_commissioner_integration[private_metadata][public_key]', f.object.public_key, class: 'form-control', required: true %>
-  <% end %>
-</div>
-
-<div class="col-6">
-  <%= f.field_container :private_key do %>
-    <%= f.label :private_key, raw(I18n.t('spree.admin.external_integrations.private_key', default: 'Private Key') + required_span_tag) %>
-    <%= text_field_tag 'spree_cm_commissioner_integration[private_metadata][private_key]', f.object.private_key, class: 'form-control', required: true %>
-  <% end %>
-</div>
-
-<div class="col-6">
-  <%= f.field_container :base_url do %>
-    <%= f.label :base_url, raw(I18n.t('spree.admin.external_integrations.base_url', default: 'Base URL') + required_span_tag) %>
-    <%= text_field_tag 'spree_cm_commissioner_integration[private_metadata][base_url]', f.object.base_url, class: 'form-control', placeholder: 'https://api.stadiumx.com', required: true %>
-  <% end %>
-</div>

data/app/views/spree/admin/integrations/edit.html.erb

@@ -1,45 +0,0 @@
-<% content_for :page_title do %>
-  <%= link_to Spree.t('external_integrations'), spree.admin_integrations_url %> / <%= @object.name %>
-<% end %>
-
-<% content_for :page_tabs do %>
-  <li class="nav-item">
-    <%= link_to Spree.t(:edit),
-      edit_admin_integration_path(@object),
-      class: "nav-link active" %>
-  </li>
-
-  <li class="nav-item">
-    <%= link_to Spree.t(:integration_sync_sessions),
-      admin_integration_sessions_path(@object),
-      class: "nav-link" %>
-  </li>
-
-  <li class="nav-item">
-    <%= link_to Spree.t(:integration_mappings),
-      admin_integration_mappings_path(@object),
-      class: "nav-link" %>
-  </li>
-<% end %>
-
-<div data-hook="admin_integrations" class="card mb-3">
-  <div class="card-header">
-    <h5 class="card-title mb-0 h6"><%= Spree.t('external_integrations') %></h5>
-  </div>
-
-  <div class="card-body">
-    <div data-hook="admin_integrations_edit_form_header">
-      <%= render partial: 'spree/admin/shared/error_messages', locals: { target: @object } %>
-    </div>
-
-    <div data-hook="admin_integrations_edit_form">
-      <%= form_with model: @object, url: admin_integration_path(@object), method: :patch, scope: :spree_cm_commissioner_integration do |f| %>
-        <%= render partial: 'form', locals: { f: f } %>
-
-        <div data-hook="admin_integrations_edit_form_buttons">
-          <%= render partial: 'spree/admin/shared/edit_resource_links', locals: { collection_url: spree.admin_integrations_path } %>
-        </div>
-      <% end %>
-    </div>
-  </div>
-</div>

data/app/views/spree/admin/integrations/index.html.erb

@@ -1,75 +0,0 @@
-<% content_for :page_title do %>
-  <%= Spree.t('external_integrations') %>
-<% end %>
-
-<% content_for :page_actions do %>
-  <%= button_link_to Spree.t(:new_external_integration), new_admin_integration_path, class: "btn-success", icon: 'add.svg' %>
-<% end if can? :create, SpreeCmCommissioner::Integration %>
-
-<% if @integrations.any? %>
-  <div class="table-responsive border rounded bg-white mb-3">
-    <table class="table" id="listing_integrations" data-hook>
-      <thead class="text-muted">
-        <tr data-hook="admin_integrations_index_headers">
-          <th><%= Spree.t(:name) %></th>
-          <th><%= Spree.t(:base_url) %></th>
-          <th><%= Spree.t(:conflict_strategy) %></th>
-          <th><%= Spree.t(:incremental_sync_interval_seconds) %></th>
-          <th><%= Spree.t(:full_sync_interval_hours) %></th>
-          <th><%= Spree.t(:vendor) %></th>
-          <th><%= Spree.t(:status) %></th>
-          <th><%= Spree.t(:updated_at) %></th>
-        </tr>
-      </thead>
-
-      <tbody>
-        <% @integrations.each do |integration| %>
-          <tr
-            id="<%= spree_dom_id integration %>"
-            data-hook="admin_integrations_index_rows"
-          >
-            <td><%= integration.name %></td>
-
-            <td>
-              <%= integration_base_url_link(integration.base_url) %>
-            </td>
-
-            <td>
-              <%= integration_conflict_strategy_badge(integration.conflict_strategy) %>
-            </td>
-
-            <td>
-              <%= integration_interval_badge(integration.incremental_sync_interval_seconds, 's') %>
-            </td>
-
-            <td>
-              <%= integration_interval_badge(integration.full_sync_interval_hours, 'h') %>
-            </td>
-
-            <td><%= integration.vendor.name %></td>
-
-            <td>
-              <%= integration_status_badge(integration.status) %>
-            </td>
-
-            <td><%= integration.updated_at %></td>
-
-            <td
-              data-hook="admin_integrations_index_row_actions"
-              class="actions"
-            >
-              <span class="d-flex justify-content-end">
-                <%= link_to_edit integration, url: edit_admin_integration_path(integration), no_text: true if can?(:edit, integration) %>
-                <%= link_to_delete integration, url: admin_integration_path(integration), no_text: true if can?(:delete, integration) %>
-              </span>
-            </td>
-          </tr>
-        <% end %>
-      </tbody>
-    </table>
-  </div>
-<% else %>
-  <small class="form-text text-muted">
-    <%= raw I18n.t('external_integrations.empty_info') %>
-  </small>
-<% end %>

data/app/views/spree/admin/integrations/new.html.erb

@@ -1,25 +0,0 @@
-<% content_for :page_title do %>
-  <%= link_to Spree.t('external_integrations'), spree.admin_integrations_url %> / <%= Spree.t(:new) %>
-<% end %>
-
-<div data-hook="admin_integrations" class="card mb-3">
-  <div class="card-header">
-    <h5 class="card-title mb-0 h6"><%= Spree.t('external_integrations') %></h5>
-  </div>
-
-  <div class="card-body">
-    <div data-hook="admin_integrations_new_form_header">
-      <%= render partial: 'spree/admin/shared/error_messages', locals: { target: @object } %>
-    </div>
-
-    <div data-hook="admin_integrations_new_form">
-      <%= form_with model: @object, url: { action: 'create' }, scope: :spree_cm_commissioner_integration do |f| %>
-        <%= render partial: 'form', locals: { f: f } %>
-
-        <div data-hook="admin_integrations_new_form_buttons">
-          <%= render partial: 'spree/admin/shared/new_resource_links' %>
-        </div>
-      <% end %>
-    </div>
-  </div>
-</div>

data/db/migrate/20251017094845_create_cm_integrations.rb

@@ -1,22 +0,0 @@
-class CreateCmIntegrations < ActiveRecord::Migration[7.0]
-  def change
-    create_table :cm_integrations, if_not_exists: true do |t|
-      t.string :type, null: false
-      t.string :name, null: false
-
-      t.integer :status, null: false, default: 0
-      t.integer :conflict_strategy, null: false, default: 0
-
-      t.integer :incremental_sync_interval_seconds, null: false, default: 10
-      t.integer :full_sync_interval_hours, null: false, default: 24
-
-      t.references :tenant, foreign_key: { to_table: :cm_tenants }, null: true
-      t.references :vendor, foreign_key: { to_table: :spree_vendors }, null: false
-
-      t.jsonb :public_metadata, default: {}
-      t.jsonb :private_metadata, default: {}
-
-      t.timestamps
-    end
-  end
-end

data/db/migrate/20251017101555_create_cm_integration_sync_sessions.rb

@@ -1,68 +0,0 @@
-class CreateCmIntegrationSyncSessions < ActiveRecord::Migration[7.0]
-  def change
-    create_table :cm_integration_sync_sessions, if_not_exists: true do |t|
-      t.integer :status, null: false
-      t.integer :sync_type, null: false
-
-      t.datetime :started_at
-      t.datetime :ended_at
-
-      t.references :integration, foreign_key: { to_table: :cm_integrations }, null: false
-      t.references :tenant, foreign_key: { to_table: :cm_tenants }, null: true
-
-      # Stores the serialized sync result (SyncResult#to_h) for this session.
-      # This field is used to persist the outcome of the sync operation,
-      # allowing for detailed tracking of results, whether the sync was successful or not.
-      #
-      # Base class: app/services/spree_cm_commissioner/integrations/base/sync_result.rb
-      #
-      # Example implementations include:
-      # - Full sync results: integrations/stadium_x_v1/sync_results/full_sync_result.rb
-      # - Incremental sync results: integrations/stadium_x_v1/sync_results/incremental_sync_result.rb
-      #
-      # The result is saved upon completion of the session in the SyncManager#run_execution method:
-      # - app/services/spree_cm_commissioner/integrations/base/sync_manager.rb
-      t.jsonb :sync_result, default: {}
-
-      t.timestamps
-    end
-
-    # Indexes for PollingScheduler queries (optimized for performance)
-    # Initially used in polling_scheduler.rb
-    #
-    # Index 1: Find in_progress/pending syncs created within last hour
-    # Used by: should_run_full_sync? and should_run_incremental_sync?
-    # Query: .where(sync_type: [...], status: [...]).where('created_at > ?', 1.hour.ago).exists?
-    # Why: Composite index on all filter columns for fast lookup of active syncs
-    unless index_exists?(:cm_integration_sync_sessions, [:integration_id, :sync_type, :status, :created_at], name: 'idx_sync_sessions_integration_type_status_created_at')
-      add_index :cm_integration_sync_sessions, [:integration_id, :sync_type, :status, :created_at],
-                name: 'idx_sync_sessions_integration_type_status_created_at'
-    end
-
-    # Index 2: Find latest completed full sync (partial index - only completed rows)
-    # Used by: should_run_full_sync?
-    # Query: .where(sync_type: :full, status: :completed).order(created_at: :desc).first
-    # Why: Partial index reduces size by ~60% (excludes in_progress/pending/failed rows)
-    #      Descending order on created_at allows DB to return latest without sorting
-    #      Only 2 columns needed since sync_type and status are filtered by WHERE clause
-    unless index_exists?(:cm_integration_sync_sessions, [:integration_id, :created_at], name: 'idx_sync_sessions_full_completed')
-      add_index :cm_integration_sync_sessions, [:integration_id, :created_at],
-                name: 'idx_sync_sessions_full_completed',
-                where: "sync_type = 0 AND status = 2", # full=0, completed=2
-                order: { created_at: :desc }
-    end
-
-    # Index 3: Find latest completed incremental sync (partial index - only completed rows)
-    # Used by: should_run_incremental_sync?
-    # Query: .where(sync_type: :incremental, status: :completed).order(created_at: :desc).first
-    # Why: Partial index reduces size by ~60% (excludes in_progress/pending/failed rows)
-    #      Descending order on created_at allows DB to return latest without sorting
-    #      Only 2 columns needed since sync_type and status are filtered by WHERE clause
-    unless index_exists?(:cm_integration_sync_sessions, [:integration_id, :created_at], name: 'idx_sync_sessions_incremental_completed')
-      add_index :cm_integration_sync_sessions, [:integration_id, :created_at],
-                name: 'idx_sync_sessions_incremental_completed',
-                where: "sync_type = 1 AND status = 2", # incremental=1, completed=2
-                order: { created_at: :desc }
-    end
-  end
-end

data/db/migrate/20251017101605_create_cm_integration_mappings.rb

@@ -1,52 +0,0 @@
-class CreateCmIntegrationMappings < ActiveRecord::Migration[7.0]
-  def change
-    create_table :cm_integration_mappings, if_not_exists: true do |t|
-      t.string :external_id, null: false
-      t.string :internal_type, null: false
-      t.bigint :internal_id, null: false
-      t.jsonb :external_payload, default: nil
-
-      # optional for record that can have different mappings on different dates
-      t.date :date, null: true
-      t.integer :status, null: false, default: 0
-      t.references :integration, null: false, foreign_key: { to_table: :cm_integrations }
-
-      t.jsonb :public_metadata, default: {}
-      t.jsonb :private_metadata, default: {}
-
-      t.datetime :last_synced_at
-      t.timestamps
-    end
-
-    # Index 1: Unique constraint for external_id mappings
-    # Used by: IntegrationMapping#validates :external_id
-    # Query: validates :external_id, uniqueness: { scope: %i[integration_id internal_type internal_id date] }
-    # Why: Ensures a single external_id maps to only one internal record per integration/date combination
-    #      Prevents duplicate mappings that could cause sync conflicts
-    #      Unique index enforces this at database level for data integrity
-    unless index_exists?(:cm_integration_mappings, [:integration_id, :external_id, :internal_type, :internal_id, :date], name: 'idx_external_internal_mapping')
-      add_index :cm_integration_mappings, [:integration_id, :external_id, :internal_type, :internal_id, :date], unique: true, name: 'idx_external_internal_mapping'
-    end
-
-    # Index 2: Efficient lookup by integration_id and external_id
-    # Used by: IntegrationMappable#find_by_integration, IntegrationMappable#find_or_initialize_by_integration,
-    # IntegrationMappable#find_or_initialize_by_integration_with_mapping
-    #
-    # Query: .find_by(integration_id: integration_id, external_id: external_id)
-    # Why: Composite index allows efficient lookup by both integration_id and external_id
-    #      These methods are called frequently during sync operations
-    unless index_exists?(:cm_integration_mappings, [:integration_id, :internal_type, :external_id], name: 'idx_integration_external_id')
-      add_index :cm_integration_mappings, [:integration_id, :internal_type, :external_id], name: 'idx_integration_external_id'
-    end
-
-    # Index 3: Incremental sync query optimization
-    # Used by: IntegrationMappable#find_oldest_active_mapping (called by IncrementalSyncStrategy every 10 seconds)
-    # Query: .where(integration_id: integration_id, internal_type: model_name).active.order(:last_synced_at).first
-    # Why: Composite index filters by integration_id, internal_type, status efficiently
-    #      Includes last_synced_at for ORDER BY to avoid expensive sort operation
-    #      Runs every 10s so performance is critical - even small optimizations matter at this frequency
-    unless index_exists?(:cm_integration_mappings, [:integration_id, :internal_type, :status, :last_synced_at], name: 'idx_incremental_sync')
-      add_index :cm_integration_mappings, [:integration_id, :internal_type, :status, :last_synced_at], name: 'idx_incremental_sync'
-    end
-  end
-end

data/lib/spree_cm_commissioner/test_helper/factories/integration_factory.rb

@@ -1,25 +0,0 @@
-FactoryBot.define do
-  factory :cm_integration, class: SpreeCmCommissioner::Integration do
-    type { 'SpreeCmCommissioner::Integrations::StadiumXV1' }
-    name { FFaker::Company.name }
-    status { :active }
-    conflict_strategy { :newest_wins }
-    incremental_sync_interval_seconds { 300 }
-    full_sync_interval_hours { 24 }
-    association :vendor, factory: :cm_vendor
-
-    trait :active do
-      status { :active }
-    end
-
-    trait :inactive do
-      status { :inactive }
-    end
-  end
-
-  factory :cm_stadium_x_v1_integration, parent: :cm_integration, class: SpreeCmCommissioner::Integrations::StadiumXV1 do
-    public_key { FFaker::Lorem.characters(20) }
-    private_key { FFaker::Lorem.characters(40) }
-    base_url { 'https://api.stadiumx.com' }
-  end
-end

data/lib/spree_cm_commissioner/test_helper/factories/integration_mapping_factory.rb

@@ -1,14 +0,0 @@
-FactoryBot.define do
-  factory :cm_integration_mapping, class: SpreeCmCommissioner::IntegrationMapping do
-    external_id { SecureRandom.alphanumeric(10) }
-    association :integration, factory: :cm_integration
-    status { :active }
-
-    transient do
-      internal { create(:cm_product) }
-    end
-
-    internal_id { internal.id }
-    internal_type { internal.class.name }
-  end
-end

data/lib/spree_cm_commissioner/test_helper/factories/integration_sync_session_factory.rb

@@ -1,7 +0,0 @@
-FactoryBot.define do
-  factory :cm_integration_sync_session, class: SpreeCmCommissioner::IntegrationSyncSession do
-    status { :pending }
-    sync_type { :full }
-    association :integration, factory: :cm_integration
-  end
-end