Rename YamlDumper to Compiler, generate/dump to compile

Author: bradgessler

lib/fixturebot.rb

@@ -7,7 +7,7 @@
 require_relative "fixturebot/row"
 require_relative "fixturebot/definition"
 require_relative "fixturebot/fixture_set"
-require_relative "fixturebot/yaml_dumper"
+require_relative "fixturebot/compiler"
 require_relative "fixturebot/cli"
 
 module FixtureBot

lib/fixturebot/cli.rb

@@ -8,7 +8,28 @@ def self.exit_on_failure?
       true
     end
 
-    desc "show DIR", "Evaluate DIR/schema.rb and DIR/fixtures.rb, then print fixture YAML to stdout"
+    desc "compile DIR", "Compile DIR/schema.rb and DIR/fixtures.rb to YAML fixture files"
+    def compile(dir)
+      schema_path = File.join(dir, "schema.rb")
+      fixtures_path = File.join(dir, "fixtures.rb")
+
+      raise Thor::Error, "Schema file not found: #{schema_path}" unless File.exist?(schema_path)
+      raise Thor::Error, "Fixtures file not found: #{fixtures_path}" unless File.exist?(fixtures_path)
+
+      schema = eval(File.read(schema_path), binding, schema_path, 1)
+      fixture_set = FixtureBot.define_from_file(schema, fixtures_path)
+
+      output_dir = File.join(dir, "fixtures")
+      Compiler.new(fixture_set).compile(output_dir)
+
+      say "Compiled fixtures to #{output_dir}/"
+      fixture_set.tables.each do |table_name, records|
+        next if records.empty?
+        say "  #{table_name}.yml (#{records.size} records)"
+      end
+    end
+
+    desc "show DIR", "Compile DIR/schema.rb and DIR/fixtures.rb, then print YAML to stdout"
     def show(dir)
       schema_path = File.join(dir, "schema.rb")
       fixtures_path = File.join(dir, "fixtures.rb")
@@ -18,12 +39,12 @@ def show(dir)
 
       schema = eval(File.read(schema_path), binding, schema_path, 1)
       fixture_set = FixtureBot.define_from_file(schema, fixtures_path)
-      dumper = FixtureBot::YamlDumper.new(fixture_set)
+      compiler = FixtureBot::Compiler.new(fixture_set)
 
       fixture_set.tables.each do |table_name, records|
         next if records.empty?
         puts "# #{table_name}.yml"
-        puts dumper.dump_table(table_name)
+        puts compiler.compile_table(table_name)
       end
     end
 

lib/fixturebot/yaml_dumper.rb lib/fixturebot/compiler.rblib/fixturebot/yaml_dumper.rb renamed to lib/fixturebot/compiler.rb

@@ -4,21 +4,21 @@
 require "fileutils"
 
 module FixtureBot
-  class YamlDumper
+  class Compiler
     def initialize(fixture_set)
       @fixture_set = fixture_set
     end
 
-    def dump(output_dir)
+    def compile(output_dir)
       FileUtils.mkdir_p(output_dir)
       @fixture_set.tables.each do |table_name, records|
         next if records.empty?
         path = File.join(output_dir, "#{table_name}.yml")
-        File.write(path, dump_table(table_name))
+        File.write(path, compile_table(table_name))
       end
     end
 
-    def dump_table(table_name)
+    def compile_table(table_name)
       records = @fixture_set.tables[table_name]
       hash = {}
       records.each do |record_name, columns|

lib/fixturebot/minitest.rb

@@ -2,4 +2,4 @@
 
 require "fixturebot/rails"
 
-FixtureBot::Rails.generate
+FixtureBot::Rails.compile

lib/fixturebot/rails.rb

@@ -7,15 +7,15 @@
 
 module FixtureBot
   module Rails
-    def self.generate(fixtures_file: nil, output_dir: nil)
+    def self.compile(fixtures_file: nil, output_dir: nil)
       fixtures_file = resolve_fixtures_file(fixtures_file)
       return unless fixtures_file && File.exist?(fixtures_file)
 
       output_dir = resolve_output_dir(output_dir, fixtures_file)
 
       schema = SchemaLoader.load
       fixture_set = FixtureBot.define_from_file(schema, fixtures_file)
-      YamlDumper.new(fixture_set).dump(output_dir)
+      Compiler.new(fixture_set).compile(output_dir)
     end
 
     def self.resolve_fixtures_file(explicit)

lib/fixturebot/rails/cli.rb

@@ -5,17 +5,17 @@
 module FixtureBot
   module Rails
     class CLI < FixtureBot::CLI
-      desc "generate FIXTURES_FILE [OUTPUT_DIR]", "Generate YAML fixtures from a DSL file"
-      def generate(fixtures_path, output_dir = "test/fixtures")
+      desc "compile FIXTURES_FILE [OUTPUT_DIR]", "Compile FixtureBot DSL to YAML fixture files"
+      def compile(fixtures_path, output_dir = "test/fixtures")
         unless File.exist?(fixtures_path)
           raise Thor::Error, "Fixtures file not found: #{fixtures_path}"
         end
 
         schema = SchemaLoader.load
         fixture_set = FixtureBot.define_from_file(schema, fixtures_path)
