diff options
| author | Can Eldem <celdem@gitlab.com> | 2020-07-22 08:45:02 +0000 |
|---|---|---|
| committer | Can Eldem <celdem@gitlab.com> | 2020-07-22 08:45:02 +0000 |
| commit | ca675527b53c2a8316c962ee1a17bc1ee1c0c156 (patch) | |
| tree | d350d872ebf69f0b07ecf6497011c345205d11f7 /lib/license/management/shell.rb | |
| parent | e1bb260b43763a36536b7d3fa4d73108ffb604d4 (diff) | |
| parent | 0e7c92f64a4f910da3266be167fcf28dccb76689 (diff) | |
Merge branch '224187-add-maven-tests' into 'master'v3.19.1
Update maven/gradle offline tests
See merge request gitlab-org/security-products/license-management!195
Diffstat (limited to 'lib/license/management/shell.rb')
| -rw-r--r-- | lib/license/management/shell.rb | 13 |
1 files changed, 6 insertions, 7 deletions
diff --git a/lib/license/management/shell.rb b/lib/license/management/shell.rb index 408c760..47639df 100644 --- a/lib/license/management/shell.rb +++ b/lib/license/management/shell.rb @@ -79,14 +79,13 @@ module License end def record(stdout, stderr, status) - logger.debug(stdout) if present?(stdout) - return unless present?(stderr) + severity = status.success? ? Logger::DEBUG : Logger::ERROR + flush(stdout, severity) + flush(stderr, severity) + end - if status.success? - logger.debug(stderr) - else - logger.error(stderr) - end + def flush(message, severity) + logger.add(severity, message) if present?(message) end def collapsible_section(header) |
