Fix #741 Add tests for ListObjectsV2 API in AWS storage module

- Introduced comprehensive tests for the ListObjectsV2 API, covering basic functionality, parameter handling (prefix, max-keys, start-after, delimiter, fetch-owner), and pagination with continuation tokens.
- Created separate test files for both model and request layers to ensure thorough validation of the API's behavior and response structure.

Author: Ankk98

tests/models/storage/files_v2_tests.rb

@@ -0,0 +1,116 @@
+Shindo.tests("Storage[:aws] | ListObjectsV2 API", ["aws"]) do
+
+  directory_attributes = {
+      :key => uniq_id('foglistobjectsv2tests')
+  }
+
+  @directory = Fog::Storage[:aws].directories.create(directory_attributes)
+
+  tests('Direct ListObjectsV2 API usage') do
+    
+    # Create some test files
+    file1 = @directory.files.create(:body => 'test1', :key => 'prefix/file1.txt')
+    file2 = @directory.files.create(:body => 'test2', :key => 'prefix/file2.txt')
+    file3 = @directory.files.create(:body => 'test3', :key => 'other/file3.txt')
+    file4 = @directory.files.create(:body => 'test4', :key => 'file4.txt')
+
+    tests('#list_objects_v2 basic functionality') do
+      response = Fog::Storage[:aws].list_objects_v2(@directory.key)
+      
+      tests('returns proper response structure').returns(true) do
+        response.body.has_key?('Contents') && 
+        response.body.has_key?('KeyCount') &&
+        response.body.has_key?('IsTruncated')
+      end
+
+      tests('returns all files').returns(4) do
+        response.body['Contents'].size
+      end
+
+      tests('has V2-specific KeyCount').returns(4) do
+        response.body['KeyCount']
+      end
+    end
+
+    tests('#list_objects_v2 with parameters') do
+      
+      tests('with prefix') do
+        response = Fog::Storage[:aws].list_objects_v2(@directory.key, 'prefix' => 'prefix/')
+        
+        tests('filters by prefix').returns(2) do
+          response.body['Contents'].size
+        end
+        
+        tests('KeyCount reflects filtered results').returns(2) do
+          response.body['KeyCount']
+        end
+      end
+
+      tests('with max-keys') do
+        response = Fog::Storage[:aws].list_objects_v2(@directory.key, 'max-keys' => 2)
+        
+        tests('limits results').returns(2) do
+          response.body['Contents'].size
+        end
+        
+        tests('is truncated').returns(true) do
+          response.body['IsTruncated']
+        end
+        
+        tests('has next continuation token').returns(true) do
+          !response.body['NextContinuationToken'].nil?
+        end
+      end
+
+      tests('with start-after') do
+        response = Fog::Storage[:aws].list_objects_v2(@directory.key, 'start-after' => 'other/file3.txt')
+        
+        tests('starts after specified key').returns(true) do
+          keys = response.body['Contents'].map { |obj| obj['Key'] }
+          keys.none? { |key| key <= 'other/file3.txt' }
+        end
+      end
+
+      tests('with delimiter') do
+        response = Fog::Storage[:aws].list_objects_v2(@directory.key, 'delimiter' => '/')
+        
+        tests('respects delimiter').returns(true) do
+          # Should have common prefixes and fewer direct contents
+          response.body.has_key?('CommonPrefixes') && response.body['CommonPrefixes'].size > 0
+        end
+      end
+
+      tests('with fetch-owner') do
+        response = Fog::Storage[:aws].list_objects_v2(@directory.key, 'fetch-owner' => true)
+        
+        tests('request succeeds').returns(true) do
+          response.body.has_key?('Contents')
+        end
+      end unless Fog.mocking?
+
+    end
+
+    tests('pagination with continuation token') do
+      first_page = Fog::Storage[:aws].list_objects_v2(@directory.key, 'max-keys' => 2)
+      
+      if first_page.body['IsTruncated'] && first_page.body['NextContinuationToken']
+        second_page = Fog::Storage[:aws].list_objects_v2(@directory.key, 'continuation-token' => first_page.body['NextContinuationToken'])
+        
+        tests('second page has different objects').returns(true) do
+          first_keys = first_page.body['Contents'].map { |obj| obj['Key'] }
+          second_keys = second_page.body['Contents'].map { |obj| obj['Key'] }
+          (first_keys & second_keys).empty?
+        end
+      end
+    end
+
+    # Clean up test files
+    file1.destroy
+    file2.destroy
+    file3.destroy
+    file4.destroy
+  end
+
+  @directory.destroy
+
+end 

