opsicle 2.12.2 → 2.12.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 6c6b93ef64ca7f88eab52c8ee74dcaf62118e685
-  data.tar.gz: 9394c8f6bd05cf4488389d61c01009cdd21bd35e
+  metadata.gz: 508652e0801f35bc78c6e29e2b990a075b32db81
+  data.tar.gz: a43909b2ff893db83f1cddca4624181e02a837c5
 SHA512:
-  metadata.gz: 0de83bd629358a9ab0a6855f0232e5b6cd246f02084ec75606218e28cb1737f97fde7477c067931a3df9c1d290b98c7d3ec23348398a070798a4e9c521011ece
-  data.tar.gz: 1a3c871b0cb9564f62c000a474b16a5719d49b421510b3f69b884f22161700c8eb4e85894c4a06fdf54a97fd021e6ceb3acab0b6969c3e53c412ad65bea858cc
+  metadata.gz: e068df9fe0e064989f92f3f5155644c5cf81c15afa6457641469e4bfae5b0bd3643d4a8ab78cb9c468fb8ce894808219fb3171d21db27a2169c8ea113f77554e
+  data.tar.gz: 6f168db27328423daddb64558e92fb3e50d7135573535652279bdc2206ce3f5ae80751de767326f68ef8ad9e67d6aee318cfcc3ab00632b08ea5e5a4f40544af

data/lib/opsicle/commands/add_tags.rb

@@ -14,7 +14,7 @@ module Opsicle
       @ec2_adapter = Ec2Adapter.new(@client)
       @opsworks_adapter = OpsworksAdapter.new(@client)
       stack_id = @client.config.opsworks_config[:stack_id]
-      @stack = ManageableStack.new(stack_id, @opsworks_adapter.client)
+      @stack = ManageableStack.new(stack_id, @opsworks_adapter)
       @cli = HighLine.new
     end
 
@@ -32,7 +32,7 @@ module Opsicle
 
     def select_layer
       puts "\nLayers:\n"
-      ops_layers = @opsworks_adapter.get_layers(@stack.id)
+      ops_layers = @opsworks_adapter.layers(@stack.id)
 
       layers = []
       ops_layers.each do |layer|

data/lib/opsicle/commands/clone_instance.rb

@@ -14,7 +14,7 @@ module Opsicle
       @ec2_adapter = Ec2Adapter.new(@client)
       @opsworks_adapter = OpsworksAdapter.new(@client)
       stack_id = @client.config.opsworks_config[:stack_id]
-      @stack = ManageableStack.new(stack_id, @opsworks_adapter.client)
+      @stack = ManageableStack.new(stack_id, @opsworks_adapter)
       @cli = HighLine.new
     end
 
@@ -38,7 +38,7 @@ module Opsicle
 
     def select_layer
       puts "\nLayers:\n"
-      ops_layers = @opsworks_adapter.get_layers(@stack.id)
+      ops_layers = @opsworks_adapter.layers(@stack.id)
 
       layers = []
       ops_layers.each do |layer|

data/lib/opsicle/commands/create_instance.rb

@@ -14,7 +14,7 @@ module Opsicle
       @ec2_adapter = Ec2Adapter.new(@client)
       @opsworks_adapter = OpsworksAdapter.new(@client)
       stack_id = @client.config.opsworks_config[:stack_id]
-      @stack = ManageableStack.new(stack_id, @opsworks_adapter.client)
+      @stack = ManageableStack.new(stack_id, @opsworks_adapter)
       @cli = HighLine.new
     end
 
@@ -33,7 +33,7 @@ module Opsicle
 
     def select_layer
       puts "\nLayers:\n"
-      ops_layers = @opsworks_adapter.get_layers(@stack.id)
+      ops_layers = @opsworks_adapter.layers(@stack.id)
 
       layers = []
       ops_layers.each do |layer|

data/lib/opsicle/commands/delete_instance.rb

@@ -14,7 +14,7 @@ module Opsicle
       @ec2_adapter = Ec2Adapter.new(@client)
       @opsworks_adapter = OpsworksAdapter.new(@client)
       stack_id = @client.config.opsworks_config[:stack_id]
