to_factory 2.1.0 → 3.0.0.pre.pre

data/spec/example_factories/admin_with_header.rb

@@ -1,9 +1,9 @@
-FactoryGirl.define do
-  factory(:admin, :parent => :"to_factory/user") do
-    birthday "2014-07-08T15:30 UTC"
-    email "admin@example.com"
-    name "Admin"
-    some_attributes({:a => 1})
-    some_id 9
+FactoryBot.define do
+  factory(:admin, parent: :"to_factory/user") do
+    birthday { "2014-07-08T15:30 UTC" }
+    email { "admin@example.com" }
+    name {"Admin" }
+    some_attributes { {a: 1 } }
+    some_id { 9 }
   end
 end

data/spec/example_factories/inherited_project_with_header.rb

@@ -0,0 +1,7 @@
+FactoryBot.define do
+  factory(:"to_factory/inherited_project") do
+    name { "My Project" }
+    objective { "easy testing" }
+    some_id { 9 }
+  end
+end

data/spec/example_factories/project_with_header.rb

@@ -1,7 +1,7 @@
-FactoryGirl.define do
+FactoryBot.define do
   factory(:"to_factory/project") do
-    name "My Project"
-    objective "easy testing"
-    some_id 9
+    name { "My Project" }
+    objective { "easy testing" }
+    some_id { 9 }
   end
 end

data/spec/example_factories/user.rb

@@ -1,7 +1,7 @@
 factory(:"to_factory/user") do
-  birthday "2014-07-08T15:30 UTC"
-  email "test@example.com"
-  name "Jeff"
-  some_attributes({:a => 1})
-  some_id 8
+  birthday {"2014-07-08T15:30 UTC"}
+  email {"test@example.com"}
+  name {"Jeff"}
+  some_attributes {{a: 1}}
+  some_id { 8 }
 end

data/spec/example_factories/user_admin.rb

@@ -1,15 +1,15 @@
 factory(:"to_factory/user") do
-  birthday "2014-07-08T15:30 UTC"
-  email "test@example.com"
-  name "Jeff"
-  some_attributes({:a => 1})
-  some_id 8
+  birthday { "2014-07-08T15:30 UTC" }
+  email { "test@example.com" }
+  name { "Jeff" }
+  some_attributes { {a: 1 } }
+  some_id { 8 }
 end
 
-factory(:admin, :parent => :"to_factory/user") do
-  birthday "2014-07-08T15:30 UTC"
-  email "admin@example.com"
-  name "Admin"
-  some_attributes({:a => 1})
-  some_id 9
+factory(:admin, parent: :"to_factory/user") do
+  birthday {"2014-07-08T15:30 UTC"}
+  email {"admin@example.com"}
+  name {"Admin"}
+  some_attributes { {a: 1 } }
+  some_id { 9 }
 end

data/spec/example_factories/user_admin_root.rb

@@ -1,22 +1,21 @@
-FactoryGirl.define do
+FactoryBot.define do
   factory(:"to_factory/user") do
-    name("User")
+    name {"User"}
   end
 
-  factory(:admin, :parent => :"to_factory/user") do
-    name("Admin")
+  factory(:admin, parent: :"to_factory/user") do
+    name {"Admin"}
   end
 
-  factory(:root, :parent => :"to_factory/user") do
-    birthday "2014-07-08T15:30 UTC"
-    email "test@example.com"
-    name "Jeff"
-    some_attributes({:a => 1})
-    some_id 8
+  factory(:root, parent: :"to_factory/user") do
+    birthday {"2014-07-08T15:30 UTC"}
+    email {"test@example.com"}
+    name {"Jeff"}
+    some_attributes {{a: 1}}
+    some_id { 8 }
   end
 
-
-  factory(:super_admin, :parent => :admin) do
-    name("Super Admin")
+  factory(:super_admin, parent: :admin) do
+    name {"Super Admin"}
   end
 end

data/spec/example_factories/user_admin_super_admin.rb

@@ -1,13 +1,13 @@
-FactoryGirl.define do
+FactoryBot.define do
   factory(:"to_factory/user") do
-    name "User"
+    name {"User"}
   end
 
-  factory(:admin, :parent => :"to_factory/user") do
-    name "Admin"
+  factory(:admin, parent: :"to_factory/user") do
+    name {"Admin" }
   end
 
