zapata 0.1.6 → 1.0.0

data/lib/zapata/predictor.rb

@@ -0,0 +1,10 @@
+# frozen_string_literal: true
+
+require_relative 'predictor/args'
+require_relative 'predictor/chooser'
+require_relative 'predictor/value'
+
+module Zapata
+  module Predictor
+  end
+end

data/lib/zapata/predictor/args.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Zapata
   module Predictor
     class Args

data/lib/zapata/predictor/chooser.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Zapata
   module Predictor
     class Chooser

data/lib/zapata/predictor/value.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Zapata
   module Predictor
     class Value
@@ -10,19 +12,21 @@ module Zapata
 
       def choose
         return Primitive::Raw.new(:nil, nil) if @name.nil?
-        return @finder if @finder and FINAL_TYPES.include?(@finder.type)
+        return @finder if @finder && FINAL_TYPES.include?(@finder.type)
         return Primitive::Raw.new(:super, @name) if possible_values.empty?
+
         Chooser.new(possible_values).by_probability
       end
 
       def a_finder?(primitive)
         return false unless @finder
-        primitive.class == @finder.class and primitive.name == @finder.name
+
+        primitive.class == @finder.class && primitive.name == @finder.name
       end
 
       def possible_values
         Revolutionist.analysis_as_array.select do |element|
-          !a_finder?(element) and element.name == @name
+          !a_finder?(element) && element.name == @name
         end
       end
       memoize :possible_values

data/lib/zapata/primitive.rb

@@ -0,0 +1,28 @@
+# frozen_string_literal: true
+
+require_relative 'primitive/base'
+require_relative 'primitive/arg'
+require_relative 'primitive/array'
+require_relative 'primitive/basic'
+require_relative 'primitive/casgn'
+require_relative 'primitive/const'
+require_relative 'primitive/const_send'
+require_relative 'primitive/def'
+require_relative 'primitive/defs'
+require_relative 'primitive/hash'
+require_relative 'primitive/ivar'
+require_relative 'primitive/klass'
+require_relative 'primitive/lvar'
+require_relative 'primitive/missing'
+require_relative 'primitive/modul'
+require_relative 'primitive/nil'
+require_relative 'primitive/optarg'
+require_relative 'primitive/raw'
+require_relative 'primitive/send'
+require_relative 'primitive/sklass'
+require_relative 'primitive/var'
+
+module Zapata
+  module Primitive
+  end
+end

data/lib/zapata/primitive/arg.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Zapata
   module Primitive
     class Arg < Base

data/lib/zapata/primitive/array.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Zapata
   module Primitive
     class Array < Base

data/lib/zapata/primitive/base.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Zapata
   module Primitive
     class Base
@@ -13,10 +15,10 @@ module Zapata
       end
 
       def dive_deeper
-        unless RETURN_TYPES.include?(node.type)
-          Diver.dive(node.args)
-          Diver.dive(node.body)
-        end
+        return if RETURN_TYPES.include?(node.type)
+
+        Diver.dive(node.args)
+        Diver.dive(node.body)
       end
 
       def to_raw

data/lib/zapata/primitive/basic.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Zapata
   module Primitive
     class Basic < Base

data/lib/zapata/primitive/casgn.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Zapata
   module Primitive
     class Casgn < Base

data/lib/zapata/primitive/const.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Zapata
   module Primitive
     class Const < Basic

data/lib/zapata/primitive/const_send.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Zapata
   module Primitive
     class ConstSend

data/lib/zapata/primitive/def.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Zapata
   module Primitive
     class Def < Base

data/lib/zapata/primitive/defs.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Zapata
   module Primitive
     class Defs < Base
@@ -19,7 +21,7 @@ module Zapata
       end
 
       def node
-        receiver, name, args, body = @code.to_a
+        _, name, args, body = @code.to_a
         type = @code.type
         OpenStruct.new(type: type, name: name, args: args, body: body)
       end

data/lib/zapata/primitive/hash.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Zapata
   module Primitive
     class Hash < Base

data/lib/zapata/primitive/ivar.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Zapata
   module Primitive
     class Ivar < Basic

data/lib/zapata/primitive/klass.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Zapata
   module Primitive
     class Klass < Base
@@ -11,11 +13,11 @@ module Zapata
       end
 
       def parent_modul_names
-        @moduls.map { |mod| mod and mod.name }.compact
+        @moduls.map { |mod| mod&.name }.compact
       end
 
       def node
