spree_cm_commissioner 2.5.0.pre.pre8 → 2.5.0.pre.pre10

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

@@ -0,0 +1,29 @@
+<!-- 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

@@ -0,0 +1,45 @@
+<% 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

@@ -0,0 +1,75 @@
+<% 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

@@ -0,0 +1,25 @@
+<% 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/bin/run_spec_group

@@ -0,0 +1,101 @@
+#!/usr/bin/env ruby
+
+# ============================================================================
+# Spec Group Runner - Parallel Test Execution
+# ============================================================================
+#
+# PURPOSE:
+#   Split all spec files into N equal groups and run a specific group.
+#   Used for parallel test execution in CI (GitHub Actions) and locally.
+#
+# HOW IT WORKS:
+#   1. Finds all *_spec.rb files in spec/ directory
+#   2. Sorts them alphabetically for consistent ordering
+#   3. Divides them into N equal groups (e.g., 10 groups)
+#   4. Runs RSpec on the files in the specified group
+#
+# USAGE:
+#   bin/run_spec_group <group_number> <total_groups>
+#
+# EXAMPLES:
+#   bin/run_spec_group 1 10   # Run group 1 out of 10 (files 1-15 if 150 total)
+#   bin/run_spec_group 5 10   # Run group 5 out of 10 (files 61-75 if 150 total)
+#   bin/run_spec_group 1 3    # Run first third of all specs
+#
+# CI USAGE (GitHub Actions):
+#   - Matrix strategy runs 10-12 parallel jobs (groups 1-12)
+#   - Each job runs ~1/10th or ~1/12th of total specs
+#   - Total test time = slowest group time (not sum of all)
+#   - Example: 150 specs × 2s each = 300s sequential
+#              150 specs ÷ 10 groups = 15 specs × 2s = 30s parallel
+#              150 specs ÷ 12 groups = 13 specs × 2s = 26s parallel
+#
+# LOCAL USAGE:
+#   - Run specific group to debug failing tests
+#   - Run multiple groups in parallel terminals
+#   - Faster iteration than running all specs
+#
+# OUTPUT:
+#   - Shows total spec count
+#   - Shows files per group
+#   - Lists all files in current group
+#   - Runs RSpec with those files
+#
+# WHERE USED:
+#   - .github/workflows/test_and_build_gem.yml (test job, matrix strategy)
+#   - Can be run locally for debugging
+#
+# ============================================================================
+
+group_num = ARGV[0]&.to_i
+total_groups = ARGV[1]&.to_i
+
+unless group_num && group_num > 0 && group_num <= total_groups
+  puts "Usage: bin/run_spec_group <group_number> [total_groups]"
+  puts "Example: bin/run_spec_group 1 10"
+  puts "  Runs group 1 out of 10 total groups"
+  exit 1
+end
+
+spec_dir = File.expand_path('../spec', __dir__)
+
+# Find all spec files and sort for consistent ordering
+all_spec_files = Dir.glob("#{spec_dir}/**/*_spec.rb").sort
+
+if all_spec_files.empty?
+  puts "❌ No spec files found in #{spec_dir}"
+  exit 1
+end
+
+# Calculate files per group
+files_per_group = (all_spec_files.length.to_f / total_groups).ceil
+
+# Get the files for this specific group
+start_idx = (group_num - 1) * files_per_group
+end_idx = [start_idx + files_per_group - 1, all_spec_files.length - 1].min
+
+if start_idx >= all_spec_files.length
+  puts "⚠️  Group #{group_num} has no spec files (total: #{all_spec_files.length} files across #{total_groups} groups)"
+  exit 0
+end
+
+group_files = all_spec_files[start_idx..end_idx]
+
+puts "=" * 80
+puts "Running Group #{group_num}/#{total_groups}"
+puts "Total spec files: #{all_spec_files.length}"
+puts "Files per group: ~#{files_per_group}"
+puts "This group: #{group_files.length} files (#{start_idx + 1}-#{end_idx + 1})"
+puts "=" * 80
+puts
+puts "📋 Files in this group:"
+group_files.each_with_index do |file, idx|
+  relative_path = file.sub("#{spec_dir}/", '')
+  puts "  #{start_idx + idx + 1}. #{relative_path}"
+end
+puts
+puts "=" * 80
+puts
+
+# Run rspec with the specific files
+exec("bundle", "exec", "rspec", *group_files)

data/config/locales/en.yml

@@ -454,6 +454,14 @@ en:
     admin:
       display_on:
         frontend_for_early_adopter: "Storefront for Early Adopter"
+      external_integrations_title: "External Integrations"
+      integration_sync_enqueued: "Integration sync has been enqueued"
+      integration_inactive: "Integration is inactive"
+      external_integrations:
+        view_result: "View Result"
+        sync_result: "Sync Result"
+        external_payload: "External Payload"
+        no_results: "No results found"
     event:
       check_in:
         success: "Guest check-in in successfully"

data/config/locales/km.yml

@@ -341,6 +341,14 @@ km:
     admin:
       display_on:
         frontend_for_early_adopter: "Storefront for Early Adopter"
+      external_integrations_title: "ការភ្ជាប់ខាងក្រៅ"
+      integration_sync_enqueued: "ការធ្វើសមកាលកម្មបានដាក់ក្នុងជួរ"
+      integration_inactive: "ការភ្ជាប់មិនដំណើរការ"
+      external_integrations:
+        view_result: "មើលលទ្ធផល"
+        sync_result: "លទ្ធផលធ្វើសមកាលកម្ម"
+        external_payload: "ទិន្នន័យខាងក្រៅ"
+        no_results: "មិនមានលទ្ធផលទេ"
     event:
       check_in:
         success: "Guest check-in in successfully"

data/config/routes.rb

@@ -283,6 +283,13 @@ Spree::Core::Engine.add_routes do
         post :set_webhook
       end
     end
+    resources :integrations do
+      member do
+        post :enqueue_polling
+      end
+      resources :sessions, only: [:index], controller: 'integration_sessions'
+      resources :mappings, only: [:index], controller: 'integration_mappings'
+    end
 
     resources :webhooks_subscribers do
       resources :rules, controller: :webhooks_subscriber_rules, except: %i[index show]
@@ -695,6 +702,7 @@ Spree::Core::Engine.add_routes do
 
         resources :seat_layouts, only: %i[show index]
         resources :inventory_items, only: %i[index]
+        resources :event_matches, only: %i[index]
         namespace :transit do
           resources :draft_orders, only: %i[create]
         end

data/db/migrate/20251017094845_create_cm_integrations.rb

@@ -0,0 +1,22 @@
+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

@@ -0,0 +1,68 @@
+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

@@ -0,0 +1,52 @@
+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/cm_app_logger.rb

@@ -1,17 +1,18 @@
 # lib/cm_app_logger.rb
 module CmAppLogger
   def self.log(label:, data: nil)
-    message = { label: label, data: data }
+    message = { label: label, data: safe_serialize(data) }
     start_time = Time.current
     Rails.logger.info(message.to_json)
 
     return unless block_given?
 
-    # Capture the block’s return value and return it to preserve existing behavior for callers expecting that value.
+    # Capture the block's return value and return it to preserve existing behavior for callers expecting that value.
     block_result = yield
 
-    message[:start_time] = start_time
+    message[:start_time] = start_time.iso8601(3)
     message[:duration_ms] = (Time.current - start_time) * 1000
+    message[:result] = safe_serialize(block_result)
     Rails.logger.info(message.to_json)
     block_result
   end
@@ -19,9 +20,40 @@ module CmAppLogger
   def self.error(label:, data: nil)
     message = {
       label: label,
-      data: data
+      data: safe_serialize(data)
     }
 
     Rails.logger.error(message.to_json)
   end
