Commit e5146354 authored by Adrian DC's avatar Adrian DC
Browse files

chore: finish #160: unify the configurations cleanup in a function

parent 30438942

gitlabci_local/features/configurations.py

+8 −3
Changes for gitlabci_local/features/configurations.py: 8 added lines, 3 removed lines.
Original line number Diff line number Diff line
@@ -27,12 +27,17 @@ class ConfigurationsFeature: 
                if Lists.match(options.names, job, ignore_case=options.ignore_case,

                               no_regex=options.no_regex):

                    self.__configuration[job] = deepcopy(jobs[job])

                    if not self.__configuration[job]['services']:

                        del self.__configuration[job]['services']

                    del self.__configuration[job]['options']

                    self.__cleanup(job)

        else:

            for job in jobs:

                self.__configuration[job] = deepcopy(jobs[job])

                self.__cleanup(job)



    # Cleanup

    def __cleanup(self, job):



        # Cleanup job configurations

        if job in self.__configuration:

            if not self.__configuration[job]['services']:

                del self.__configuration[job]['services']

            del self.__configuration[job]['options']