feat(tests): add tests to ensure integer descriptors in srcset for responsive image attributes

Author: manu4543

lib/imagekit/helpers/helper.rb

@@ -303,16 +303,18 @@ def get_responsive_image_attributes(options)
 
         # Build srcset
         src_set_entries = candidates.map.with_index do |w, i|
-          descriptor = descriptor_kind == :w ? "#{w}w" : "#{i + 1}x"
-          "#{build_url_fn.call(w)} #{descriptor}"
+          # Ensure width is an integer for proper descriptor format (e.g., "640w" not "640.0w")
+          width_int = w.to_i
+          descriptor = descriptor_kind == :w ? "#{width_int}w" : "#{i + 1}x"
+          "#{build_url_fn.call(width_int)} #{descriptor}"
         end
         src_set = src_set_entries.empty? ? nil : src_set_entries.join(", ")
 
         final_sizes = sizes || (descriptor_kind == :w ? "100vw" : nil)
 
         # Build and return ResponsiveImageAttributes model
         Imagekit::Models::ResponsiveImageAttributes.new(
-          src: build_url_fn.call(candidates.last), # largest candidate
+          src: build_url_fn.call(candidates.last.to_i), # largest candidate as integer
           src_set: src_set,
           sizes: final_sizes,
           width: width

test/imagekit/custom-tests/responsive_image_attributes_test.rb

@@ -214,6 +214,56 @@ def test_should_work_with_plain_hashes_for_responsive_image_attributes
     assert_equal(expected, result.to_h)
   end
 
+  # Test to verify that srcset uses integer descriptors (640w) not float (640.0w)
+  # This tests both model object and hash approaches
+  def test_should_use_integer_descriptors_in_srcset_with_model_object
+    # Using GetImageAttributesOptions model with Float array (as defined in schema)
+    result = @client.helper.get_responsive_image_attributes(
+      Imagekit::Models::GetImageAttributesOptions.new(
+        src: "sample.jpg",
+        url_endpoint: "https://ik.imagekit.io/demo",
+        device_breakpoints: [320.0, 640.0, 1280.0], # SDK converts to floats
+        image_breakpoints: []
+      )
+    )
+
+    src_set = result.src_set
+
+    # Should have integer descriptors like "320w", NOT "320.0w"
+    assert_includes(src_set, "320w")
+    assert_includes(src_set, "640w")
+    assert_includes(src_set, "1280w")
+
+    # Should NOT have float descriptors
+    refute_includes(src_set, "320.0w")
+    refute_includes(src_set, "640.0w")
+    refute_includes(src_set, "1280.0w")
+  end
+
+  def test_should_use_integer_descriptors_in_srcset_with_hash
+    # Using plain hash with integer breakpoints
+    result = @client.helper.get_responsive_image_attributes(
+      {
+        src: "sample.jpg",
+        url_endpoint: "https://ik.imagekit.io/demo",
+        device_breakpoints: [320, 640, 1280],
+        image_breakpoints: []
+      }
+    )
+
+    src_set = result.src_set
+
+    # Should have integer descriptors like "320w", NOT "320.0w"
+    assert_includes(src_set, "320w")
+    assert_includes(src_set, "640w")
+    assert_includes(src_set, "1280w")
+
+    # Should NOT have float descriptors
+    refute_includes(src_set, "320.0w")
+    refute_includes(src_set, "640.0w")
+    refute_includes(src_set, "1280.0w")
+  end
+
   def test_fallback_when_no_usable_vw_tokens
     result = @client.helper.get_responsive_image_attributes(
       src: "sample.jpg",