-      @stack = ManageableStack.new(stack_id, @opsworks_adapter.client)
+      @stack = ManageableStack.new(stack_id, @opsworks_adapter)
       @cli = HighLine.new
     end
 
@@ -38,7 +38,7 @@ module Opsicle
 
     def select_layer
       puts "\nLayers:\n"
-      ops_layers = @opsworks_adapter.get_layers(@stack.id)
+      ops_layers = @opsworks_adapter.layers(@stack.id)
 
       layers = []
       ops_layers.each do |layer|

data/lib/opsicle/commands/move_eip.rb

@@ -4,21 +4,37 @@ require "opsicle/opsworks_adapter"
 require "opsicle/manageable_layer"
 require "opsicle/manageable_instance"
 require "opsicle/manageable_stack"
+require "opsicle/questionnaire/eip_inquiry"
 
 module Opsicle
   class MoveEip
 
     def initialize(environment)
       @client = Client.new(environment)
-      @opsworks_adpater = OpsworksAdapter.new(@client)
+      @opsworks_adapter = OpsworksAdapter.new(@client)
       stack_id = @client.config.opsworks_config[:stack_id]
       @cli = HighLine.new
-      @stack = ManageableStack.new(stack_id, @opsworks_adpater.client, @cli)
+      @stack = ManageableStack.new(stack_id, @opsworks_adapter, @cli)
+
+      @eip_inquiry = Questionnaire::EipInquiry.new(
+        opsworks_adapter: @opsworks_adapter,
+        highline_client: @cli
+      )
     end
 
     def execute(options={})
       puts "Stack ID = #{@stack.id}"
-      @stack.move_eip
+      moved_values = move_eip
+      puts "\nEIP #{moved_values[:ip_address]} was moved to instance #{moved_values[:target_instance_id]}"
+    end
+
+    def move_eip
+      eip_information = @stack.eips
+      moveable_eip = @eip_inquiry.which_eip_should_move(eip_information)
+      target_instance_id = @eip_inquiry.which_instance_should_get_eip(moveable_eip)
+      @stack.transfer_eip(moveable_eip, target_instance_id)
+      { ip_address: moveable_eip[:ip_address], target_instance_id: target_instance_id }
     end
+    private :move_eip
   end
 end

data/lib/opsicle/commands/stop_instance.rb

@@ -14,7 +14,7 @@ module Opsicle
       @ec2_adapter = Ec2Adapter.new(@client)
       @opsworks_adapter = OpsworksAdapter.new(@client)
       stack_id = @client.config.opsworks_config[:stack_id]
-      @stack = ManageableStack.new(stack_id, @opsworks_adapter.client)
+      @stack = ManageableStack.new(stack_id, @opsworks_adapter)
       @cli = HighLine.new
     end
 
@@ -38,7 +38,7 @@ module Opsicle
 
     def select_layer
       puts "\nLayers:\n"
-      ops_layers = @opsworks_adapter.get_layers(@stack.id)
+      ops_layers = @opsworks_adapter.layers(@stack.id)
 
       layers = []
       ops_layers.each do |layer|

data/lib/opsicle/manageable_stack.rb

@@ -1,34 +1,28 @@
 module Opsicle
   class ManageableStack
-    attr_accessor :id, :opsworks, :stack, :vpc_id, :eips, :cli
+    attr_accessor :id, :opsworks_adapter, :stack, :vpc_id, :eips, :cli
 
-    def initialize(stack_id, opsworks, cli=nil)
+    STOPPABLE_STATES = %w(start_failed stop_failed online running_setup setup_failed booting rebooting)
+
+    def initialize(stack_id, opsworks_adapter, cli=nil)
       self.id = stack_id
-      self.opsworks = opsworks
+      self.opsworks_adapter = opsworks_adapter
       self.cli = cli
-      self.stack = get_stack
+      self.stack = @opsworks_adapter.stack(id)
       self.vpc_id = self.stack.vpc_id
-      self.eips = nil
-    end
-
-    def get_stack
-      @opsworks.describe_stacks({ :stack_ids => [self.id.to_s] }).stacks.first
-    end
-
-    def get_eips
-      self.eips = @opsworks.describe_elastic_ips(stack_id: self.id.to_s).elastic_ips
     end
 