-        YamlDumper.new(fixture_set).dump(output_dir)
+        Compiler.new(fixture_set).compile(output_dir)
 
-        say "Generated fixtures in #{output_dir}/"
+        say "Compiled fixtures to #{output_dir}/"
         fixture_set.tables.each do |table_name, records|
           next if records.empty?
           say "  #{table_name}.yml (#{records.size} records)"

lib/fixturebot/rails/railtie.rb

@@ -7,9 +7,9 @@ class Railtie < ::Rails::Railtie
 
       rake_tasks do
         namespace :fixturebot do
-          desc "Generate YAML fixture files from FixtureBot DSL"
-          task generate: :environment do
-            FixtureBot::Rails.generate
+          desc "Compile FixtureBot DSL to YAML fixture files"
+          task compile: :environment do
+            FixtureBot::Rails.compile
           end
         end
       end

lib/fixturebot/rspec.rb

@@ -4,6 +4,6 @@
 
 RSpec.configure do |config|
   config.before(:suite) do
-    FixtureBot::Rails.generate
+    FixtureBot::Rails.compile
   end
 end

spec/fixturebot/yaml_dumper_spec.rb spec/fixturebot/compiler_spec.rbspec/fixturebot/yaml_dumper_spec.rb renamed to spec/fixturebot/compiler_spec.rb

@@ -3,7 +3,7 @@
 require "tmpdir"
 require "yaml"
 
-RSpec.describe FixtureBot::YamlDumper do
+RSpec.describe FixtureBot::Compiler do
   let(:schema) do
     FixtureBot::Schema.define do
       table :users, singular: :user, columns: [:name, :email]
@@ -27,11 +27,11 @@
     end
   end
 
-  let(:dumper) { described_class.new(fixture_set) }
+  let(:compiler) { described_class.new(fixture_set) }
 
-  describe "#dump_table" do
+  describe "#compile_table" do
     it "produces valid YAML with string keys" do
-      yaml = dumper.dump_table(:users)
+      yaml = compiler.compile_table(:users)
       parsed = YAML.safe_load(yaml)
 
       expect(parsed).to have_key("admin")
@@ -41,11 +41,11 @@
     end
   end
 
-  describe "#dump" do
+  describe "#compile" do
     it "writes per-table YAML files to the output directory" do
       dir = Dir.mktmpdir
       begin
-        dumper.dump(dir)
+        compiler.compile(dir)
 
         expect(File.exist?(File.join(dir, "users.yml"))).to be true
         expect(File.exist?(File.join(dir, "posts.yml"))).to be true
@@ -72,7 +72,7 @@
           end
         end
 
-        described_class.new(fs).dump(dir)
+        described_class.new(fs).compile(dir)
 
         expect(File.exist?(File.join(dir, "users.yml"))).to be true
         expect(File.exist?(File.join(dir, "posts.yml"))).to be false

spec/fixturebot/rails/generate_spec.rb spec/fixturebot/rails/compile_spec.rbspec/fixturebot/rails/generate_spec.rb renamed to spec/fixturebot/rails/compile_spec.rb

@@ -3,7 +3,7 @@
 require "fixturebot/rails"
 require "tmpdir"
 
-RSpec.describe FixtureBot::Rails, ".generate" do
+RSpec.describe FixtureBot::Rails, ".compile" do
   before do
     ActiveRecord::Base.establish_connection(adapter: "sqlite3", database: ":memory:")
 
@@ -29,7 +29,7 @@
     ActiveRecord::Base.connection_pool.disconnect!
   end
 
-  it "generates YAML fixture files from a DSL file" do
+  it "compiles YAML fixture files from a DSL file" do
     Dir.mktmpdir do |tmpdir|
       fixtures_file = File.join(tmpdir, "fixtures.rb")
       output_dir = File.join(tmpdir, "fixtures")
@@ -54,7 +54,7 @@
         end
       RUBY
 
-      described_class.generate(fixtures_file: fixtures_file, output_dir: output_dir)
+      described_class.compile(fixtures_file: fixtures_file, output_dir: output_dir)
 
       users_yaml = YAML.load_file(File.join(output_dir, "users.yml"))
       expect(users_yaml.keys).to contain_exactly("alice", "bob")
@@ -82,7 +82,7 @@
         end
       RUBY
 
-      described_class.generate(fixtures_file: fixtures_file, output_dir: output_dir)
+      described_class.compile(fixtures_file: fixtures_file, output_dir: output_dir)
 
       expect(File.exist?(File.join(output_dir, "users.yml"))).to be true
       expect(File.exist?(File.join(output_dir, "posts.yml"))).to be false
@@ -93,7 +93,7 @@
     Dir.mktmpdir do |tmpdir|
       output_dir = File.join(tmpdir, "fixtures")
 
-      described_class.generate(
+      described_class.compile(
         fixtures_file: File.join(tmpdir, "nonexistent.rb"),
         output_dir: output_dir
       )