Fix breakpoints format parameter

Author: Amir Tocker

lib/api.js

@@ -969,13 +969,9 @@ exports.generate_responsive_breakpoints_string = (breakpoints)->
 
   for breakpoint_settings in breakpoints
     if breakpoint_settings?
-      transformation = breakpoint_settings.transformation
-      delete breakpoint_settings.transformation
-      format = breakpoint_settings.format
-      delete breakpoint_settings.format
-      if transformation
-        transformation = utils.generate_transformation_string(_.clone(transformation))
-      breakpoint_settings.transformation = _.compact([transformation, format]).join("/")
+      if breakpoint_settings.transformation
+        breakpoint_settings.transformation = utils.generate_transformation_string(_.clone(breakpoint_settings.transformation))
+
   JSON.stringify(breakpoints)
 
 exports.build_streaming_profiles_param = (options={})->

lib/api.js.map

@@ -557,14 +557,30 @@ describe "uploader", ->
   context ":responsive_breakpoints", ->
     context ":create_derived with different transformation settings", ->
       it 'should return a responsive_breakpoints in the response', (done)->
-        cloudinary.v2.uploader.upload IMAGE_FILE, responsive_breakpoints: [{transformation:{effect: "sepia"}, format:"jpg", bytes_step:20000, create_derived: true, min_width: 200, max_width: 1000, max_images: 20}, {format:"gif", create_derived:true, bytes_step:20000, min_width: 200, max_width: 1000, max_images: 20}], tags: UPLOAD_TAGS, (error, result)->
+        cloudinary.v2.uploader.upload IMAGE_FILE, responsive_breakpoints: [{
+          transformation: {effect: "sepia"},
+          format: "jpg",
+          bytes_step: 20000,
+          create_derived: true,
+          min_width: 200,
+          max_width: 1000,
+          max_images: 20
+        }, {
+          transformation: {angle: 10},
+          format: "gif",
+          create_derived: true,
+          bytes_step: 20000,
+          min_width: 200,
+          max_width: 1000,
+          max_images: 20
+        }], tags: UPLOAD_TAGS, (error, result)->
           return done(new Error error.message) if error?
           expect(result).to.have.key('responsive_breakpoints')
           expect(result.responsive_breakpoints).to.have.length(2)
-          expect(result.responsive_breakpoints[0]).to.have.key("transformation")
-          expect(result.responsive_breakpoints[0].transformation).to.eql("e_sepia/jpg")
-          expect(result.responsive_breakpoints[1]).to.have.key("transformation")
-          expect(result.responsive_breakpoints[1].transformation).to.eql("gif")
+          expect(_.at(result, "responsive_breakpoints[0].transformation")[0]).to.eql("e_sepia")
+          expect(_.at(result, "responsive_breakpoints[0].breakpoints[0].url")[0]).to.match(/\.jpg$/)
+          expect(_.at(result, "responsive_breakpoints[1].transformation")[0]).to.eql("a_10")
+          expect(_.at(result, "responsive_breakpoints[1].breakpoints[0].url")[0]).to.match(/\.gif$/)
           done()
         true