-  factory(:super_admin, :parent => :admin) do
-    name "Super Admin"
+  factory(:super_admin, parent: :admin) do
+    name {"Super Admin"}
   end
 end

data/spec/example_factories/user_admin_with_header.rb

@@ -1,17 +1,17 @@
-FactoryGirl.define do
+FactoryBot.define do
   factory(:"to_factory/user") do
-    birthday "2014-07-08T15:30 UTC"
-    email "test@example.com"
-    name "Jeff"
-    some_attributes({:a => 1})
-    some_id 8
+    birthday {"2014-07-08T15:30 UTC"}
+    email {"test@example.com"}
+    name {"Jeff"}
+    some_attributes {{a: 1}}
+    some_id { 8 }
   end
 
-  factory(:admin, :parent => :"to_factory/user") do
-    birthday "2014-07-08T15:30 UTC"
-    email "admin@example.com"
-    name "Admin"
-    some_attributes({:a => 1})
-    some_id 9
+  factory(:admin, parent: :"to_factory/user") do
+    birthday {"2014-07-08T15:30 UTC"}
+    email {"admin@example.com"}
+    name {"Admin"}
+    some_attributes {{a: 1}}
+    some_id { 9 }
   end
 end

data/spec/example_factories/user_with_header.rb

@@ -1,9 +1,9 @@
-FactoryGirl.define do
+FactoryBot.define do
   factory(:"to_factory/user") do
-    birthday "2014-07-08T15:30 UTC"
-    email "test@example.com"
-    name "Jeff"
-    some_attributes({:a => 1})
-    some_id 8
+    birthday { "2014-07-08T15:30 UTC" }
+    email { "test@example.com" }
+    name { "Jeff" }
+    some_attributes { {:a => 1} }
+    some_id { 8 }
   end
 end

data/spec/integration/empty_factory_file_spec.rb

@@ -1,5 +1,5 @@
 describe "Empty File spec" do
-  let(:user) { create_user!  }
+  let(:user) { create_user! }
 
   context "with an empty factory file" do
     before do
@@ -13,7 +13,7 @@ describe "Empty File spec" do
     end
 
     it "doesn't blow up" do
-      ToFactory(:empty => OpenStruct.new({attributes: {}}), :user => user)
+      ToFactory(empty: OpenStruct.new({ attributes: {} }), user: user)
     end
   end
 end

data/spec/integration/file_sync_spec.rb

@@ -1,7 +1,7 @@
 describe "FileSync" do
-  let(:user) { create_user!  }
-  let(:admin) { create_admin!  }
-  let(:project) { create_project!  }
+  let(:user) { create_user! }
+  let(:admin) { create_admin! }
+  let(:project) { create_project! }
   let(:expected_user_file)             { File.read("./spec/example_factories/user.rb") }
   let(:expected_user_with_header_file) { File.read("./spec/example_factories/user_with_header.rb") }
   let(:expected_admin_file)            { File.read("./spec/example_factories/admin.rb") }
@@ -9,11 +9,15 @@ describe "FileSync" do
   let(:user_admin_with_header)         { File.read("./spec/example_factories/user_admin_with_header.rb") }
 
   def user_file
-    File.read("./tmp/factories/to_factory/user.rb") rescue nil
+    File.read("./tmp/factories/to_factory/user.rb")
+  rescue
+    nil
   end
 
   def project_file
-    File.read("./tmp/factories/to_factory/project.rb") rescue nil
+    File.read("./tmp/factories/to_factory/project.rb")
+  rescue
+    nil
   end
 
   context "with no arguments" do
@@ -26,7 +30,7 @@ describe "FileSync" do
       sync = ToFactory::FileSync.new
       sync.perform
 
-      expect(user_file ).to match_sexp expected_user_with_header_file
+      expect(user_file).to match_sexp expected_user_with_header_file
     end
   end
 
@@ -35,13 +39,11 @@ describe "FileSync" do
       sync = ToFactory::FileSync.new(user)
       sync.perform
 
-      expect(user_file   ).to match_sexp user_with_header
+      expect(user_file).to match_sexp user_with_header
       expect(project_file).to eq nil
     end
   end
 
-
-
   context "with a pre-existing file" do
     let(:sync) { ToFactory::FileSync.new(user) }
     before do
@@ -50,26 +52,23 @@ describe "FileSync" do
     end
 
     it "raises an error" do