tests/requests/storage/list_objects_v2_tests.rb

@@ -0,0 +1,193 @@
+Shindo.tests('Fog::Storage[:aws] | list_objects_v2 requests', ["aws"]) do
+  @aws_bucket_name = 'foglistobjectsv2tests-' + Time.now.to_i.to_s(32)
+
+  tests('success') do
+
+    @bucket_v2_format = {
+      'CommonPrefixes'        => [],
+      'IsTruncated'          => Fog::Boolean,
+      'ContinuationToken'    => Fog::Nullable::String,
+      'NextContinuationToken' => Fog::Nullable::String,
+      'KeyCount'             => Integer,
+      'MaxKeys'              => Integer,
+      'Name'                 => String,
+      'Prefix'               => Fog::Nullable::String,
+      'StartAfter'           => Fog::Nullable::String,
+      'Contents'             => [{
+        'ETag'          => String,
+        'Key'           => String,
+        'LastModified'  => Time,
+        'Owner' => Fog::Nullable::Hash,
+        'Size' => Integer,
+        'StorageClass' => String
+      }]
+    }
+
+    tests("#put_bucket('#{@aws_bucket_name}')").succeeds do
+      Fog::Storage[:aws].put_bucket(@aws_bucket_name)
+    end
+
+    file = Fog::Storage[:aws].directories.get(@aws_bucket_name).files.create(:body => 'y', :key => 'x')
+
+    tests("#list_objects_v2('#{@aws_bucket_name}')").formats(@bucket_v2_format) do
+      Fog::Storage[:aws].list_objects_v2(@aws_bucket_name).body
+    end
+
+    file.destroy
+
+    file1 = Fog::Storage[:aws].directories.get(@aws_bucket_name).files.create(:body => 'a',    :key => 'a/a1/file1')
+    file2 = Fog::Storage[:aws].directories.get(@aws_bucket_name).files.create(:body => 'ab',   :key => 'a/file2')
+    file3 = Fog::Storage[:aws].directories.get(@aws_bucket_name).files.create(:body => 'abc',  :key => 'b/file3')
+    file4 = Fog::Storage[:aws].directories.get(@aws_bucket_name).files.create(:body => 'abcd', :key => 'file4')
+
+    tests("#list_objects_v2('#{@aws_bucket_name}')") do
+      before do
+        @bucket = Fog::Storage[:aws].list_objects_v2(@aws_bucket_name)
+      end
+
+      tests(".body['Contents'].map{|n| n['Key']}").returns(["a/a1/file1", "a/file2", "b/file3", "file4"]) do
+        @bucket.body['Contents'].map{|n| n['Key']}
+      end
+
+      tests(".body['Contents'].map{|n| n['Size']}").returns([1, 2, 3, 4]) do
+        @bucket.body['Contents'].map{|n| n['Size']}
+      end
+
+      tests(".body['CommonPrefixes']").returns([]) do
+        @bucket.body['CommonPrefixes']
+      end
+
+      tests(".body['KeyCount']").returns(4) do
+        @bucket.body['KeyCount']
+      end
+
+      tests(".body['IsTruncated']").returns(false) do
+        @bucket.body['IsTruncated']
+      end
+    end
+
+    tests("#list_objects_v2('#{@aws_bucket_name}', 'delimiter' => '/')") do
+      before do
+        @bucket = Fog::Storage[:aws].list_objects_v2(@aws_bucket_name, 'delimiter' => '/')
+      end
+
+      tests(".body['Contents'].map{|n| n['Key']}").returns(['file4']) do
+        @bucket.body['Contents'].map{|n| n['Key']}
+      end
+
+      tests(".body['CommonPrefixes']").returns(['a/', 'b/']) do
+        @bucket.body['CommonPrefixes']
+      end
+
+      tests(".body['KeyCount']").returns(3) do
+        @bucket.body['KeyCount']
+      end
+    end
+
+    tests("#list_objects_v2('#{@aws_bucket_name}', 'delimiter' => '/', 'prefix' => 'a/')") do
+      before do
+        @bucket = Fog::Storage[:aws].list_objects_v2(@aws_bucket_name, 'delimiter' => '/', 'prefix' => 'a/')
+      end
+
+      tests(".body['Contents'].map{|n| n['Key']}").returns(['a/file2']) do
+        @bucket.body['Contents'].map{|n| n['Key']}
+      end
+
+      tests(".body['CommonPrefixes']").returns(['a/a1/']) do
+        @bucket.body['CommonPrefixes']
+      end
+
+      tests(".body['KeyCount']").returns(2) do
+        @bucket.body['KeyCount']
+      end
+    end
+
+    tests("#list_objects_v2('#{@aws_bucket_name}', 'start-after' => 'a/file2')") do
+      before do
+        @bucket = Fog::Storage[:aws].list_objects_v2(@aws_bucket_name, 'start-after' => 'a/file2')
+      end
+
+      tests(".body['Contents'].map{|n| n['Key']}").returns(['b/file3', 'file4']) do
+        @bucket.body['Contents'].map{|n| n['Key']}
+      end
+
+      tests(".body['KeyCount']").returns(2) do
+        @bucket.body['KeyCount']
+      end
+    end
+
+    tests("#list_objects_v2('#{@aws_bucket_name}', 'max-keys' => 2)") do
+      before do
+        @bucket = Fog::Storage[:aws].list_objects_v2(@aws_bucket_name, 'max-keys' => 2)
+      end
+
+      tests(".body['Contents'].size").returns(2) do
+        @bucket.body['Contents'].size
+      end
+
+      tests(".body['KeyCount']").returns(2) do
+        @bucket.body['KeyCount']
+      end
+
+      tests(".body['IsTruncated']").returns(true) do
+        @bucket.body['IsTruncated']
+      end
+
+      tests(".body['NextContinuationToken']").returns(true) do
+        !@bucket.body['NextContinuationToken'].nil?
+      end
+    end
+
+    # Test pagination with continuation token
+    tests("pagination with continuation token") do
+      first_page = Fog::Storage[:aws].list_objects_v2(@aws_bucket_name, 'max-keys' => 2)
+      next_token = first_page.body['NextContinuationToken']
+
+      tests("#list_objects_v2 with continuation-token").succeeds do
+        second_page = Fog::Storage[:aws].list_objects_v2(@aws_bucket_name, 'continuation-token' => next_token)
+        tests("second page has remaining objects").returns(true) do
+          second_page.body['Contents'].size > 0
+        end
+      end if next_token
+    end
+
+    # Test fetch-owner parameter
+    tests("#list_objects_v2('#{@aws_bucket_name}', 'fetch-owner' => true)") do
+      before do
+        @bucket = Fog::Storage[:aws].list_objects_v2(@aws_bucket_name, 'fetch-owner' => true)
+      end
+
+      tests("owner information is included").returns(true) do
+        @bucket.body['Contents'].first['Owner'] != nil
+      end
+    end unless Fog.mocking?
+
+    tests("#list_objects_v2('#{@aws_bucket_name}', 'fetch-owner' => false)") do
+      before do
+        @bucket = Fog::Storage[:aws].list_objects_v2(@aws_bucket_name, 'fetch-owner' => false)
+      end
+
+      tests("owner information is not included").returns(true) do
+        @bucket.body['Contents'].first['Owner'].nil?
+      end
+    end unless Fog.mocking?
+
+    file1.destroy; file2.destroy; file3.destroy; file4.destroy
+
+    tests("#delete_bucket('#{@aws_bucket_name}')").succeeds do
+      Fog::Storage[:aws].delete_bucket(@aws_bucket_name)
+    end
+
+  end
+
+  tests('failure') do
+    tests("#list_objects_v2('fognonbucket')").raises(Excon::Errors::NotFound) do
+      Fog::Storage[:aws].list_objects_v2('fognonbucket')
+    end
+
+    tests("#list_objects_v2 without bucket name").raises(ArgumentError) do
+      Fog::Storage[:aws].list_objects_v2(nil)
+    end
+  end
+
+end