fixup! Organize Factories into Fixtures

spec/amber/cli/commands/generator_spec.cr

@@ -6,7 +6,7 @@ include CLIHelper
 include CLIFixtures
 
 module Amber::CLI
-  describe MainCommand::Generate do
+  describe "amber generate" do
     ENV["AMBER_ENV"] = "test"
     camel_case = "PostComment"
     snake_case = "post_comment"
@@ -15,199 +15,197 @@ module Amber::CLI
     class_definition_prefix = "class #{camel_case}"
     spec_definition_prefix = "describe #{camel_case}"
 
-    context "amber generate" do
-      cleanup
-      scaffold_app(TESTING_APP)
+    cleanup
+    scaffold_app(TESTING_APP)
 
-      describe "controller" do
-        options = %w(add:post list:get remove:delete)
-        MainCommand.run %w(generate controller animal) | options
-        route_file = File.read("./config/routes.cr")
+    describe "controller" do
+      options = %w(add:post list:get remove:delete)
+      MainCommand.run %w(generate controller animal) | options
+      route_file = File.read("./config/routes.cr")
 
-        it "generates controller with correct verbs and actions" do
-          generated_controller = "./src/controllers/animal_controller.cr"
-          File.read(generated_controller).should eq expected_animal_controller
-        end
+      it "generates controller with correct verbs and actions" do
+        generated_controller = "./src/controllers/animal_controller.cr"
+        File.read(generated_controller).should eq expected_animal_controller
+      end
 
-        options.each do |route|
-          action, method = route.split(":")
-          it "creates a valid #{method} route for #{action}" do
-            route_file.includes?(build_route("animal", action, method)).should be_true
-          end
+      options.each do |route|
+        action, method = route.split(":")
+        it "creates a valid #{method} route for #{action}" do
+          route_file.includes?(build_route("animal", action, method)).should be_true
         end
+      end
 
-        it "follows naming conventions for all files and class names" do
-          [camel_case, snake_case].each do |arg|
-            MainCommand.run ["generate", "controller", arg]
-            filename = snake_case
-            src_filepath = "./src/controllers/#{filename}_controller.cr"
-            spec_filepath = "./spec/controllers/#{filename}_controller_spec.cr"
-
-            File.exists?(src_filepath).should be_true
-            File.exists?(spec_filepath).should be_true
-            File.read(src_filepath).should contain class_definition_prefix
-            File.read(spec_filepath).should contain spec_definition_prefix
-            File.delete(src_filepath)
-            File.delete(spec_filepath)
-          end
+      it "follows naming conventions for all files and class names" do
+        [camel_case, snake_case].each do |arg|
+          MainCommand.run ["generate", "controller", arg]
+          filename = snake_case
+          src_filepath = "./src/controllers/#{filename}_controller.cr"
+          spec_filepath = "./spec/controllers/#{filename}_controller_spec.cr"
+
+          File.exists?(src_filepath).should be_true
+          File.exists?(spec_filepath).should be_true
+          File.read(src_filepath).should contain class_definition_prefix
+          File.read(spec_filepath).should contain spec_definition_prefix
+          File.delete(src_filepath)
+          File.delete(spec_filepath)
         end
       end
+    end
 
-      describe "model" do
-        MainCommand.run %w(
-          generate model Post title:string body:text published:bool likes:int user:references
-        )
+    describe "model" do
+      MainCommand.run %w(
+        generate model Post title:string body:text published:bool likes:int user:references
+      )
 
-        it "creates Post migration file" do
-          generated_migration = Dir["./db/migrations/*_post.sql"].first
-          File.read(generated_migration).should eq expected_post_model_migration
-        end
+      it "creates Post migration file" do
+        generated_migration = Dir["./db/migrations/*_post.sql"].first
+        File.read(generated_migration).should eq expected_post_model_migration
+      end
 
-        it "generates Post model spec file" do
-          generated_spec = "./spec/models/post_spec.cr"
-          File.read(generated_spec).should eq expected_post_model_spec
-        end
+      it "generates Post model spec file" do
+        generated_spec = "./spec/models/post_spec.cr"
+        File.read(generated_spec).should eq expected_post_model_spec
+      end
 
-        it "generates Post model" do
-          generated_model = "src/models/post.cr"
-          File.read(generated_model).should eq expected_post_model
-        end
+      it "generates Post model" do
+        generated_model = "src/models/post.cr"
+        File.read(generated_model).should eq expected_post_model
       end
+    end
 
-      describe "scaffold" do
-        it "follows naming conventions for all files and class names" do
-          [camel_case, snake_case].each do |arg|
-            MainCommand.run ["generate", "scaffold", arg, "name:string"]
-
-            File.exists?("./spec/models/#{snake_case}_spec.cr").should be_true
-            File.exists?("./src/models/#{snake_case}.cr").should be_true
-            File.exists?("./spec/controllers/#{snake_case}_controller_spec.cr").should be_true
-            File.exists?("./src/controllers/#{snake_case}_controller.cr").should be_true
-            File.exists?("./src/views/#{snake_case}/_form.slang").should be_true
-            File.exists?("./src/views/#{snake_case}/edit.slang").should be_true
-            File.exists?("./src/views/#{snake_case}/index.slang").should be_true
-            File.exists?("./src/views/#{snake_case}/new.slang").should be_true
-            File.exists?("./src/views/#{snake_case}/show.slang").should be_true
-            File.read("./spec/models/#{snake_case}_spec.cr").should contain spec_definition_prefix
-            File.read("./src/models/#{snake_case}.cr").should contain class_definition_prefix
-            File.read("./spec/controllers/#{snake_case}_controller_spec.cr").should contain spec_definition_prefix
-            File.read("./src/controllers/#{snake_case}_controller.cr").should contain class_definition_prefix
-            File.read("./src/views/#{snake_case}/_form.slang").should contain snake_case
-            File.read("./src/views/#{snake_case}/edit.slang").should contain display
-            File.read("./src/views/#{snake_case}/index.slang").should contain display
-            File.read("./src/views/#{snake_case}/new.slang").should contain display
-            File.read("./src/views/#{snake_case}/show.slang").should contain snake_case
-            File.read("./config/routes.cr").should contain "#{camel_case}Controller"
-            File.read("./config/routes.cr").should_not contain "#{incorrect_case}Controller"
-          end
+    describe "scaffold" do
+      it "follows naming conventions for all files and class names" do
+        [camel_case, snake_case].each do |arg|
+          MainCommand.run ["generate", "scaffold", arg, "name:string"]
+
+          File.exists?("./spec/models/#{snake_case}_spec.cr").should be_true
+          File.exists?("./src/models/#{snake_case}.cr").should be_true
+          File.exists?("./spec/controllers/#{snake_case}_controller_spec.cr").should be_true
+          File.exists?("./src/controllers/#{snake_case}_controller.cr").should be_true
+          File.exists?("./src/views/#{snake_case}/_form.slang").should be_true
+          File.exists?("./src/views/#{snake_case}/edit.slang").should be_true
+          File.exists?("./src/views/#{snake_case}/index.slang").should be_true
+          File.exists?("./src/views/#{snake_case}/new.slang").should be_true
+          File.exists?("./src/views/#{snake_case}/show.slang").should be_true
+          File.read("./spec/models/#{snake_case}_spec.cr").should contain spec_definition_prefix
+          File.read("./src/models/#{snake_case}.cr").should contain class_definition_prefix
+          File.read("./spec/controllers/#{snake_case}_controller_spec.cr").should contain spec_definition_prefix
+          File.read("./src/controllers/#{snake_case}_controller.cr").should contain class_definition_prefix
+          File.read("./src/views/#{snake_case}/_form.slang").should contain snake_case
+          File.read("./src/views/#{snake_case}/edit.slang").should contain display
+          File.read("./src/views/#{snake_case}/index.slang").should contain display
+          File.read("./src/views/#{snake_case}/new.slang").should contain display
+          File.read("./src/views/#{snake_case}/show.slang").should contain snake_case
+          File.read("./config/routes.cr").should contain "#{camel_case}Controller"
+          File.read("./config/routes.cr").should_not contain "#{incorrect_case}Controller"
         end
       end
+    end
 