-      expect(lambda{ sync.perform }).to raise_error ToFactory::AlreadyExists
+      expect { sync.perform }.to raise_error ToFactory::AlreadyExists
     end
 
     context "with a named factory" do
       it do
-        sync = ToFactory::FileSync.new({:admin => admin})
+        sync = ToFactory::FileSync.new(admin: admin)
         sync.perform
 
         parser = ToFactory::Parsing::File.new(user_file)
         result = parser.parse
-        admin= result.find{|r| r.name == "admin"}
-        user= result.find{|r| r.name == "to_factory/user"}
-
+        admin = result.find { |r| r.name == "admin" }
+        user = result.find { |r| r.name == "to_factory/user" }
 
         expect(admin.definition).to match_sexp expected_admin_file
         expect(user.definition).to match_sexp expected_user_file
 
-        expect(lambda{
-          sync.perform
-        }).to raise_error ToFactory::AlreadyExists
+        expect { sync.perform }.to raise_error ToFactory::AlreadyExists
       end
     end
   end

data/spec/integration/file_writer_spec.rb

@@ -1,18 +1,22 @@
 describe ToFactory::FileWriter do
-  let(:fw) { ToFactory::FileWriter.new }
-  let(:expected)    { File.read "./spec/example_factories/user_with_header.rb"}
-  let(:user_file_contents)    { File.read "./tmp/factories/to_factory/user.rb"}
+  let(:file_writer) { ToFactory::FileWriter.new }
+  let(:user_file_contents)    { File.read "./tmp/factories/to_factory/user.rb" }
   let!(:user) { create_user!  }
-  let!(:admin) { create_admin!  }
+  let!(:admin) { create_admin! }
+  let(:file_sync) { ToFactory::FileSync.new }
+  let(:representations) { file_sync.all_representations }
 
-  it do
-    fs = ToFactory::FileSync.new
-    representations = fs.all_representations
-    #sanity check generation isn't broken
+  let(:expected) { File.read "./spec/example_factories/user_with_header.rb" }
+
+  before do
+    # sanity check generation isn't broken
     expect(representations.keys).to eq ["to_factory/user"]
     expect(representations.values[0][0]).to be_a ToFactory::Representation
+  end
+
+  it 'writes correctly' do
+    file_writer.write representations
 
-    fw.write representations
     expect(user_file_contents).to match_sexp expected
   end
 end

data/spec/integration/lint_spec.rb

@@ -1,12 +1,13 @@
 context "full integration" do
   let(:user) do
-    ToFactory::User.create :name => "Jeff", :email => "test@example.com", :some_id => 8, :birthday => Time.now
+    ToFactory::User.create name: "Jeff", email: "test@example.com", some_id: 8, birthday: Time.now
   end
 
   it "#to_factory linting the output" do
     ToFactory(user)
+    require 'factory_bot'
+
     load "./tmp/factories/to_factory/user.rb"
-    FactoryGirl.lint
+    FactoryBot.lint
   end
 end
-

data/spec/integration/multiple_to_factory_calls_spec.rb

@@ -1,11 +1,13 @@
 describe ToFactory do
-  let!(:user)    { create_user! }
+  let!(:user) { create_user! }
 
   def user_file
-    File.read("./tmp/factories/to_factory/user.rb") rescue nil
+    File.read("./tmp/factories/to_factory/user.rb")
+  rescue
+    nil
   end
 
-  let(:expected_user_file) { File.read "./spec/example_factories/user_with_header.rb"}
+  let(:expected_user_file) { File.read "./spec/example_factories/user_with_header.rb" }
 
   context "single call" do
     before do
@@ -24,21 +26,21 @@ describe ToFactory do
 
     it "renders two factories correctly" do
       expect(user_file).to eq <<-FACTORY.strip_heredoc
-        FactoryGirl.define do
+        FactoryBot.define do
           factory(:"to_factory/user") do
-            birthday("2014-07-08T15:30 UTC")
-            email("test@example.com")
-            name("Jeff")
-            some_attributes(:a => 1)
-            some_id(8)
+            birthday { "2014-07-08T15:30 UTC" }
+            email { "test@example.com" }
+            name { "Jeff" }
+            some_attributes { { :a => 1 } }
+            some_id { 8 }
           end
 
           factory(:user_2, :parent => :"to_factory/user") do
