producer-core 0.1.8 → 0.1.9

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: e267920fce6f6536aa7aeabe841d440bd3ffae7b
-  data.tar.gz: d9f71061074b02737726ccbbb56f468fdc5e8432
+  metadata.gz: 890c789a5de327b39836f468441f8c75b890faeb
+  data.tar.gz: c524443e3b5882f32346b144a237bebdc60a477f
 SHA512:
-  metadata.gz: c4fd1d85b8191a94838c39f866f3fcb583c9e4a59c7db09b7a37c84d5c0cf096d55ae92d11bba1362cee35f3c7262b1e044075cba0ff82bb6ba2387284aec9d3
-  data.tar.gz: 5b9e9ea1ec0f4f5ae937c23d53ba115fd9f4427a810b4ea8a674c39dd6289ce59e3389c601244bdc71b0984451ba12e614baafa17e0aea6bebcaa1b90b91887e
+  metadata.gz: 3a21fe320fa3ac8c89c92c7def97284413ad31afd620784a255daeeb38fa8db1b005fc1f5db02cd55d801d4ba159985845a87e23addea989ce6a25aa42135629
+  data.tar.gz: d4d1389cfe3de827c45d2d6c0982644827d5356352699e2d41d8424772c17abb3d602c04e6b3d8aeacfee599de391c9fc33ddc1c2d245b71f70a63e66bb155f7

data/lib/producer/core/cli.rb

@@ -5,19 +5,22 @@ module Producer
 
       USAGE = "Usage: #{File.basename $0} recipe_file"
 
+      EX_USAGE = 64
+
       class << self
         def run!(arguments, output: $stderr)
           begin
             cli = new(arguments)
           rescue ArgumentError
             output.puts USAGE
-            exit 64
+            exit EX_USAGE
           end
-          cli.run!
+          cli.run
         end
       end
 
       attr_reader :arguments, :stdout
+      attr_accessor :recipe
 
       def initialize(arguments, stdout: $stdout)
         raise ArgumentError unless arguments.any?
@@ -25,20 +28,13 @@ module Producer
         @stdout     = stdout
       end
 
-      def run!
-        interpreter.process recipe.tasks
-      end
-
-      def env
-        @env ||= Env.new
-      end
-
-      def recipe
-        @recipe ||= Recipe.evaluate_from_file(@arguments.first, env)
+      def run(worker: Interpreter.new)
+        load_recipe
+        worker.process recipe.tasks
       end
 
-      def interpreter
-        @interpreter ||= Interpreter.new
+      def load_recipe
+        @recipe = Recipe.evaluate_from_file(@arguments.first, Env.new)
       end
     end
   end

data/lib/producer/core/version.rb

@@ -1,5 +1,5 @@
 module Producer
   module Core
-    VERSION = '0.1.8'
+    VERSION = '0.1.9'
   end
 end

data/spec/fixtures/recipes/some_recipe.rb

@@ -0,0 +1,5 @@
+task :some_task do
+end
+
+task :another_task do
+end

data/spec/producer/core/cli_spec.rb

@@ -5,7 +5,7 @@ module Producer::Core
     include ExitHelpers
     include FixturesHelpers
 
-    let(:recipe_file) { fixture_path_for 'recipes/empty.rb' }
+    let(:recipe_file) { fixture_path_for 'recipes/some_recipe.rb' }
     let(:arguments)   { [recipe_file] }
     let(:stdout)      { StringIO.new }
 
@@ -23,7 +23,7 @@ module Producer::Core
       it 'runs the CLI' do
         cli = double 'cli'
         allow(CLI).to receive(:new) { cli }
-        expect(cli).to receive :run!
+        expect(cli).to receive :run
         run
       end
 
@@ -69,51 +69,37 @@ module Producer::Core
       end
     end
 
-    describe '#run!' do
-      it 'processes the tasks with the interpreter' do
-        allow(cli.recipe).to receive(:tasks) { [:some_task] }
-        expect(cli.interpreter).to receive(:process).with [:some_task]
-        cli.run!
+    describe '#recipe' do
+      it 'returns the assigned recipe' do
+        recipe = double 'recipe'
+        cli.recipe = recipe
+        expect(cli.recipe).to be recipe
       end
     end
 
-    describe '#env' do
-      it 'builds an environment with the current recipe' do
-        expect(Env).to receive :new
-        cli.env
+    describe '#run' do
+      it 'loads the recipe' do
+        cli.run
+        expect(cli.recipe).to be_a Recipe
       end
 
-      it 'returns the env' do
-        env = double 'env'
-        allow(Env).to receive(:new) { env }
-        expect(cli.env).to be env
+      it 'process recipe tasks with given worker' do
+        worker = double 'worker'
+        expect(worker).to receive(:process).with [kind_of(Task), kind_of(Task)]
+        cli.run worker: worker
       end
     end
 
-    describe '#recipe' do
-      it 'builds a recipe' do
+    describe '#load_recipe' do
+      it 'evaluates the recipe file with an env' do
         expect(Recipe)
-          .to receive(:evaluate_from_file).with(recipe_file, cli.env)
-        cli.recipe
-      end
-
-      it 'returns the recipe' do
-        recipe = double('recipe').as_null_object
-        allow(Recipe).to receive(:new) { recipe }
-        expect(cli.recipe).to be recipe
-      end
-    end
-
-    describe '#interpreter' do
-      it 'builds a interpreter' do
-        expect(Interpreter).to receive :new
-        cli.interpreter
+          .to receive(:evaluate_from_file).with(recipe_file, kind_of(Env))
+        cli.load_recipe
       end
 
-      it 'returns the interpreter' do
-        interpreter = double 'interpreter'
-        allow(Interpreter).to receive(:new) { interpreter }
-        expect(cli.interpreter).to be interpreter
+      it 'assigns the evaluated recipe' do
+        cli.load_recipe
+        expect(cli.recipe.tasks.count).to be 2
       end
     end
   end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: producer-core
 version: !ruby/object:Gem::Version
-  version: 0.1.8
+  version: 0.1.9
 platform: ruby
 authors:
 - Thibault Jouan
@@ -154,6 +154,7 @@ files:
 - lib/producer/core/version.rb
 - producer-core.gemspec
 - spec/fixtures/recipes/empty.rb
+- spec/fixtures/recipes/some_recipe.rb
 - spec/fixtures/recipes/throw.rb
 - spec/producer/core/action_spec.rb
 - spec/producer/core/actions/echo_spec.rb
@@ -226,6 +227,7 @@ test_files:
 - features/tests/has_file.feature
 - features/tests/negated_test.feature
 - spec/fixtures/recipes/empty.rb
+- spec/fixtures/recipes/some_recipe.rb
 - spec/fixtures/recipes/throw.rb
 - spec/producer/core/action_spec.rb
 - spec/producer/core/actions/echo_spec.rb