tapsoob 0.8.5 → 0.8.7

data/spec/unit/tapsoob/schema_spec.rb

@@ -2,6 +2,20 @@ require 'spec_helper'
 require 'tapsoob/schema'
 
 RSpec.describe Tapsoob::Schema do
+  # SQLite file-based URL helpers — memory: URLs open a fresh DB on each connect,
+  # so methods that open their own connection (dump, foreign_keys, indexes, …)
+  # must use a file-backed database.
+  def with_sqlite_file
+    path = File.join(Dir.tmpdir, "tapsoob_schema_#{Process.pid}_#{rand(9999)}.db")
+    url  = DbHelpers.adapt_url("sqlite://#{path}")
+    db   = DbHelpers.connect(url)
+    db.extension :schema_dumper
+    yield url, db
+  ensure
+    DbHelpers.disconnect_all
+    File.delete(path) rescue nil
+  end
+
   let(:db) do
     d = connect_sqlite
     d.extension :schema_dumper
@@ -99,4 +113,144 @@ RSpec.describe Tapsoob::Schema do
       }.not_to raise_error
     end
   end
+
+  # ── dump ─────────────────────────────────────────────────────────────────────
+
+  describe '.dump' do
+    it 'returns a migration string covering all tables' do
+      with_sqlite_file do |url, tmp|
+        tmp.create_table(:things) { primary_key :id; String :name }
+        result = described_class.dump(url)
+        expect(result).to include('Sequel::Migration')
+        expect(result).to include('things')
+      end
+    end
+
+    it 'includes both up and down blocks' do
+      with_sqlite_file do |url, tmp|
+        tmp.create_table(:items) { primary_key :id }
+        result = described_class.dump(url)
+        expect(result).to include('def up')
+        expect(result).to include('def down')
+      end
+    end
+  end
+
+  # ── foreign_keys ─────────────────────────────────────────────────────────────
+
+  describe '.foreign_keys' do
+    it 'returns a string (even when there are no FK constraints)' do
+      with_sqlite_file do |url, _|
+        result = described_class.foreign_keys(url)
+        expect(result).to be_a(String)
+      end
+    end
+  end
+
+  # ── indexes ──────────────────────────────────────────────────────────────────
+
+  describe '.indexes' do
+    it 'returns a string' do
+      with_sqlite_file do |url, tmp|
+        tmp.create_table(:idx_things) { primary_key :id; String :slug }
+        tmp.add_index(:idx_things, :slug)
+        result = described_class.indexes(url)
+        expect(result).to be_a(String)
+      end
+    end
+  end
+
+  # ── load via URL ─────────────────────────────────────────────────────────────
+
+  describe '.load (URL path)' do
+    it 'creates the table when passed a URL string' do
+      with_sqlite_file do |url, tmp|
+        schema_str = described_class.dump_table(db, :articles, {})
+        described_class.load(url, schema_str)
+        expect(tmp.table_exists?(:articles)).to be true
+      end
+    end
+
+    it 'drops then recreates table when drop: true and passed a URL' do
+      with_sqlite_file do |url, tmp|
+        schema_str = described_class.dump_table(db, :articles, {})
+        described_class.load(url, schema_str)
+        described_class.load(url, schema_str, drop: true)
+        expect(tmp.table_exists?(:articles)).to be true
+      end
+    end
+  end
+
+  # ── load_indexes ─────────────────────────────────────────────────────────────
+
+  describe '.load_indexes' do
+    it 'applies an index migration without error' do
+      with_sqlite_file do |url, tmp|
+        tmp.create_table(:things) { primary_key :id; String :slug }
+        index_migration = <<~RUBY
+          Class.new(Sequel::Migration) do
+            def up
+              add_index :things, :slug
+            end
+          end
+        RUBY
+        expect { described_class.load_indexes(url, index_migration) }.not_to raise_error
+        expect(tmp.indexes(:things)).to have_key(:things_slug_index)
+      end
+    end
+  end
+
+  # ── load_foreign_keys ────────────────────────────────────────────────────────
+
+  describe '.load_foreign_keys' do
+    it 'applies a foreign key migration without error' do
+      with_sqlite_file do |url, tmp|
+        tmp.create_table(:parents) { primary_key :id }
+        tmp.create_table(:children) { primary_key :id; Integer :parent_id }
+        fk_migration = <<~RUBY
+          Class.new(Sequel::Migration) do
+            def up
+              alter_table(:children) { add_foreign_key [:parent_id], :parents }
+            end
+          end
+        RUBY
+        expect { described_class.load_foreign_keys(url, fk_migration) }.not_to raise_error
+      end
+    end
+  end
+
+  # ── rewrite_non_integer_primary_keys ─────────────────────────────────────────
+
+  describe '.rewrite_non_integer_primary_keys' do
+    it 'leaves integer primary keys unchanged' do
+      schema = '  primary_key :id, :type=>"integer"'
+      expect(described_class.rewrite_non_integer_primary_keys(schema)).to eq(schema)
+    end
+
+    it 'leaves bigint primary keys unchanged' do
+      schema = '  primary_key :id, :type=>"bigint"'
+      expect(described_class.rewrite_non_integer_primary_keys(schema)).to eq(schema)
+    end
+
+    it 'rewrites varchar primary keys to column form' do
+      schema = '  primary_key :code, :type=>"varchar(10)"'
+      result = described_class.rewrite_non_integer_primary_keys(schema)
+      expect(result).to include('column :code')
+      expect(result).to include('"varchar(10)"')
+      expect(result).to include('primary_key: true')
+      expect(result).not_to include('primary_key :code,')
+    end
+
+    it 'rewrites uuid primary keys to column form' do
+      schema = '  primary_key :id, :type=>"uuid"'
+      result = described_class.rewrite_non_integer_primary_keys(schema)
+      expect(result).to include('column :id')
+      expect(result).to include('primary_key: true')
+    end
+
+    it 'passes through schema with no primary_key lines unchanged' do
+      schema = "  String :name, size: 50\n  Integer :score"
+      expect(described_class.rewrite_non_integer_primary_keys(schema)).to eq(schema)
+    end
+  end
 end