+
+  # Safely serializes objects for JSON logging.
+  #
+  # @param obj [Object] The object to serialize
+  # @param depth [Integer] Internal parameter tracking recursion depth (max: 50)
+  # @return [Object] A JSON-safe representation of the input object
+  def self.safe_serialize(obj, depth: 0) # rubocop:disable Metrics/CyclomaticComplexity,Metrics/PerceivedComplexity
+    return '[Max Depth Exceeded]' if depth > 50
+
+    if obj.is_a?(Hash)
+      obj.each_with_object({}) do |(k, v), memo|
+        memo[safe_serialize(k, depth: depth + 1)] = safe_serialize(v, depth: depth + 1)
+      end
+    elsif obj.is_a?(Array)
+      obj.map { |item| safe_serialize(item, depth: depth + 1) }
+    elsif obj.is_a?(Date)
+      obj.iso8601
+    elsif obj.is_a?(Time) || obj.is_a?(DateTime) || obj.is_a?(ActiveSupport::TimeWithZone)
+      obj.iso8601(3)
+    elsif obj.is_a?(ActiveJob::Base)
+      {
+        job_class: obj.class.name,
+        job_id: obj.job_id,
+        arguments: safe_serialize(obj.arguments, depth: depth + 1)
+      }
+    elsif obj.respond_to?(:id)
+      { class: obj.class.name, id: obj.id }
+    else
+      obj.to_s
+    end
+  end
 end

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

@@ -0,0 +1,25 @@
+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

@@ -0,0 +1,14 @@
+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

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

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

@@ -5,6 +5,7 @@ FactoryBot.define do
   end
 
   factory :cm_vendor, parent: :vendor do
+    sequence(:name) { |n| "#{FFaker::Company.name} #{n}#{Kernel.rand(9999)}" }
     state { :active }
     default_state_id { Spree::State.first&.id }
     primary_product_type { :ecommerce }

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

@@ -1,6 +1,6 @@
 FactoryBot.define do
   factory :cm_location_vendor_place, class: SpreeCmCommissioner::VendorPlace do
-    association :vendor, factory: :vendor
+    association :vendor, factory: :cm_vendor
     association :place, factory: :cm_place
 
     distance  { FFaker::Number.decimal }
@@ -10,7 +10,7 @@ FactoryBot.define do
   end
 
   factory :cm_vendor_place, class: SpreeCmCommissioner::VendorPlace do
-    association :vendor, factory: :vendor
+    association :vendor, factory: :cm_vendor
     association :place, factory: :cm_place
     association :location, factory: :cm_location_vendor_place
 
@@ -28,6 +28,7 @@ FactoryBot.define do
 
     trait :location do
       place_type { :location }
+      location { nil }
     end
   end
 end

data/lib/spree_cm_commissioner/version.rb

@@ -1,5 +1,5 @@
 module SpreeCmCommissioner
-  VERSION = '2.5.0.pre.pre8'.freeze
+  VERSION = '2.5.0.pre.pre10'.freeze
 
   module_function
 

data/lib/spree_cm_commissioner.rb

@@ -55,16 +55,17 @@ require 'byebug' if Rails.env.development? || Rails.env.test?
 
 module SpreeCmCommissioner
   class << self
-    # Allows overriding the default Redis connection pool with a custom one
-    attr_writer :redis_pool
+    # Allows overriding the default Redis connection pools with custom ones
+    attr_writer :inventory_redis_pool
 
-    def redis_pool
-      @redis_pool ||= default_redis_pool
+    # Inventory Redis pool for inventory management
+    def inventory_redis_pool
+      @inventory_redis_pool ||= default_redis_pool
     end
 
-    # Resets the Redis pool, useful for testing or reinitialization
-    def reset_redis_pool
-      @redis_pool = nil
+    # Resets all Redis pools, useful for testing or reinitialization
+    def reset_redis_pools
+      @inventory_redis_pool = nil
     end
 
     private