extend support of url_suffix for different resource types

Author: nadavofir

lib/utils.js

@@ -544,11 +544,20 @@ finalize_resource_type = (resource_type, type, url_suffix, use_root_path, shorte
     if resource_type == 'image' && type == 'upload'
       resource_type = "images"
       type = null
+    else if resource_type == 'image' && type == 'private'
+      resource_type = 'private_images'
+      type = null
+    else if resource_type == 'image' && type == 'authenticated'
+      resource_type = 'authenticated_images'
+      type = null
     else if resource_type == 'raw' && type == 'upload'
       resource_type = 'files'
       type = null
+    else if resource_type == 'video' && type == 'upload'
+      resource_type = 'videos'
+      type = null
     else
-      throw new Error("URL Suffix only supported for image/upload and raw/upload")
+      throw new Error("URL Suffix only supported for image/upload, image/private, image/authenticated, video/upload and raw/upload")
   if use_root_path
     if (resource_type == 'image' && type == 'upload') || (resource_type == 'images' && !type?)
       resource_type = null

src/utils.coffee

@@ -84,7 +84,7 @@ describe "utils", ->
     expect(()-> utils.url("test", {url_suffix:"hello"})).to.be.throwError(/URL Suffix only supported in private CDN/)
 
   it "should disallow url_suffix in non upload types" , ->
-    expect(()-> utils.url("test", {url_suffix:"hello", private_cdn:true, type:'facebook'})).to.be.throwError(/URL Suffix only supported for image\/upload and raw\/upload/)
+    expect(()-> utils.url("test", {url_suffix:"hello", private_cdn:true, type:'facebook'})).to.be.throwError(/URL Suffix only supported for image\/upload, image\/private, image\/authenticated, video\/upload and raw\/upload/)
 
   it "should disallow url_suffix with / or ." , ->
     expect(()-> utils.url("test", {url_suffix:"hello/world", private_cdn:true})).to.be.throwError(/url_suffix should not include . or \//)
@@ -111,6 +111,12 @@ describe "utils", ->
   it "should support url_suffix for raw uploads" , ->
     test_cloudinary_url("test", {url_suffix:"hello", private_cdn:true, resource_type:'raw'}, "http://#{cloud_name}-res.cloudinary.com/files/test/hello", {})
 
+  it "should support url_suffix for raw uploads" , ->
+    test_cloudinary_url("test", {url_suffix:"hello", private_cdn:true, resource_type:'video'}, "http://#{cloud_name}-res.cloudinary.com/videos/test/hello", {})
+
+  it "should support url_suffix for raw uploads" , ->
+    test_cloudinary_url("test", {url_suffix:"hello", private_cdn:true, type:'authenticated'}, "http://#{cloud_name}-res.cloudinary.com/authenticated_images/test/hello", {})
+
   it "should support use_root_path in shared distribution" , ->
     test_cloudinary_url("test", {use_root_path:true, private_cdn:false}, "http://res.cloudinary.com/#{cloud_name}/test", {})
     test_cloudinary_url("test", {use_root_path:true, private_cdn:false, angle:0}, "http://res.cloudinary.com/#{cloud_name}/a_0/test", {})