data/spec/unit/tapsoob/utils_spec.rb

@@ -220,6 +220,70 @@ RSpec.describe Tapsoob::Utils do
     end
   end
 
+  # ── windows? / bin ───────────────────────────────────────────────────────────
+
+  describe '.windows? / .bin' do
+    it 'returns a boolean for windows?' do
+      expect([true, false]).to include(described_class.windows?)
+    end
+
+    it 'returns the command unchanged on non-Windows' do
+      allow(described_class).to receive(:windows?).and_return(false)
+      expect(described_class.bin("mytool")).to eq("mytool")
+    end
+
+    it 'appends .cmd on Windows' do
+      allow(described_class).to receive(:windows?).and_return(true)
+      expect(described_class.bin("mytool")).to eq("mytool.cmd")
+    end
+  end
+
+  # ── incorrect_blobs ──────────────────────────────────────────────────────────
+
+  describe '.incorrect_blobs' do
+    let(:db) do
+      d = connect_sqlite
+      d.create_table(:blob_test) { primary_key :id; File :data; String :name }
+      d
+    end
+    after { db.disconnect }
+
+    it 'returns blob-typed column names' do
+      expect(described_class.incorrect_blobs(db, :blob_test)).to include(:data)
+    end
+
+    it 'does not include non-blob columns' do
+      expect(described_class.incorrect_blobs(db, :blob_test)).not_to include(:name)
+    end
+  end
+
+  # ── encode_blobs ASCII-8BIT branch ──────────────────────────────────────────
+
+  describe '.encode_blobs (ASCII-8BIT branch)' do
+    it 'encodes raw binary strings that are not Sequel::SQL::Blob' do
+      raw = "binary".force_encoding(Encoding::ASCII_8BIT)
+      row = { data: raw }
+      described_class.encode_blobs(row, [:data])
+      expect(row[:data]).to eq(described_class.base64encode(raw))
+    end
+  end
+
+  # ── load_schema with a DB object ─────────────────────────────────────────────
+
+  describe '.load_schema' do
+    let(:dir) { Dir.mktmpdir }
+    after { FileUtils.rm_rf(dir) }
+
+    it 'accepts a Sequel::Database object directly' do
+      db = connect_sqlite
+      FileUtils.mkdir_p(File.join(dir, 'schemas'))
+      schema_content = "Sequel.migration { change { create_table(:load_test) { primary_key :id } } }"
+      File.write(File.join(dir, 'schemas', 'load_test.rb'), schema_content)
+      expect { described_class.load_schema(dir, db, :load_test) }.not_to raise_error
+      db.disconnect
+    end
+  end
+
   # ── export_rows / export_schema / export_indexes (filesystem) ────────────────
 
   describe 'filesystem export helpers' do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: tapsoob
 version: !ruby/object:Gem::Version
-  version: 0.8.5
+  version: 0.8.7
 platform: ruby
 authors:
 - Félix Bellanger
@@ -131,11 +131,21 @@ files:
 - spec/spec_helper.rb
 - spec/support/db_helpers.rb
 - spec/support/fixtures.rb
+- spec/support/operation_helpers.rb
 - spec/support/round_trip_helper.rb
 - spec/support/shared_examples/round_trip.rb
+- spec/unit/tapsoob/base_spec.rb
 - spec/unit/tapsoob/chunksize_spec.rb
+- spec/unit/tapsoob/cli_pipeline_spec.rb
+- spec/unit/tapsoob/config_spec.rb
 - spec/unit/tapsoob/data_stream_spec.rb
+- spec/unit/tapsoob/file_partition_spec.rb
+- spec/unit/tapsoob/keyed_spec.rb
 - spec/unit/tapsoob/operation_base_spec.rb
+- spec/unit/tapsoob/progress_event_spec.rb
+- spec/unit/tapsoob/progress_spec.rb
+- spec/unit/tapsoob/pull_spec.rb
+- spec/unit/tapsoob/push_spec.rb
 - spec/unit/tapsoob/schema_spec.rb
 - spec/unit/tapsoob/utils_spec.rb
 - spec/unit/tapsoob/version_spec.rb