-        const, inherited_from_klass, body = @code.to_a
+        const, _, body = @code.to_a
         immediate_modul, klass = const.to_a
         chain = parent_modul_names + [immediate_modul, klass]
         name = chain.compact.join('::')

data/lib/zapata/primitive/lvar.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Zapata
   module Primitive
     class Lvar < Base

data/lib/zapata/primitive/missing.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Zapata
   module Primitive
     class Missing

data/lib/zapata/primitive/modul.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Zapata
   module Primitive
     class Modul < Base

data/lib/zapata/primitive/nil.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Zapata
   module Primitive
     class Nil < Basic

data/lib/zapata/primitive/optarg.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Zapata
   module Primitive
     class Optarg < Base

data/lib/zapata/primitive/raw.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Zapata
   module Primitive
     class Raw

data/lib/zapata/primitive/send.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Zapata
   module Primitive
     class Send < Base
@@ -25,11 +27,12 @@ module Zapata
 
       def raw_receiver
         return unless node.receiver
+
         Diver.dive(node.receiver).to_raw
       end
 
       def to_raw
-        if raw_receiver and raw_receiver.type == :const
+        if raw_receiver && raw_receiver.type == :const
           ConstSend.new(raw_receiver, node.name, node.args).to_raw
         else
           missing_name = if node.receiver

data/lib/zapata/primitive/sklass.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Zapata
   module Primitive
     class Sklass < Base
@@ -10,7 +12,7 @@ module Zapata
       end
 
       def node
-        receiver_self, body = @code.to_a
+        _, body = @code.to_a
         type = @code.type
 
         OpenStruct.new(type: type, body: body)

data/lib/zapata/primitive/var.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Zapata
   module Primitive
     class Var < Base

data/lib/zapata/printer.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Zapata
   class Printer
     class << self
@@ -86,7 +88,6 @@ module Zapata
         values = unnested.map do |key, val|
           print_hash_pair(key, val, all_keys_symbols?(unnested))
         end
-
         "{ #{values.join(', ')} }"
       end
 

data/lib/zapata/rzpec/runner.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Zapata
   module RZpec
     class Runner
@@ -43,7 +45,7 @@ module Zapata
             Printer.print(Diver.dive(Parser::CurrentRuby.parse(clean_expected_line)).to_raw)
           end
         else
-         "'Exception in RSpec'"
+          "'Exception in RSpec'"
         end
       end
 

data/lib/zapata/rzpec/writer.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Zapata
   module RZpec
     class Writer
@@ -9,7 +11,7 @@ module Zapata
         end
       end
 
-      def initialize(file, code, subject_analysis, whole_analysis, spec_analysis = nil)
+      def initialize(file, _code, subject_analysis, whole_analysis, spec_analysis = nil)
         self.class.reset_ivars
         @subject_analysis = subject_analysis
         @whole_analysis = whole_analysis
@@ -36,8 +38,8 @@ module Zapata
 
       def klass_defs(klass)
         @subject_analysis.select do |method|
-          [Primitive::Def, Primitive::Defs].include?(method.class) and
-            method.public? and method.klass.name == klass.name
+          [Primitive::Def, Primitive::Defs].include?(method.class) &&
+            method.public? && method.klass.name == klass.name
         end
       end
 

data/lib/zapata/version.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Zapata
-  VERSION = '0.1.6'
+  VERSION = '1.0.0'
 end

data/spec/array_spec.rb

@@ -1,4 +1,4 @@
-require 'spec_helper'
+# frozen_string_literal: true
 
 describe Zapata::Revolutionist do
   before(:all) do
@@ -6,31 +6,31 @@ describe Zapata::Revolutionist do
   end
 
   it '#test_in_arg' do
-    has_block('#test_in_arg', %Q{
+    has_block('#test_in_arg', %{
       expect(test_array.test_in_arg([2, 7.1, 8])).to eq([2, 7.1, 8])
     })
   end
 
   it '#test_nested_one_level' do
-    has_block('#test_nested_one_level', %Q{
+    has_block('#test_nested_one_level', %{
       expect(test_array.test_nested_one_level([[2, 7.1, 8], :mexico])).to eq([[2, 7.1, 8], :mexico])
     })
   end
 
   it '#test_nested_two_levels' do
-    has_block('#test_nested_two_levels', %Q{
+    has_block('#test_nested_two_levels', %{
       expect(test_array.test_nested_two_levels([[[2, 7.1, 8], :mexico], [2, 7.1, 8], :mexico])).to eq([[[2, 7.1, 8], :mexico], [2, 7.1, 8], :mexico])
     })
   end
 
   it '#test_nested_three_levels' do
-    has_block('#test_nested_three_levels', %Q{
+    has_block('#test_nested_three_levels', %{
       expect(test_array.test_nested_three_levels([[[[2, 7.1, 8], :mexico], [2, 7.1, 8], :mexico], [[2, 7.1, 8], :mexico], [2, 7.1, 8], :mexico])).to eq([[[[2, 7.1, 8], :mexico], [2, 7.1, 8], :mexico], [[2, 7.1, 8], :mexico], [2, 7.1, 8], :mexico])
     })
   end
 
   it '#test_hash_nested' do
-    has_block('#test_hash_nested', %Q{
+    has_block('#test_hash_nested', %{
       expect(test_array.test_hash_nested([{ emiliano: [2, 7.1, 8] }])).to eq([{ emiliano: [2, 7.1, 8] }])
     })
   end

data/spec/definition_spec.rb

@@ -1,4 +1,4 @@
-require 'spec_helper'
+# frozen_string_literal: true
 
 describe Zapata::Revolutionist do
   before(:all) do
@@ -6,13 +6,13 @@ describe Zapata::Revolutionist do
   end
 
   it '#in_optional_args' do
-    has_block('#in_optional_args', %Q{
+    has_block('#in_optional_args', %{
       expect(test_definition.in_optional_args(:audioslave)).to eq(:audioslave)
     })
   end
 
   it '#use_optional' do
-    has_block('#use_optional', %Q{
+    has_block('#use_optional', %{
       expect(test_definition.use_optional(:audioslave)).to eq(:audioslave)
     })
   end
@@ -30,13 +30,13 @@ describe Zapata::Revolutionist do
   # end
 
   it '#call_method_result_in_optional_args' do
-    has_block('#call_method_result_in_optional_args', %Q{
+    has_block('#call_method_result_in_optional_args', %{
       expect(test_definition.call_method_result_in_optional_args('Missing "fall_meth.first"')).to eq('Missing "fall_meth.first"')
     })
   end
 
   it '#resursive_method' do
-    has_block('#recursive_method', %Q{
+    has_block('#recursive_method', %{
       expect(test_definition.recursive_method).to eq('Exception in RSpec')
     })
   end

data/spec/generation_spec.rb

@@ -1,10 +1,10 @@
-require 'spec_helper'
+# frozen_string_literal: true
 
 describe Zapata::Revolutionist do
   describe '#generate_with_friendly_output' do
     let(:file_name) { 'app/models/test_array.rb' }
 
-    context "with the generate command" do
+    context 'with the generate command' do
       context 'with single specified' do
         it 'returns a single file generation message' do
           output = execution_output("cd #{RAILS_TEST_APP_DIR} && bundle exec zapata generate #{file_name} -s")
@@ -20,39 +20,5 @@ describe Zapata::Revolutionist do
         end
       end
     end
-
-    context "without the generate command" do
-      context 'with single (-s) specified' do
-        it 'returns a single file generation message' do
-          output = execution_output("cd #{RAILS_TEST_APP_DIR} && bundle exec zapata #{file_name} -s")
-          expect(output.count).to eq 1
-          expect(output.first).to eq "Its done, comrades. File spec/models/test_array_spec.rb was generated.\n"
-        end
-      end
-
-      context 'with single (--single) specified' do
-        it 'returns a single file generation message' do
-          output = execution_output("cd #{RAILS_TEST_APP_DIR} && bundle exec zapata #{file_name} --single")
-          expect(output.count).to eq 1
-          expect(output.first).to eq "Its done, comrades. File spec/models/test_array_spec.rb was generated.\n"
-        end
-      end
-
-      context 'with no single specified' do
-        it 'returns multiple file generation messages' do
-          output = execution_output("cd #{RAILS_TEST_APP_DIR} && bundle exec zapata #{file_name}")
-          expect(output.count).to be > 1
-        end
-      end
-    end
-
-    def execution_output(command)
-      stdout = Bundler.with_clean_env do
-        Open3.pipeline_r(
-          command
-        )
-      end
-      stdout.first.readlines
-    end
   end
 end