Removed validation for price in pack metadata

demisto_sdk/commands/common/hook_validations/pack_unique_files.py

@@ -6,18 +6,12 @@
 import os
 import re
 
-from demisto_sdk.commands.common.constants import (API_MODULES_PACK,
-                                                   PACK_METADATA_CATEGORIES,
-                                                   PACK_METADATA_DEPENDENCIES,
-                                                   PACK_METADATA_FIELDS,
-                                                   PACK_METADATA_KEYWORDS,
-                                                   PACK_METADATA_PRICE,
-                                                   PACK_METADATA_TAGS,
-                                                   PACK_METADATA_USE_CASES,
-                                                   PACKS_PACK_IGNORE_FILE_NAME,
-                                                   PACKS_PACK_META_FILE_NAME,
-                                                   PACKS_README_FILE_NAME,
-                                                   PACKS_WHITELIST_FILE_NAME)
+from demisto_sdk.commands.common.constants import (  # PACK_METADATA_PRICE,
+    API_MODULES_PACK, PACK_METADATA_CATEGORIES, PACK_METADATA_DEPENDENCIES,
+    PACK_METADATA_FIELDS, PACK_METADATA_KEYWORDS, PACK_METADATA_TAGS,
+    PACK_METADATA_USE_CASES, PACKS_PACK_IGNORE_FILE_NAME,
+    PACKS_PACK_META_FILE_NAME, PACKS_README_FILE_NAME,
+    PACKS_WHITELIST_FILE_NAME)
 from demisto_sdk.commands.common.tools import pack_name_to_path
 
 
@@ -151,12 +145,13 @@ def _is_pack_meta_file_structure_valid(self):
             if not isinstance(dependencies_field, dict):
                 self._add_error('The dependencies field in the pack must be a dictionary.')
                 return False
-            price_field = metadata[PACK_METADATA_PRICE]
-            try:
-                int(price_field)
-            except Exception:
-                self._add_error('The price field in the pack must be a number.')
-                return False
+            # TODO: add it back after #23546 is ready.
+            # price_field = metadata[PACK_METADATA_PRICE]
+            # try:
+            #     int(price_field)
+            # except Exception:
+            #     self._add_error('The price field in the pack must be a number.')
+            #     return False
             for list_field in (PACK_METADATA_KEYWORDS, PACK_METADATA_TAGS, PACK_METADATA_CATEGORIES,
                                PACK_METADATA_USE_CASES):
                 field = metadata[list_field]

demisto_sdk/commands/common/tests/pack_metadata_validator_test.py

@@ -20,8 +20,9 @@ def test_metadata_validator_valid(self, mocker):
         validator = PackUniqueFilesValidator('fake')
         assert validator.validate_pack_meta_file()
 
+    # TODO: add the validation for price after #23546 is ready.
     @pytest.mark.parametrize('metadata', [os.path.join(FILES_PATH, 'pack_metadata_missing_fields.json'),
-                                          os.path.join(FILES_PATH, 'pack_metadata_invalid_price.json'),
+                                          # os.path.join(FILES_PATH, 'pack_metadata_invalid_price.json'),
                                           os.path.join(FILES_PATH, 'pack_metadata_invalid_tags.json'),
                                           os.path.join(FILES_PATH, 'pack_metadata_list.json')])
     def test_metadata_validator_invalid(self, mocker, metadata):

demisto_sdk/commands/init/initiator.py

@@ -207,7 +207,7 @@ def create_metadata(fill_manually: bool) -> Dict:
             'certification': 'certified',
             'useCases': [],
             'keywords': [],
-            'price': '0',
+            # 'price': '0',
             'dependencies': {},
         }
 
@@ -232,9 +232,9 @@ def create_metadata(fill_manually: bool) -> Dict:
         tags = input("\nTags of the pack, comma separated values: ")
         tags_list = [t.strip() for t in tags.split(',')]
         metadata['tags'] = tags_list
-
-        price = input("\nThe price of the pack: ")
-        metadata['price'] = price
+        # TODO: add it back after #23546 is ready.
+        # price = input("\nThe price of the pack: ")
+        # metadata['price'] = price
 
         return metadata
 

demisto_sdk/commands/init/tests/initiator_test.py

@@ -17,7 +17,7 @@
 PACK_URL = 'https://www.github.com/pack'
 PACK_EMAIL = 'author@mail.com'
 PACK_TAGS = 'Tag1,Tag2'
-PACK_PRICE = '0'
+# PACK_PRICE = '0'
 
 
 @pytest.fixture
@@ -61,6 +61,7 @@ def test_get_object_id(monkeypatch, initiator):
     assert initiator.id == 'SomeIntegrationID'
 
 
+# TODO: add the validation for price after #23546 is ready.
 def test_create_metadata(monkeypatch, initiator):
     # test create_metadata without user filling manually
     pack_metadata = initiator.create_metadata(False)
@@ -82,7 +83,7 @@ def test_create_metadata(monkeypatch, initiator):
         'certification': 'certified',
         'useCases': [],
         'keywords': [],
-        'price': '0',
+        # 'price': '0',
         'dependencies': {},
     }
 
@@ -92,7 +93,8 @@ def test_create_metadata(monkeypatch, initiator):
         generate_multiple_inputs(
             deque([
                 PACK_NAME, PACK_DESC, '1', PACK_SERVER_MIN_VERSION, PACK_AUTHOR,
-                PACK_URL, PACK_EMAIL, '1', PACK_TAGS, PACK_PRICE
+                PACK_URL, PACK_EMAIL, '1', PACK_TAGS
+                # PACK_PRICE
             ])
         )
     )
@@ -109,7 +111,7 @@ def test_create_metadata(monkeypatch, initiator):
         'email': PACK_EMAIL,
         'keywords': [],
         'name': PACK_NAME,
-        'price': PACK_PRICE,
+        # 'price': PACK_PRICE,
         'serverMinVersion': PACK_SERVER_MIN_VERSION,
         'support': PACK_SUPPORT_OPTIONS[0],
         'tags': ['Tag1', 'Tag2'],