brightbox-cli 1.0.0.rc1 → 1.0.0.rc2

data/spec/commands/config/user_add_spec.rb

@@ -6,7 +6,7 @@ require "spec_helper"
 #
 describe "brightbox config" do
 
-  describe "user_add" do
+  describe "user_add", :broken_1_8 do
     let(:output) { FauxIO.new { Brightbox::run(argv) } }
     let(:stdout) { output.stdout }
     let(:stderr) { output.stderr }

data/spec/spec_helper.rb

@@ -29,6 +29,10 @@ RSpec.configure do |config|
   config.include TokenHelpers
   config.include PasswordPromptHelpers
 
+  if RUBY_VERSION >= "1.8.7"
+    config.filter_run_excluding :broken_1_8 => true
+  end
+
   # For each test, point to the testing endpoint to make it safer and easier to
   # record from dev endpoints. Devs can DNS api.brightbox.dev to their dev service
   config.before do

data/spec/support/vcr.rb

@@ -11,8 +11,8 @@ VCR.configure do |vcr|
 
   vcr.configure_rspec_metadata!
   vcr.default_cassette_options = {
-    :record => :new_episodes,
-    #:record => :none,
+    #:record => :new_episodes,
+    :record => :none,
     :match_requests_on => [:method, :path, :body],
     :update_content_length_header => true
   }

data/spec/unit/brightbox/bb_config/add_section_spec.rb

@@ -3,7 +3,7 @@ require "spec_helper"
 describe Brightbox::BBConfig do
 
   describe "#add_section" do
-    context "when first and only client", :vcr do
+    context "when first and only client", :vcr, :broken_1_8 do
       let(:config) { Brightbox::BBConfig.new }
       let(:client_alias) { "dev" }
       let(:client_id) { "app-12345" }

data/spec/unit/brightbox/bb_config/renew_tokens_spec.rb

@@ -11,7 +11,7 @@ describe Brightbox::BBConfig do
   let(:cli_id) { "cli-12345" }
   let(:cli_secret) { "qy6xxnvy4o0tgv5" }
 
-  describe "#renew_tokens" do
+  describe "#renew_tokens", :broken_1_8 do
     context "when using a user app with no tokens", :vcr do
       before do
         contents = <<-EOS

data/spec/unit/brightbox/firewall_policy/create_spec.rb

@@ -2,7 +2,7 @@ require "spec_helper"
 
 describe Brightbox::FirewallPolicy do
 
-  describe "#create", :vcr do
+  describe "#create", :vcr, :broken_1_8 do
 
     it "should create firewall policy" do
       params = { :name => "rspec_tests"}

data/spec/unit/brightbox/firewall_policy/destroy_spec.rb

@@ -2,7 +2,7 @@ require "spec_helper"
 
 describe Brightbox::FirewallPolicy do
 
-  describe "#destroy", :vcr do
+  describe "#destroy", :vcr, :broken_1_8 do
     it "should destroy firewall policy" do
       params = { :name => "rspec_tests"}
       @group = Brightbox::ServerGroup.create(params)

data/spec/unit/brightbox/firewall_rule/create_spec.rb

@@ -2,7 +2,7 @@ require "spec_helper"
 
 describe Brightbox::FirewallRule do
 