-    def gather_eip_information
+    def eips
+      eips = @opsworks_adapter.elastic_ips(id)
       eip_information = []
 
-      @eips.each do |eip|
+      eips.each do |eip|
         next unless eip.instance_id
         instance_id = eip.instance_id
-        instance = @opsworks.describe_instances(instance_ids: [instance_id]).instances.first
+        instance = @opsworks_adapter.instance(instance_id)
         instance_name = instance.hostname
         layer_id = instance.layer_ids.first
-        layer = @opsworks.describe_layers(layer_ids: [layer_id]).layers.first
+        layer = @opsworks_adapter.layer(layer_id)
         layer_name = layer.name
         eip_information << { eip: eip, ip_address: eip.ip, instance_name: instance_name, layer_id: layer_id }
       end
@@ -36,49 +30,28 @@ module Opsicle
       eip_information
     end
 
-    def ask_which_eip_to_move(eip_information)
-      puts "\nHere are all of the EIPs for this stack:"
-      eip_information.each_with_index { |h, index| puts "#{index.to_i + 1}) #{h[:ip_address]} connected to #{h[:instance_name]}" }
-      eip_index = @cli.ask("Which EIP would you like to move?\n", Integer) { |q| q.in = 1..eip_information.length.to_i } - 1
-      eip_information[eip_index]
-    end
-
-    def ask_which_target_instance(moveable_eip)
-      puts "\nHere are all of the instances in the current instance's layer:"
-      instances = @opsworks.describe_instances(layer_id: moveable_eip[:layer_id]).instances
-      instances = instances.select { |instance| instance.elastic_ip.nil? && instance.auto_scaling_type.nil? }
-      instances.each_with_index { |instance, index| puts "#{index.to_i + 1}) #{instance.status} - #{instance.hostname}" }
-      instance_index = @cli.ask("What is your target instance?\n", Integer) { |q| q.in = 1..instances.length.to_i } - 1
-      instances[instance_index].instance_id
-    end
-
     def transfer_eip(moveable_eip, target_instance_id)
-      @opsworks.associate_elastic_ip({ elastic_ip: moveable_eip[:ip_address], instance_id: target_instance_id })
-      puts "\nEIP #{moveable_eip[:ip_address]} was moved to instance #{target_instance_id}"
-    end
-
-    def move_eip
-      get_eips
-      eip_information = gather_eip_information
-      moveable_eip = ask_which_eip_to_move(eip_information)
-      target_instance_id = ask_which_target_instance(moveable_eip)
-      transfer_eip(moveable_eip, target_instance_id)
+      @opsworks_adapter.associate_elastic_ip(moveable_eip[:ip_address], target_instance_id)
     end
 
     def instances
-      @opsworks.describe_instances(stack_id: self.id).instances
+      @opsworks_adapter.instances_by_stack(id)
     end
 
     def deleteable_instances(layer)
-      instances.select{ |instance| instance.auto_scaling_type.nil?  && instance.status == "stopped" && instance.layer_ids.include?(layer.layer_id) }
-    end
-
-    def stoppable_states
-      %w(start_failed stop_failed online running_setup setup_failed booting rebooting)
+      instances.select do |instance|
+        instance.auto_scaling_type.nil? &&
+        instance.status == "stopped" &&
+        instance.layer_ids.include?(layer.layer_id)
+      end
     end
 
     def stoppable_instances(layer)
-      instances.select{ |instance| instance.elastic_ip.nil?  && stoppable_states.include?(instance.status) && instance.layer_ids.include?(layer.layer_id) }
+      instances.select do |instance|
+        instance.elastic_ip.nil? &&
+        STOPPABLE_STATES.include?(instance.status) &&
+        instance.layer_ids.include?(layer.layer_id)
+      end
     end
   end
 end

data/lib/opsicle/opsworks_adapter.rb

@@ -1,14 +1,45 @@
 class Opsicle::OpsworksAdapter
   attr_reader :client
 
-  def initialize(client)
-    @client = client.opsworks
+  def initialize(opsicle_client)
+    @client = opsicle_client.opsworks
   end
 
