cli spec; reorg repl a bit

Author: tpitale

bin/tryruby

@@ -1,6 +1,6 @@
 #!/usr/bin/env ruby
 require 'tryruby'
 
-TryRuby::CLI.run
+TryRuby::CLI.new(ARGV).run
 
 # add -Ilib to run during development

lib/try_ruby/cli.rb

@@ -2,50 +2,18 @@ module TryRuby
   class CLI
     attr_accessor :options
 
-    def initialize(options = {})
-      self.options = options
+    def initialize(args)
+      self.options = parse_options(args)
     end
 
-    # def create_config_file
-    #   `open 'https://api.put.io/v2/oauth2/authenticate?client_id=183&response_type=code&redirect_uri=#{Getit::REDIRECT}'`
-    #   # stdin ask for code from the url
-    #   print "Please paste what comes after the 'code=': "
-    #   code = STDIN.gets
-
-    #   # request access_token with code
-    #   uri = URI.parse("https://api.put.io/v2/oauth2/access_token?client_id=#{Getit::ID}&client_secret=#{Getit::SECRET}&grant_type=authorization_code&redirect_uri=#{Getit::REDIRECT}&code=#{code}")
-    #   http = Net::HTTP.new(uri.host, uri.port)
-    #   http.use_ssl = true
-    #   http.verify_mode = OpenSSL::SSL::VERIFY_NONE
-
-    #   request = Net::HTTP::Get.new(uri.request_uri)
-
-    #   response = http.request(request)
-
-    #   new_access_token = ''
-
-    #   if response.code.to_i == 200
-    #     new_access_token = JSON.parse(response.body)['access_token']
-    #   end
-
-    #   # write file
-    #   @config = {
-    #     'access_token' => 'access_token',
-    #     'last_accessed_at' => Time.now.to_i
-    #   }
-
-    #   write_config_file
-    # end
-
     def run
       TryRuby::REPL.new(options).start
     end
 
-    def self.run
-      new(parse_options).run
-    end
+    private
 
-    def self.parse_options
+    # @private
+    def parse_options(args)
       {}.tap do |options|
 
         OptionParser.new do |opts|
@@ -79,7 +47,7 @@ def self.parse_options
             puts opts
             exit
           end
-        end.parse!
+        end.parse!(args)
 
       end
     end

lib/try_ruby/repl.rb

@@ -13,35 +13,34 @@ def start
       display script.intro.formatted
 
       while command = Readline.readline("> ", true)
-        # print instructions if code == 'help'
-
         case command
         when "help"
           display "Soon, I will have help for you."
         when "next", "continue"
           display script.next.formatted
         when "prev", "previous", "back"
           display script.previous.formatted
-        when "exit"
-          display "Goodbye, I hope you had fun!\n"
-          exit 0
+        when "exit", "quit"
+          break
         else
-          begin
-            result = eval(command)
+          process(command)
+        end
+      end
 
-            display "=> #{result}"
+      display "Goodbye, I hope you had fun!\n"
+    end
 
-            display(script.next.formatted) if script.continue?
-          rescue SyntaxError
-            display "Oops, seems to have been some error. Care to try again?"
+    def process(command)
+      begin
+        result = eval(command)
 
-            next
-          rescue NameError
-            display "Oops, you tried to use a method or variable that doesn't exist. Care to try again?"
+        display "=> #{result}"
 
-            next
-          end
-        end
+        display(script.next.formatted) if script.continue?
+      rescue SyntaxError
+        display "Oops, seems to have been some error. Care to try again?"
+      rescue NameError
+        display "Oops, you tried to use a method or variable that doesn't exist. Care to try again?"
       end
     end
 

spec/lib/try_ruby/cli_spec.rb

@@ -0,0 +1,17 @@
+require 'spec_helper'
+
+describe TryRuby::CLI do
+  context "a cli" do
+    let(:cli) {TryRuby::CLI.new([])}
+
+    it 'runs the repl with parsed options' do
+      repl = stub(:start)
+      TryRuby::REPL.stubs(:new).returns(repl)
+
+      cli.run
+
+      expect(repl).to have_received(:start)
+      expect(TryRuby::REPL).to have_received(:new).with({})
+    end
+  end
+end

spec/lib/try_ruby/repl_spec.rb

@@ -0,0 +1,5 @@
+require 'spec_helper'
+
+describe TryRuby::REPL do
+
+end