-  describe "#create" do
+  describe "#create", :broken_1_8 do
     context "when policy exists" do
       before do
         policy_options = {

data/spec/unit/brightbox/firewall_rule/destroy_spec.rb

@@ -2,7 +2,7 @@ require "spec_helper"
 
 describe Brightbox::FirewallRule do
 
-  describe "#destroy", :vcr do
+  describe "#destroy", :vcr, :broken_1_8 do
     context "when rule exists" do
       before do
         policy_options = {}

data/spec/unit/brightbox/firewall_rule/find_spec.rb

@@ -2,7 +2,7 @@ require "spec_helper"
 
 describe Brightbox::FirewallRule do
 
-  describe "#find", :vcr do
+  describe "#find", :vcr, :broken_1_8 do
     context "when rule exists" do
       before do
         policy_options = {}

data/spec/unit/brightbox/firewall_rule/from_policy_spec.rb

@@ -2,7 +2,7 @@ require "spec_helper"
 
 describe Brightbox::FirewallRule do
 
-  describe ".from_policy", :vcr do
+  describe ".from_policy", :vcr, :broken_1_8 do
     context "when policy exists with a rule" do
       before do
         policy_options = {}

data/spec/unit/brightbox/server/destroy_spec.rb

@@ -4,7 +4,7 @@ describe Brightbox::Server do
   include ServerHelpers
 
   describe "#destroy" do
-    context "when server exists", :vcr do
+    context "when server exists", :vcr, :broken_1_8 do
       it "should work" do
         #FIXME Spec never actually calls destroy, just checks output of creation!!
 

data/spec/unit/brightbox/server/find_or_call_spec.rb

@@ -2,7 +2,7 @@ require "spec_helper"
 
 describe Brightbox::Server do
 
-  describe "#show", :vcr do
+  describe "#show", :vcr, :broken_1_8 do
     context "when server exists" do
       before do
         options = {

data/spec/unit/brightbox/server/shutdown_spec.rb

@@ -3,7 +3,7 @@ require "spec_helper"
 describe Brightbox::Server do
   include ServerHelpers
 
-  describe "#shutdown", :vcr do
+  describe "#shutdown", :vcr, :broken_1_8 do
     it "should work" do
       type = Brightbox::Type.find_by_handle "nano"
       options = server_params("rspec_server_shutdown",type)

data/spec/unit/brightbox/server/start_spec.rb

@@ -3,7 +3,7 @@ require "spec_helper"
 describe Brightbox::Server do
   include ServerHelpers
 
-  describe "#start", :vcr do
+  describe "#start", :vcr, :broken_1_8 do
     it "should work" do
       type = Brightbox::Type.find_by_handle "nano"
       options = server_params("rspec_server_start",type)

data/spec/unit/brightbox/server/stop_spec.rb

@@ -3,7 +3,7 @@ require "spec_helper"
 describe Brightbox::Server do
   include ServerHelpers
 
-  describe "#stop", :vcr do
+  describe "#stop", :vcr, :broken_1_8 do
     it "should work" do
       type = Brightbox::Type.find_by_handle "nano"
       options = server_params("rspec_server_stop",type)

data/spec/unit/brightbox/server/update_spec.rb

@@ -4,7 +4,7 @@ describe Brightbox::Server do
   let(:image_id) { "img-12345" }
 
   describe "#update" do
-    context "when passing new group membership", :vcr do
+    context "when passing new group membership", :vcr, :broken_1_8 do
       before do
         options = {
           :image_id => image_id

data/spec/unit/brightbox/server_group/find_spec.rb

@@ -2,7 +2,7 @@ require "spec_helper"
 
 describe Brightbox::ServerGroup do
 
-  describe "#find(:all)" do
+  describe "#find(:all)", :broken_1_8 do
     context "when a group exists", :vcr do
       before do
         options = {

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: brightbox-cli
 version: !ruby/object:Gem::Version
-  version: 1.0.0.rc1
+  version: 1.0.0.rc2
   prerelease: 6
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-09-25 00:00:00.000000000 Z
+date: 2013-10-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: gli
@@ -236,7 +236,6 @@ files:
 - lib/brightbox-cli/cloud_ips.rb
 - lib/brightbox-cli/collaborating_account.rb
 - lib/brightbox-cli/collaboration.rb
-- lib/brightbox-cli/command_generator.rb
 - lib/brightbox-cli/commands/accounts-accept-invite.rb
 - lib/brightbox-cli/commands/accounts-default.rb
 - lib/brightbox-cli/commands/accounts-list.rb
@@ -382,6 +381,7 @@ files:
 - spec/cassettes/Brightbox_Server/_stop/should_work.yml
 - spec/cassettes/Brightbox_Server/_update/when_passing_new_group_membership/should_update_with_group.yml
 - spec/cassettes/Brightbox_ServerGroup/_find_all_/when_a_group_exists/list_server_groups.yml
+- spec/cassettes/Firewall_policies/update/when_the_policy_does_not_exist/prints_error_to_STDERR.yml
 - spec/cassettes/brightbox_accounts/list/_when_access_token_expired_/does_not_report_invalid_token_errors.yml
 - spec/cassettes/brightbox_accounts/list/_when_both_tokens_expired_/does_not_report_invalid_token_errors.yml
 - spec/cassettes/brightbox_accounts/list/_when_invalid_tokens_/does_not_report_invalid_token_errors.yml
@@ -610,7 +610,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 1351040522938516048
+      hash: -4345128981479648464
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
@@ -658,6 +658,7 @@ test_files:
 - spec/cassettes/Brightbox_Server/_stop/should_work.yml
 - spec/cassettes/Brightbox_Server/_update/when_passing_new_group_membership/should_update_with_group.yml
 - spec/cassettes/Brightbox_ServerGroup/_find_all_/when_a_group_exists/list_server_groups.yml
+- spec/cassettes/Firewall_policies/update/when_the_policy_does_not_exist/prints_error_to_STDERR.yml
 - spec/cassettes/brightbox_accounts/list/_when_access_token_expired_/does_not_report_invalid_token_errors.yml
 - spec/cassettes/brightbox_accounts/list/_when_both_tokens_expired_/does_not_report_invalid_token_errors.yml
 - spec/cassettes/brightbox_accounts/list/_when_invalid_tokens_/does_not_report_invalid_token_errors.yml

data/lib/brightbox-cli/command_generator.rb

@@ -1,53 +0,0 @@
-module Brightbox
-  class CommandGenerator
-    COMMAND_LIST = [
-      'brightbox-accounts',
-      'brightbox-cloudips',
-      'brightbox-config',
-      'brightbox-groups',
-      'brightbox-images',
-      'brightbox-lbs',
-      'brightbox-servers',
-      'brightbox-firewall-policies',
-      'brightbox-firewall-rules',
-      'brightbox-types',
-      'brightbox-users',
-      'brightbox-zones'
-    ]
-
-    def initialize(brightbox_cli_path)
-      @brightbox_cli_path = brightbox_cli_path
-      COMMAND_LIST.each do |command|
-        File.open("#{cli_binary_path}/#{command}","w+") do |fl|
-          fl.write(command_template(command))
-        end
-      end
-    end
-
-    private
-    def command_template(file_name)
-      cmd_name = file_name.sub("brightbox-", "")
-      brightbox_command =<<-EOF
-#!/usr/bin/env ruby
-
-begin
-  require "brightbox_cli"
-  require "brightbox-cli/legacy/args_adjuster"
-rescue LoadError
-  brightbox_cli_path = File.expand_path('../../lib', __FILE__)
-  $:.unshift(brightbox_cli_path)
-  require "brightbox_cli"
-  require "brightbox-cli/legacy/args_adjuster"
-end
-
-Brightbox::run Brightbox::Legacy::ArgsAdjuster.new(ARGV).for_command("#{cmd_name}")
-      EOF
-      brightbox_command
-    end
-
-    def cli_binary_path
-      File.join(@brightbox_cli_path,"..","bin")
-    end
-
-  end
-end