-            birthday "2014-07-08T15:30 UTC"
-            email "test@example.com"
-            name "Jeff"
-            some_attributes({:a => 1})
-            some_id 8
+            birthday { "2014-07-08T15:30 UTC" }
+            email { "test@example.com" }
+            name { "Jeff" }
+            some_attributes { {:a => 1} }
+            some_id { 8 }
           end
         end
       FACTORY
@@ -55,37 +57,37 @@ describe ToFactory do
 
     it "renders multiple factories correctly" do
       expect(user_file).to match_sexp <<-FACTORY
-       FactoryGirl.define do
+       FactoryBot.define do
          factory(:"to_factory/user") do
-           birthday("2014-07-08T15:30 UTC")
-           email("test@example.com")
-           name("Jeff")
-           some_attributes(:a => 1)
-           some_id(8)
+           birthday { "2014-07-08T15:30 UTC" }
+           email { "test@example.com" }
+           name { "Jeff" }
+           some_attributes { {:a => 1} }
+           some_id { 8 }
          end
 
          factory(:user_2, :parent => :"to_factory/user") do
-           birthday("2014-07-08T15:30 UTC")
-           email("test@example.com")
-           name("Jeff")
-           some_attributes(:a => 1)
-           some_id(8)
+           birthday { "2014-07-08T15:30 UTC" }
+           email { "test@example.com" }
+           name { "Jeff" }
+           some_attributes { {:a => 1} }
+           some_id { 8 }
          end
 
          factory(:user_3, :parent => :"to_factory/user") do
-           birthday("2014-07-08T15:30 UTC")
-           email("test@example.com")
-           name("Jeff")
-           some_attributes(:a => 1)
-           some_id(8)
+           birthday { "2014-07-08T15:30 UTC" }
+           email { "test@example.com" }
+           name { "Jeff" }
+           some_attributes { {:a => 1} }
+           some_id { 8 }
          end
 
          factory(:user_4, :parent => :"to_factory/user") do
-           birthday "2014-07-08T15:30 UTC"
-           email "test@example.com"
-           name "Jeff"
-           some_attributes({:a => 1})
-           some_id 8
+           birthday { "2014-07-08T15:30 UTC" }
+           email { "test@example.com" }
+           name { "Jeff" }
+           some_attributes { {:a => 1} }
+           some_id { 8 }
          end
        end
       FACTORY

data/spec/integration/non_active_record_classes_spec.rb

@@ -0,0 +1,40 @@
+describe "non active record classes" do
+  let!(:project) { create_project! }
+
+  def project_file
+    File.read("./tmp/factories/to_factory/project.rb")
+  rescue
+    nil
+  end
+
+  def inherited_project_file
+    File.read("./tmp/factories/to_factory/inherited_project.rb")
+  rescue
+    nil
+  end
+
+  let(:expected_project_file) { File.read "./spec/example_factories/project_with_header.rb" }
+  let(:expected_inherited_project_file) { File.read "./spec/example_factories/inherited_project_with_header.rb" }
+
+  context "given a folder with non active record models in" do
+    before do
+      ToFactory.models = "./spec/support/non_active_record"
+      ToFactory()
+    end
+
+    it "creates an ordinary factory correctly" do
+      expect(project_file).to match_sexp expected_project_file
+    end
+
+    context "STI" do
+      it "creates a single factory correctly" do
+        expect(inherited_project_file).to match_sexp expected_inherited_project_file
+      end
+    end
+
+    it "doesn't create other factories" do
+      count = Dir.glob("./tmp/factories/*.rb").length
+      expect(count).to eq 0
+    end
+  end
+end

data/spec/integration/to_factory_method_spec.rb

@@ -1,28 +1,31 @@
 describe ToFactory do
   let!(:user)    { create_user! }
-  let!(:project) { create_project!}
+  let!(:project) { create_project! }
 
   def user_file
-    File.read("./tmp/factories/to_factory/user.rb") rescue nil
+    File.read("./tmp/factories/to_factory/user.rb")
+  rescue
+    nil
   end
 
   def project_file
-    File.read("./tmp/factories/to_factory/project.rb") rescue nil
+    File.read("./tmp/factories/to_factory/project.rb")
+  rescue
+    nil
   end
 
-  let(:expected_user_file) { File.read "./spec/example_factories/user_with_header.rb"}
-  let(:expected_project_file) { File.read "./spec/example_factories/project_with_header.rb"}
+  let(:expected_user_file) { File.read "./spec/example_factories/user_with_header.rb" }
+  let(:expected_project_file) { File.read "./spec/example_factories/project_with_header.rb" }
 
   describe "ToFactory.definitions" do
     it do
       ToFactory()
       expect(ToFactory.definitions).to match_array ["to_factory/user", "to_factory/project"]
     end