-  def get_layers(stack_id)
+  def stack(stack_id)
+    client.describe_stacks(stack_ids: [ stack_id ]).stacks.first
+  end
+
+  def layers(stack_id)
     client.describe_layers(stack_id: stack_id).layers
   end
 
+  def layer(layer_id)
+    client.describe_layers(layer_ids: [ layer_id ]).layers.first
+  end
+
+  def instances_by_stack(stack_id)
+    client.describe_instances(stack_id: stack_id).instances
+  end
+
+  def instances_by_layer(layer_id)
+    client.describe_instances(layer_id: layer_id).instances
+  end
+
+  def instance(instance_id)
+    client.describe_instances(instance_ids: [ instance_id ]).instances.first
+  end
+
+  def elastic_ips(stack_id)
+    client.describe_elastic_ips(stack_id: stack_id).elastic_ips
+  end
+
+  def associate_elastic_ip(elastic_ip, target_instance_id)
+    client.associate_elastic_ip(
+      elastic_ip: elastic_ip,
+      instance_id: target_instance_id
+    )
+  end
+
   def start_instance(instance_id)
     client.start_instance(instance_id: instance_id)
   end

data/lib/opsicle/questionnaire/eip_inquiry.rb

@@ -0,0 +1,48 @@
+module Opsicle
+  module Questionnaire
+    class EipInquiry
+      attr_accessor :opsworks_adapter, :cli
+
+      def initialize(options)
+        self.opsworks_adapter = options[:opsworks_adapter]
+        self.cli = options[:highline_client]
+      end
+
+      def which_eip_should_move(eip_information)
+        puts "\nHere are all of the EIPs for this stack:"
+        print_current_eips(eip_information)
+        eip_index = ask_eip_question("Which EIP would you like to move?\n", eip_information)
+        eip_information[eip_index]
+      end
+
+      def which_instance_should_get_eip(moveable_eip)
+        puts "\nHere are all of the instances in the current instance's layer:"
+        instances = get_potential_target_instances(moveable_eip)
+        print_potential_target_instances(instances)
+        instance_index = ask_eip_question("What is your target instance?\n", instances)
+        instances[instance_index].instance_id
+      end
+
+      def ask_eip_question(prompt, choices)
+        @cli.ask(prompt, Integer) { |q| q.in = 1..choices.length.to_i } - 1
+      end
+      private :ask_eip_question
+
+      def print_current_eips(eip_information)
+        eip_information.each_with_index { |eip, index| puts "#{index.to_i + 1}) #{eip[:ip_address]} connected to #{eip[:instance_name]}" }
+      end
+      private :print_current_eips
+
+      def print_potential_target_instances(instances)
+        instances.each_with_index { |instance, index| puts "#{index.to_i + 1}) #{instance.status} - #{instance.hostname}" }
+      end
+      private :print_potential_target_instances
+
+      def get_potential_target_instances(moveable_eip)
+        instances = @opsworks_adapter.instances_by_layer(moveable_eip[:layer_id])
+        instances.select { |instance| instance.elastic_ip.nil? && instance.auto_scaling_type.nil? }
+      end
+      private :get_potential_target_instances
+    end
+  end
+end

data/lib/opsicle/version.rb

@@ -1,3 +1,3 @@
 module Opsicle
-  VERSION = "2.12.2"
+  VERSION = "2.12.3"
 end

data/spec/opsicle/commands/move_eip_spec.rb

@@ -0,0 +1,47 @@
+describe Opsicle::MoveEip do
+  let(:old_eip) { 'old_eip' }
+  let(:target_instance) { 'target_instance' }
+  let(:opsworks_eip) { double(:eip, ip: "123", instance_id: "1234") }
+  let(:opsworks_stack) { double(:stack, id: "1234", vpc_id: "21") }
+  let(:config) { double(:config, opsworks_config: {stack_id: "123"}) }
+  let(:opsicle_client) { double(:client, config: config, opsworks: opsworks_client) }
+
+  let(:opsworks_client) do
+    double(:opsworks_client,
+      associate_elastic_ip: true,
+      describe_stacks: double(stack: [opsworks_stack])
+    )
+  end
+
+  let(:opsworks_adapter) do
+    double(:opsworks_adapter,
+      client: opsworks_client,
+      stack: opsworks_stack,
+      elastic_ips: [opsworks_eip, opsworks_eip],
+      instance: double(hostname: 'hostname', layer_ids: ['layer_id']),
+      layer: double(name: 'layer_name')
+    )
+  end
+
+  let(:eip_inquiry) do
+    double(:eip_inquiry,
+      which_eip_should_move: { ip_address: old_eip },
+      which_instance_should_get_eip: target_instance
+    )
+  end
+
+  before do
+    allow(Opsicle::Client).to receive(:new).with("staging").and_return(opsicle_client)
+    allow(Opsicle::OpsworksAdapter).to receive(:new).and_return(opsworks_adapter)
+    allow(Opsicle::Questionnaire::EipInquiry).to receive(:new).and_return(eip_inquiry)
+  end
+
+  subject { described_class.new("staging") }
+
+  describe "#execute" do
+    it "should make an API call to opsworks to change the EIP" do
+      expect(opsworks_adapter).to receive(:associate_elastic_ip).with(old_eip, target_instance)
+      subject.execute
+    end
+  end
+end

data/spec/opsicle/manageable_stack_spec.rb

@@ -1,23 +1,96 @@
-require "spec_helper"
-require "opsicle"
-require 'gli'
-require "opsicle/user_profile"
-
-module Opsicle
-  describe ManageableStack do
-    before do
-    @stack = double('stack', :vpc_id => 'vpc-123456')
-      @stacks = double('stacks', :stacks => [@stack])
-      @opsworks = double('opsworks', :describe_stacks => @stacks)
+describe Opsicle::ManageableStack do
+  let(:deleteable_instance) do
+    double(:deleteable_instance,
+      auto_scaling_type: nil,
+      status: "stopped",
+      layer_ids: ["456"],
+      elastic_ip: nil,
+      hostname: "example-hostname"
+    )
+  end
+
+  let(:stoppable_instance) do
+    double(:stoppable_instance,
+      elastic_ip: nil,
+      status: "start_failed",
+      layer_ids: ["456"],
+      auto_scaling_type: nil
+    )
+  end
+
+  let(:stack) { double(:stack, vpc_id: "id") }
+
+  let(:layer) do
+    double(:layer,
+      layer_id: "456",
+      name: "layername"
+    )
+  end
+
+  let(:eip) do
+    double(:eip,
+      instance_id: "123",
+      ip: "ip-123",
+    )
+  end
+
+  let(:client) { double(:client, opsworks: aws_opsworks_client) }
+
+  let(:aws_opsworks_client) do
+    double(:aws_opsworks_client,
+      describe_stacks: double(:stacks, stacks: [stack]),
+      describe_layers: double(:layers, layers: [layer]),
+      describe_instances: double(:instances, instances: [deleteable_instance, stoppable_instance]),
+      describe_elastic_ips: double(:eips, elastic_ips: [eip]),
+      associate_elastic_ip: :associated_eip,
+      start_instance: :started,
+      stop_instance: :stopped,
+      delete_instance: :deleted
+    )
+  end
+
+  let(:opsworks_adapter) { Opsicle::OpsworksAdapter.new(client) }
+  let(:stack_id) { '123' }
+  
+  subject { described_class.new(stack_id, opsworks_adapter) }
+
+  describe "#eips" do
+    let(:eips) { subject.eips }
+
+    it "should properly find and format EIPs" do
+      expect(eips).to eq([{eip: eip, ip_address: "ip-123", instance_name: "example-hostname", layer_id: "456"}])
     end
+  end
+
+  describe "#transfer_eip" do
+    let(:transfer) { subject.transfer_eip({ip_address: true}, "target_instance_id") }
+
+    it "should properly transfer the EIP" do
+      expect(transfer).to eq(:associated_eip)
+    end
+  end
+
+  describe "#instances" do
+    let(:instances) { subject.instances }
+
+    it "should properly gather a list of instances" do
+      expect(instances).to eq([deleteable_instance, stoppable_instance])
+    end
+  end
+
+  describe "#deleteable_instances" do
+    let(:instances) { subject.deleteable_instances(layer) }
+
+    it "should look for any instances that are deleteable" do
+      expect(instances).to eq([deleteable_instance])
+    end
+  end
+
+  describe "#stoppable_instances" do
+    let(:instances) { subject.stoppable_instances(layer) }
 
