Skip to content
This repository has been archived by the owner on Mar 17, 2021. It is now read-only.

SW360Updater now catches and logs exceptions during processing. #504

Conversation

oheger-bosch
Copy link
Contributor

Issue: eclipse#502.

This PR makes the updater of the compliance tool more robust against exceptions during artifact upload. Exceptions while processing an artifact no longer abort the process, but are just logged.

Request Reviewer

You can add desired reviewers here with an @mention.
@neubs-bsi

Type of Change

Mention one of the following:
bug fix | new feature | improvements | documentation update | CI | Other

Type of change:
improvements

How Has This Been Tested?

If you have added any changes that require additional tests, or changes in tests, you should implement them and describe them here.
All test that passed before your contribution should pass after it as well.
The existing test case has been extended to test exception handling.

Checklist

Must:

  • All related issues are referenced in commit messages

Optional: (delete if not applicable)

  • I have provided tests for the changes (if there are changes that need additional tests)

Issue: eclipse#502.

If an exception during processing of an artifact occurs, the update
process should not be stopped, but continue with the next artifact.

Signed-off-by: Oliver Heger <[email protected]>
@sonarqubecloud
Copy link

sonarqubecloud bot commented May 5, 2020

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 0 Code Smells

100.0% 100.0% Coverage
0.0% 0.0% Duplication

@oheger-bosch oheger-bosch merged commit 208e9e8 into eclipse-archived:master May 5, 2020
@oheger-bosch oheger-bosch deleted the oheger-bosch/sw360/#502_multiple_external_ids branch May 5, 2020 11:25
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants