cf_script 0.0.1.beta.1 → 0.0.1

data/test/lib/cf_script/command/cf/general/login_test.rb

@@ -24,16 +24,15 @@ describe 'LoginCommand' do
       space:  'staging'
     }
 
-    fake_cf do |stdout, stderr|
-      command.run('user', 'pass', options)
-
-      trace = stdout.lines.last
-      assert_match /cf login/, trace
-      assert_match /-u user -p \[PASSWORD HIDDEN\]/, trace
-      assert_match /-a https:\/\/api.example.io/, trace
-      assert_match /-o ACME/, trace
-      assert_match /-s staging/, trace
-    end
+    assert_command_args command,
+      ['user', 'pass', options],
+      [{
+        u: 'user',
+        p: 'pass',
+        a: 'https://api.example.io',
+        o: 'ACME',
+        s: 'staging'
+      }]
   end
 
   it "returns false when credentials get rejected" do

data/test/lib/cf_script/command/cf/routes/create_route_test.rb

@@ -16,4 +16,16 @@ describe 'CreateRouteCommand' do
       assert_equal true, command.run(:staging, :api, 'example.com')
     end
   end
+
+  it "host option defaults to nil" do
+    assert_command_args command,
+      [:staging, 'example.com'],
+      [:staging, 'example.com', {}]
+  end
+
+  it "adds option for host when given" do
+    assert_command_args command,
+      [:staging, 'example.com', 'www'],
+      [:staging, 'example.com', { n: 'www' }]
+  end
 end

data/test/lib/cf_script/command/cf/routes/delete_route_test.rb

@@ -16,4 +16,22 @@ describe 'DeleteRouteCommand' do
       assert_equal true, command.run('example.com', :api)
     end
   end
+
+  it "adds flag for force by default" do
+    assert_command_args command,
+      ['example.com'],
+      ['example.com', { flags: [:f] }]
+  end
+
+  it "skips flag for force when false" do
+    assert_command_args command,
+      ['example.com', 'www', false],
+      ['example.com', { n: 'www' }]
+  end
+
+  it "adds option for host and flag for force when both are given" do
+    assert_command_args command,
+      ['example.com', 'www', true],
+      ['example.com', { n: 'www', flags: [:f] }]
+  end
 end

data/test/lib/cf_script/command/cf/routes/map_route_test.rb

@@ -18,6 +18,18 @@ describe 'MapRouteCommand' do
     end
   end
 
+  it "host option defaults to nil" do
+    assert_command_args command,
+      [:worker, 'example.com'],
+      [:worker, 'example.com', {}]
+  end
+
+  it "adds option for host when given" do
+    assert_command_args command,
+      [:worker, 'example.com', 'www'],
+      [:worker, 'example.com', { n: 'www' }]
+  end
+
   it "prints an error and returns false when the route is invalid" do
     fake_cf map_route: :invalid do |stdout, stderr|
       assert_equal false, command.run(:api, 'example.com')

data/test/lib/cf_script/command/cf/routes/routes_test.rb

@@ -21,4 +21,12 @@ describe 'RoutesCommand' do
       end
     end
   end
+
+  it "returns nil if the routes table was not found" do
+    fake_cf routes: :empty do
+      routes = command.run
+
+      assert_equal nil, routes
+    end
+  end
 end

data/test/lib/cf_script/command/cf/routes/unmap_route_test.rb

@@ -18,6 +18,18 @@ describe 'UnmapRouteCommand' do
     end
   end
 
+  it "host option defaults to nil" do
+    assert_command_args command,
+      [:worker, 'example.com'],
+      [:worker, 'example.com', {}]
+  end
+
+  it "adds option for host when given" do
+    assert_command_args command,
+      [:worker, 'example.com', 'www'],
+      [:worker, 'example.com', { n: 'www' }]
+  end
+
   it "prints an error and returns false when the app can not be found" do
     fake_cf unmap_route: :no_app do |stdout, stderr|
       assert_equal false, command.run(:api, 'example.com')

data/test/lib/cf_script/object/app_info_test.rb

@@ -103,11 +103,11 @@ describe CfScript::AppInfo do
     assert_respond_to info, :stopped?
     assert_respond_to info, :cold?
 
-    assert info.started?
-    assert info.hot?
+    assert_equal true, info.started?
+    assert_equal true, info.hot?
 
-    refute info.stopped?
-    refute info.cold?
+    refute_equal true, info.stopped?
+    refute_equal true, info.cold?
   end
 
   it "defines set_intance_status method that takes attribute lists and builds InstanceStatus objects" do

data/test/lib/cf_script/object/app_list_test.rb

@@ -16,9 +16,15 @@ describe CfScript::AppList do
       urls:            'example.com',
     )
 
+    api_attrs = build_attribute_list(
+      requested_state: 'started',
+      urls:            'example.com',
+    )
+
     @list = CfScript::AppList.new([
       CfScript::AppInfo.new('one-worker', one_attrs),
       CfScript::AppInfo.new('two-engine', two_attrs),
+      CfScript::AppInfo.new('org-api', api_attrs),
     ])
   end
 
@@ -28,14 +34,74 @@ describe CfScript::AppList do
     assert_equal 0, app_list.length
   end
 
-  it "selects by prefix" do
+  it "defines a names method that returns an array of names" do
+    assert_instance_of Array, list.names
+    assert_equal ['one-worker', 'two-engine', 'org-api'], list.names
+  end
+
+  describe '==' do
+    it "returns true when the lists contain the same names" do
+      other = CfScript::AppList.new([
+        CfScript::AppInfo.new('one-worker', {}),
+        CfScript::AppInfo.new('two-engine', {}),
+        CfScript::AppInfo.new('org-api',    {}),
+      ])
+
+      assert_equal true, other == list
+    end
+
+    it "returns false when the lists have different names" do
+      other = CfScript::AppList.new([
+        CfScript::AppInfo.new('one-engine', {}),
+        CfScript::AppInfo.new('two-worker', {}),
+        CfScript::AppInfo.new('org-api',    {}),
+      ])
+
+      assert_equal false, other == list
+    end
+  end
+
+  describe 'each_name' do
+    it "yields the app name in the list" do
+      names = []
+
+      list.each_name do |name|
+        names << name
+      end
+
+      assert_equal names, list.names
+    end
+  end
+
+  describe 'each' do
+    it "yields each AppInfo instance in the list" do
+      infos = []
+
+      list.each do |info|
+        infos << info
+      end
+
+      infos.each_with_index do |info, index|
+        assert_equal info, list[index]
+      end
+    end
+  end
+
+  it "selects by suffix" do
     list.select!(ending_with: 'worker')
 
     assert_equal 1, list.length
     assert_equal 'one-worker', list.first.name
   end
 
-  it "selects by suffix" do
+  it "selects by matching" do
+    list.select!(matching: 'api')
+
+    assert_equal 1, list.length
+    assert_equal 'org-api', list.first.name
+  end
+
+  it "selects by prefix" do
     list.select!(starting_with: 'two')
 
     assert_equal 1, list.length

data/test/lib/cf_script/output/buffer_test.rb

@@ -13,17 +13,17 @@ describe CfScript::Output::Buffer do
   end
 
   it "responds to raw and returns a original text" do
-    assert buffer.respond_to?(:raw)
+    assert buffer.respond_to?(:raw), "Expected Buffer to respond to raw"
     assert_equal raw_text, buffer.raw
   end
 
   it "responds to content and returns a sanitized copy" do
-    assert buffer.respond_to?(:content)
+    assert buffer.respond_to?(:content), "Expected Buffer to respond to content"
     assert_equal "line 1\nline 2\nline 3\n", buffer.content
   end
 
   it "responds to lines and returns sanitized lines" do
-    assert buffer.respond_to?(:lines)
+    assert buffer.respond_to?(:lines), "Expected Buffer to respond to lines"
     assert_equal ['line 1', 'line 2', 'line 3'], buffer.lines
   end
 
@@ -57,17 +57,17 @@ describe CfScript::Output::Buffer do
   end
 
   it "responds to matches? and returns true or false" do
-    assert buffer.matches?(/2/)
-    refute buffer.matches?(/4/)
+    assert_equal true, buffer.matches?(/2/)
+    refute_equal true, buffer.matches?(/4/)
   end
 
   it "responds to contains? and returns true or false" do
-    assert buffer.contains?('2')
-    refute buffer.contains?('4')
+    assert_equal true, buffer.contains?('2')
+    refute_equal true, buffer.contains?('4')
   end
 
   it "responds to last_line_matches? and returns true or false" do
-    assert buffer.last_line_matches?(/3/)
-    refute buffer.last_line_matches?(/4/)
+    assert_equal true, buffer.last_line_matches?(/3/)
+    refute_equal true, buffer.last_line_matches?(/4/)
   end
 end

data/test/lib/cf_script/output/parser/section_test.rb

@@ -33,8 +33,10 @@ describe CfScript::Output::Parser::Section do
 
   describe "parse_section_attributes" do
     it "calls parse_attribute_list" do
-      parser.stub :parse_attribute_list, 'called' do
-        assert_equal 'called', parser.parse_section_attributes(text_buffer, 'Section Title')
+      parser.stub :parse_attribute_list, :called do
+        assert_equal :called, parser.parse_section_attributes(
+          text_buffer, 'Section Title'
+        )
       end
     end
 

data/test/lib/cf_script/output/tests_test.rb

@@ -6,86 +6,86 @@ describe CfScript::Output::Tests do
   it "defines matches? and calls the out buffer matches?" do
     output = fake_output
 
-    output.out.stub :matches?, 'called' do
-      assert_equal 'called', output.matches?('a')
+    output.out.stub :matches?, :called do
+      assert_equal :called, output.matches?('a')
     end
   end
 
   it "defines contains? and calls the out buffer matches?" do
     output = fake_output
 
-    output.out.stub :contains?, 'called' do
-      assert_equal 'called', output.contains?('a')
+    output.out.stub :contains?, :called do
+      assert_equal :called, output.contains?('a')
     end
   end
 
   it "defines ok?, calls the out buffer last_line_matches?" do
     output = fake_output('OK')
 
-    assert output.ok?
-    refute fake_output('KO').ok?
+    assert_equal true, output.ok?
+    refute_equal true, fake_output('KO').ok?
   end
 
   it "defines authenticated? and calls matches?" do
     output = fake_output("Authenticating...\nOK")
 
-    assert output.authenticated?
-    refute fake_output('NOPE').authenticated?
+    assert_equal true, output.authenticated?
+    refute_equal true, fake_output('NOPE').authenticated?
   end
 
   it "defines failed? and calls contains?" do
     output = fake_output("FAILED")
 
-    assert output.failed?
-    refute fake_output('OK').failed?
+    assert_equal true, output.failed?
+    refute_equal true, fake_output('OK').failed?
   end
 
   it "defines no_api_endpoint? and calls contains?" do
     output = fake_output("No API endpoint set.")
 
-    assert output.no_api_endpoint?
-    refute fake_output('API endpoint set').no_api_endpoint?
+    assert_equal true, output.no_api_endpoint?
+    refute_equal true, fake_output('API endpoint set').no_api_endpoint?
   end
 
   it "defines not_logged_in? and calls contains?" do
     output = fake_output("Not logged in")
 
-    assert output.not_logged_in?
-    refute fake_output('logged in').not_logged_in?
+    assert_equal true, output.not_logged_in?
+    refute_equal true, fake_output('logged in').not_logged_in?
   end
 
   it "defines not_authorized? and calls contains?" do
     output = fake_output("You are not authorized")
 
-    assert output.not_authorized?
-    refute fake_output('You are authorized').not_authorized?
+    assert_equal true, output.not_authorized?
+    refute_equal true, fake_output('You are authorized').not_authorized?
   end
 
   it "defines credentials_rejected? and calls contains?" do
     output = fake_output("Credentials were rejected")
 
-    assert output.credentials_rejected?
-    refute fake_output('Credentials accepted').credentials_rejected?
+    assert_equal true, output.credentials_rejected?
+    refute_equal true, fake_output('Credentials accepted').credentials_rejected?
   end
 
   it "defines not_found? and calls out buffer's last_line_matches?" do
     output = fake_output("App api not found")
 
-    assert output.not_found?('App', 'api')
-    refute fake_output('Found').not_found?('App', 'api')
+    assert_equal true, output.not_found?('App', 'api')
+    refute_equal true, fake_output('Found').not_found?('App', 'api')
   end
 
   it "defines is_already? and calls out buffer's last_line_matches?" do
     output = fake_output("app is already started")
 