-      describe "migration" do
-        it "follows naming conventions for all files" do
-          migration_definition_prefix = "CREATE TABLE #{snake_case}"
+    describe "migration" do
+      it "follows naming conventions for all files" do
+        migration_definition_prefix = "CREATE TABLE #{snake_case}"
 
-          [camel_case, snake_case].each do |arg|
-            MainCommand.run ["generate", "migration", arg]
+        [camel_case, snake_case].each do |arg|
+          MainCommand.run ["generate", "migration", arg]
 
-            migration_filename = Dir.entries("./db/migrations").sort.last
-            migration_filename.should contain snake_case
-            File.delete("./db/migrations/#{migration_filename}")
-          end
+          migration_filename = Dir.entries("./db/migrations").sort.last
+          migration_filename.should contain snake_case
+          File.delete("./db/migrations/#{migration_filename}")
         end
       end
+    end
+
+    describe "mailer" do
+      it "follows naming conventions for all files and class names" do
+        [camel_case, snake_case].each do |arg|
+          MainCommand.run ["generate", "mailer", arg]
+          filename = snake_case
+          src_filepath = "./src/mailers/#{filename}_mailer.cr"
 
-      describe "mailer" do
-        it "follows naming conventions for all files and class names" do
-          [camel_case, snake_case].each do |arg|
-            MainCommand.run ["generate", "mailer", arg]
-            filename = snake_case
-            src_filepath = "./src/mailers/#{filename}_mailer.cr"
-
-            File.exists?(src_filepath).should be_true
-            File.read(src_filepath).should contain class_definition_prefix
-            File.delete(src_filepath)
-          end
+          File.exists?(src_filepath).should be_true
+          File.read(src_filepath).should contain class_definition_prefix
+          File.delete(src_filepath)
         end
       end
+    end
 
-      describe "socket" do
-        struct_definition_prefix = "struct #{camel_case}"
+    describe "socket" do
+      struct_definition_prefix = "struct #{camel_case}"
 
-        it "follows naming conventions for all files and class names" do
-          [camel_case, snake_case].each do |arg|
-            MainCommand.run ["generate", "socket", arg]
-            filename = snake_case
-            src_filepath = "./src/sockets/#{filename}_socket.cr"
+      it "follows naming conventions for all files and class names" do
+        [camel_case, snake_case].each do |arg|
+          MainCommand.run ["generate", "socket", arg]
+          filename = snake_case
+          src_filepath = "./src/sockets/#{filename}_socket.cr"
 
-            File.exists?(src_filepath).should be_true
-            File.read(src_filepath).should contain struct_definition_prefix
-            File.delete(src_filepath)
-          end
+          File.exists?(src_filepath).should be_true
+          File.read(src_filepath).should contain struct_definition_prefix
+          File.delete(src_filepath)
         end
       end
+    end
 
-      describe "channel" do
-        it "follows naming conventions for all files and class names" do
-          [camel_case, snake_case].each do |arg|
-            MainCommand.run ["generate", "channel", arg]
-            filename = snake_case
-            src_filepath = "./src/channels/#{filename}_channel.cr"
-
-            File.exists?(src_filepath).should be_true
-            File.read(src_filepath).should contain class_definition_prefix
-            File.delete(src_filepath)
-          end
-        end
-      end
+    describe "channel" do
+      it "follows naming conventions for all files and class names" do
+        [camel_case, snake_case].each do |arg|
+          MainCommand.run ["generate", "channel", arg]
+          filename = snake_case
+          src_filepath = "./src/channels/#{filename}_channel.cr"
 
