-
Notifications
You must be signed in to change notification settings - Fork 71
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
### Changes: 1. Logging option -v -vv -vv (verbosity levels - logging levels verbose-critical- .. - debug) 2. Logging path option - Until debug level. 3. Docker-sdk, git-python usage for better exceptions handling and output, 4. overall optimizations and code readability. 5. JSON output including all collected data for analytics. 6. Adding the following flags: * test-xml - export test results as xml. * json-report - export json report on all results. * log - save the log to file. 7. Git python pack filtering - using set operations (save 19 sec- - each time). 8. Facts gathering optimization - Pipfile collecting, modules from master, additional packages adding to each package 'test-requirements.txt` in the package. 9. Context manager for mandatory test modules (CommonServerPython etc). 10. Code readability. 11. Docker image builds and not commit (Not the recommended way), Now we are using only Single Docker file to the setup test image.
- Loading branch information
Showing
32 changed files
with
3,068 additions
and
1,241 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
28 changes: 0 additions & 28 deletions
28
demisto_sdk/commands/common/dev_sh_scripts/pkg_dev_container_setup.sh
This file was deleted.
Oops, something went wrong.
18 changes: 0 additions & 18 deletions
18
demisto_sdk/commands/common/dev_sh_scripts/pkg_dev_container_setup_pwsh.sh
This file was deleted.
Oops, something went wrong.
60 changes: 0 additions & 60 deletions
60
demisto_sdk/commands/common/dev_sh_scripts/run_dev_tasks.sh
This file was deleted.
Oops, something went wrong.
34 changes: 0 additions & 34 deletions
34
demisto_sdk/commands/common/dev_sh_scripts/run_dev_tasks_pwsh.sh
This file was deleted.
Oops, something went wrong.
This file was deleted.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,85 @@ | ||
| import logging | ||
| import os | ||
|
|
||
|
|
||
| def logging_setup(verbose: int, quiet: bool, log_path: str) -> logging.Logger: | ||
| """ Init logger object for logging in demisto-sdk | ||
| For more info - https://docs.python.org/3/library/logging.html | ||
| Args: | ||
| verbose(int) verosity level - 1-3 | ||
| quiet(bool): Whether to output a quiet response. | ||
| log_path(str): Path to save log of all levels | ||
| Returns: | ||
| logging.Logger: logger object | ||
| """ | ||
| if quiet: | ||
| verbose = 0 | ||
| logger: logging.Logger = logging.getLogger('demisto-sdk') | ||
| logger.setLevel(logging.DEBUG) | ||
| log_level = logging.getLevelName((6 - 2 * verbose) * 10) | ||
| fmt = logging.Formatter('%(message)s') | ||
|
|
||
| if verbose: | ||
| console_handler = logging.StreamHandler() | ||
| console_handler.setLevel(log_level) | ||
| console_handler.setFormatter(fmt) | ||
| logger.addHandler(console_handler) | ||
|
|
||
| # Setting debug log file if in circleci | ||
| if log_path: | ||
| file_handler = logging.FileHandler(filename=os.path.join(log_path, 'lint_debug_log.log')) | ||
| file_handler.setFormatter(fmt) | ||
| file_handler.setLevel(level=logging.DEBUG) | ||
| logger.addHandler(file_handler) | ||
|
|
||
| logger.propagate = False | ||
|
|
||
| return logger | ||
|
|
||
|
|
||
| # Python program to print | ||
| # colored text and background | ||
| class Colors: | ||
| """Colors class:reset all colors with colors.reset; two | ||
| sub classes fg for foreground | ||
| and bg for background; use as colors.subclass.colorname. | ||
| i.e. colors.fg.red or colors.bg.greenalso, the generic bold, disable, | ||
| underline, reverse, strike through, | ||
| and invisible work with the main class i.e. colors.bold""" | ||
| reset = '\033[0m' | ||
| bold = '\033[01m' | ||
| disable = '\033[02m' | ||
| underline = '\033[04m' | ||
| reverse = '\033[07m' | ||
| strikethrough = '\033[09m' | ||
| invisible = '\033[08m' | ||
|
|
||
| class Fg: | ||
| """Forgrownd""" | ||
| black = '\033[30m' | ||
| red = '\033[31m' | ||
| green = '\033[32m' | ||
| orange = '\033[33m' | ||
| blue = '\033[34m' | ||
| purple = '\033[35m' | ||
| cyan = '\033[36m' | ||
| lightgrey = '\033[37m' | ||
| darkgrey = '\033[90m' | ||
| lightred = '\033[91m' | ||
| lightgreen = '\033[92m' | ||
| yellow = '\033[93m' | ||
| lightblue = '\033[94m' | ||
| pink = '\033[95m' | ||
| lightcyan = '\033[96m' | ||
|
|
||
| class Bg: | ||
| """Backgrownd""" | ||
| black = '\033[40m' | ||
| red = '\033[41m' | ||
| green = '\033[42m' | ||
| orange = '\033[43m' | ||
| blue = '\033[44m' | ||
| purple = '\033[45m' | ||
| cyan = '\033[46m' |
Oops, something went wrong.