diff --git a/lib/fog/aws/requests/ec2/revoke_security_group_ingress.rb b/lib/fog/aws/requests/ec2/revoke_security_group_ingress.rb index c40292d55..e9b6ff8c7 100644 --- a/lib/fog/aws/requests/ec2/revoke_security_group_ingress.rb +++ b/lib/fog/aws/requests/ec2/revoke_security_group_ingress.rb @@ -33,7 +33,6 @@ module Fog class Mock - # TODO: handle the GroupName/Source/Source case def revoke_security_group_ingress(options = {}) if options['GroupName'] && options['SourceSecurityGroupName'] && options['SourceSecurityGroupOwnerId'] raise MockNotImplemented.new("Contributions welcome!") diff --git a/lib/fog/aws/requests/ec2/run_instances.rb b/lib/fog/aws/requests/ec2/run_instances.rb index 1f90a0430..d8a284697 100644 --- a/lib/fog/aws/requests/ec2/run_instances.rb +++ b/lib/fog/aws/requests/ec2/run_instances.rb @@ -92,8 +92,6 @@ module Fog class Mock - # TODO: allow for block device mapping in mocks - # TODO: allow for rootDeviceType specification def run_instances(image_id, min_count, max_count, options = {}) response = Excon::Response.new response.status = 200 diff --git a/lib/fog/aws/requests/ec2/terminate_instances.rb b/lib/fog/aws/requests/ec2/terminate_instances.rb index 6b000ad58..c76877fbb 100644 --- a/lib/fog/aws/requests/ec2/terminate_instances.rb +++ b/lib/fog/aws/requests/ec2/terminate_instances.rb @@ -44,7 +44,6 @@ module Fog for id in instance_id instance = @data[:instances][id] @data[:deleted_at][id] = Time.now - # TODO: the codes are mostly educated guessing, not certainty code = case instance['instanceState']['name'] when 'pending' 0 diff --git a/lib/fog/aws/requests/s3/copy_object.rb b/lib/fog/aws/requests/s3/copy_object.rb index 84b1b487c..bd54f3318 100644 --- a/lib/fog/aws/requests/s3/copy_object.rb +++ b/lib/fog/aws/requests/s3/copy_object.rb @@ -23,7 +23,6 @@ module Fog # * 'ETag'<~String> - etag of new object # * 'LastModified'<~Time> - date object was last modified # - # TODO: allow specifying new metadata (support all/some of put_object?) def copy_object(source_bucket_name, source_object_name, target_bucket_name, target_object_name, options = {}) headers = { 'x-amz-copy-source' => "/#{source_bucket_name}/#{source_object_name}" }.merge!(options) request({ diff --git a/lib/fog/aws/requests/s3/get_bucket.rb b/lib/fog/aws/requests/s3/get_bucket.rb index 220f5016a..34ffa147a 100644 --- a/lib/fog/aws/requests/s3/get_bucket.rb +++ b/lib/fog/aws/requests/s3/get_bucket.rb @@ -57,7 +57,6 @@ module Fog class Mock - # FIXME: implement delimiter def get_bucket(bucket_name, options = {}) unless bucket_name raise ArgumentError.new('bucket_name is required') diff --git a/lib/fog/terremark/requests/shared/delete_internet_service.rb b/lib/fog/terremark/requests/shared/delete_internet_service.rb index 5813b78aa..e141a8e5a 100644 --- a/lib/fog/terremark/requests/shared/delete_internet_service.rb +++ b/lib/fog/terremark/requests/shared/delete_internet_service.rb @@ -8,18 +8,6 @@ module Fog # ==== Parameters # * internet_service_id<~Integer> - Id of service to destroy # - # ==== Returns - # * response<~Excon::Response>: - # * body<~Hash>: - - # FIXME - - # * 'CatalogItems'<~Array> - # * 'href'<~String> - linke to item - # * 'name'<~String> - name of item - # * 'type'<~String> - type of item - # * 'description'<~String> - Description of catalog - # * 'name'<~String> - Name of catalog def delete_internet_service(internet_service_id) request( :expects => 200, diff --git a/lib/fog/terremark/requests/shared/delete_node_service.rb b/lib/fog/terremark/requests/shared/delete_node_service.rb index 3216db916..a6e694b48 100644 --- a/lib/fog/terremark/requests/shared/delete_node_service.rb +++ b/lib/fog/terremark/requests/shared/delete_node_service.rb @@ -8,18 +8,6 @@ module Fog # ==== Parameters # * node_service_id<~Integer> - Id of node to destroy # - # ==== Returns - # * response<~Excon::Response>: - # * body<~Hash>: - - # FIXME - - # * 'CatalogItems'<~Array> - # * 'href'<~String> - linke to item - # * 'name'<~String> - name of item - # * 'type'<~String> - type of item - # * 'description'<~String> - Description of catalog - # * 'name'<~String> - Name of catalog def delete_node_service(node_service_id) request( :expects => 200, diff --git a/lib/fog/terremark/requests/shared/delete_vapp.rb b/lib/fog/terremark/requests/shared/delete_vapp.rb index c3e5802dd..8070bbab7 100644 --- a/lib/fog/terremark/requests/shared/delete_vapp.rb +++ b/lib/fog/terremark/requests/shared/delete_vapp.rb @@ -8,18 +8,6 @@ module Fog # ==== Parameters # * vapp_id<~Integer> - Id of vapp to destroy # - # ==== Returns - # * response<~Excon::Response>: - # * body<~Hash>: - - # FIXME - - # * 'CatalogItems'<~Array> - # * 'href'<~String> - linke to item - # * 'name'<~String> - name of item - # * 'type'<~String> - type of item - # * 'description'<~String> - Description of catalog - # * 'name'<~String> - Name of catalog def delete_vapp(vapp_id) request( :expects => 202, diff --git a/lib/fog/terremark/shared.rb b/lib/fog/terremark/shared.rb index 196c36ffe..0e4f475fb 100644 --- a/lib/fog/terremark/shared.rb +++ b/lib/fog/terremark/shared.rb @@ -5,7 +5,6 @@ module Fog # Commond methods shared by Real and Mock module Common - # TODO: bust cache on organization creation? def default_organization_id @default_organization_id ||= begin org_list = get_organizations.body['OrgList'] diff --git a/tests/slicehost/requests/get_backups_tests.rb b/tests/slicehost/requests/get_backups_tests.rb index d74bb124e..e542fdf8c 100644 --- a/tests/slicehost/requests/get_backups_tests.rb +++ b/tests/slicehost/requests/get_backups_tests.rb @@ -5,7 +5,6 @@ Shindo.tests('Slicehost#get_backups', 'slicehost') do @data = Slicehost[:slices].get_backups.body end - # TODO: ensure this still works with a non-empty list test('has proper output format') do has_format(@data, { 'backups' => [Slicehost::Formats::BACKUP] }) end