mirror of
https://github.com/fog/fog.git
synced 2022-11-09 13:51:43 -05:00
Merge pull request #2335 from faber/aws-internet-gateway-parser-bugfix
Fix parser typo in AWS #describe_internet_gateways
This commit is contained in:
commit
c64f71921b
7 changed files with 15 additions and 8 deletions
|
@ -223,7 +223,6 @@ module Fog
|
||||||
:subnets => [],
|
:subnets => [],
|
||||||
:vpcs => [],
|
:vpcs => [],
|
||||||
:dhcp_options => [],
|
:dhcp_options => [],
|
||||||
:internet_gateways => [],
|
|
||||||
:route_tables => [],
|
:route_tables => [],
|
||||||
:account_attributes => [
|
:account_attributes => [
|
||||||
{
|
{
|
||||||
|
|
|
@ -26,7 +26,7 @@ module Fog
|
||||||
if @in_tag_set
|
if @in_tag_set
|
||||||
case name
|
case name
|
||||||
when 'item'
|
when 'item'
|
||||||
@vpc['tagSet'][@tag['key']] = @tag['value']
|
@internet_gateway['tagSet'][@tag['key']] = @tag['value']
|
||||||
@tag = {}
|
@tag = {}
|
||||||
when 'key', 'value'
|
when 'key', 'value'
|
||||||
@tag[name] = value
|
@tag[name] = value
|
||||||
|
|
|
@ -33,16 +33,17 @@ module Fog
|
||||||
|
|
||||||
class Mock
|
class Mock
|
||||||
def create_internet_gateway()
|
def create_internet_gateway()
|
||||||
self.data[:internet_gateways].push({
|
gateway_id = Fog::AWS::Mock.internet_gateway_id
|
||||||
'internetGatewayId' => Fog::AWS::Mock.request_id,
|
self.data[:internet_gateways][gateway_id] = {
|
||||||
|
'internetGatewayId' => gateway_id,
|
||||||
'attachmentSet' => {},
|
'attachmentSet' => {},
|
||||||
'tagSet' => {}
|
'tagSet' => {}
|
||||||
})
|
}
|
||||||
Excon::Response.new(
|
Excon::Response.new(
|
||||||
:status => 200,
|
:status => 200,
|
||||||
:body => {
|
:body => {
|
||||||
'requestId' => Fog::AWS::Mock.request_id,
|
'requestId' => Fog::AWS::Mock.request_id,
|
||||||
'internetGatewaySet' => self.data[:internet_gateways]
|
'internetGatewaySet' => [self.data[:internet_gateways][gateway_id]]
|
||||||
}
|
}
|
||||||
)
|
)
|
||||||
end
|
end
|
||||||
|
|
|
@ -53,6 +53,8 @@ module Fog
|
||||||
'snapshot'
|
'snapshot'
|
||||||
when /^vol\-[a-z0-9]{8}$/i
|
when /^vol\-[a-z0-9]{8}$/i
|
||||||
'volume'
|
'volume'
|
||||||
|
when /^igw\-[a-z0-9]{8}$/i
|
||||||
|
'internet_gateway'
|
||||||
end
|
end
|
||||||
if type && ((type == 'image' && visible_images[resource_id]) || self.data[:"#{type}s"][resource_id])
|
if type && ((type == 'image' && visible_images[resource_id]) || self.data[:"#{type}s"][resource_id])
|
||||||
{ 'resourceId' => resource_id, 'resourceType' => type }
|
{ 'resourceId' => resource_id, 'resourceType' => type }
|
||||||
|
|
|
@ -30,7 +30,7 @@ module Fog
|
||||||
Excon::Response.new.tap do |response|
|
Excon::Response.new.tap do |response|
|
||||||
if internet_gateway_id
|
if internet_gateway_id
|
||||||
response.status = 200
|
response.status = 200
|
||||||
self.data[:internet_gateways].reject! { |v| v['internetGatewayId'] == internet_gateway_id }
|
self.data[:internet_gateways].delete(internet_gateway_id)
|
||||||
|
|
||||||
response.body = {
|
response.body = {
|
||||||
'requestId' => Fog::AWS::Mock.request_id,
|
'requestId' => Fog::AWS::Mock.request_id,
|
||||||
|
|
|
@ -40,7 +40,7 @@ module Fog
|
||||||
|
|
||||||
class Mock
|
class Mock
|
||||||
def describe_internet_gateways(filters = {})
|
def describe_internet_gateways(filters = {})
|
||||||
internet_gateways = self.data[:internet_gateways]
|
internet_gateways = self.data[:internet_gateways].values
|
||||||
|
|
||||||
if filters['internet-gateway-id']
|
if filters['internet-gateway-id']
|
||||||
internet_gateways = internet_gateways.reject {|internet_gateway| internet_gateway['internetGatewayId'] != filters['internet-gateway-id']}
|
internet_gateways = internet_gateways.reject {|internet_gateway| internet_gateway['internetGatewayId'] != filters['internet-gateway-id']}
|
||||||
|
|
|
@ -26,6 +26,11 @@ Shindo.tests('Fog::Compute[:aws] | internet_gateway requests', ['aws']) do
|
||||||
tests('#describe_internet_gateways').formats(@internet_gateways_format) do
|
tests('#describe_internet_gateways').formats(@internet_gateways_format) do
|
||||||
Fog::Compute[:aws].describe_internet_gateways.body
|
Fog::Compute[:aws].describe_internet_gateways.body
|
||||||
end
|
end
|
||||||
|
|
||||||
|
tests('#describe_internet_gateways with tags').formats(@internet_gateways_format) do
|
||||||
|
Fog::Compute[:aws].create_tags @igw_id, {"environment" => "production"}
|
||||||
|
Fog::Compute[:aws].describe_internet_gateways.body
|
||||||
|
end
|
||||||
|
|
||||||
tests("#attach_internet_gateway('#{@igw_id}, #{@vpc_id}')").formats(AWS::Compute::Formats::BASIC) do
|
tests("#attach_internet_gateway('#{@igw_id}, #{@vpc_id}')").formats(AWS::Compute::Formats::BASIC) do
|
||||||
Fog::Compute[:aws].attach_internet_gateway(@igw_id, @vpc_id).body
|
Fog::Compute[:aws].attach_internet_gateway(@igw_id, @vpc_id).body
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue