def __init__(self, is_backward_check=True, prev_ver=None, use_git=False, is_circle=False, print_ignored_files=False, validate_conf_json=True, validate_id_set=False, file_path=None, validate_all=False, configuration=Configuration()): self.validate_all = validate_all self.branch_name = '' self.use_git = use_git if self.use_git: print('Using git') self.branch_name = self.get_current_working_branch() print(f'Running validation on branch {self.branch_name}') self.prev_ver = prev_ver self._is_valid = True self.configuration = configuration self.is_backward_check = is_backward_check self.is_circle = is_circle self.print_ignored_files = print_ignored_files self.validate_conf_json = validate_conf_json self.validate_id_set = validate_id_set self.file_path = file_path if self.validate_conf_json: self.conf_json_validator = ConfJsonValidator() if self.validate_id_set: self.id_set_validator = IDSetValidator( is_circle=self.is_circle, configuration=self.configuration)
def test_validness_in_set(): validator = IDSetValidator(is_circle=False, is_test_run=True, configuration=CONFIG) obj_data = {"test": {"name": "test"}} obj_set = [ obj_data, ] assert validator.is_valid_in_id_set(file_path="test", obj_data=obj_data, obj_set=obj_set), \ "The id validator couldn't find id as valid one"
def test_obj_not_found_in_set(): validator = IDSetValidator(is_circle=False, is_test_run=True, configuration=CONFIG) obj_data = {"test": {"name": "test"}} actual_obj_set = {"test": {"name": "test", "fromversion": "1.2.2"}} obj_set = [ actual_obj_set, ] assert validator.is_valid_in_id_set(file_path="test", obj_data=obj_data, obj_set=obj_set) is False, \ "The id validator couldn't find id as valid one"
def test_duplicated_id_with_diff_versioning(): validator = IDSetValidator(is_circle=False, is_test_run=True, configuration=CONFIG) obj_data = {"test": {"name": "test", "fromversion": "1.0.0"}} actual_obj_set = {"test": {"name": "test", "toversion": "2.0.0"}} obj_set = [ actual_obj_set, ] validator.id_set = {"testing_set": obj_set} assert validator.is_id_duplicated(obj_id="test", obj_data=obj_data, obj_type="testing_set"), \ "The id validator couldn't find id as duplicated one(In different sets)"
def test_duplicated_id_same_set(): validator = IDSetValidator(configuration=CONFIG, is_test_run=True) obj_data = {"test": {"name": "test"}} actual_obj_set = { "test": { "name": "test", } } obj_set = [ actual_obj_set, ] validator.id_set = {"testing_set": obj_set} assert validator.is_id_duplicated(obj_id="test", obj_data=obj_data, obj_type="testing_set") is False, \ "The id validator found the id as duplicated although it is not"
def __init__(self, is_backward_check=True, prev_ver=None, use_git=False, only_committed_files=False, print_ignored_files=False, skip_conf_json=True, validate_id_set=False, file_path=None, validate_all=False, is_external_repo=False, skip_pack_rn_validation=False, print_ignored_errors=False, silence_init_prints=False, no_docker_checks=False): # General configuration self.skip_docker_checks = False self.no_configuration_prints = silence_init_prints self.skip_conf_json = skip_conf_json self.is_backward_check = is_backward_check self.validate_in_id_set = validate_id_set self.is_circle = only_committed_files self.validate_all = validate_all self.use_git = use_git self.skip_pack_rn_validation = skip_pack_rn_validation self.prev_ver = prev_ver if prev_ver else 'origin/master' self.print_ignored_files = print_ignored_files self.print_ignored_errors = print_ignored_errors self.compare_type = '...' # Class constants self.handle_error = BaseValidator(print_as_warnings=print_ignored_errors).handle_error self.file_path = file_path self.branch_name = '' self.changes_in_schema = False self.check_only_schema = False self.always_valid = False self.ignored_files = set() self.new_packs = set() self.skipped_file_types = (FileType.CHANGELOG, FileType.DESCRIPTION, FileType.TEST_PLAYBOOK) if is_external_repo: if not self.no_configuration_prints: click.echo('Running in a private repository') self.skip_conf_json = True if validate_all: # No need to check docker images on build branch hence we do not check on -a mode self.skip_docker_checks = True self.skip_pack_rn_validation = True if self.validate_in_id_set: self.id_set_validator = IDSetValidator(is_circle=self.is_circle, configuration=Configuration()) if no_docker_checks: self.skip_docker_checks = True
class FilesValidator: """FilesValidator is a class that's designed to validate all the changed files on your branch, and all files in case you are on master, this class will be used on your local env as the validation hook(pre-commit), and on CircleCi to make sure you did not bypass the hooks as a safety precaution. Attributes: is_backward_check (bool): Whether to check for backwards compatibility. prev_ver (str): If using git, holds the branch to compare the current one to. Default is origin/master. use_git (bool): Whether to use git or not. is_circle: (bool): Whether the validation was initiated by CircleCI or not. print_ignored_files (bool): Whether to print the files that were ignored during the validation or not. validate_conf_json (bool): Whether to validate conf.json or not. validate_id_set (bool): Whether to validate id_set or not. file_path (string): If validating a specific file, golds it's path. configuration (Configuration): Configurations for IDSetValidator. """ def __init__(self, is_backward_check=True, prev_ver='origin/master', use_git=False, is_circle=False, print_ignored_files=False, validate_conf_json=True, validate_id_set=False, file_path=None, configuration=Configuration()): self.branch_name = '' self.use_git = use_git if self.use_git: print('Using git') self.branch_name = self.get_current_working_branch() print(f'Running validation on branch {self.branch_name}') self.prev_ver = prev_ver if not self.prev_ver: # validate against master if no version was provided self.prev_ver = 'origin/master' self._is_valid = True self.configuration = configuration self.is_backward_check = is_backward_check self.is_circle = is_circle self.print_ignored_files = print_ignored_files self.validate_conf_json = validate_conf_json self.validate_id_set = validate_id_set self.file_path = file_path if self.validate_conf_json: self.conf_json_validator = ConfJsonValidator() if self.validate_id_set: self.id_set_validator = IDSetValidator( is_circle=self.is_circle, configuration=self.configuration) def run(self): print_color('Starting validating files structure', LOG_COLORS.GREEN) if self.is_valid_structure(): print_color('The files are valid', LOG_COLORS.GREEN) return 0 else: print_color( 'The files were found as invalid, the exact error message can be located above', LOG_COLORS.RED) return 1 @staticmethod def get_current_working_branch(): branches = run_command('git branch') branch_name_reg = re.search(r'\* (.*)', branches) return branch_name_reg.group(1) @staticmethod def get_modified_files(files_string, tag='master', print_ignored_files=False): """Get lists of the modified files in your branch according to the files string. Args: files_string (string): String that was calculated by git using `git diff` command. tag (string): String of git tag used to update modified files. print_ignored_files (bool): should print ignored files. Returns: (modified_files_list, added_files_list, deleted_files). Tuple of sets. """ all_files = files_string.split('\n') deleted_files = set([]) added_files_list = set([]) modified_files_list = set([]) old_format_files = set([]) for f in all_files: file_data = f.split() if not file_data: continue file_status = file_data[0] file_path = file_data[1] if file_status.lower().startswith('r'): file_status = 'r' file_path = file_data[2] if checked_type(file_path, CODE_FILES_REGEX) and file_status.lower() != 'd' \ and not file_path.endswith('_test.py'): # naming convention - code file and yml file in packages must have same name. file_path = os.path.splitext(file_path)[0] + '.yml' elif file_path.endswith('.js') or file_path.endswith('.py'): continue if file_status.lower() == 'd' and checked_type( file_path) and not file_path.startswith('.'): deleted_files.add(file_path) elif not os.path.isfile(file_path): continue elif file_status.lower() in ['m', 'a', 'r'] and checked_type(file_path, OLD_YML_FORMAT_FILE) and \ FilesValidator._is_py_script_or_integration(file_path): old_format_files.add(file_path) elif file_status.lower() == 'm' and checked_type( file_path) and not file_path.startswith('.'): modified_files_list.add(file_path) elif file_status.lower() == 'a' and checked_type( file_path) and not file_path.startswith('.'): added_files_list.add(file_path) elif file_status.lower().startswith('r') and checked_type( file_path): # if a code file changed, take the associated yml file. if checked_type(file_data[2], CODE_FILES_REGEX): modified_files_list.add(file_path) else: modified_files_list.add((file_data[1], file_data[2])) elif checked_type(file_path, [SCHEMA_REGEX]): modified_files_list.add(file_path) elif file_status.lower() not in KNOWN_FILE_STATUSES: print_error( '{} file status is an unknown one, please check. File status was: {}' .format(file_path, file_status)) elif print_ignored_files and not checked_type( file_path, IGNORED_TYPES_REGEXES): print_warning('Ignoring file path: {}'.format(file_path)) modified_files_list, added_files_list, deleted_files = filter_packagify_changes( modified_files_list, added_files_list, deleted_files, tag) return modified_files_list, added_files_list, deleted_files, old_format_files def get_modified_and_added_files(self, tag='origin/master'): """Get lists of the modified and added files in your branch according to the git diff output. Args: tag (string): String of git tag used to update modified files Returns: (modified_files, added_files). Tuple of sets. """ # Two dots is the default in git diff, it will compare with the last known commit as the base # Three dots will compare with the last known shared commit as the base compare_type = '.' if 'master' in tag else '' all_changed_files_string = run_command( 'git diff --name-status {tag}..{compare_type}refs/heads/{branch}'. format(tag=tag, branch=self.branch_name, compare_type=compare_type)) modified_files, added_files, _, old_format_files = self.get_modified_files( all_changed_files_string, tag=tag, print_ignored_files=self.print_ignored_files) if not self.is_circle: files_string = run_command( 'git diff --name-status --no-merges HEAD') nc_modified_files, nc_added_files, nc_deleted_files, nc_old_format_files = self.get_modified_files( files_string, print_ignored_files=self.print_ignored_files) all_changed_files_string = run_command( 'git diff --name-status {}'.format(tag)) modified_files_from_tag, added_files_from_tag, _, _ = \ self.get_modified_files(all_changed_files_string, print_ignored_files=self.print_ignored_files) if self.file_path: if F'M\t{self.file_path}' in files_string: modified_files = {self.file_path} added_files = set() else: modified_files = set() added_files = {self.file_path} return modified_files, added_files, set(), set() old_format_files = old_format_files.union(nc_old_format_files) modified_files = modified_files.union( modified_files_from_tag.intersection(nc_modified_files)) added_files = added_files.union( added_files_from_tag.intersection(nc_added_files)) modified_files = modified_files - set(nc_deleted_files) added_files = added_files - set(nc_modified_files) - set( nc_deleted_files) packs = self.get_packs(modified_files, added_files) return modified_files, added_files, old_format_files, packs @staticmethod def get_packs(modified_files, added_files): packs = set() changed_files = modified_files.union(added_files) for changed_file in changed_files: if isinstance(changed_file, tuple): changed_file = changed_file[1] pack = get_pack_name(changed_file) if pack and is_file_path_in_pack(changed_file): packs.add(pack) return packs def is_valid_release_notes(self, file_path): release_notes_validator = ReleaseNotesValidator(file_path) if not release_notes_validator.is_file_valid(): self._is_valid = False def validate_modified_files(self, modified_files): # noqa: C901 """Validate the modified files from your branch. In case we encounter an invalid file we set the self._is_valid param to False. Args: modified_files (set): A set of the modified files in the current branch. """ for file_path in modified_files: old_file_path = None if isinstance(file_path, tuple): old_file_path, file_path = file_path print('Validating {}'.format(file_path)) if not checked_type(file_path): print_warning( '- Skipping validation of non-content entity file.') continue if re.match(TEST_PLAYBOOK_REGEX, file_path, re.IGNORECASE): continue elif 'README' in file_path: readme_validator = ReadMeValidator(file_path) if not readme_validator.is_valid_file(): self._is_valid = False continue structure_validator = StructureValidator( file_path, old_file_path=old_file_path) if not structure_validator.is_valid_file(): self._is_valid = False if self.validate_id_set: if not self.id_set_validator.is_file_valid_in_set(file_path): self._is_valid = False elif checked_type(file_path, YML_INTEGRATION_REGEXES): image_validator = ImageValidator(file_path) if not image_validator.is_valid(): self._is_valid = False description_validator = DescriptionValidator(file_path) if not description_validator.is_valid(): self._is_valid = False integration_validator = IntegrationValidator( structure_validator) if self.is_backward_check and not integration_validator.is_backward_compatible( ): self._is_valid = False if not integration_validator.is_valid_file(): self._is_valid = False elif checked_type(file_path, YML_BETA_INTEGRATIONS_REGEXES): image_validator = ImageValidator(file_path) if not image_validator.is_valid(): self._is_valid = False description_validator = DescriptionValidator(file_path) if not description_validator.is_valid_beta_description(): self._is_valid = False integration_validator = IntegrationValidator( structure_validator) if not integration_validator.is_valid_beta_integration(): self._is_valid = False elif checked_type(file_path, [SCRIPT_REGEX]): script_validator = ScriptValidator(structure_validator) if self.is_backward_check and not script_validator.is_backward_compatible( ): self._is_valid = False if not script_validator.is_valid_file(): self._is_valid = False elif checked_type(file_path, PLAYBOOKS_REGEXES_LIST): playbook_validator = PlaybookValidator(structure_validator) if not playbook_validator.is_valid_playbook( is_new_playbook=False): self._is_valid = False elif checked_type(file_path, PACKAGE_SCRIPTS_REGEXES): unifier = Unifier(os.path.dirname(file_path)) yml_path, _ = unifier.get_script_package_data() # Set file path to the yml file structure_validator.file_path = yml_path script_validator = ScriptValidator(structure_validator) if self.is_backward_check and not script_validator.is_backward_compatible( ): self._is_valid = False if not script_validator.is_valid_file(): self._is_valid = False elif re.match(IMAGE_REGEX, file_path, re.IGNORECASE): image_validator = ImageValidator(file_path) if not image_validator.is_valid(): self._is_valid = False # incident fields and indicator fields are using the same scheme. elif checked_type(file_path, JSON_INDICATOR_AND_INCIDENT_FIELDS): incident_field_validator = IncidentFieldValidator( structure_validator) if not incident_field_validator.is_valid_file( validate_rn=True): self._is_valid = False if self.is_backward_check and not incident_field_validator.is_backward_compatible( ): self._is_valid = False elif checked_type(file_path, [REPUTATION_REGEX]): reputation_validator = ReputationValidator(structure_validator) if not reputation_validator.is_valid_file(validate_rn=True): self._is_valid = False elif checked_type(file_path, JSON_ALL_LAYOUT_REGEXES): layout_validator = LayoutValidator(structure_validator) if not layout_validator.is_valid_layout(validate_rn=True): self._is_valid = False elif checked_type(file_path, JSON_ALL_DASHBOARDS_REGEXES): dashboard_validator = DashboardValidator(structure_validator) if not dashboard_validator.is_valid_dashboard( validate_rn=True): self._is_valid = False elif checked_type(file_path, JSON_ALL_INCIDENT_TYPES_REGEXES): incident_type_validator = IncidentTypeValidator( structure_validator) if not incident_type_validator.is_valid_incident_type( validate_rn=True): self._is_valid = False if self.is_backward_check and not incident_type_validator.is_backward_compatible( ): self._is_valid = False elif 'CHANGELOG' in file_path: self.is_valid_release_notes(file_path) elif checked_type(file_path, CHECKED_TYPES_REGEXES): pass else: print_error( "The file type of {} is not supported in validate command". format(file_path)) print_error( "'validate' command supports: Integrations, Scripts, Playbooks, " "Incident fields, Indicator fields, Images, Release notes, Layouts and Descriptions" ) self._is_valid = False def validate_added_files(self, added_files, file_type: str = None): # noqa: C901 """Validate the added files from your branch. In case we encounter an invalid file we set the self._is_valid param to False. Args: added_files (set): A set of the modified files in the current branch. file_type (str): Used only with -p flag (the type of the file). """ for file_path in added_files: print('Validating {}'.format(file_path)) if re.match(TEST_PLAYBOOK_REGEX, file_path, re.IGNORECASE) and not file_type: continue elif 'README' in file_path: readme_validator = ReadMeValidator(file_path) if not readme_validator.is_valid_file(): self._is_valid = False continue structure_validator = StructureValidator( file_path, is_new_file=True, predefined_scheme=file_type) if not structure_validator.is_valid_file(): self._is_valid = False if self.validate_id_set: if not self.id_set_validator.is_file_valid_in_set(file_path): self._is_valid = False if self.id_set_validator.is_file_has_used_id(file_path): self._is_valid = False elif re.match(PLAYBOOK_REGEX, file_path, re.IGNORECASE) or file_type == 'playbook': playbook_validator = PlaybookValidator(structure_validator) if not playbook_validator.is_valid_playbook(): self._is_valid = False elif checked_type( file_path, YML_INTEGRATION_REGEXES) or file_type == 'integration': image_validator = ImageValidator(file_path) # if file_type(non git path) the image is not in a separate path image_validator.file_path = file_path if file_type else image_validator.file_path if not image_validator.is_valid(): self._is_valid = False description_validator = DescriptionValidator(file_path) if not description_validator.is_valid(): self._is_valid = False integration_validator = IntegrationValidator( structure_validator) if not integration_validator.is_valid_file( validate_rn=not file_type): self._is_valid = False elif checked_type( file_path, PACKAGE_SCRIPTS_REGEXES) or file_type == 'script': unifier = Unifier(os.path.dirname(file_path)) yml_path, _ = unifier.get_script_package_data() # Set file path to the yml file structure_validator.file_path = yml_path script_validator = ScriptValidator(structure_validator) if not script_validator.is_valid_file( validate_rn=not file_type): self._is_valid = False elif re.match(BETA_INTEGRATION_REGEX, file_path, re.IGNORECASE) or \ re.match(BETA_INTEGRATION_YML_REGEX, file_path, re.IGNORECASE): description_validator = DescriptionValidator(file_path) if not description_validator.is_valid_beta_description(): self._is_valid = False integration_validator = IntegrationValidator( structure_validator) if not integration_validator.is_valid_beta_integration(): self._is_valid = False elif re.match(IMAGE_REGEX, file_path, re.IGNORECASE): image_validator = ImageValidator(file_path) if not image_validator.is_valid(): self._is_valid = False # incident fields and indicator fields are using the same scheme. elif checked_type(file_path, JSON_INDICATOR_AND_INCIDENT_FIELDS) or \ file_type in ('incidentfield', 'indicatorfield'): incident_field_validator = IncidentFieldValidator( structure_validator) if not incident_field_validator.is_valid_file( validate_rn=not file_type): self._is_valid = False elif checked_type(file_path, [REPUTATION_REGEX]) or file_type == 'reputation': reputation_validator = ReputationValidator(structure_validator) if not reputation_validator.is_valid_file( validate_rn=not file_type): self._is_valid = False elif checked_type( file_path, JSON_ALL_LAYOUT_REGEXES) or file_type == 'layout': layout_validator = LayoutValidator(structure_validator) if not layout_validator.is_valid_layout( validate_rn=not file_type): self._is_valid = False elif checked_type( file_path, JSON_ALL_DASHBOARDS_REGEXES) or file_type == 'dashboard': dashboard_validator = DashboardValidator(structure_validator) if not dashboard_validator.is_valid_dashboard( validate_rn=not file_type): self._is_valid = False elif checked_type(file_path, JSON_ALL_INCIDENT_TYPES_REGEXES): incident_type_validator = IncidentTypeValidator( structure_validator) if not incident_type_validator.is_valid_incident_type( validate_rn=not file_type): self._is_valid = False elif 'CHANGELOG' in file_path: self.is_valid_release_notes(file_path) elif checked_type(file_path, CHECKED_TYPES_REGEXES): pass else: print_error( "The file type of {} is not supported in validate command". format(file_path)) print_error( "validate command supports: Integrations, Scripts, Playbooks, " "Incident fields, Indicator fields, Images, Release notes, Layouts and Descriptions" ) self._is_valid = False def validate_no_old_format(self, old_format_files): """ Validate there are no files in the old format(unified yml file for the code and configuration). Args: old_format_files(set): file names which are in the old format. """ invalid_files = [] for f in old_format_files: yaml_data = get_yaml(f) if 'toversion' not in yaml_data: # we only fail on old format if no toversion (meaning it is latest) invalid_files.append(f) if invalid_files: print_error( 'You should update the following files to the package format, for further details please visit ' 'https://github.com/demisto/content/tree/master/docs/package_directory_structure. ' 'The files are:\n{}'.format('\n'.join(list(invalid_files)))) self._is_valid = False def validate_committed_files(self): """Validate that all the committed files in your branch are valid""" modified_files, added_files, old_format_files, packs = self.get_modified_and_added_files( ) schema_changed = False for f in modified_files: if isinstance(f, tuple): _, f = f if checked_type(f, [SCHEMA_REGEX]): schema_changed = True # Ensure schema change did not break BC if schema_changed: print("Schema changed, validating all files") self.validate_all_files() else: self.validate_modified_files(modified_files) self.validate_added_files(added_files) self.validate_no_old_format(old_format_files) self.validate_pack_unique_files(packs) def validate_pack_unique_files(self, packs): for pack in packs: pack_unique_files_validator = PackUniqueFilesValidator(pack) pack_errors = pack_unique_files_validator.validate_pack_unique_files( ) if pack_errors: print_error(pack_errors) self._is_valid = False def validate_all_files(self): """Validate all files in the repo are in the right format.""" # go over packs for root, dirs, _ in os.walk(PACKS_DIR): for dir_in_dirs in dirs: for directory in PACKS_DIRECTORIES: for inner_root, inner_dirs, files in os.walk( os.path.join(root, dir_in_dirs, directory)): for inner_dir in inner_dirs: if inner_dir.startswith('.'): continue project_dir = os.path.join(inner_root, inner_dir) _, file_path = get_yml_paths_in_dir( os.path.normpath(project_dir), Errors.no_yml_file(project_dir)) if file_path: # check if the file_path is part of test_data yml if any(test_file in file_path.lower() for test_file in TESTS_DIRECTORIES): continue print("Validating {}".format(file_path)) structure_validator = StructureValidator( file_path) if not structure_validator.is_valid_scheme(): self._is_valid = False # go over regular content entities for directory in DIR_LIST_FOR_REGULAR_ENTETIES: print_color('Validating {} directory:'.format(directory), LOG_COLORS.GREEN) for root, dirs, files in os.walk(directory): for file_name in files: file_path = os.path.join(root, file_name) # skipping hidden files if not file_name.endswith('.yml'): continue print('Validating ' + file_name) structure_validator = StructureValidator(file_path) if not structure_validator.is_valid_scheme(): self._is_valid = False # go over regular PACKAGE_SUPPORTING_DIRECTORIES entities for directory in PACKAGE_SUPPORTING_DIRECTORIES: for root, dirs, files in os.walk(directory): for inner_dir in dirs: if inner_dir.startswith('.'): continue project_dir = os.path.join(root, inner_dir) _, file_path = get_yml_paths_in_dir( project_dir, Errors.no_yml_file(project_dir)) if file_path: # check if the file_path is part of test_data yml if any(test_file in file_path.lower() for test_file in TESTS_DIRECTORIES): continue print('Validating ' + file_path) structure_validator = StructureValidator(file_path) if not structure_validator.is_valid_scheme(): self._is_valid = False def is_valid_structure(self): """Check if the structure is valid for the case we are in, master - all files, branch - changed files. Returns: (bool). Whether the structure is valid or not. """ if self.validate_conf_json: if not self.conf_json_validator.is_valid_conf_json(): self._is_valid = False if self.use_git: if self.branch_name != 'master' and ( not self.branch_name.startswith('19.') and not self.branch_name.startswith('20.')): print('Validates only committed files') self.validate_committed_files() self.validate_against_previous_version(no_error=True) else: self.validate_against_previous_version(no_error=True) print( 'Validates all of Content repo directories according to their schemas' ) self.validate_all_files() else: if self.file_path: print('Not using git, validating file: {}'.format( self.file_path)) self.is_backward_check = False # if not using git, no need for BC checks self.validate_added_files({self.file_path}, file_type=find_type(self.file_path)) else: print('Not using git, validating all files.') self.validate_all_files() return self._is_valid def validate_against_previous_version(self, no_error=False): """Validate all files that were changed between previous version and branch_sha Args: no_error (bool): If set to true will restore self._is_valid after run (will not return new errors) """ if self.prev_ver and self.prev_ver != 'master': print_color('Starting validation against {}'.format(self.prev_ver), LOG_COLORS.GREEN) modified_files, _, _, _ = self.get_modified_and_added_files( self.prev_ver) prev_self_valid = self._is_valid self.validate_modified_files(modified_files) if no_error: self._is_valid = prev_self_valid # parser.add_argument('-t', '--test-filter', type=str2bool, default=False, # help='Check that tests are valid.') # TODO: after validation there was a step to run the configure_tests script to check that each changed file # had a relevant test - was used as part of the hooks. @staticmethod def _is_py_script_or_integration(file_path): file_yml = get_yaml(file_path) if re.match(INTEGRATION_REGEX, file_path, re.IGNORECASE): if file_yml.get('script', {}).get('type', 'javascript') != 'python': return False return True if re.match(SCRIPT_REGEX, file_path, re.IGNORECASE): if file_yml.get('type', 'javascript') != 'python': return False return True return False
class ValidateManager: def __init__(self, is_backward_check=True, prev_ver=None, use_git=False, only_committed_files=False, print_ignored_files=False, skip_conf_json=True, validate_id_set=False, file_path=None, validate_all=False, is_external_repo=False, skip_pack_rn_validation=False, print_ignored_errors=False, silence_init_prints=False, no_docker_checks=False): # General configuration self.skip_docker_checks = False self.no_configuration_prints = silence_init_prints self.skip_conf_json = skip_conf_json self.is_backward_check = is_backward_check self.validate_in_id_set = validate_id_set self.is_circle = only_committed_files self.validate_all = validate_all self.use_git = use_git self.skip_pack_rn_validation = skip_pack_rn_validation self.prev_ver = prev_ver if prev_ver else 'origin/master' self.print_ignored_files = print_ignored_files self.print_ignored_errors = print_ignored_errors self.compare_type = '...' # Class constants self.handle_error = BaseValidator(print_as_warnings=print_ignored_errors).handle_error self.file_path = file_path self.branch_name = '' self.changes_in_schema = False self.check_only_schema = False self.always_valid = False self.ignored_files = set() self.new_packs = set() self.skipped_file_types = (FileType.CHANGELOG, FileType.DESCRIPTION, FileType.TEST_PLAYBOOK) if is_external_repo: if not self.no_configuration_prints: click.echo('Running in a private repository') self.skip_conf_json = True if validate_all: # No need to check docker images on build branch hence we do not check on -a mode self.skip_docker_checks = True self.skip_pack_rn_validation = True if self.validate_in_id_set: self.id_set_validator = IDSetValidator(is_circle=self.is_circle, configuration=Configuration()) if no_docker_checks: self.skip_docker_checks = True def print_final_report(self, valid): self.print_ignored_files_report(self.print_ignored_files) self.print_ignored_errors_report(self.print_ignored_errors) if valid: click.secho('\nThe files are valid', fg='green') return 0 else: all_failing_files = '\n'.join(FOUND_FILES_AND_ERRORS) click.secho(f"\n=========== Found errors in the following files ===========\n\n{all_failing_files}\n", fg="bright_red") if self.always_valid: click.secho('Found the errors above, but not failing build', fg='yellow') return 0 click.secho('The files were found as invalid, the exact error message can be located above', fg='red') return 1 def run_validation(self): """Initiates validation in accordance with mode (i,g,a) """ if self.validate_all: is_valid = self.run_validation_on_all_packs() elif self.use_git: is_valid = self.run_validation_using_git() elif self.file_path: is_valid = self.run_validation_on_specific_files() else: # default validate to -g --post-commit self.use_git = True self.is_circle = True is_valid = self.run_validation_using_git() return self.print_final_report(is_valid) def run_validation_on_specific_files(self): """Run validations only on specific files """ files_validation_result = set() for path in self.file_path.split(','): error_ignore_list = self.get_error_ignore_list(get_pack_name(path)) if os.path.isfile(path): click.secho('\n================= Validating file =================', fg="bright_cyan") files_validation_result.add(self.run_validations_on_file(path, error_ignore_list)) else: path = path.rstrip('/') dir_name = os.path.basename(path) if dir_name in CONTENT_ENTITIES_DIRS: click.secho(f'\n================= Validating content directory {path} =================', fg="bright_cyan") files_validation_result.add(self.run_validation_on_content_entities(path, error_ignore_list)) else: if os.path.basename(os.path.dirname(path)) == PACKS_DIR: click.secho(f'\n================= Validating pack {path} =================', fg="bright_cyan") files_validation_result.add(self.run_validations_on_pack(path)) else: click.secho(f'\n================= Validating package {path} =================', fg="bright_cyan") files_validation_result.add(self.run_validation_on_package(path, error_ignore_list)) return all(files_validation_result) def run_validation_on_all_packs(self): """Runs validations on all files in all packs in repo (-a option) Returns: bool. true if all files are valid, false otherwise. """ click.secho('\n================= Validating all files =================', fg="bright_cyan") all_packs_valid = set() if not self.skip_conf_json: conf_json_validator = ConfJsonValidator() all_packs_valid.add(conf_json_validator.is_valid_conf_json()) for pack_name in os.listdir(PACKS_DIR): pack_path = os.path.join(PACKS_DIR, pack_name) all_packs_valid.add(self.run_validations_on_pack(pack_path)) return all(all_packs_valid) def run_validations_on_pack(self, pack_path): """Runs validation on all files in given pack. (i,g,a) Args: pack_path: the path to the pack. Returns: bool. true if all files in pack are valid, false otherwise. """ pack_entities_validation_results = set() pack_error_ignore_list = self.get_error_ignore_list(os.path.basename(pack_path)) pack_entities_validation_results.add(self.validate_pack_unique_files(pack_path, pack_error_ignore_list)) for content_dir in os.listdir(pack_path): content_entity_path = os.path.join(pack_path, content_dir) if content_dir in CONTENT_ENTITIES_DIRS: pack_entities_validation_results.add(self.run_validation_on_content_entities(content_entity_path, pack_error_ignore_list)) else: self.ignored_files.add(content_entity_path) return all(pack_entities_validation_results) def run_validation_on_content_entities(self, content_entity_dir_path, pack_error_ignore_list): """Gets non-pack folder and runs validation within it (Scripts, Integrations...) Returns: bool. true if all files in directory are valid, false otherwise. """ content_entities_validation_results = set() for file_name in os.listdir(content_entity_dir_path): file_path = os.path.join(content_entity_dir_path, file_name) if os.path.isfile(file_path): if file_path.endswith('.json') or file_path.endswith('.yml') or file_path.endswith('.md'): content_entities_validation_results.add(self.run_validations_on_file(file_path, pack_error_ignore_list)) else: self.ignored_files.add(file_path) else: content_entities_validation_results.add(self.run_validation_on_package(file_path, pack_error_ignore_list)) return all(content_entities_validation_results) def run_validation_on_package(self, package_path, pack_error_ignore_list): package_entities_validation_results = set() for file_name in os.listdir(package_path): file_path = os.path.join(package_path, file_name) if file_path.endswith('.yml') or file_path.endswith('.md'): package_entities_validation_results.add(self.run_validations_on_file(file_path, pack_error_ignore_list)) else: self.ignored_files.add(file_path) return all(package_entities_validation_results) # flake8: noqa: C901 def run_validations_on_file(self, file_path, pack_error_ignore_list, is_modified=False, old_file_path=None, modified_files=None, added_files=None): """Choose a validator to run for a single file. (i) Args: modified_files: A set of modified files - used for RN validation added_files: A set of added files - used for RN validation old_file_path: The old file path for renamed files pack_error_ignore_list: A dictionary of all pack ignored errors file_path: the file on which to run. is_modified: whether the file is modified or added. Returns: bool. true if file is valid, false otherwise. """ file_type = find_type(file_path) if file_type in self.skipped_file_types: self.ignored_files.add(file_path) return True elif file_type is None: error_message, error_code = Errors.file_type_not_supported() if self.handle_error(error_message=error_message, error_code=error_code, file_path=file_path, drop_line=True): return False if not self.check_only_schema: click.echo(f"\nValidating {file_path} as {file_type.value}") structure_validator = StructureValidator(file_path, predefined_scheme=file_type, ignored_errors=pack_error_ignore_list, print_as_warnings=self.print_ignored_errors, tag=self.prev_ver, old_file_path=old_file_path) click.secho(f'Validating scheme for {file_path}') if not structure_validator.is_valid_file(): return False elif self.check_only_schema: return True if self.validate_in_id_set: click.echo(f"Validating id set registration for {file_path}") if not self.id_set_validator.is_file_valid_in_set(file_path): return False # Note: these file are not ignored but there are no additional validators for reports nor connections if file_type in {FileType.REPORT, FileType.CONNECTION}: return True elif file_type == FileType.RELEASE_NOTES: if not self.skip_pack_rn_validation: return self.validate_release_notes(file_path, added_files, modified_files, pack_error_ignore_list, is_modified) elif file_type == FileType.README: return self.validate_readme(file_path, pack_error_ignore_list) elif file_type == FileType.PLAYBOOK: return self.validate_playbook(structure_validator, pack_error_ignore_list) elif file_type == FileType.INTEGRATION: return self.validate_integration(structure_validator, pack_error_ignore_list, is_modified) elif file_type in (FileType.SCRIPT, FileType.TEST_SCRIPT): return self.validate_script(structure_validator, pack_error_ignore_list, is_modified) elif file_type == FileType.BETA_INTEGRATION: return self.validate_beta_integration(structure_validator, pack_error_ignore_list) elif file_type == FileType.IMAGE: return self.validate_image(file_path, pack_error_ignore_list) # incident fields and indicator fields are using the same scheme. elif file_type in (FileType.INCIDENT_FIELD, FileType.INDICATOR_FIELD): return self.validate_incident_field(structure_validator, pack_error_ignore_list, is_modified) elif file_type == FileType.REPUTATION: return self.validate_reputation(structure_validator, pack_error_ignore_list) elif file_type == FileType.LAYOUT: return self.validate_layout(structure_validator, pack_error_ignore_list) elif file_type == FileType.LAYOUTS_CONTAINER: return self.validate_layoutscontainer(structure_validator, pack_error_ignore_list) elif file_type == FileType.DASHBOARD: return self.validate_dashboard(structure_validator, pack_error_ignore_list) elif file_type == FileType.INCIDENT_TYPE: return self.validate_incident_type(structure_validator, pack_error_ignore_list, is_modified) elif file_type == FileType.MAPPER: return self.validate_mapper(structure_validator, pack_error_ignore_list) elif file_type in (FileType.OLD_CLASSIFIER, FileType.CLASSIFIER): return self.validate_classifier(structure_validator, pack_error_ignore_list, file_type) elif file_type == FileType.WIDGET: return self.validate_widget(structure_validator, pack_error_ignore_list) else: error_message, error_code = Errors.file_type_not_supported() if self.handle_error(error_message=error_message, error_code=error_code, file_path=file_path): return False return True def run_validation_using_git(self): """Runs validation on only changed packs/files (g) """ self.setup_git_params() click.secho(f'\n================= Running validation on branch {self.branch_name} =================', fg="bright_cyan") if not self.no_configuration_prints: click.echo(f"Validating against {self.prev_ver}") modified_files, added_files, old_format_files, changed_meta_files = \ self.get_modified_and_added_files(self.compare_type, self.prev_ver) validation_results = set() validation_results.add(self.validate_modified_files(modified_files)) validation_results.add(self.validate_added_files(added_files, modified_files)) validation_results.add(self.validate_changed_packs_unique_files(modified_files, added_files, changed_meta_files)) if not self.skip_pack_rn_validation: validation_results.add(self.validate_no_duplicated_release_notes(added_files)) validation_results.add(self.validate_no_missing_release_notes(modified_files, added_files)) if old_format_files: click.secho(f'\n================= Running validation on old format files =================', fg="bright_cyan") validation_results.add(self.validate_no_old_format(old_format_files)) if self.changes_in_schema: self.check_only_schema = True click.secho(f'\n================= Detected changes in schema - Running validation on all files ' f'=================', fg="bright_cyan") validation_results.add(self.run_validation_on_all_packs()) return all(validation_results) """ ######################################## Unique Validations ####################################### """ def validate_readme(self, file_path, pack_error_ignore_list): readme_validator = ReadMeValidator(file_path, ignored_errors=pack_error_ignore_list, print_as_warnings=self.print_ignored_errors) return readme_validator.is_valid_file() def validate_release_notes(self, file_path, added_files, modified_files, pack_error_ignore_list, is_modified): pack_name = get_pack_name(file_path) # modified existing RN if is_modified: error_message, error_code = Errors.modified_existing_release_notes(pack_name) if self.handle_error(error_message=error_message, error_code=error_code, file_path=file_path): return False # added new RN to a new pack if pack_name in self.new_packs: error_message, error_code = Errors.added_release_notes_for_new_pack(pack_name) if self.handle_error(error_message=error_message, error_code=error_code, file_path=file_path): return False if pack_name != 'NonSupported': if not added_files: added_files = {file_path} release_notes_validator = ReleaseNotesValidator(file_path, pack_name=pack_name, modified_files=modified_files, added_files=added_files, ignored_errors=pack_error_ignore_list, print_as_warnings=self.print_ignored_errors) return release_notes_validator.is_file_valid() return True def validate_playbook(self, structure_validator, pack_error_ignore_list): playbook_validator = PlaybookValidator(structure_validator, ignored_errors=pack_error_ignore_list, print_as_warnings=self.print_ignored_errors) return playbook_validator.is_valid_playbook(validate_rn=False) def validate_integration(self, structure_validator, pack_error_ignore_list, is_modified): integration_validator = IntegrationValidator(structure_validator, ignored_errors=pack_error_ignore_list, print_as_warnings=self.print_ignored_errors, skip_docker_check=self.skip_docker_checks) if is_modified and self.is_backward_check: return all([integration_validator.is_valid_file(validate_rn=False, skip_test_conf=self.skip_conf_json), integration_validator.is_backward_compatible()]) else: return integration_validator.is_valid_file(validate_rn=False, skip_test_conf=self.skip_conf_json) def validate_script(self, structure_validator, pack_error_ignore_list, is_modified): script_validator = ScriptValidator(structure_validator, ignored_errors=pack_error_ignore_list, print_as_warnings=self.print_ignored_errors, skip_docker_check=self.skip_docker_checks) if is_modified and self.is_backward_check: return all([script_validator.is_valid_file(validate_rn=False), script_validator.is_backward_compatible()]) else: return script_validator.is_valid_file(validate_rn=False) def validate_beta_integration(self, structure_validator, pack_error_ignore_list): integration_validator = IntegrationValidator(structure_validator, ignored_errors=pack_error_ignore_list, print_as_warnings=self.print_ignored_errors, skip_docker_check=self.skip_docker_checks) return integration_validator.is_valid_beta_integration() def validate_image(self, file_path, pack_error_ignore_list): image_validator = ImageValidator(file_path, ignored_errors=pack_error_ignore_list, print_as_warnings=self.print_ignored_errors) return image_validator.is_valid() def validate_incident_field(self, structure_validator, pack_error_ignore_list, is_modified): incident_field_validator = IncidentFieldValidator(structure_validator, ignored_errors=pack_error_ignore_list, print_as_warnings=self.print_ignored_errors) if is_modified and self.is_backward_check: return all([incident_field_validator.is_valid_file(validate_rn=False), incident_field_validator.is_backward_compatible()]) else: return incident_field_validator.is_valid_file(validate_rn=False) def validate_reputation(self, structure_validator, pack_error_ignore_list): reputation_validator = ReputationValidator(structure_validator, ignored_errors=pack_error_ignore_list, print_as_warnings=self.print_ignored_errors) return reputation_validator.is_valid_file(validate_rn=False) def validate_layout(self, structure_validator, pack_error_ignore_list): layout_validator = LayoutValidator(structure_validator, ignored_errors=pack_error_ignore_list, print_as_warnings=self.print_ignored_errors) return layout_validator.is_valid_layout(validate_rn=False) def validate_layoutscontainer(self, structure_validator, pack_error_ignore_list): layout_validator = LayoutsContainerValidator(structure_validator, ignored_errors=pack_error_ignore_list, print_as_warnings=self.print_ignored_errors) return layout_validator.is_valid_layout(validate_rn=False) def validate_dashboard(self, structure_validator, pack_error_ignore_list): dashboard_validator = DashboardValidator(structure_validator, ignored_errors=pack_error_ignore_list, print_as_warnings=self.print_ignored_errors) return dashboard_validator.is_valid_dashboard(validate_rn=False) def validate_incident_type(self, structure_validator, pack_error_ignore_list, is_modified): incident_type_validator = IncidentTypeValidator(structure_validator, ignored_errors=pack_error_ignore_list, print_as_warnings=self.print_ignored_errors) if is_modified and self.is_backward_check: return all([incident_type_validator.is_valid_incident_type(validate_rn=False), incident_type_validator.is_backward_compatible()]) else: return incident_type_validator.is_valid_incident_type(validate_rn=False) def validate_mapper(self, structure_validator, pack_error_ignore_list): mapper_validator = MapperValidator(structure_validator, ignored_errors=pack_error_ignore_list, print_as_warnings=self.print_ignored_errors) return mapper_validator.is_valid_mapper(validate_rn=False) def validate_classifier(self, structure_validator, pack_error_ignore_list, file_type): if file_type == FileType.CLASSIFIER: new_classifier_version = True else: new_classifier_version = False classifier_validator = ClassifierValidator(structure_validator, new_classifier_version=new_classifier_version, ignored_errors=pack_error_ignore_list, print_as_warnings=self.print_ignored_errors) return classifier_validator.is_valid_classifier(validate_rn=False) def validate_widget(self, structure_validator, pack_error_ignore_list): widget_validator = WidgetValidator(structure_validator, ignored_errors=pack_error_ignore_list, print_as_warnings=self.print_ignored_errors) return widget_validator.is_valid_file(validate_rn=False) def validate_pack_unique_files(self, pack_path: str, pack_error_ignore_list: dict, should_version_raise=False) -> bool: """ Runs validations on the following pack files: * .secret-ignore: Validates that the file exist and that the file's secrets can be parsed as a list delimited by '\n' * .pack-ignore: Validates that the file exists and that all regexes in it can be compiled * README.md file: Validates that the file exists * pack_metadata.json: Validates that the file exists and that it has a valid structure Args: should_version_raise: Whether we should check if the version of the metadata was raised pack_error_ignore_list: A dictionary of all pack ignored errors pack_path: A path to a pack """ print(f'\nValidating {pack_path} unique pack files') pack_unique_files_validator = PackUniqueFilesValidator(pack=os.path.basename(pack_path), pack_path=pack_path, ignored_errors=pack_error_ignore_list, print_as_warnings=self.print_ignored_errors, should_version_raise=should_version_raise) pack_errors = pack_unique_files_validator.validate_pack_unique_files() if pack_errors: click.secho(pack_errors, fg="bright_red") return False return True def validate_modified_files(self, modified_files): click.secho(f'\n================= Running validation on modified files =================', fg="bright_cyan") valid_files = set() for file_path in modified_files: # handle renamed files if isinstance(file_path, tuple): old_file_path = file_path[0] file_path = file_path[1] else: old_file_path = None pack_name = get_pack_name(file_path) valid_files.add(self.run_validations_on_file(file_path, self.get_error_ignore_list(pack_name), is_modified=True, old_file_path=old_file_path)) return all(valid_files) def validate_added_files(self, added_files, modified_files): click.secho(f'\n================= Running validation on newly added files =================', fg="bright_cyan") valid_files = set() for file_path in added_files: pack_name = get_pack_name(file_path) valid_files.add(self.run_validations_on_file(file_path, self.get_error_ignore_list(pack_name), is_modified=False, modified_files=modified_files, added_files=added_files)) return all(valid_files) def validate_changed_packs_unique_files(self, modified_files, added_files, changed_meta_files): click.secho(f'\n================= Running validation on changed pack unique files =================', fg="bright_cyan") valid_pack_files = set() added_packs = get_pack_names_from_files(added_files) modified_packs = get_pack_names_from_files(modified_files) changed_meta_packs = get_pack_names_from_files(changed_meta_files) packs_that_should_have_version_raised = self.get_packs_that_should_have_version_raised(modified_files, changed_meta_packs) changed_packs = modified_packs.union(added_packs).union(changed_meta_packs) for pack in changed_packs: raise_version = False pack_path = os.path.join(PACKS_DIR, pack) if pack in packs_that_should_have_version_raised: raise_version = True valid_pack_files.add(self.validate_pack_unique_files(pack_path, self.get_error_ignore_list(pack), should_version_raise=raise_version)) return all(valid_pack_files) def validate_no_old_format(self, old_format_files): """ Validate there are no files in the old format(unified yml file for the code and configuration). Args: old_format_files(set): file names which are in the old format. """ invalid_files = [] for file_path in old_format_files: click.echo(f"Validating old-format file {file_path}") yaml_data = get_yaml(file_path) if 'toversion' not in yaml_data: # we only fail on old format if no toversion (meaning it is latest) invalid_files.append(file_path) if invalid_files: error_message, error_code = Errors.invalid_package_structure(invalid_files) if self.handle_error(error_message, error_code, file_path=file_path): return False return True def validate_no_duplicated_release_notes(self, added_files): """Validated that among the added files - there are no duplicated RN for the same pack. Args: added_files(set): The added files Returns: bool. True if no duplications found, false otherwise """ click.secho(f'\n================= Verifying no duplicated release notes =================', fg="bright_cyan") added_rn = set() for file in added_files: if find_type(file) == FileType.RELEASE_NOTES: pack_name = get_pack_name(file) if pack_name not in added_rn: added_rn.add(pack_name) else: error_message, error_code = Errors.multiple_release_notes_files() if self.handle_error(error_message, error_code, file_path=pack_name): return False click.secho("\nNo duplicated release notes found.\n", fg="bright_green") return True def validate_no_missing_release_notes(self, modified_files, added_files): """Validate that there are no missing RN for changed files Args: modified_files (set): a set of modified files. added_files (set): a set of files that were added. Returns: bool. True if no missing RN found, False otherwise """ click.secho("\n================= Checking for missing release notes =================\n", fg="bright_cyan") # existing packs that have files changed (which are not RN, README nor test files) - should have new RN packs_that_should_have_new_rn = get_pack_names_from_files(modified_files, skip_file_types={FileType.RELEASE_NOTES, FileType.README, FileType.TEST_PLAYBOOK, FileType.TEST_SCRIPT}) packs_that_have_new_rn = self.get_packs_with_added_release_notes(added_files) packs_that_have_missing_rn = packs_that_should_have_new_rn.difference(packs_that_have_new_rn) if len(packs_that_have_missing_rn) > 0: is_valid = set() for pack in packs_that_have_missing_rn: # # ignore RN in NonSupported pack if 'NonSupported' in pack: continue ignored_errors_list = self.get_error_ignore_list(pack) error_message, error_code = Errors.missing_release_notes_for_pack(pack) if not BaseValidator(ignored_errors=ignored_errors_list, print_as_warnings=self.print_ignored_errors).handle_error( error_message, error_code, file_path=os.path.join(PACKS_DIR, pack)): is_valid.add(True) else: is_valid.add(False) return all(is_valid) else: click.secho("No missing release notes found.\n", fg="bright_green") return True """ ######################################## Git Tools and filtering ####################################### """ def setup_git_params(self): self.branch_name = self.get_current_working_branch() if self.branch_name != 'master' and (not self.branch_name.startswith('19.') and not self.branch_name.startswith('20.')): # on a non-master branch - we use '...' comparison range to check changes from origin/master. # if not in master or release branch use the pre-existing prev_ver (The branch against which we compare) self.compare_type = '...' else: self.skip_pack_rn_validation = True # on master branch - we use '..' comparison range to check changes from the last release branch. self.compare_type = '..' self.prev_ver = get_content_release_identifier(self.branch_name) # when running against git while on release branch - show errors but don't fail the validation if self.branch_name.startswith('20.'): self.always_valid = True def get_modified_and_added_files(self, compare_type, prev_ver): """Get the modified and added files from a specific branch Args: compare_type (str): whether to run diff with two dots (..) or three (...) prev_ver (str): Against which branch to run the comparision - master/last releaese Returns: tuple. 3 sets representing modified files, added files and files of old format who have changed. """ if not self.no_configuration_prints: click.echo("Collecting all committed files") # all committed changes of the current branch vs the prev_ver all_committed_files_string = run_command( f'git diff --name-status {prev_ver}{compare_type}refs/heads/{self.branch_name}') modified_files, added_files, _, old_format_files, changed_meta_files = \ self.filter_changed_files(all_committed_files_string, prev_ver) if not self.is_circle: remote_configured = has_remote_configured() is_origin_demisto = is_origin_content_repo() if remote_configured and not is_origin_demisto: if not self.no_configuration_prints: click.echo("Collecting all local changed files from fork against the content master") # all local non-committed changes and changes against prev_ver all_changed_files_string = run_command( 'git diff --name-status upstream/master...HEAD') modified_files_from_tag, added_files_from_tag, _, _, changed_meta_files_from_tag = \ self.filter_changed_files(all_changed_files_string, print_ignored_files=self.print_ignored_files) # only changes against prev_ver (without local changes) outer_changes_files_string = run_command('git diff --name-status --no-merges upstream/master...HEAD') nc_modified_files, nc_added_files, nc_deleted_files, nc_old_format_files, nc_changed_meta_files = \ self.filter_changed_files(outer_changes_files_string, print_ignored_files=self.print_ignored_files) else: if (not is_origin_demisto and not remote_configured) and not self.no_configuration_prints: error_message, error_code = Errors.changes_may_fail_validation() self.handle_error(error_message, error_code, file_path="General-Error", warning=True, drop_line=True) if not self.no_configuration_prints: click.echo("Collecting all local changed files against the content master") # all local non-committed changes and changes against prev_ver all_changed_files_string = run_command('git diff --name-status {}'.format(prev_ver)) modified_files_from_tag, added_files_from_tag, _, _, changed_meta_files_from_tag = \ self.filter_changed_files(all_changed_files_string, print_ignored_files=self.print_ignored_files) # only changes against prev_ver (without local changes) outer_changes_files_string = run_command('git diff --name-status --no-merges HEAD') nc_modified_files, nc_added_files, nc_deleted_files, nc_old_format_files, nc_changed_meta_files = \ self.filter_changed_files(outer_changes_files_string, print_ignored_files=self.print_ignored_files) old_format_files = old_format_files.union(nc_old_format_files) modified_files = modified_files.union( modified_files_from_tag.intersection(nc_modified_files)) added_files = added_files.union( added_files_from_tag.intersection(nc_added_files)) changed_meta_files = changed_meta_files.union( changed_meta_files_from_tag.intersection(nc_changed_meta_files)) modified_files = modified_files - set(nc_deleted_files) added_files = added_files - set(nc_modified_files) - set(nc_deleted_files) changed_meta_files = changed_meta_files - set(nc_deleted_files) return modified_files, added_files, old_format_files, changed_meta_files def filter_changed_files(self, files_string, tag='master', print_ignored_files=False): """Get lists of the modified files in your branch according to the files string. Args: files_string (string): String that was calculated by git using `git diff` command. tag (string): String of git tag used to update modified files. print_ignored_files (bool): should print ignored files. Returns: Tuple of sets. """ all_files = files_string.split('\n') deleted_files = set() added_files_list = set() modified_files_list = set() old_format_files = set() changed_meta_files = set() for f in all_files: file_data = list(filter(None, f.split('\t'))) if not file_data: continue file_status = file_data[0] file_path = file_data[1] if file_status.lower().startswith('r'): file_status = 'r' file_path = file_data[2] # if the file is a code file - change path to the associated yml path. if checked_type(file_path, CODE_FILES_REGEX) and file_status.lower() != 'd' \ and not (file_path.endswith('_test.py') or file_path.endswith('.Tests.ps1')): # naming convention - code file and yml file in packages must have same name. file_path = os.path.splitext(file_path)[0] + '.yml' # ignore changes in JS files and unit test files. elif file_path.endswith('.js') or file_path.endswith('.py') or file_path.endswith('.ps1'): self.ignored_files.add(file_path) continue # identify deleted files if file_status.lower() == 'd' and checked_type(file_path) and not file_path.startswith('.'): deleted_files.add(file_path) # ignore directories elif not os.path.isfile(file_path): continue # changes in old scripts and integrations - unified python scripts/integrations elif file_status.lower() in ['m', 'a', 'r'] and checked_type(file_path, OLD_YML_FORMAT_FILE) and \ self._is_py_script_or_integration(file_path): old_format_files.add(file_path) # identify modified files elif file_status.lower() == 'm' and checked_type(file_path) and not file_path.startswith('.'): modified_files_list.add(file_path) # identify added files elif file_status.lower() == 'a' and checked_type(file_path) and not file_path.startswith('.'): added_files_list.add(file_path) # identify renamed files elif file_status.lower().startswith('r') and checked_type(file_path): # if a code file changed, take the associated yml file. if checked_type(file_data[2], CODE_FILES_REGEX): modified_files_list.add(file_path) else: # file_data[1] = old name, file_data[2] = new name modified_files_list.add((file_data[1], file_data[2])) # detect changes in schema elif checked_type(file_path, [SCHEMA_REGEX]): modified_files_list.add(file_path) self.changes_in_schema = True elif file_status.lower() not in KNOWN_FILE_STATUSES: click.secho('{} file status is an unknown one, please check. File status was: {}' .format(file_path, file_status), fg="bright_red") elif file_path.endswith(PACKS_PACK_META_FILE_NAME): if file_status.lower() == 'a': self.new_packs.add(get_pack_name(file_path)) elif file_status.lower() == 'm': changed_meta_files.add(file_path) elif print_ignored_files and not checked_type(file_path, IGNORED_TYPES_REGEXES): if file_path not in self.ignored_files: self.ignored_files.add(file_path) click.secho('Ignoring file path: {}'.format(file_path), fg="yellow") modified_files_list, added_files_list, deleted_files = filter_packagify_changes( modified_files_list, added_files_list, deleted_files, tag) return modified_files_list, added_files_list, deleted_files, old_format_files, changed_meta_files """ ######################################## Validate Tools ############################################### """ @staticmethod def get_pack_ignore_file_path(pack_name): return os.path.join(PACKS_DIR, pack_name, PACKS_PACK_IGNORE_FILE_NAME) @staticmethod def create_ignored_errors_list(errors_to_check): ignored_error_list = [] all_errors = ERROR_CODE.values() for error_code in all_errors: error_type = error_code[:2] if error_code not in errors_to_check and error_type not in errors_to_check: ignored_error_list.append(error_code) return ignored_error_list @staticmethod def get_allowed_ignored_errors_from_list(error_list): allowed_ignore_list = [] for error in error_list: if error in ALLOWED_IGNORE_ERRORS: allowed_ignore_list.append(error) return allowed_ignore_list def add_ignored_errors_to_list(self, config, section, key, ignored_errors_list): if key == 'ignore': ignored_errors_list.extend(self.get_allowed_ignored_errors_from_list(str(config[section][key]).split(','))) if key in PRESET_ERROR_TO_IGNORE: ignored_errors_list.extend(PRESET_ERROR_TO_IGNORE.get(key)) if key in PRESET_ERROR_TO_CHECK: ignored_errors_list.extend( self.create_ignored_errors_list(PRESET_ERROR_TO_CHECK.get(key))) def get_error_ignore_list(self, pack_name): ignored_errors_list = {} if pack_name: pack_ignore_path = self.get_pack_ignore_file_path(pack_name) if os.path.isfile(pack_ignore_path): try: config = ConfigParser(allow_no_value=True) config.read(pack_ignore_path) # create file specific ignored errors list for section in config.sections(): if section.startswith("file:"): file_name = section[5:] ignored_errors_list[file_name] = [] for key in config[section]: self.add_ignored_errors_to_list(config, section, key, ignored_errors_list[file_name]) except MissingSectionHeaderError: pass return ignored_errors_list @staticmethod def get_current_working_branch(): branches = run_command('git branch') branch_name_reg = re.search(r'\* (.*)', branches) return branch_name_reg.group(1) def get_content_release_identifier(self) -> Optional[str]: return tools.get_content_release_identifier(self.branch_name) @staticmethod def _is_py_script_or_integration(file_path): file_yml = get_yaml(file_path) if re.match(PACKS_INTEGRATION_NON_SPLIT_YML_REGEX, file_path, re.IGNORECASE): if file_yml.get('script', {}).get('type', 'javascript') != 'python': return False return True if re.match(PACKS_SCRIPT_NON_SPLIT_YML_REGEX, file_path, re.IGNORECASE): if file_yml.get('type', 'javascript') != 'python': return False return True return False @staticmethod def get_packs_with_added_release_notes(added_files): added_rn = set() for file in added_files: if find_type(path=file) == FileType.RELEASE_NOTES: added_rn.add(get_pack_name(file)) return added_rn def print_ignored_errors_report(self, print_ignored_errors): if print_ignored_errors: all_ignored_errors = '\n'.join(FOUND_FILES_AND_IGNORED_ERRORS) click.secho(f"\n=========== Found ignored errors " f"in the following files ===========\n\n{all_ignored_errors}", fg="yellow") def print_ignored_files_report(self, print_ignored_files): if print_ignored_files: all_ignored_files = '\n'.join(list(self.ignored_files)) click.secho(f"\n=========== Ignored the following files ===========\n\n{all_ignored_files}", fg="yellow") @staticmethod def get_packs_that_should_have_version_raised(modified_files, changed_meta_packs): # modified packs (where the change is not test-playbook, test-script, readme or release notes) # and packs where the meta file changed should have their version raised modified_packs_that_should_have_version_raised = get_pack_names_from_files(modified_files, skip_file_types={ FileType.RELEASE_NOTES, FileType.README, FileType.TEST_PLAYBOOK, FileType.TEST_SCRIPT }) return changed_meta_packs.union(modified_packs_that_should_have_version_raised)