Closed JCHacking closed 2 weeks ago
I like it. Can you add a test with tags populated?
Sure! Tests added
I see that the test has failed, I imagine that it is because it is not the same object since one is the tags only with name and the others are the tags obtained from the persistence that will have more properties.
I should only make an assertEquals of the list of strings with the name of the tags, right?
I should only make an assertEquals of the list of strings with the name of the tags, right?
Yes. Consider using AssertJ's fluent assertions, like so:
assertThat(project.getTags())
.extracting(Tag::getName)
.containsExactlyInAnyOrder("foo", "bar");
Fixed it with your suggestion, thanks!!
Coverage variation | Diff coverage |
---|---|
:white_check_mark: +0.02% (target: -1.00%) | :white_check_mark: 100.00% (target: 70.00%) |
:rocket: Don’t miss a bit, follow what’s new on Codacy.
Codacy stopped sending the deprecated coverage status on June 5th, 2024. Learn more
Description
To be able to upload a bom and auto-create the project with specific tags
Addressed Issue
https://github.com/DependencyTrack/dependency-track/issues/1674
Additional Details
I have added the optional property to the PUT method to upload BOM files, I have not programmed much in Java so any suggestion to improve this PR is welcome. Thanks!!!
Checklist