🐛 Fix Ruby SDK E2E tests for both TDD and cloud modes

- Add cloud_mode? helper to detect when running with VIZZLY_TOKEN
- Add assert_screenshot_success helper for mode-aware assertions
- TDD mode: asserts status is 'new' or 'match'
- Cloud mode: asserts success is true (no local comparison)

Author: Robdel12

clients/ruby/test/integration_test.rb

@@ -51,6 +51,26 @@ def stop_server
     @server_pid = nil
   end
 
+  # Check if running in cloud mode (vizzly run) vs TDD mode (vizzly tdd run)
+  # Cloud mode returns { success: true } without a status field
+  # TDD mode returns comparison results with status field
+  def cloud_mode?
+    # In cloud mode, VIZZLY_TOKEN is typically set
+    token = ENV.fetch('VIZZLY_TOKEN', nil)
+    token && !token.empty?
+  end
+
+  # Assert that a screenshot result indicates success
+  # In TDD mode: expects 'new' or 'match' status
+  # In cloud mode: expects 'success' to be true
+  def assert_screenshot_success(result)
+    if cloud_mode?
+      assert result['success'], "Expected success to be true in cloud mode, got: #{result.inspect}"
+    else
+      assert %w[new match].include?(result['status']), "Expected status 'new' or 'match', got: #{result['status']}"
+    end
+  end
+
   # Create a minimal valid PNG (1x1 red pixel)
   def create_test_png
     [
@@ -118,7 +138,7 @@ def test_basic_screenshot
     result = Vizzly.screenshot('basic-screenshot', image_data)
 
     assert result, 'Expected result to be non-nil'
-    assert %w[new match].include?(result['status']), "Expected status 'new' or 'match', got: #{result['status']}"
+    assert_screenshot_success(result)
   end
 
   def test_screenshot_with_properties
@@ -133,7 +153,7 @@ def test_screenshot_with_properties
                                })
 
     assert result, 'Expected result to be non-nil'
-    assert %w[new match].include?(result['status']), "Expected status 'new' or 'match', got: #{result['status']}"
+    assert_screenshot_success(result)
   end
 
   def test_screenshot_with_threshold
@@ -143,7 +163,7 @@ def test_screenshot_with_threshold
     result = Vizzly.screenshot('screenshot-threshold', image_data, threshold: 5)
 
     assert result, 'Expected result to be non-nil'
-    assert %w[new match].include?(result['status']), "Expected status 'new' or 'match', got: #{result['status']}"
+    assert_screenshot_success(result)
   end
 
   def test_screenshot_with_full_page
@@ -153,7 +173,7 @@ def test_screenshot_with_full_page
     result = Vizzly.screenshot('screenshot-fullpage', image_data, full_page: true)
 
     assert result, 'Expected result to be non-nil'
-    assert %w[new match].include?(result['status']), "Expected status 'new' or 'match', got: #{result['status']}"
+    assert_screenshot_success(result)
   end
 
   def test_screenshot_with_all_options
@@ -170,7 +190,7 @@ def test_screenshot_with_all_options
                                full_page: false)
 
     assert result, 'Expected result to be non-nil'
-    assert %w[new match].include?(result['status']), "Expected status 'new' or 'match', got: #{result['status']}"
+    assert_screenshot_success(result)
   end
 
   # ===========================================================================
@@ -195,7 +215,7 @@ def test_auto_discovery_via_server_json
     result = client.screenshot('auto-discovered', image_data)
 
     assert result, 'Expected result to be non-nil'
-    assert %w[new match].include?(result['status']), "Expected status 'new' or 'match', got: #{result['status']}"
+    assert_screenshot_success(result)
   end
 
   # ===========================================================================