Merge branch 'bug/seo-suffix-validation'

Author: Amir Tocker

Cloudinary.Test/ApiTest.cs

@@ -845,13 +845,6 @@ public void TestDisallowUseRootPathIfNotImageUploadForRaw()
             m_api.UrlImgUp.UseRootPath(true).PrivateCdn(true).ResourceType("raw").BuildUrl("test");
         }
 
-        [Test]
-        [ExpectedException(ExpectedException = typeof(NotSupportedException), ExpectedMessage = "URL Suffix only supported in private CDN!")]
-        public void TestDisallowUrlSuffixInSharedDistribution()
-        {
-            m_api.UrlImgUp.Suffix("hello").BuildUrl("test");
-        }
-
         [Test]
         [ExpectedException(ExpectedException = typeof(NotSupportedException), ExpectedMessage = "URL Suffix only supported for image/upload and raw/upload!")]
         public void TestDisallowUrlSuffixInNonUploadTypes()
@@ -920,6 +913,13 @@ public void TestSupportUrlSuffixForRawUploads()
             Assert.AreEqual("http://testcloud-res.cloudinary.com/files/test/hello", actual);
         }
 
+        [Test]
+        public void TestSupportUrlSuffixForSharedCdn()
+        {
+            string actual = m_api.UrlImgUp.Suffix("hello").Signed(true).Format("jpg").BuildUrl("test");
+            Assert.AreEqual("http://res.cloudinary.com/testcloud/images/s--1TMilNWq--/test/hello.jpg", actual);
+        }
+
         [Test]
         public void TestResponsiveWidthTransform()
         {

Cloudinary/Url.cs

@@ -492,9 +492,6 @@ public string BuildUrl(string source)
             if (String.IsNullOrEmpty(m_cloudName))
                 throw new ArgumentException("cloudName must be specified!");
 
-            if (!m_usePrivateCdn && !String.IsNullOrEmpty(m_suffix))
-                throw new NotSupportedException("URL Suffix only supported in private CDN!");
-
             if (source == null)
                 source = m_source;