-    assert output.is_already?('app', 'started')
-    refute fake_output('app not started').is_already?('app', 'started')
+    assert_equal true, output.is_already?('app', 'started')
+    refute_equal true, fake_output('app not started').is_already?('app', 'started')
   end
 
   it "defines already_exists? and calls out buffer's last_line_matches?" do
     output = fake_output("App api already exists")
 
-    assert output.already_exists?('App', 'api')
-    refute fake_output('App api does not exist').already_exists?('App', 'api')
+    assert_equal true, output.already_exists?('App', 'api')
+    refute_equal true, fake_output('App api does not exist').already_exists?('App', 'api')
   end
 end

data/test/lib/cf_script/output_test.rb

@@ -21,12 +21,12 @@ describe CfScript::Output do
   end
 
   it "responds to good? and returns true if exit status == 0" do
-    assert output.respond_to?(:good?)
-    assert output.good?
+    assert output.respond_to?(:good?), "Expected Output to respond to good?"
+    assert_equal true, output.good?
   end
 
   it "responds to good? and returns false if exit status != 0" do
-    refute fake_output('', '', 1).good?
+    refute_equal true, fake_output('', '', 1).good?
   end
 
   it "responds to out and returns a Buffer object" do
@@ -43,10 +43,11 @@ describe CfScript::Output do
 
   describe "attributes" do
     it "calls parse_attribute_list" do
-      assert output.respond_to?(:attributes)
+      assert output.respond_to?(:attributes),
+        "Expected Output to respond to attributes"
 
-      output.stub :parse_attribute_list, 'called' do
-        assert_equal 'called', output.attributes
+      output.stub :parse_attribute_list, :called do
+        assert_equal :called, output.attributes
       end
     end
 
@@ -58,10 +59,11 @@ describe CfScript::Output do
 
   describe "line_attributes" do
     it "calls parse_line_attributes" do
-      assert output.respond_to?(:line_attributes)
+      assert output.respond_to?(:line_attributes),
+        "Expected Output to respond to line_attributes"
 
-      output.stub :parse_line_attributes, 'called' do
-        assert_equal 'called', output.line_attributes(/a/)
+      output.stub :parse_line_attributes, :called do
+        assert_equal :called, output.line_attributes(/a/)
       end
     end
 
@@ -87,10 +89,11 @@ describe CfScript::Output do
     end
 
     it "calls parse_attribute_list" do
-      assert output.respond_to?(:attributes_from)
+      assert output.respond_to?(:attributes_from),
+        "Expected Output to respond to attributes_from"
 
-      output.stub :parse_attribute_list, 'called' do
-        assert_equal 'called', output.attributes_from('two')
+      output.stub :parse_attribute_list, :called do
+        assert_equal :called, output.attributes_from('two')
       end
     end
 
@@ -106,10 +109,11 @@ describe CfScript::Output do
 
   describe "table" do
     it "calls parse_table" do
-      assert output.respond_to?(:table)
+      assert output.respond_to?(:table),
+        "Expected Output to respond to table"
 
-      output.stub :parse_table, 'called' do
-        assert_equal 'called', output.table([])
+      output.stub :parse_table, :called do
+        assert_equal :called, output.table([])
       end
     end
 
@@ -124,10 +128,11 @@ describe CfScript::Output do
 
   describe "section_attributes" do
     it "calls parse_section_attributes" do
-      assert output.respond_to?(:section_attributes)
+      assert output.respond_to?(:section_attributes),
+        "Expected Output to respond to section_attributes"
 
-      output.stub :parse_section_attributes, 'called' do
-        assert_equal 'called', output.section_attributes('header')
+      output.stub :parse_section_attributes, :called do
+        assert_equal :called, output.section_attributes('header')
       end
     end
   end

data/test/lib/cf_script/scope/app/env_test.rb

@@ -0,0 +1,58 @@
+require 'test_helper'
+
+describe CfScript::Scope::App::Env do
+  include MockExecution
+
+  let(:target) { CfScript::Target.new('API', 'ORG', 'staging') }
+
+  def create_app(name)
+    app = nil
+    fake_cf { app = CfScript::Scope::App.new(:api, target) }
+    app
+  end
+
+  it "defines an env method that calls Command.env with the app name" do
+    app = create_app(:api)
+
+    arg_catcher = lambda do |command, *args, &block|
+      assert_equal :env, command
+      assert_equal [:api], args
+
+      return :called
+    end
+
+    CfScript::Command.stub :run, arg_catcher do
+      assert_equal :called, app.env
+    end
+  end
+
+  it "defines a set_env method that calls Command.set_env with the app name" do
+    app = create_app(:api)
+
+    arg_catcher = lambda do |command, *args, &block|
+      assert_equal :set_env, command
+      assert_equal [:api, 'name', 'value'], args
+
+      return :called
+    end
+
+    CfScript::Command.stub :run, arg_catcher do
+      assert_equal :called, app.set_env('name', 'value')
+    end
+  end
+
+  it "defines an unset_env method that calls Command.unset_env with the app name" do
+    app = create_app(:api)
+
+    arg_catcher = lambda do |command, *args, &block|
+      assert_equal :unset_env, command
+      assert_equal [:api, 'name'], args
+
+      return :called
+    end
+
+    CfScript::Command.stub :run, arg_catcher do
+      assert_equal :called, app.unset_env('name')
+    end
+  end
+end

data/test/lib/cf_script/scope/app/routes_test.rb

@@ -0,0 +1,56 @@
+require 'test_helper'
+
+describe CfScript::Scope::App::Routes do
+  include MockExecution
+
+  let(:target) { CfScript::Target.new('API', 'ORG', 'staging') }
+
+  def create_app(name)
+    app = nil
+    fake_cf { app = CfScript::Scope::App.new(:api, target) }
+    app
+  end
+
+  it "defines a has_route? method" do
+    app = create_app(:api)
+
+    app.stub :urls, ['api.example.com'] do
+      assert_equal false, app.has_route?('www.example.com')
+      assert_equal true,  app.has_route?('api.example.com')
+    end
+  end
+
+  it "defines a map_route method that calls Command.map_route with the app name" do
+    app = create_app(:api)
+
+    arg_catcher = lambda do |command, *args, &block|
+      assert_equal :map_route, command
+      assert_equal [:api, 'domain', nil], args
+
+      return :called
+    end
+
+    app.stub :cf_self, true do
+      CfScript::Command.stub :run, arg_catcher do
+        assert_equal :called, app.map_route('domain')
+      end
+    end
+  end
+
+  it "defines an unmap_route method that calls Command.unmap_route with the app name" do
+    app = create_app(:api)
+
+    arg_catcher = lambda do |command, *args, &block|
+      assert_equal :unmap_route, command
+      assert_equal [:api, 'domain', nil], args
+
+      return :called
+    end
+
+    app.stub :cf_self, true do
+      CfScript::Command.stub :run, arg_catcher do
+        assert_equal :called, app.unmap_route('domain')
+      end
+    end
+  end
+end

data/test/lib/cf_script/scope/app/state_test.rb

@@ -43,6 +43,21 @@ describe CfScript::Scope::App::State do
     end
   end
 
+  it "defines a restart method that calls Command.restart with the app name" do
+    app = create_app(:api)
+
+    arg_catcher = lambda do |command, *args, &block|
+      assert_equal :restart,  command
+      assert_equal [:api], args
+
+      return :called
+    end
+
+    CfScript::Command.stub :run, arg_catcher do
+      assert_equal :called, app.restart
+    end
+  end
+
   it "defines a push method that calls Command.push with the app name and options" do
     app = create_app(:api)
 
@@ -58,4 +73,19 @@ describe CfScript::Scope::App::State do
       assert_equal :called, app.push({ memory: '128MB' })
     end
   end
+
+  it "defines a restage method that calls Command.restage with the app name" do
+    app = create_app(:api)
+
+    arg_catcher = lambda do |command, *args, &block|
+      assert_equal :restage,  command
+      assert_equal [:api], args
+
+      return :called
+    end
+
+    CfScript::Command.stub :run, arg_catcher do
+      assert_equal :called, app.restage
+    end
+  end
 end

data/test/lib/cf_script/scope/script_test.rb

@@ -21,14 +21,15 @@ describe CfScript::Scope::Script do
       script = CfScript::Scope::Script.new
     end
 
-    assert script.respond_to?(:run)
+    assert script.respond_to?(:run), "Expected Scope::Script to respond to run"
 
     arg_catcher = lambda do |scope, args, &block|
       assert_same script, scope
+      return :called
     end
 
     script.stub :exec_in, arg_catcher do
-      assert script.run { nil }
+      assert_equal :called, script.run { nil }
     end
   end