-      describe "auth" do
-        camel_case = "AdminUser"
-        snake_case = "admin_user"
-        class_definition_prefix = "class #{camel_case}"
-        spec_definition_prefix = "describe #{camel_case}"
-        migration_definition_prefix = "CREATE TABLE #{snake_case}"
-
-        it "follows naming conventions for all files and class names" do
-          [camel_case, snake_case].each do |arg|
-            MainCommand.run ["generate", "auth", arg]
-
-            File.exists?("./db/seeds.cr").should be_true
-            File.exists?("./spec/models/admin_user_spec.cr").should be_true
-            File.exists?("./src/controllers/registration_controller.cr").should be_true
-            File.exists?("./src/controllers/session_controller.cr").should be_true
-            File.exists?("./src/handlers/authenticate.cr").should be_true
-            File.exists?("./src/models/admin_user.cr").should be_true
-            File.exists?("./src/views/registration/new.slang").should be_true
-            File.exists?("./src/views/session/new.slang").should be_true
-
-            migration_filename = Dir["./db/migrations/*_#{snake_case}.sql"].first
-            File.read("#{migration_filename}").should contain migration_definition_prefix
-            File.read("./db/seeds.cr").should contain camel_case
-            File.read("./db/seeds.cr").should contain snake_case
-            File.read("./spec/models/admin_user_spec.cr").should contain spec_definition_prefix
-            File.read("./src/controllers/registration_controller.cr").should contain camel_case
-            File.read("./src/controllers/registration_controller.cr").should contain snake_case
-            File.read("./src/controllers/session_controller.cr").should contain camel_case
-            File.read("./src/controllers/session_controller.cr").should contain snake_case
-            File.read("./src/handlers/authenticate.cr").should contain camel_case
-            File.read("./src/handlers/authenticate.cr").should contain snake_case
-            File.read("./src/models/admin_user.cr").should contain class_definition_prefix
-            File.read("./src/models/admin_user.cr").should contain snake_case
-            File.read("./src/views/registration/new.slang").should contain snake_case
-            File.read("./src/views/session/new.slang").should contain snake_case
-          end
+          File.exists?(src_filepath).should be_true
+          File.read(src_filepath).should contain class_definition_prefix
+          File.delete(src_filepath)
         end
       end
+    end
 
-      context "with the above generated files" do
-        it "builds and compile the App in bin directory" do
-          `shards build`
-          File.exists?("bin/#{TEST_APP_NAME}").should be_true
+    describe "auth" do
+      camel_case = "AdminUser"
+      snake_case = "admin_user"
+      class_definition_prefix = "class #{camel_case}"
+      spec_definition_prefix = "describe #{camel_case}"
+      migration_definition_prefix = "CREATE TABLE #{snake_case}"
+
+      it "follows naming conventions for all files and class names" do
+        [camel_case, snake_case].each do |arg|
+          MainCommand.run ["generate", "auth", arg]
+
+          File.exists?("./db/seeds.cr").should be_true
+          File.exists?("./spec/models/admin_user_spec.cr").should be_true
+          File.exists?("./src/controllers/registration_controller.cr").should be_true
+          File.exists?("./src/controllers/session_controller.cr").should be_true
+          File.exists?("./src/handlers/authenticate.cr").should be_true
+          File.exists?("./src/models/admin_user.cr").should be_true
+          File.exists?("./src/views/registration/new.slang").should be_true
+          File.exists?("./src/views/session/new.slang").should be_true
+
+          migration_filename = Dir["./db/migrations/*_#{snake_case}.sql"].first
+          File.read("#{migration_filename}").should contain migration_definition_prefix
+          File.read("./db/seeds.cr").should contain camel_case
+          File.read("./db/seeds.cr").should contain snake_case
+          File.read("./spec/models/admin_user_spec.cr").should contain spec_definition_prefix
+          File.read("./src/controllers/registration_controller.cr").should contain camel_case
+          File.read("./src/controllers/registration_controller.cr").should contain snake_case
+          File.read("./src/controllers/session_controller.cr").should contain camel_case
+          File.read("./src/controllers/session_controller.cr").should contain snake_case
+          File.read("./src/handlers/authenticate.cr").should contain camel_case
+          File.read("./src/handlers/authenticate.cr").should contain snake_case
+          File.read("./src/models/admin_user.cr").should contain class_definition_prefix
+          File.read("./src/models/admin_user.cr").should contain snake_case
+          File.read("./src/views/registration/new.slang").should contain snake_case
+          File.read("./src/views/session/new.slang").should contain snake_case
         end
+      end
+    end
 
-        cleanup
+    context "with the above generated files" do
+      it "builds and compile the App in bin directory" do
+        `shards build`
+        File.exists?("bin/#{TEST_APP_NAME}").should be_true
       end
+
+      cleanup
     end
   end
 end