Merge pull request #240 from selmanj/works_with_empty_snapshots

Check for empty snapshots when listing

Author: icco

lib/fog/compute/google/models/snapshots.rb

@@ -9,7 +9,7 @@ def all
           next_page_token = nil
           loop do
             data = service.list_snapshots(nil, next_page_token)
-            items.concat(data.body["items"])
+            items.concat(data.body["items"]) unless data.body["items"].nil?
             next_page_token = data.body["nextPageToken"]
             break if next_page_token.nil? || next_page_token.empty?
           end

test/integration/compute/test_snapshots.rb

@@ -0,0 +1,21 @@
+require "helpers/integration_test_helper"
+require "securerandom"
+require "base64"
+
+class TestComputeSnapshots < FogIntegrationTest
+  def setup
+    @client = Fog::Compute::Google.new
+    # Ensure any resources we create with test prefixes are removed
+    Minitest.after_run do
+      delete_test_resources
+    end
+  end
+
+  def test_list_empty_snapshots
+    assert_empty @client.snapshots.all
+  end
+
+  def delete_test_resources
+    # nothing to do
+  end
+end