fix: parsing urls check

Author: javiercavlop

src/main/java/io/github/isagroup/services/parsing/FeatureParser.java

@@ -105,7 +105,17 @@ private static Integration parseMapToIntegration(String featureName, Map<String,
         }
 
         if (integration.getIntegrationType().equals(IntegrationType.WEB_SAAS)) {
-            integration.setPricingUrls((List<String>) map.get("pricingUrls"));
+            if (map.get("pricingUrls") != null) {
+                if (!(map.get("pricingUrls") instanceof List) || ((List<String>) map.get("pricingUrls")).isEmpty()
+                        || ((List<String>) map.get("pricingUrls")).stream().anyMatch(url -> !url.matches("^(http|https)://.*"))) {
+                    throw new PricingParsingException("The feature " + featureName
+                            + " is from type INTEGRATION with integrationType WEB_SAAS but does not have a valid pricingUrls list (each item must be a valid URL with the http or https protocol). Current value: " + map.get("pricingUrls")
+                            + ". To specify a list you must use dash (-) before each item. Remember, it is an optional field so you can remove it from the input.");
+                }
+                integration.setPricingUrls((List<String>) map.get("pricingUrls"));
+            } else {
+                integration.setPricingUrls(List.of());
+            }
         }
 
         return integration;
@@ -160,7 +170,11 @@ private static Guarantee parseMapToGuarantee(String featureName, Map<String, Obj
 
         if (map.get("docUrl") != null && !(map.get("docUrl") instanceof String)) {
             throw new PricingParsingException("\'docUrl\' must be a String but found a "
-                    + map.get("docUrl").getClass().getSimpleName() + " instead");
+                    + map.get("docUrl").getClass().getSimpleName() + " instead (feature affected: '" + featureName + "'). Remember, it is an optional field so you can remove it from the input.");
+        }
+
+        if (map.get("docUrl") != null && !((String) map.get("docUrl")).matches("^(http|https)://.*")) {
+            throw new PricingParsingException("The docUrl field (from feature '" + featureName + "') must be a valid URL with the http or https protocol. Received: " + map.get("docUrl") + ". Remember, it is an optional field so you can remove it from the input.");
         }
 
         guarantee.setDocURL((String) map.get("docUrl"));

src/test/resources/negative-parsing-tests.csv

@@ -31,7 +31,7 @@ Throw an error if 'currency' is missing;parsing/negative/currency/currency-is-nu
 Throw an error if a feature is not a map of attributes;parsing/negative/feature/feature-is-key-value.yml;Feature 'foo' must be a Map but found String instead
 Throw an error if a feature is named 'null';parsing/negative/feature/feature-null-as-key.yml;A feature cannot have the name null
 # feature.docUrl
-Throw an error if feature 'docUrl' is not a string;parsing/negative/feature/docUrl/is-bool.yml;'docUrl' must be a String but found a Boolean instead
+Throw an error if feature 'docUrl' is not a string;parsing/negative/feature/docUrl/is-bool.yml;'docUrl' must be a String but found a Boolean instead (feature affected: 'guarantee'). Remember, it is an optional field so you can remove it from the input.
 # feature.type
 Throw an error if feature 'type' is missing or null;parsing/negative/feature/type/null-type.yml;feature 'type' is mandatory
 # feature.valueType