-    context "#get_stack" do
-      it "should gather opsworks instances for that layer" do
-        stack = ManageableStack.new(12345, @opsworks)
-        expect(@opsworks).to receive(:describe_stacks).and_return(@stacks)
-        expect(@stacks).to receive(:stacks)
-        stack.get_stack
-      end
+    it "should look for any instances that are stoppable" do
+      expect(instances).to eq([stoppable_instance])
     end
   end
 end

data/spec/opsicle/opsworks_adapter_spec.rb

@@ -1,7 +1,11 @@
 describe Opsicle::OpsworksAdapter do
   let(:aws_opsworks_client) do
     double(:aws_opsworks_client,
-      describe_layers: double(:layers, layers: []),
+      describe_stacks: double(:stacks, stacks: [:stack_one]),
+      describe_layers: double(:layers, layers: [:layer_one]),
+      describe_instances: double(:instances, instances: [:instance_one]),
+      describe_elastic_ips: double(:eips, elastic_ips: []),
+      associate_elastic_ip: :associated_eip,
       start_instance: :started,
       stop_instance: :stopped,
       delete_instance: :deleted
@@ -11,11 +15,67 @@ describe Opsicle::OpsworksAdapter do
 
   subject { described_class.new(client) }
 
-  describe "#get_layers" do
-    let(:get_layers) { subject.get_layers(:stack_id) }
+  describe "#stack" do
+    let(:stack) { subject.stack(:stack_id) }
+
+    it "should return the stack in question" do
+      expect(stack).to eq(:stack_one)
+    end
+  end
+
+  describe "#layers" do
+    let(:layers) { subject.layers(:stack_id) }
 
     it "should gather an array of layers for this opsworks client" do
-      expect(get_layers).to be_empty
+      expect(layers).to eq([:layer_one])
+    end
+  end
+
+  describe "#layer" do
+    let (:layer) { subject.layer(:layer_id) }
+
+    it "should get the layer in question" do
+      expect(layer).to eq(:layer_one)
+    end
+  end
+
+  describe "#instances_by_stack" do
+    let (:instances) { subject.instances_by_stack(:stack_id) }
+
+    it "should get a list of instances in the stack" do
+      expect(instances).to eq([:instance_one])
+    end
+  end
+
+  describe "#instances_by_layer" do
+    let (:instances) { subject.instances_by_layer(:layer_id) }
+
+    it "should get a list of instances in the_layer" do
+      expect(instances).to eq([:instance_one])
+    end
+  end
+
+  describe "#instance" do
+    let (:instance) { subject.instance(:instance_id) }
+
+    it "should get the instance in question" do
+      expect(instance).to eq(:instance_one)
+    end
+  end
+
+  describe "#elastic_ips" do
+    let (:eips) { subject.elastic_ips(:stack_id) }
+
+    it "should get a list of the EIPs in the stack" do
+      expect(eips).to be_empty
+    end
+  end
+
+  describe "#associate_elastic_ip" do
+    let (:associated) { subject.associate_elastic_ip(:eip, :target_instance_id) }
+
+    it "should get a list of the EIPs in the stack" do
+      expect(associated).to be(:associated_eip)
     end
   end
 

data/spec/opsicle/questionnaire/eip_inquiry_spec.rb

@@ -0,0 +1,70 @@
+describe Opsicle::Questionnaire::EipInquiry do
+  let(:cli) { double(:cli, ask: 1) }
+
+  let(:online_instance_with_eip) do
+    double(:instance,
+      elastic_ip: true,
+      auto_scaling_type: nil,
+      status: "online",
+      hostname: "example",
+      instance_id: "instance-id"
+    )
+  end
+
+  let(:online_instance_without_eip) do
+    double(:instance,
+      elastic_ip: nil,
+      auto_scaling_type: nil,
+      status: "online",
+      hostname: "example",
+      instance_id: "instance-id"
+    )
+  end
+
+  let(:stopped_instance) do
+    double(:instance,
+      elastic_ip: nil,
+      auto_scaling_type: nil,
+      status: "stopped",
+      hostname: "example",
+      instance_id: "instance-id"
+    )
+  end
+
+  let(:client) { double(:client, opsworks: aws_opsworks_client) }
+
+  let(:aws_opsworks_client) do
+    double(:aws_opsworks_client,
+      describe_instances: double(:instances, instances: [online_instance_with_eip, online_instance_without_eip, stopped_instance])
+    )
+  end
+
+  let(:opsworks_adapter) { Opsicle::OpsworksAdapter.new(client) }
+
+  let(:eip_info) do
+    { eip: true, ip_address: "ip-123", instance_name: "example-hostname", layer_id: "id1" }
+  end
+  
+  subject { described_class.new(opsworks_adapter: opsworks_adapter, highline_client: cli) }
+
+  describe "#which_eip_should_move" do
+    let(:eip_response) { subject.which_eip_should_move([eip_info]) }
+
+    it "should ask the user a question" do
+      expect(cli).to receive(:ask)
+      subject.which_eip_should_move([eip_info])
+    end
+
+    it "should return the EIP to move" do
+      expect(eip_response).to eq(eip_info)
+    end
+  end
+
+  describe "#which_instance_should_get_eip" do
+    let(:instance_response) { subject.which_instance_should_get_eip(eip_info) }
+
+    it "should return a single instance" do
+      expect(instance_response).to eq("instance-id")
+    end
+  end
+end

data/spec/spec_helper.rb

@@ -15,4 +15,5 @@ RSpec.configure do |config|
   # the seed, which is printed after each run.
   #     --seed 1234
   config.order = 'random'
+  config.filter_run focus: true
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: opsicle
 version: !ruby/object:Gem::Version
-  version: 2.12.2
+  version: 2.12.3
 platform: ruby
 authors:
 - Andy Fleener
@@ -238,6 +238,7 @@ files:
 - lib/opsicle/monitor/translatable.rb
 - lib/opsicle/opsworks_adapter.rb
 - lib/opsicle/output.rb
+- lib/opsicle/questionnaire/eip_inquiry.rb
 - lib/opsicle/s3_bucket.rb
 - lib/opsicle/stack.rb
 - lib/opsicle/user_profile.rb
@@ -251,6 +252,7 @@ files:
 - spec/opsicle/commands/failure_log_spec.rb
 - spec/opsicle/commands/list_instances_spec.rb
 - spec/opsicle/commands/list_spec.rb
+- spec/opsicle/commands/move_eip_spec.rb
 - spec/opsicle/commands/ssh_key_spec.rb
 - spec/opsicle/commands/ssh_spec.rb
 - spec/opsicle/commands/update_spec.rb
@@ -270,6 +272,7 @@ files:
 - spec/opsicle/monitor/subpanel_spec.rb
 - spec/opsicle/opsworks_adapter_spec.rb
 - spec/opsicle/output_spec.rb
+- spec/opsicle/questionnaire/eip_inquiry_spec.rb
 - spec/opsicle/s3_bucket_spec.rb
 - spec/opsicle/user_profile_spec.rb
 - spec/spec_helper.rb
@@ -311,6 +314,7 @@ test_files:
 - spec/opsicle/manageable_stack_spec.rb
 - spec/opsicle/s3_bucket_spec.rb
 - spec/opsicle/manageable_layer_spec.rb
+- spec/opsicle/questionnaire/eip_inquiry_spec.rb
 - spec/opsicle/opsworks_adapter_spec.rb
 - spec/opsicle/ec2_adapter_spec.rb
 - spec/opsicle/commands/deploy_spec.rb
@@ -321,6 +325,7 @@ test_files:
 - spec/opsicle/commands/update_spec.rb
 - spec/opsicle/commands/failure_log_spec.rb
 - spec/opsicle/commands/delete_instance_spec.rb
+- spec/opsicle/commands/move_eip_spec.rb
 - spec/opsicle/commands/list_instances_spec.rb
 - spec/opsicle/commands/execute_recipes_spec.rb
 - spec/opsicle/commands/ssh_spec.rb