-
   end
 
   describe "ToFactory.definition_for" do
-  let(:expected_user_file) { File.read "./spec/example_factories/user.rb"}
+    let(:expected_user_file) { File.read "./spec/example_factories/user.rb" }
     it do
       expect(ToFactory.definition_for user).to match_sexp expected_user_file
     end
@@ -33,35 +36,35 @@ describe ToFactory do
     end
 
     it "raises a not found error" do
-      expect(lambda{ToFactory.definition_for :"to_factory/user"}).to raise_error ToFactory::NotFoundError
+      expect { ToFactory.definition_for :"to_factory/user" }.to raise_error ToFactory::NotFoundError
     end
   end
 
   describe "Object#ToFactory" do
     context "with multiple levels of parent classes" do
-      let(:filename) { "spec/example_factories/#{'user_admin_super_admin'}.rb"}
+      let(:filename) { "spec/example_factories/#{'user_admin_super_admin'}.rb" }
 
       it "gets the output order correct" do
         output = "./tmp/factories/to_factory/user.rb"
         `mkdir -p ./tmp/factories/to_factory`
         `cp #{filename} #{output}`
 
-        ToFactory(:root => user)
+        ToFactory(root: user)
 
         expected = File.read "spec/example_factories/#{'user_admin_root'}.rb"
 
-        #user, admin, super_admin, root
+        # user, admin, super_admin, root
         expect(File.read(output)).to match_sexp expected
       end
     end
 
     it "generates all factories" do
       ToFactory()
-      #simple check for equivalent ruby
+      # simple check for equivalent ruby
       expect(user_file)   .to match_sexp expected_user_file
       expect(project_file).to match_sexp expected_project_file
 
-      #once we are sure output is equivalent ruby, check output is identical
+      # once we are sure output is equivalent ruby, check output is identical
       expect(user_file.chomp)   .to eq expected_user_file.chomp
       expect(project_file.chomp).to eq expected_project_file.chomp
     end
@@ -77,13 +80,13 @@ describe ToFactory do
       end
 
       it "ignores specified classes" do
-        ToFactory(:exclude => ToFactory::User)
+        ToFactory(exclude: ToFactory::User)
         expect(user_file).to be_nil
         expect(project_file).to be_present
       end
 
       it "ignores specified classes - sanity check" do
-        ToFactory(:exclude => ToFactory::Project)
+        ToFactory(exclude: ToFactory::Project)
         expect(user_file).to be_present
         expect(project_file).to be_nil
       end
@@ -115,14 +118,14 @@ describe ToFactory do
       context "with a name for the factory" do
         it "appends to the file" do
           user_file_includes('factory(:"to_factory/user"')
-          ToFactory(:specific_user => user)
+          ToFactory(specific_user: user)
           user_file_includes('factory(:specific_user, :parent => :"to_factory/user"')
         end
       end
 
       it "without a name" do
-        expect(lambda{ToFactory(user)}).
-               to raise_error ToFactory::AlreadyExists
+        expect { ToFactory(user) }
+          .to raise_error ToFactory::AlreadyExists
       end
     end
   end

data/spec/spec_helper.rb

@@ -1,13 +1,13 @@
 begin
-  require "codeclimate-test-reporter"
-  CodeClimate::TestReporter.start
+  require 'simplecov'
+  SimpleCov.start
 rescue LoadError
-  #ignore on ruby 1.8.x
+  # ignore on ruby 1.8.x
 end
 
-require 'active_record'
-require 'active_support/core_ext/string'
-require 'active_support/core_ext/hash'
+require "active_record"
+require "active_support/core_ext/string"
+require "active_support/core_ext/hash"
 require "sqlite3"
 require "database_cleaner"
 
@@ -24,8 +24,8 @@ RSpec.configure do |config|
 
   config.before :suite do
     ActiveRecord::Base.tap do |base|
-      config = {:adapter => "sqlite3", :database => "spec/db/test.sqlite3"}
-      base.configurations = {:test => config}.with_indifferent_access
+      config = { adapter: "sqlite3", database: "spec/db/test.sqlite3" }
+      base.configurations = { test: config }.with_indifferent_access
       base.establish_connection :test
     end