Fix a assignment method issue

Replaced attr_accessor with instance varianble

Author: hoppergee

lib/matrixeval/ruby/command_line.rb

@@ -6,7 +6,7 @@ module Ruby
 
     class CommandLine
 
-      attr_accessor :argv
+      attr_reader :argv
 
       def initialize(argv)
         @argv = argv
@@ -19,7 +19,7 @@ def valid?
       end
 
       def init?
-        argv[0] == 'init'
+        @argv[0] == 'init'
       end
 
       def all?
@@ -31,19 +31,19 @@ def context_options
       end
 
       def context_arguments
-        arguments = argv[0...seperator_index]
-        arguments << "-h" if argv.empty?
+        arguments = @argv[0...seperator_index]
+        arguments << "-h" if @argv.empty?
         arguments
       end
 
       def rest_arguments
-        argv[seperator_index..-1]
+        @argv[seperator_index..-1]
       end
 
       private
 
       def seperator_index
-        argv.index do |argument|
+        @argv.index do |argument|
           Config.commands.include?(argument)
         end
       end

lib/matrixeval/ruby/runner.rb

@@ -16,7 +16,6 @@ def start(argv)
       end
 
       attr_reader :argv, :command
-      attr_accessor :threads, :matrixeval_results
 
       def initialize(argv)
         @argv = argv
@@ -88,7 +87,7 @@ def run_all_contexts_sequentially
           docker_compose = DockerCompose.new(context)
           success = docker_compose.run(command.rest_arguments)
 
-          self.matrixeval_results << [context, !!success]
+          @matrixeval_results << [context, !!success]
         end
 
         report
@@ -145,7 +144,7 @@ def report
           table.add_row headers.map { |value| { value: value, alignment: :center } }
           table.add_separator 
 
-          matrixeval_results.each do |context, success|
+          @matrixeval_results.each do |context, success|
             success_cell = [success ? Rainbow('Success').green : Rainbow('Failed').red]
             row = (context.variants.map(&:key) + success_cell).map do |value|
               { value: value, alignment: :center }
@@ -160,19 +159,19 @@ def report
       end
 
       def parallel(collection)
-        threads = [] unless threads.empty?
-        matrixeval_results = [] unless matrixeval_results.empty?
+        @threads = [] unless @threads.empty?
+        @matrixeval_results = [] unless @matrixeval_results.empty?
 
         collection.each_slice(per_worker_contexts_count) do |sub_collection|
-          threads << Thread.new do
+          @threads << Thread.new do
             yield sub_collection
           end
         end
 
-        threads.each(&:join)
+        @threads.each(&:join)
 
-        threads.each do |thread|
-          self.matrixeval_results += (thread[:matrixeval_results] || [])
+        @threads.each do |thread|
+          @matrixeval_results += (thread[:matrixeval_results] || [])
         end
       end