test plan controller to migrate tms #2191

imashkanov · 2025-03-05T17:04:31Z

No description provided.

github-actions

Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit

reviewdog

src/main/java/com/epam/ta/reportportal/core/tms/controller/TmsTestPlanController.java

github-actions · 2025-03-05T17:05:42Z

src/main/java/com/epam/ta/reportportal/core/tms/controller/TmsTestPlanController.java

+
+import static com.epam.ta.reportportal.auth.permissions.Permissions.ASSIGNED_TO_PROJECT;
+
+@RestController


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.javadoc.MissingJavadocTypeCheck> _{reported by reviewdog 🐶}
Missing a Javadoc comment.

src/main/java/com/epam/ta/reportportal/core/tms/controller/TmsTestPlanController.java

github-actions

Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit

reviewdog

src/main/java/com/epam/ta/reportportal/core/tms/controller/TmsTestPlanController.java

github-actions · 2025-03-07T08:04:40Z

src/main/java/com/epam/ta/reportportal/core/tms/controller/TmsTestPlanController.java

+		return tmsTestPlanService.update(projectId, testPlanId, testPlan);
+	}
+
+	@GetMapping("/{id}")


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.indentation.IndentationCheck> _{reported by reviewdog 🐶}
'method def modifier' has incorrect indentation level 8, expected level should be 2.

github-actions

Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit

reviewdog

src/main/java/com/epam/ta/reportportal/core/tms/controller/TmsTestPlanController.java

github-actions · 2025-03-07T08:42:36Z

src/main/java/com/epam/ta/reportportal/core/tms/controller/TmsTestPlanController.java

+
+	@GetMapping("/{id}")
+//	@PreAuthorize(ASSIGNED_TO_PROJECT)
+	public TmsTestPlanRS getTestPlanById(@PathVariable Long projectId, @PathVariable("id") Long testPlanId) {


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.whitespace.FileTabCharacterCheck> _{reported by reviewdog 🐶}
Line contains a tab character.

github-actions · 2025-03-07T08:42:36Z

src/main/java/com/epam/ta/reportportal/core/tms/controller/TmsTestPlanController.java

+	@GetMapping("/{id}")
+//	@PreAuthorize(ASSIGNED_TO_PROJECT)
+	public TmsTestPlanRS getTestPlanById(@PathVariable Long projectId, @PathVariable("id") Long testPlanId) {
+		return tmsTestPlanService.getById(projectId, testPlanId);


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.whitespace.FileTabCharacterCheck> _{reported by reviewdog 🐶}
Line contains a tab character.

github-actions · 2025-03-07T08:42:38Z

src/main/java/com/epam/ta/reportportal/core/tms/controller/TmsTestPlanController.java

+//	@PreAuthorize(ASSIGNED_TO_PROJECT)
+	public TmsTestPlanRS patchTestPlan(@PathVariable Long projectId, @PathVariable("id") Long testPlanId,
+			@RequestBody TmsTestPlanRQ updatedTestPlan) {
+		return tmsTestPlanService.patch(projectId, testPlanId, updatedTestPlan);


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.indentation.IndentationCheck> _{reported by reviewdog 🐶}
'method def' child has incorrect indentation level 16, expected level should be 4.

github-actions · 2025-03-07T08:42:38Z

src/main/java/com/epam/ta/reportportal/core/tms/controller/TmsTestPlanController.java

+	public TmsTestPlanRS patchTestPlan(@PathVariable Long projectId, @PathVariable("id") Long testPlanId,
+			@RequestBody TmsTestPlanRQ updatedTestPlan) {
+		return tmsTestPlanService.patch(projectId, testPlanId, updatedTestPlan);
+	}


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.whitespace.FileTabCharacterCheck> _{reported by reviewdog 🐶}
Line contains a tab character.

github-actions · 2025-03-07T08:42:38Z

src/main/java/com/epam/ta/reportportal/core/tms/controller/TmsTestPlanController.java

+	public TmsTestPlanRS patchTestPlan(@PathVariable Long projectId, @PathVariable("id") Long testPlanId,
+			@RequestBody TmsTestPlanRQ updatedTestPlan) {
+		return tmsTestPlanService.patch(projectId, testPlanId, updatedTestPlan);
+	}


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.indentation.IndentationCheck> _{reported by reviewdog 🐶}
'method def rcurly' has incorrect indentation level 8, expected level should be 2.

github-actions · 2025-03-07T08:42:38Z

src/main/java/com/epam/ta/reportportal/core/tms/dto/TmsAttributeRQ.java

+import com.fasterxml.jackson.annotation.JsonInclude;
+import lombok.Data;
+
+@Data


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.javadoc.MissingJavadocTypeCheck> _{reported by reviewdog 🐶}
Missing a Javadoc comment.

github-actions · 2025-03-07T08:42:38Z

src/main/java/com/epam/ta/reportportal/core/tms/dto/TmsAttributeRQ.java

+
+@Data
+@JsonInclude(JsonInclude.Include.NON_EMPTY)
+public class TmsAttributeRQ {


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.naming.AbbreviationAsWordInNameCheck> _{reported by reviewdog 🐶}
Abbreviation in name 'TmsAttributeRQ' must contain no more than '1' consecutive capital letters.

github-actions

Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit

reviewdog

github-actions · 2025-03-07T09:13:57Z

src/main/java/com/epam/ta/reportportal/core/tms/controller/TmsTestPlanController.java

+	private final TmsTestPlanService tmsTestPlanService;
+
+	@PostMapping
+//	@PreAuthorize(ASSIGNED_TO_PROJECT) //TODO revert


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.indentation.CommentsIndentationCheck> _{reported by reviewdog 🐶}
Comment has incorrect indentation level 0, expected is 1, indentation should be the same level as line 36.

github-actions · 2025-03-07T09:13:57Z

src/main/java/com/epam/ta/reportportal/core/tms/controller/TmsTestPlanController.java

+	private final TmsTestPlanService tmsTestPlanService;
+
+	@PostMapping
+//	@PreAuthorize(ASSIGNED_TO_PROJECT) //TODO revert


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.whitespace.FileTabCharacterCheck> _{reported by reviewdog 🐶}
Line contains a tab character.

github-actions · 2025-03-07T09:13:57Z

src/main/java/com/epam/ta/reportportal/core/tms/controller/TmsTestPlanController.java

+	}
+
+	@PutMapping("/{id}")
+//	@PreAuthorize(ASSIGNED_TO_PROJECT) //TODO revert


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.indentation.CommentsIndentationCheck> _{reported by reviewdog 🐶}
Comment has incorrect indentation level 0, expected is 1, indentation should be the same level as line 55.

github-actions · 2025-03-07T09:13:57Z

src/main/java/com/epam/ta/reportportal/core/tms/controller/TmsTestPlanController.java

+	}
+
+	@PutMapping("/{id}")
+//	@PreAuthorize(ASSIGNED_TO_PROJECT) //TODO revert


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.whitespace.FileTabCharacterCheck> _{reported by reviewdog 🐶}
Line contains a tab character.

github-actions · 2025-03-07T09:13:57Z

src/main/java/com/epam/ta/reportportal/core/tms/controller/TmsTestPlanController.java

+	}
+
+	@GetMapping("/{id}")
+//	@PreAuthorize(ASSIGNED_TO_PROJECT) //TODO revert


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.indentation.CommentsIndentationCheck> _{reported by reviewdog 🐶}
Comment has incorrect indentation level 0, expected is 1, indentation should be the same level as line 62.

github-actions · 2025-03-07T09:13:59Z

src/main/java/com/epam/ta/reportportal/core/tms/dto/TmsMilestoneRS.java

+@JsonInclude(JsonInclude.Include.NON_EMPTY)
+public class TmsMilestoneRS {
+	private Long id;
+	private String name;


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.whitespace.FileTabCharacterCheck> _{reported by reviewdog 🐶}
Line contains a tab character.

github-actions · 2025-03-07T09:13:59Z

src/main/java/com/epam/ta/reportportal/core/tms/dto/TmsMilestoneRS.java

+@JsonInclude(JsonInclude.Include.NON_EMPTY)
+public class TmsMilestoneRS {
+	private Long id;
+	private String name;


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.indentation.IndentationCheck> _{reported by reviewdog 🐶}
'member def modifier' has incorrect indentation level 8, expected level should be 2.

github-actions · 2025-03-07T09:13:59Z

src/main/java/com/epam/ta/reportportal/core/tms/dto/TmsMilestoneRS.java

+public class TmsMilestoneRS {
+	private Long id;
+	private String name;
+	private String type;


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.whitespace.FileTabCharacterCheck> _{reported by reviewdog 🐶}
Line contains a tab character.

github-actions · 2025-03-07T09:13:59Z

src/main/java/com/epam/ta/reportportal/core/tms/dto/TmsMilestoneRS.java

+public class TmsMilestoneRS {
+	private Long id;
+	private String name;
+	private String type;


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.indentation.IndentationCheck> _{reported by reviewdog 🐶}
'member def modifier' has incorrect indentation level 8, expected level should be 2.

github-actions · 2025-03-07T09:13:59Z

src/main/java/com/epam/ta/reportportal/core/tms/dto/TmsMilestoneRS.java

+	private Long id;
+	private String name;
+	private String type;
+	private String startDate;


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.whitespace.FileTabCharacterCheck> _{reported by reviewdog 🐶}
Line contains a tab character.

github-actions

Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit

reviewdog

github-actions · 2025-03-07T09:53:23Z

src/main/java/com/epam/ta/reportportal/core/tms/dto/TmsMilestoneRS.java

+	private Long id;
+	private String name;
+	private String type;
+	private String startDate;


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.indentation.IndentationCheck> _{reported by reviewdog 🐶}
'member def modifier' has incorrect indentation level 8, expected level should be 2.

github-actions · 2025-03-07T09:53:23Z

src/main/java/com/epam/ta/reportportal/core/tms/dto/TmsMilestoneRS.java

+	private String name;
+	private String type;
+	private String startDate;
+	private String endDate;


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.whitespace.FileTabCharacterCheck> _{reported by reviewdog 🐶}
Line contains a tab character.

github-actions · 2025-03-07T09:53:23Z

src/main/java/com/epam/ta/reportportal/core/tms/dto/TmsMilestoneRS.java

+	private String name;
+	private String type;
+	private String startDate;
+	private String endDate;


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.indentation.IndentationCheck> _{reported by reviewdog 🐶}
'member def modifier' has incorrect indentation level 8, expected level should be 2.

github-actions · 2025-03-07T09:53:23Z

src/main/java/com/epam/ta/reportportal/core/tms/dto/TmsMilestoneRS.java

+	private String type;
+	private String startDate;
+	private String endDate;
+	private TmsProductVersionRS productVersion;


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.whitespace.FileTabCharacterCheck> _{reported by reviewdog 🐶}
Line contains a tab character.

github-actions · 2025-03-07T09:53:23Z

src/main/java/com/epam/ta/reportportal/core/tms/dto/TmsMilestoneRS.java

+	private String type;
+	private String startDate;
+	private String endDate;
+	private TmsProductVersionRS productVersion;


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.indentation.IndentationCheck> _{reported by reviewdog 🐶}
'member def modifier' has incorrect indentation level 8, expected level should be 2.

github-actions · 2025-03-07T09:53:25Z

src/main/java/com/epam/ta/reportportal/core/tms/dto/TmsProductVersionRS.java

+@Data
+@JsonInclude(JsonInclude.Include.NON_EMPTY)
+public class TmsProductVersionRS {
+	private Long id;


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.whitespace.FileTabCharacterCheck> _{reported by reviewdog 🐶}
Line contains a tab character.

github-actions · 2025-03-07T09:53:25Z

src/main/java/com/epam/ta/reportportal/core/tms/dto/TmsProductVersionRS.java

+@Data
+@JsonInclude(JsonInclude.Include.NON_EMPTY)
+public class TmsProductVersionRS {
+	private Long id;


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.indentation.IndentationCheck> _{reported by reviewdog 🐶}
'member def modifier' has incorrect indentation level 8, expected level should be 2.

github-actions · 2025-03-07T09:53:25Z

src/main/java/com/epam/ta/reportportal/core/tms/dto/TmsProductVersionRS.java

+public class TmsProductVersionRS {
+	private Long id;
+
+	private String version;


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.whitespace.FileTabCharacterCheck> _{reported by reviewdog 🐶}
Line contains a tab character.

github-actions · 2025-03-07T09:53:25Z

src/main/java/com/epam/ta/reportportal/core/tms/dto/TmsProductVersionRS.java

+public class TmsProductVersionRS {
+	private Long id;
+
+	private String version;


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.indentation.IndentationCheck> _{reported by reviewdog 🐶}
'member def modifier' has incorrect indentation level 8, expected level should be 2.

github-actions · 2025-03-07T09:53:25Z

src/main/java/com/epam/ta/reportportal/core/tms/dto/TmsEnvironmentRS.java

+import com.fasterxml.jackson.annotation.JsonInclude;
+import lombok.Data;
+
+@Data


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.javadoc.MissingJavadocTypeCheck> _{reported by reviewdog 🐶}
Missing a Javadoc comment.

github-actions

Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit

reviewdog

github-actions · 2025-03-09T18:54:35Z

src/main/java/com/epam/ta/reportportal/core/tms/controller/TmsTestPlanController.java

+			@RequestParam(required = false) List<Long> productVersionId,
+			Pageable pageable) {
+		return tmsTestPlanService.getByCriteria(
+				projectId, environmentId, productVersionId, pageable


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.whitespace.FileTabCharacterCheck> _{reported by reviewdog 🐶}
Line contains a tab character.

github-actions · 2025-03-09T18:54:35Z

src/main/java/com/epam/ta/reportportal/core/tms/controller/TmsTestPlanController.java

+	}
+
+	@PutMapping("/{id}")
+//	@PreAuthorize(ASSIGNED_TO_PROJECT) //TODO revert


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.indentation.CommentsIndentationCheck> _{reported by reviewdog 🐶}
Comment has incorrect indentation level 0, expected is 1, indentation should be the same level as line 53.

github-actions · 2025-03-09T18:54:36Z

src/main/java/com/epam/ta/reportportal/core/tms/controller/TmsTestPlanController.java

+	}
+
+	@GetMapping("/{id}")
+//	@PreAuthorize(ASSIGNED_TO_PROJECT) //TODO revert


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.indentation.CommentsIndentationCheck> _{reported by reviewdog 🐶}
Comment has incorrect indentation level 0, expected is 1, indentation should be the same level as line 60.

github-actions · 2025-03-09T18:54:36Z

src/main/java/com/epam/ta/reportportal/core/tms/controller/TmsTestPlanController.java

+	}
+
+	@DeleteMapping("/{id}")
+//	@PreAuthorize(ASSIGNED_TO_PROJECT) //TODO revert


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.indentation.CommentsIndentationCheck> _{reported by reviewdog 🐶}
Comment has incorrect indentation level 0, expected is 1, indentation should be the same level as line 66.

github-actions · 2025-03-09T18:54:36Z

src/main/java/com/epam/ta/reportportal/core/tms/controller/TmsTestPlanController.java

+	}
+
+	@PatchMapping("/{id}")
+//	@PreAuthorize(ASSIGNED_TO_PROJECT) //TODO revert


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.indentation.CommentsIndentationCheck> _{reported by reviewdog 🐶}
Comment has incorrect indentation level 0, expected is 1, indentation should be the same level as line 72.

github-actions · 2025-03-09T18:54:38Z

src/main/java/com/epam/ta/reportportal/core/tms/dto/TmsTestPlanRQ.java

+
+import com.fasterxml.jackson.annotation.JsonInclude;
+import lombok.Data;
+@Data


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.javadoc.MissingJavadocTypeCheck> _{reported by reviewdog 🐶}
Missing a Javadoc comment.

github-actions · 2025-03-09T18:54:38Z

src/main/java/com/epam/ta/reportportal/core/tms/dto/TmsTestPlanRQ.java

+import lombok.Data;
+@Data
+@JsonInclude(JsonInclude.Include.NON_EMPTY)
+public class TmsTestPlanRQ {


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.naming.AbbreviationAsWordInNameCheck> _{reported by reviewdog 🐶}
Abbreviation in name 'TmsTestPlanRQ' must contain no more than '1' consecutive capital letters.

github-actions · 2025-03-09T18:54:38Z

src/main/java/com/epam/ta/reportportal/core/tms/dto/TmsTestPlanRQ.java

+@Data
+@JsonInclude(JsonInclude.Include.NON_EMPTY)
+public class TmsTestPlanRQ {
+	private String name;


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.whitespace.FileTabCharacterCheck> _{reported by reviewdog 🐶}
Line contains a tab character.

github-actions · 2025-03-09T18:54:38Z

src/main/java/com/epam/ta/reportportal/core/tms/dto/TmsTestPlanRQ.java

+@Data
+@JsonInclude(JsonInclude.Include.NON_EMPTY)
+public class TmsTestPlanRQ {
+	private String name;


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.indentation.IndentationCheck> _{reported by reviewdog 🐶}
'member def modifier' has incorrect indentation level 8, expected level should be 2.

github-actions · 2025-03-09T18:54:38Z

src/main/java/com/epam/ta/reportportal/core/tms/dto/TmsTestPlanRQ.java

+@JsonInclude(JsonInclude.Include.NON_EMPTY)
+public class TmsTestPlanRQ {
+	private String name;
+	private String description;


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.whitespace.FileTabCharacterCheck> _{reported by reviewdog 🐶}
Line contains a tab character.

github-actions

Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit

reviewdog

github-actions · 2025-03-09T19:03:52Z

src/main/java/com/epam/ta/reportportal/core/tms/dto/TmsTestPlanRQ.java

+@JsonInclude(JsonInclude.Include.NON_EMPTY)
+public class TmsTestPlanRQ {
+	private String name;
+	private String description;


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.indentation.IndentationCheck> _{reported by reviewdog 🐶}
'member def modifier' has incorrect indentation level 8, expected level should be 2.

github-actions · 2025-03-09T19:03:52Z

src/main/java/com/epam/ta/reportportal/core/tms/dto/TmsTestPlanRQ.java

+public class TmsTestPlanRQ {
+	private String name;
+	private String description;
+	private Long environmentId;


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.whitespace.FileTabCharacterCheck> _{reported by reviewdog 🐶}
Line contains a tab character.

github-actions · 2025-03-09T19:03:52Z

src/main/java/com/epam/ta/reportportal/core/tms/dto/TmsTestPlanRQ.java

+public class TmsTestPlanRQ {
+	private String name;
+	private String description;
+	private Long environmentId;


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.indentation.IndentationCheck> _{reported by reviewdog 🐶}
'member def modifier' has incorrect indentation level 8, expected level should be 2.

github-actions · 2025-03-09T19:03:53Z

src/main/java/com/epam/ta/reportportal/core/tms/dto/TmsTestPlanRQ.java

+	private String name;
+	private String description;
+	private Long environmentId;
+	private Long productVersionId;


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.whitespace.FileTabCharacterCheck> _{reported by reviewdog 🐶}
Line contains a tab character.

github-actions · 2025-03-09T19:03:53Z

src/main/java/com/epam/ta/reportportal/core/tms/dto/TmsTestPlanRQ.java

+	private String name;
+	private String description;
+	private Long environmentId;
+	private Long productVersionId;


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.indentation.IndentationCheck> _{reported by reviewdog 🐶}
'member def modifier' has incorrect indentation level 8, expected level should be 2.

github-actions · 2025-03-09T19:03:55Z

src/main/java/com/epam/ta/reportportal/core/tms/mapper/TmsTestPlanAttributeMapper.java

+@Mapper(config = CommonMapperConfig.class)
+public abstract class TmsTestPlanAttributeMapper {
+	public abstract Set<TmsTestPlanAttribute> convertToTmsTestPlanAttributes(
+			List<TmsTestPlanAttributeRQ> tmsTestPlanAttributeRQList);


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.whitespace.FileTabCharacterCheck> _{reported by reviewdog 🐶}
Line contains a tab character.

github-actions · 2025-03-09T19:03:55Z

src/main/java/com/epam/ta/reportportal/core/tms/mapper/TmsTestPlanAttributeMapper.java

+@Mapper(config = CommonMapperConfig.class)
+public abstract class TmsTestPlanAttributeMapper {
+	public abstract Set<TmsTestPlanAttribute> convertToTmsTestPlanAttributes(
+			List<TmsTestPlanAttributeRQ> tmsTestPlanAttributeRQList);


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.naming.AbbreviationAsWordInNameCheck> _{reported by reviewdog 🐶}
Abbreviation in name 'tmsTestPlanAttributeRQList' must contain no more than '1' consecutive capital letters.

github-actions · 2025-03-09T19:03:55Z

src/main/java/com/epam/ta/reportportal/core/tms/mapper/TmsTestPlanAttributeMapper.java

+	public abstract Set<TmsTestPlanAttribute> convertToTmsTestPlanAttributes(
+			List<TmsTestPlanAttributeRQ> tmsTestPlanAttributeRQList);
+
+	@Mapping(target = "attribute.id", source = "tmsTestPlanAttributeRQ.attributeId")


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.whitespace.FileTabCharacterCheck> _{reported by reviewdog 🐶}
Line contains a tab character.

github-actions · 2025-03-09T19:03:55Z

src/main/java/com/epam/ta/reportportal/core/tms/mapper/TmsTestPlanAttributeMapper.java

+	public abstract Set<TmsTestPlanAttribute> convertToTmsTestPlanAttributes(
+			List<TmsTestPlanAttributeRQ> tmsTestPlanAttributeRQList);
+
+	@Mapping(target = "attribute.id", source = "tmsTestPlanAttributeRQ.attributeId")


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.indentation.IndentationCheck> _{reported by reviewdog 🐶}
'method def modifier' has incorrect indentation level 8, expected level should be 2.

github-actions · 2025-03-09T19:03:55Z

src/main/java/com/epam/ta/reportportal/core/tms/mapper/TmsTestPlanAttributeMapper.java

+			List<TmsTestPlanAttributeRQ> tmsTestPlanAttributeRQList);
+
+	@Mapping(target = "attribute.id", source = "tmsTestPlanAttributeRQ.attributeId")
+	@Mapping(target = "id.attributeId", source = "tmsTestPlanAttributeRQ.attributeId")


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.whitespace.FileTabCharacterCheck> _{reported by reviewdog 🐶}
Line contains a tab character.

github-actions

Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit

reviewdog

github-actions · 2025-03-09T19:52:01Z

src/main/java/com/epam/ta/reportportal/core/tms/mapper/TmsTestPlanAttributeMapper.java

+
+	@Mapping(target = "attribute.id", source = "tmsTestPlanAttributeRQ.attributeId")
+	@Mapping(target = "id.attributeId", source = "tmsTestPlanAttributeRQ.attributeId")
+	public abstract TmsTestPlanAttribute convertToTmsTestPlanAttribute(TmsTestPlanAttributeRQ tmsTestPlanAttributeRQ);


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.sizes.LineLengthCheck> _{reported by reviewdog 🐶}
Line is longer than 100 characters (found 122).

github-actions · 2025-03-09T19:52:01Z

src/main/java/com/epam/ta/reportportal/core/tms/mapper/TmsTestPlanAttributeMapper.java

+
+	@Mapping(target = "attribute.id", source = "tmsTestPlanAttributeRQ.attributeId")
+	@Mapping(target = "id.attributeId", source = "tmsTestPlanAttributeRQ.attributeId")
+	public abstract TmsTestPlanAttribute convertToTmsTestPlanAttribute(TmsTestPlanAttributeRQ tmsTestPlanAttributeRQ);


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.whitespace.FileTabCharacterCheck> _{reported by reviewdog 🐶}
Line contains a tab character.

github-actions · 2025-03-09T19:52:01Z

src/main/java/com/epam/ta/reportportal/core/tms/mapper/TmsTestPlanAttributeMapper.java

+
+	@Mapping(target = "attribute.id", source = "tmsTestPlanAttributeRQ.attributeId")
+	@Mapping(target = "id.attributeId", source = "tmsTestPlanAttributeRQ.attributeId")
+	public abstract TmsTestPlanAttribute convertToTmsTestPlanAttribute(TmsTestPlanAttributeRQ tmsTestPlanAttributeRQ);


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.naming.AbbreviationAsWordInNameCheck> _{reported by reviewdog 🐶}
Abbreviation in name 'tmsTestPlanAttributeRQ' must contain no more than '1' consecutive capital letters.

github-actions · 2025-03-09T19:52:01Z

src/main/java/com/epam/ta/reportportal/core/tms/mapper/TmsMilestoneMapper.java

+import org.mapstruct.Mapper;
+import org.mapstruct.Mapping;
+
+import java.util.List;


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.imports.CustomImportOrderCheck> _{reported by reviewdog 🐶}
Extra separation in import group before 'java.util.List'

github-actions · 2025-03-09T19:52:01Z

src/main/java/com/epam/ta/reportportal/core/tms/mapper/TmsMilestoneMapper.java

+import org.mapstruct.Mapper;
+import org.mapstruct.Mapping;
+
+import java.util.List;


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.imports.CustomImportOrderCheck> _{reported by reviewdog 🐶}
Wrong lexicographical order for 'java.util.List' import. Should be before 'org.mapstruct.Mapping'.

github-actions · 2025-03-09T19:52:03Z

src/main/java/com/epam/ta/reportportal/core/tms/mapper/TmsTestPlanMapper.java

+	@Mapping(target = "productVersion", source = "testPlanRQ.productVersionId")
+	@Mapping(target = "milestones", ignore = true)
+	@Mapping(target = "attributes", ignore = true)
+	public abstract TmsTestPlan convertFromRQ(Long projectId, TmsTestPlanRQ testPlanRQ);


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.naming.AbbreviationAsWordInNameCheck> _{reported by reviewdog 🐶}
Abbreviation in name 'convertFromRQ' must contain no more than '1' consecutive capital letters.

github-actions · 2025-03-09T19:52:03Z

src/main/java/com/epam/ta/reportportal/core/tms/mapper/TmsTestPlanMapper.java

+	@Mapping(target = "productVersion", source = "testPlanRQ.productVersionId")
+	@Mapping(target = "milestones", ignore = true)
+	@Mapping(target = "attributes", ignore = true)
+	public abstract TmsTestPlan convertFromRQ(Long projectId, TmsTestPlanRQ testPlanRQ);


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.naming.AbbreviationAsWordInNameCheck> _{reported by reviewdog 🐶}
Abbreviation in name 'testPlanRQ' must contain no more than '1' consecutive capital letters.

github-actions · 2025-03-09T19:52:03Z

src/main/java/com/epam/ta/reportportal/core/tms/mapper/TmsTestPlanMapper.java

+	@Mapping(target = "attributes", ignore = true)
+	public abstract TmsTestPlan convertFromRQ(Long projectId, TmsTestPlanRQ testPlanRQ);
+
+	@BeanMapping(nullValuePropertyMappingStrategy = NullValuePropertyMappingStrategy.SET_TO_NULL, nullValueCheckStrategy = NullValueCheckStrategy.ON_IMPLICIT_CONVERSION)


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.sizes.LineLengthCheck> _{reported by reviewdog 🐶}
Line is longer than 100 characters (found 173).

github-actions · 2025-03-09T19:52:03Z

src/main/java/com/epam/ta/reportportal/core/tms/mapper/TmsTestPlanMapper.java

+	@Mapping(target = "attributes", ignore = true)
+	public abstract TmsTestPlan convertFromRQ(Long projectId, TmsTestPlanRQ testPlanRQ);
+
+	@BeanMapping(nullValuePropertyMappingStrategy = NullValuePropertyMappingStrategy.SET_TO_NULL, nullValueCheckStrategy = NullValueCheckStrategy.ON_IMPLICIT_CONVERSION)


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.whitespace.FileTabCharacterCheck> _{reported by reviewdog 🐶}
Line contains a tab character.

github-actions · 2025-03-09T19:52:03Z

src/main/java/com/epam/ta/reportportal/core/tms/mapper/TmsTestPlanMapper.java

+	@Mapping(target = "attributes", ignore = true)
+	public abstract TmsTestPlan convertFromRQ(Long projectId, TmsTestPlanRQ testPlanRQ);
+
+	@BeanMapping(nullValuePropertyMappingStrategy = NullValuePropertyMappingStrategy.SET_TO_NULL, nullValueCheckStrategy = NullValueCheckStrategy.ON_IMPLICIT_CONVERSION)


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.indentation.IndentationCheck> _{reported by reviewdog 🐶}
'method def modifier' has incorrect indentation level 8, expected level should be 2.

github-actions

Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit

reviewdog

github-actions · 2025-03-09T19:54:38Z

src/main/java/com/epam/ta/reportportal/core/tms/mapper/TmsTestPlanMapper.java

+	public abstract TmsTestPlan convertFromRQ(Long projectId, TmsTestPlanRQ testPlanRQ);
+
+	@BeanMapping(nullValuePropertyMappingStrategy = NullValuePropertyMappingStrategy.SET_TO_NULL, nullValueCheckStrategy = NullValueCheckStrategy.ON_IMPLICIT_CONVERSION)
+	@Mapping(target = "id", ignore = true)


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.whitespace.FileTabCharacterCheck> _{reported by reviewdog 🐶}
Line contains a tab character.

github-actions · 2025-03-09T19:54:38Z

src/main/java/com/epam/ta/reportportal/core/tms/mapper/TmsTestPlanMapper.java

+
+	@BeanMapping(nullValuePropertyMappingStrategy = NullValuePropertyMappingStrategy.SET_TO_NULL, nullValueCheckStrategy = NullValueCheckStrategy.ON_IMPLICIT_CONVERSION)
+	@Mapping(target = "id", ignore = true)
+	public abstract void update(@MappingTarget TmsTestPlan targetTestPlan, TmsTestPlan tmsTestPlan);


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.sizes.LineLengthCheck> _{reported by reviewdog 🐶}
Line is longer than 100 characters (found 104).

github-actions · 2025-03-09T19:54:38Z

src/main/java/com/epam/ta/reportportal/core/tms/mapper/TmsTestPlanMapper.java

+
+	@BeanMapping(nullValuePropertyMappingStrategy = NullValuePropertyMappingStrategy.SET_TO_NULL, nullValueCheckStrategy = NullValueCheckStrategy.ON_IMPLICIT_CONVERSION)
+	@Mapping(target = "id", ignore = true)
+	public abstract void update(@MappingTarget TmsTestPlan targetTestPlan, TmsTestPlan tmsTestPlan);


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.whitespace.FileTabCharacterCheck> _{reported by reviewdog 🐶}
Line contains a tab character.

github-actions · 2025-03-09T19:54:38Z

src/main/java/com/epam/ta/reportportal/core/tms/mapper/TmsTestPlanMapper.java

+	@Mapping(target = "id", ignore = true)
+	public abstract void update(@MappingTarget TmsTestPlan targetTestPlan, TmsTestPlan tmsTestPlan);
+
+	@BeanMapping(nullValuePropertyMappingStrategy = NullValuePropertyMappingStrategy.IGNORE, nullValueCheckStrategy = NullValueCheckStrategy.ALWAYS)


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.sizes.LineLengthCheck> _{reported by reviewdog 🐶}
Line is longer than 100 characters (found 152).

github-actions · 2025-03-09T19:54:38Z

src/main/java/com/epam/ta/reportportal/core/tms/mapper/TmsTestPlanMapper.java

+	@Mapping(target = "id", ignore = true)
+	public abstract void update(@MappingTarget TmsTestPlan targetTestPlan, TmsTestPlan tmsTestPlan);
+
+	@BeanMapping(nullValuePropertyMappingStrategy = NullValuePropertyMappingStrategy.IGNORE, nullValueCheckStrategy = NullValueCheckStrategy.ALWAYS)


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.whitespace.FileTabCharacterCheck> _{reported by reviewdog 🐶}
Line contains a tab character.

github-actions · 2025-03-09T19:54:40Z

src/main/java/com/epam/ta/reportportal/core/tms/service/TmsTestPlanAttributeService.java

+
+import java.util.List;
+
+public interface TmsTestPlanAttributeService {


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.javadoc.MissingJavadocTypeCheck> _{reported by reviewdog 🐶}
Missing a Javadoc comment.

github-actions · 2025-03-09T19:54:40Z

src/main/java/com/epam/ta/reportportal/core/tms/service/TmsTestPlanAttributeService.java

+
+public interface TmsTestPlanAttributeService {
+
+	void createTestPlanAttributes(TmsTestPlan tmsTestPlan, List<TmsTestPlanAttributeRQ> attributes);


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.sizes.LineLengthCheck> _{reported by reviewdog 🐶}
Line is longer than 100 characters (found 104).

github-actions · 2025-03-09T19:54:40Z

src/main/java/com/epam/ta/reportportal/core/tms/service/TmsTestPlanAttributeService.java

+
+public interface TmsTestPlanAttributeService {
+
+	void createTestPlanAttributes(TmsTestPlan tmsTestPlan, List<TmsTestPlanAttributeRQ> attributes);


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.whitespace.FileTabCharacterCheck> _{reported by reviewdog 🐶}
Line contains a tab character.

github-actions · 2025-03-09T19:54:41Z

src/main/java/com/epam/ta/reportportal/core/tms/service/TmsTestPlanAttributeService.java

+
+public interface TmsTestPlanAttributeService {
+
+	void createTestPlanAttributes(TmsTestPlan tmsTestPlan, List<TmsTestPlanAttributeRQ> attributes);


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.indentation.IndentationCheck> _{reported by reviewdog 🐶}
'method def modifier' has incorrect indentation level 8, expected level should be 2.

github-actions · 2025-03-09T19:54:41Z

src/main/java/com/epam/ta/reportportal/core/tms/service/TmsTestPlanAttributeService.java

+
+	void createTestPlanAttributes(TmsTestPlan tmsTestPlan, List<TmsTestPlanAttributeRQ> attributes);
+
+	void updateTestPlanAttributes(TmsTestPlan existingTestPlan,


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.whitespace.FileTabCharacterCheck> _{reported by reviewdog 🐶}
Line contains a tab character.

github-actions

Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit

reviewdog

github-actions · 2025-03-10T13:44:36Z

src/main/java/com/epam/ta/reportportal/core/tms/controller/TmsTestPlanController.java

+  private final TmsTestPlanService tmsTestPlanService;
+
+  @PostMapping
+//	@PreAuthorize(ASSIGNED_TO_PROJECT) //TODO revert


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.indentation.CommentsIndentationCheck> _{reported by reviewdog 🐶}
Comment has incorrect indentation level 0, expected is 2, indentation should be the same level as line 32.

github-actions · 2025-03-10T13:44:36Z

src/main/java/com/epam/ta/reportportal/core/tms/controller/TmsTestPlanController.java

+    return tmsTestPlanService.create(projectId, testPlan);
+  }
+
+  @GetMapping


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.javadoc.MissingJavadocMethodCheck> _{reported by reviewdog 🐶}
Missing a Javadoc comment.

github-actions · 2025-03-10T13:44:36Z

src/main/java/com/epam/ta/reportportal/core/tms/controller/TmsTestPlanController.java

+  }
+
+  @PutMapping("/{id}")
+//	@PreAuthorize(ASSIGNED_TO_PROJECT) //TODO revert


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.indentation.CommentsIndentationCheck> _{reported by reviewdog 🐶}
Comment has incorrect indentation level 0, expected is 2, indentation should be the same level as line 50.

github-actions · 2025-03-10T13:44:36Z

src/main/java/com/epam/ta/reportportal/core/tms/controller/TmsTestPlanController.java

+  }
+
+  @GetMapping("/{id}")
+//	@PreAuthorize(ASSIGNED_TO_PROJECT) //TODO revert


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.indentation.CommentsIndentationCheck> _{reported by reviewdog 🐶}
Comment has incorrect indentation level 0, expected is 2, indentation should be the same level as line 58.

github-actions · 2025-03-10T13:44:36Z

src/main/java/com/epam/ta/reportportal/core/tms/controller/TmsTestPlanController.java

+  }
+
+  @DeleteMapping("/{id}")
+//	@PreAuthorize(ASSIGNED_TO_PROJECT) //TODO revert


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.indentation.CommentsIndentationCheck> _{reported by reviewdog 🐶}
Comment has incorrect indentation level 0, expected is 2, indentation should be the same level as line 65.

github-actions · 2025-03-10T13:44:39Z

src/main/java/com/epam/ta/reportportal/core/tms/service/ProductVersionServiceImpl.java

@@ -61,6 +63,11 @@ public ProductVersionRS update(long projectID,final Long productVersionID, final

        return productVersionMapper.convert(productVersionRepository.save(productVersion));    }

+    @Override
+    public ProductVersionRS patch(long projectId, Long aLong, ProductVersionRQ t) {
+        throw new UnsupportedOperationException();


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.indentation.IndentationCheck> _{reported by reviewdog 🐶}
'method def' child has incorrect indentation level 8, expected level should be 4.

github-actions · 2025-03-10T13:44:39Z

src/main/java/com/epam/ta/reportportal/core/tms/service/ProductVersionServiceImpl.java

+    @Override
+    public ProductVersionRS patch(long projectId, Long aLong, ProductVersionRQ t) {
+        throw new UnsupportedOperationException();
+    }


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.indentation.IndentationCheck> _{reported by reviewdog 🐶}
'method def rcurly' has incorrect indentation level 4, expected level should be 2.

github-actions · 2025-03-10T13:44:39Z

src/main/java/com/epam/ta/reportportal/core/tms/service/TmsTestPlanServiceImpl.java

+import org.springframework.stereotype.Service;
+import org.springframework.transaction.annotation.Transactional;
+
+@Service


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.javadoc.MissingJavadocTypeCheck> _{reported by reviewdog 🐶}
Missing a Javadoc comment.

github-actions · 2025-03-10T13:44:39Z

src/main/java/com/epam/ta/reportportal/core/tms/service/TmsTestPlanServiceImpl.java

+@RequiredArgsConstructor
+public class TmsTestPlanServiceImpl implements TmsTestPlanService {
+
+  private static final String TMS_TEST_PLAN_NOT_FOUND_BY_ID = "TMS test pan cannot be found by id: {0} for project: {1}";


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.sizes.LineLengthCheck> _{reported by reviewdog 🐶}
Line is longer than 100 characters (found 121).

github-actions · 2025-03-10T13:44:39Z

src/main/java/com/epam/ta/reportportal/core/tms/service/TmsTestPlanServiceImpl.java

+
+  @Override
+  @Transactional
+  public TmsTestPlanRS create(long projectId, TmsTestPlanRQ testPlanRQ) {


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.naming.AbbreviationAsWordInNameCheck> _{reported by reviewdog 🐶}
Abbreviation in name 'testPlanRQ' must contain no more than '1' consecutive capital letters.

github-actions

Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit

reviewdog

src/test/java/com/epam/ta/reportportal/core/tms/service/TmsTestPlanServiceImplTest.java|177 col 9| Abbreviation in name 'testPlanRS' must contain no more than '1' consecutive capital letters.
src/test/java/com/epam/ta/reportportal/core/tms/service/TmsTestPlanServiceImplTest.java|198 col 9| Abbreviation in name 'testPlanRQ' must contain no more than '1' consecutive capital letters.
src/test/java/com/epam/ta/reportportal/core/tms/service/TmsTestPlanServiceImplTest.java|201 col 9| Abbreviation in name 'testPlanRS' must contain no more than '1' consecutive capital letters.
src/test/java/com/epam/ta/reportportal/core/tms/service/TmsTestPlanServiceImplTest.java|223 col 9| Abbreviation in name 'testPlanRQ' must contain no more than '1' consecutive capital letters.
src/test/java/com/epam/ta/reportportal/core/tms/service/TmsTestPlanAttributeServiceImplTest.java|50 col 5| Distance between variable 'tmsTestPlan' declaration and its first usage is 4, but allowed 3. Consider making that variable final if you still need to store its value in advance (before method calls that might have side effects on the original value).
src/test/java/com/epam/ta/reportportal/core/tms/service/TmsTestPlanAttributeServiceImplTest.java|51 col 9| Abbreviation in name 'attributesRQ' must contain no more than '1' consecutive capital letters.
src/test/java/com/epam/ta/reportportal/core/tms/service/TmsTestPlanAttributeServiceImplTest.java|83 col 5| Distance between variable 'existingTestPlan' declaration and its first usage is 4, but allowed 3. Consider making that variable final if you still need to store its value in advance (before method calls that might have side effects on the original value).
src/test/java/com/epam/ta/reportportal/core/tms/service/TmsTestPlanAttributeServiceImplTest.java|84 col 9| Abbreviation in name 'attributesRQ' must contain no more than '1' consecutive capital letters.
src/test/java/com/epam/ta/reportportal/core/tms/service/TmsTestPlanAttributeServiceImplTest.java|124 col 9| Abbreviation in name 'attributesRQ' must contain no more than '1' consecutive capital letters.

github-actions · 2025-03-10T13:46:07Z

src/main/java/com/epam/ta/reportportal/core/tms/service/TmsTestPlanServiceImpl.java

+
+  @Override
+  @Transactional
+  public TmsTestPlanRS update(long projectId, Long testPlanId, TmsTestPlanRQ testPlanRQ) {


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.naming.AbbreviationAsWordInNameCheck> _{reported by reviewdog 🐶}
Abbreviation in name 'testPlanRQ' must contain no more than '1' consecutive capital letters.

github-actions · 2025-03-10T13:46:07Z

src/main/java/com/epam/ta/reportportal/core/tms/service/TmsTestPlanServiceImpl.java

+
+  @Override
+  @Transactional
+  public TmsTestPlanRS patch(long projectId, Long testPlanId, TmsTestPlanRQ testPlanRQ) {


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.naming.AbbreviationAsWordInNameCheck> _{reported by reviewdog 🐶}
Abbreviation in name 'testPlanRQ' must contain no more than '1' consecutive capital letters.

github-actions · 2025-03-10T13:46:07Z

src/main/java/com/epam/ta/reportportal/core/tms/service/TmsMilestoneServiceImpl.java

+import org.springframework.stereotype.Service;
+import org.springframework.transaction.annotation.Transactional;
+
+@Service


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.javadoc.MissingJavadocTypeCheck> _{reported by reviewdog 🐶}
Missing a Javadoc comment.

github-actions · 2025-03-10T13:46:07Z

src/main/java/com/epam/ta/reportportal/core/tms/db/repository/TmsTestPlanRepository.java

+import org.springframework.data.repository.query.Param;
+import org.springframework.stereotype.Repository;
+
+@Repository


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.javadoc.MissingJavadocTypeCheck> _{reported by reviewdog 🐶}
Missing a Javadoc comment.

github-actions · 2025-03-10T13:46:07Z

src/main/java/com/epam/ta/reportportal/core/tms/db/repository/TmsTestPlanRepository.java

+@Repository
+public interface TmsTestPlanRepository extends ReportPortalRepository<TmsTestPlan, Long> {
+
+  @Query("SELECT tp FROM TmsTestPlan tp " +


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.whitespace.OperatorWrapCheck> _{reported by reviewdog 🐶}
'+' should be on a new line.

github-actions · 2025-03-10T13:46:09Z

src/test/java/com/epam/ta/reportportal/core/tms/service/TmsTestPlanServiceImplTest.java

+    var projectId = 1L;
+    var testPlanRQ = new TmsTestPlanRQ();
+    var testPlan = new TmsTestPlan();
+    var testPlanRS = new TmsTestPlanRS();


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.naming.AbbreviationAsWordInNameCheck> _{reported by reviewdog 🐶}
Abbreviation in name 'testPlanRS' must contain no more than '1' consecutive capital letters.

github-actions · 2025-03-10T13:46:09Z

src/test/java/com/epam/ta/reportportal/core/tms/service/TmsTestPlanServiceImplTest.java

+    var pageable = PageRequest.of(0, 10);
+
+    var testPlan = new TmsTestPlan();
+    var testPlanRS = new TmsTestPlanRS();


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.naming.AbbreviationAsWordInNameCheck> _{reported by reviewdog 🐶}
Abbreviation in name 'testPlanRS' must contain no more than '1' consecutive capital letters.

github-actions · 2025-03-10T13:46:09Z

src/test/java/com/epam/ta/reportportal/core/tms/service/TmsTestPlanServiceImplTest.java

+  void shouldUpdateExisting() {
+    var projectId = 1L;
+    var testPlanId = 2L;
+    var testPlanRQ = new TmsTestPlanRQ();


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.naming.AbbreviationAsWordInNameCheck> _{reported by reviewdog 🐶}
Abbreviation in name 'testPlanRQ' must contain no more than '1' consecutive capital letters.

github-actions · 2025-03-10T13:46:09Z

src/test/java/com/epam/ta/reportportal/core/tms/service/TmsTestPlanServiceImplTest.java

+    var testPlanRQ = new TmsTestPlanRQ();
+    var existingTestPlan = new TmsTestPlan();
+    var updatedTestPlan = new TmsTestPlan();
+    var testPlanRS = new TmsTestPlanRS();


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.naming.AbbreviationAsWordInNameCheck> _{reported by reviewdog 🐶}
Abbreviation in name 'testPlanRS' must contain no more than '1' consecutive capital letters.

github-actions · 2025-03-10T13:46:09Z

src/test/java/com/epam/ta/reportportal/core/tms/service/TmsTestPlanServiceImplTest.java

+  void shouldCreateWhenUpdateButNotFound() {
+    var projectId = 1L;
+    var testPlanId = 2L;
+    var testPlanRQ = new TmsTestPlanRQ();


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.naming.AbbreviationAsWordInNameCheck> _{reported by reviewdog 🐶}
Abbreviation in name 'testPlanRQ' must contain no more than '1' consecutive capital letters.

github-actions · 2025-03-10T13:58:41Z

src/test/java/com/epam/ta/reportportal/core/tms/service/TmsTestPlanServiceImplTest.java

+    var testPlanId = 2L;
+    var testPlanRQ = new TmsTestPlanRQ();
+    var testPlan = new TmsTestPlan();
+    var testPlanRS = new TmsTestPlanRS();


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.naming.AbbreviationAsWordInNameCheck> _{reported by reviewdog 🐶}
Abbreviation in name 'testPlanRS' must contain no more than '1' consecutive capital letters.

github-actions · 2025-03-10T13:58:41Z

src/test/java/com/epam/ta/reportportal/core/tms/service/TmsTestPlanServiceImplTest.java

+  void shouldPatchExisting() {
+    var projectId = 1L;
+    var testPlanId = 2L;
+    var testPlanRQ = new TmsTestPlanRQ();


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.naming.AbbreviationAsWordInNameCheck> _{reported by reviewdog 🐶}
Abbreviation in name 'testPlanRQ' must contain no more than '1' consecutive capital letters.

github-actions · 2025-03-10T13:58:41Z

src/test/java/com/epam/ta/reportportal/core/tms/service/TmsTestPlanServiceImplTest.java

+    var testPlanRQ = new TmsTestPlanRQ();
+    var existingTestPlan = new TmsTestPlan();
+    var patchedTestPlan = new TmsTestPlan();
+    var testPlanRS = new TmsTestPlanRS();


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.naming.AbbreviationAsWordInNameCheck> _{reported by reviewdog 🐶}
Abbreviation in name 'testPlanRS' must contain no more than '1' consecutive capital letters.

github-actions · 2025-03-10T13:58:41Z

src/test/java/com/epam/ta/reportportal/core/tms/service/TmsTestPlanServiceImplTest.java

+  void shouldThrowNotFoundExceptionWhenPatchAndNotFound() {
+    var projectId = 1L;
+    var testPlanId = 2L;
+    var testPlanRQ = new TmsTestPlanRQ();


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.naming.AbbreviationAsWordInNameCheck> _{reported by reviewdog 🐶}
Abbreviation in name 'testPlanRQ' must contain no more than '1' consecutive capital letters.

github-actions · 2025-03-10T13:58:41Z

...test/java/com/epam/ta/reportportal/core/tms/service/TmsTestPlanAttributeServiceImplTest.java

+
+  @Test
+  void shouldCreateTestPlanAttributes() {
+    var tmsTestPlan = new TmsTestPlan();


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck> _{reported by reviewdog 🐶}
Distance between variable 'tmsTestPlan' declaration and its first usage is 4, but allowed 3. Consider making that variable final if you still need to store its value in advance (before method calls that might have side effects on the original value).

github-actions · 2025-03-10T13:58:41Z

...test/java/com/epam/ta/reportportal/core/tms/service/TmsTestPlanAttributeServiceImplTest.java

+  @Test
+  void shouldCreateTestPlanAttributes() {
+    var tmsTestPlan = new TmsTestPlan();
+    var attributesRQ = List.of(new TmsTestPlanAttributeRQ(), new TmsTestPlanAttributeRQ());


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.naming.AbbreviationAsWordInNameCheck> _{reported by reviewdog 🐶}
Abbreviation in name 'attributesRQ' must contain no more than '1' consecutive capital letters.

github-actions · 2025-03-10T13:58:42Z

...test/java/com/epam/ta/reportportal/core/tms/service/TmsTestPlanAttributeServiceImplTest.java

+
+  @Test
+  void shouldTestUpdateTestPlanAttributes() {
+    var existingTestPlan = new TmsTestPlan();


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.coding.VariableDeclarationUsageDistanceCheck> _{reported by reviewdog 🐶}
Distance between variable 'existingTestPlan' declaration and its first usage is 4, but allowed 3. Consider making that variable final if you still need to store its value in advance (before method calls that might have side effects on the original value).

github-actions · 2025-03-10T13:58:42Z

...test/java/com/epam/ta/reportportal/core/tms/service/TmsTestPlanAttributeServiceImplTest.java

+  @Test
+  void shouldTestUpdateTestPlanAttributes() {
+    var existingTestPlan = new TmsTestPlan();
+    var attributesRQ = List.of(new TmsTestPlanAttributeRQ(), new TmsTestPlanAttributeRQ());


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.naming.AbbreviationAsWordInNameCheck> _{reported by reviewdog 🐶}
Abbreviation in name 'attributesRQ' must contain no more than '1' consecutive capital letters.

github-actions · 2025-03-10T13:58:42Z

...test/java/com/epam/ta/reportportal/core/tms/service/TmsTestPlanAttributeServiceImplTest.java

+    existingAttributes.add(new TmsTestPlanAttribute());
+    existingTestPlan.setAttributes(existingAttributes);
+
+    var attributesRQ = List.of(new TmsTestPlanAttributeRQ(), new TmsTestPlanAttributeRQ());


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.naming.AbbreviationAsWordInNameCheck> _{reported by reviewdog 🐶}
Abbreviation in name 'attributesRQ' must contain no more than '1' consecutive capital letters.

github-actions

Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit

reviewdog

github-actions · 2025-03-10T14:18:24Z

src/main/java/com/epam/ta/reportportal/core/tms/controller/TmsTestPlanController.java

+@RequiredArgsConstructor
+public class TmsTestPlanController {
+
+    private final TmsTestPlanService tmsTestPlanService;


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.indentation.IndentationCheck> _{reported by reviewdog 🐶}
'member def modifier' has incorrect indentation level 4, expected level should be 2.

github-actions · 2025-03-10T14:18:24Z

src/main/java/com/epam/ta/reportportal/core/tms/controller/TmsTestPlanController.java

+
+    private final TmsTestPlanService tmsTestPlanService;
+
+    @PostMapping


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.indentation.IndentationCheck> _{reported by reviewdog 🐶}
'method def modifier' has incorrect indentation level 4, expected level should be 2.

github-actions · 2025-03-10T14:18:24Z

src/main/java/com/epam/ta/reportportal/core/tms/controller/TmsTestPlanController.java

+    @PreAuthorize(ASSIGNED_TO_PROJECT)
+    public TmsTestPlanRS createTestPlan(@PathVariable Long projectId,
+        @RequestBody TmsTestPlanRQ testPlan) {
+        return tmsTestPlanService.create(projectId, testPlan);


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.indentation.IndentationCheck> _{reported by reviewdog 🐶}
'method def' child has incorrect indentation level 8, expected level should be 4.

github-actions · 2025-03-10T14:18:24Z

src/main/java/com/epam/ta/reportportal/core/tms/controller/TmsTestPlanController.java

+    public TmsTestPlanRS createTestPlan(@PathVariable Long projectId,
+        @RequestBody TmsTestPlanRQ testPlan) {
+        return tmsTestPlanService.create(projectId, testPlan);
+    }


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.indentation.IndentationCheck> _{reported by reviewdog 🐶}
'method def rcurly' has incorrect indentation level 4, expected level should be 2.

github-actions · 2025-03-10T14:18:25Z

src/main/java/com/epam/ta/reportportal/core/tms/controller/TmsTestPlanController.java

+        return tmsTestPlanService.create(projectId, testPlan);
+    }
+
+    @GetMapping


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.indentation.IndentationCheck> _{reported by reviewdog 🐶}
'method def modifier' has incorrect indentation level 4, expected level should be 2.

github-actions · 2025-03-10T14:18:27Z

src/main/java/com/epam/ta/reportportal/core/tms/dto/TmsMilestoneRS.java

+public class TmsMilestoneRS {
+
+    private Long id;
+    private String name;


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.indentation.IndentationCheck> _{reported by reviewdog 🐶}
'member def modifier' has incorrect indentation level 4, expected level should be 2.

github-actions · 2025-03-10T14:18:27Z

src/main/java/com/epam/ta/reportportal/core/tms/dto/TmsMilestoneRS.java

+
+    private Long id;
+    private String name;
+    private String type;


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.indentation.IndentationCheck> _{reported by reviewdog 🐶}
'member def modifier' has incorrect indentation level 4, expected level should be 2.

src/main/java/com/epam/ta/reportportal/core/tms/dto/TmsMilestoneRS.java

+    private Long id;
+    private String name;
+    private String type;
+    private String startDate;


github-actions · 2025-03-10T14:18:27Z

src/main/java/com/epam/ta/reportportal/core/tms/dto/TmsMilestoneRS.java

+    private String name;
+    private String type;
+    private String startDate;
+    private String endDate;


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.indentation.IndentationCheck> _{reported by reviewdog 🐶}
'member def modifier' has incorrect indentation level 4, expected level should be 2.

github-actions · 2025-03-10T14:18:27Z

src/main/java/com/epam/ta/reportportal/core/tms/dto/TmsMilestoneRS.java

+    private String type;
+    private String startDate;
+    private String endDate;
+    private TmsProductVersionRS productVersion;


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.indentation.IndentationCheck> _{reported by reviewdog 🐶}
'member def modifier' has incorrect indentation level 4, expected level should be 2.

github-actions

Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit

reviewdog

github-actions · 2025-03-10T14:21:44Z

src/main/java/com/epam/ta/reportportal/core/tms/dto/TmsTestPlanRS.java

+@JsonInclude(JsonInclude.Include.NON_EMPTY)
+public class TmsTestPlanRS {
+
+    private Long id;


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.indentation.IndentationCheck> _{reported by reviewdog 🐶}
'member def modifier' has incorrect indentation level 4, expected level should be 2.

github-actions · 2025-03-10T14:21:44Z

src/main/java/com/epam/ta/reportportal/core/tms/dto/TmsTestPlanRS.java

+public class TmsTestPlanRS {
+
+    private Long id;
+    private String name;


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.indentation.IndentationCheck> _{reported by reviewdog 🐶}
'member def modifier' has incorrect indentation level 4, expected level should be 2.

github-actions · 2025-03-10T14:21:44Z

src/main/java/com/epam/ta/reportportal/core/tms/dto/TmsTestPlanRS.java

+
+    private Long id;
+    private String name;
+    private String description;


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.indentation.IndentationCheck> _{reported by reviewdog 🐶}
'member def modifier' has incorrect indentation level 4, expected level should be 2.

github-actions · 2025-03-10T14:21:44Z

src/main/java/com/epam/ta/reportportal/core/tms/dto/TmsTestPlanRS.java

+    private Long id;
+    private String name;
+    private String description;
+    private TmsEnvironmentRS environment;


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.indentation.IndentationCheck> _{reported by reviewdog 🐶}
'member def modifier' has incorrect indentation level 4, expected level should be 2.

github-actions · 2025-03-10T14:21:44Z

src/main/java/com/epam/ta/reportportal/core/tms/dto/TmsTestPlanRS.java

+    private String name;
+    private String description;
+    private TmsEnvironmentRS environment;
+    private TmsProductVersionRS productVersion;


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.indentation.IndentationCheck> _{reported by reviewdog 🐶}
'member def modifier' has incorrect indentation level 4, expected level should be 2.

github-actions · 2025-03-10T14:21:47Z

src/main/java/com/epam/ta/reportportal/core/tms/mapper/TmsTestPlanAttributeMapper.java

+    @Mapping(target = "attribute.id", source = "tmsTestPlanAttributeRQ.attributeId")
+    @Mapping(target = "id.attributeId", source = "tmsTestPlanAttributeRQ.attributeId")
+    public abstract TmsTestPlanAttribute convertToTmsTestPlanAttribute(
+        TmsTestPlanAttributeRQ tmsTestPlanAttributeRQ);


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.naming.AbbreviationAsWordInNameCheck> _{reported by reviewdog 🐶}
Abbreviation in name 'tmsTestPlanAttributeRQ' must contain no more than '1' consecutive capital letters.

github-actions · 2025-03-10T14:21:47Z

src/main/java/com/epam/ta/reportportal/core/tms/mapper/TmsMilestoneMapper.java

+@Mapper(config = CommonMapperConfig.class)
+public abstract class TmsMilestoneMapper {
+
+    public abstract Set<TmsMilestone> convertToTmsMilestones(List<Long> milestoneIds);


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.indentation.IndentationCheck> _{reported by reviewdog 🐶}
'method def modifier' has incorrect indentation level 4, expected level should be 2.

github-actions · 2025-03-10T14:21:47Z

src/main/java/com/epam/ta/reportportal/core/tms/mapper/TmsMilestoneMapper.java

+
+    public abstract Set<TmsMilestone> convertToTmsMilestones(List<Long> milestoneIds);
+
+    @Mapping(target = "id", source = "tmsMilestoneId")


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.indentation.IndentationCheck> _{reported by reviewdog 🐶}
'method def modifier' has incorrect indentation level 4, expected level should be 2.

github-actions · 2025-03-10T14:21:47Z

src/main/java/com/epam/ta/reportportal/core/tms/mapper/TmsTestPlanMapper.java

+    TmsProductVersionMapper.class})
+public abstract class TmsTestPlanMapper {
+
+    public abstract TmsTestPlanRS convertToRS(TmsTestPlan tmsTestPlan);


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.indentation.IndentationCheck> _{reported by reviewdog 🐶}
'method def modifier' has incorrect indentation level 4, expected level should be 2.

github-actions · 2025-03-10T14:21:47Z

src/main/java/com/epam/ta/reportportal/core/tms/mapper/TmsTestPlanMapper.java

+    TmsProductVersionMapper.class})
+public abstract class TmsTestPlanMapper {
+
+    public abstract TmsTestPlanRS convertToRS(TmsTestPlan tmsTestPlan);


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.naming.AbbreviationAsWordInNameCheck> _{reported by reviewdog 🐶}
Abbreviation in name 'convertToRS' must contain no more than '1' consecutive capital letters.

github-actions

Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit

reviewdog

github-actions · 2025-03-11T07:59:04Z

src/main/java/com/epam/ta/reportportal/core/tms/dto/TmsTestPlanRS.java

+    private Long id;
+    private String name;
+    private String description;
+    private TmsTestPlanEnvironmentRS environment;


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.indentation.IndentationCheck> _{reported by reviewdog 🐶}
'member def modifier' has incorrect indentation level 4, expected level should be 2.

github-actions · 2025-03-11T07:59:04Z

src/main/java/com/epam/ta/reportportal/core/tms/dto/TmsTestPlanRS.java

+    private String name;
+    private String description;
+    private TmsTestPlanEnvironmentRS environment;
+    private TmsTestPlanProductVersionRS productVersion;


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.indentation.IndentationCheck> _{reported by reviewdog 🐶}
'member def modifier' has incorrect indentation level 4, expected level should be 2.

github-actions · 2025-03-11T07:59:05Z

src/main/java/com/epam/ta/reportportal/core/tms/dto/TmsTestPlanRS.java

+    private TmsTestPlanEnvironmentRS environment;
+    private TmsTestPlanProductVersionRS productVersion;
+    private List<TmsTestPlanAttributeRS> attributes;
+    private List<TmsTestPlanMilestoneRS> milestones;


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.indentation.IndentationCheck> _{reported by reviewdog 🐶}
'member def modifier' has incorrect indentation level 4, expected level should be 2.

github-actions · 2025-03-11T07:59:05Z

src/main/java/com/epam/ta/reportportal/core/tms/dto/TmsTestPlanProductVersionRS.java

+import com.fasterxml.jackson.annotation.JsonInclude;
+import lombok.Data;
+
+@Data


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.javadoc.MissingJavadocTypeCheck> _{reported by reviewdog 🐶}
Missing a Javadoc comment.

github-actions · 2025-03-11T07:59:05Z

src/main/java/com/epam/ta/reportportal/core/tms/dto/TmsTestPlanProductVersionRS.java

+
+@Data
+@JsonInclude(JsonInclude.Include.NON_EMPTY)
+public class TmsTestPlanProductVersionRS {


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.naming.AbbreviationAsWordInNameCheck> _{reported by reviewdog 🐶}
Abbreviation in name 'TmsTestPlanProductVersionRS' must contain no more than '1' consecutive capital letters.

github-actions · 2025-03-11T07:59:07Z

src/main/java/com/epam/ta/reportportal/core/tms/mapper/TmsTestPlanMapper.java

+    @Mapping(target = "id", ignore = true)
+    public abstract void update(@MappingTarget TmsTestPlan targetTestPlan, TmsTestPlan tmsTestPlan);
+
+    @BeanMapping(nullValuePropertyMappingStrategy = NullValuePropertyMappingStrategy.IGNORE, nullValueCheckStrategy = NullValueCheckStrategy.ALWAYS)


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.sizes.LineLengthCheck> _{reported by reviewdog 🐶}
Line is longer than 100 characters (found 148).

github-actions · 2025-03-11T07:59:07Z

src/main/java/com/epam/ta/reportportal/core/tms/mapper/TmsTestPlanMapper.java

+    @Mapping(target = "id", ignore = true)
+    public abstract void update(@MappingTarget TmsTestPlan targetTestPlan, TmsTestPlan tmsTestPlan);
+
+    @BeanMapping(nullValuePropertyMappingStrategy = NullValuePropertyMappingStrategy.IGNORE, nullValueCheckStrategy = NullValueCheckStrategy.ALWAYS)


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.indentation.IndentationCheck> _{reported by reviewdog 🐶}
'method def modifier' has incorrect indentation level 4, expected level should be 2.

github-actions · 2025-03-11T07:59:07Z

src/main/java/com/epam/ta/reportportal/core/tms/mapper/TmsTestPlanMapper.java

+    public abstract void patch(@MappingTarget TmsTestPlan existingTestPlan,
+        TmsTestPlan tmsTestPlan);
+
+    public Page<TmsTestPlanRS> convertToRS(Page<TmsTestPlan> testPlansByCriteria) {


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.indentation.IndentationCheck> _{reported by reviewdog 🐶}
'method def modifier' has incorrect indentation level 4, expected level should be 2.

github-actions · 2025-03-11T07:59:07Z

src/main/java/com/epam/ta/reportportal/core/tms/mapper/TmsTestPlanMapper.java

+    public abstract void patch(@MappingTarget TmsTestPlan existingTestPlan,
+        TmsTestPlan tmsTestPlan);
+
+    public Page<TmsTestPlanRS> convertToRS(Page<TmsTestPlan> testPlansByCriteria) {


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.coding.OverloadMethodsDeclarationOrderCheck> _{reported by reviewdog 🐶}
All overloaded methods should be placed next to each other. Placing non-overloaded methods in between overloaded methods with the same type is a violation. Previous overloaded method located at line '20'.

github-actions · 2025-03-11T07:59:07Z

src/main/java/com/epam/ta/reportportal/core/tms/mapper/TmsTestPlanMapper.java

+    public abstract void patch(@MappingTarget TmsTestPlan existingTestPlan,
+        TmsTestPlan tmsTestPlan);
+
+    public Page<TmsTestPlanRS> convertToRS(Page<TmsTestPlan> testPlansByCriteria) {


⚠️ [reviewdog] <com.puppycrawl.tools.checkstyle.checks.javadoc.MissingJavadocMethodCheck> _{reported by reviewdog 🐶}
Missing a Javadoc comment.

github-actions bot reviewed Mar 5, 2025

View reviewed changes

github-actions bot reviewed Mar 7, 2025

View reviewed changes

github-actions bot reviewed Mar 9, 2025

View reviewed changes

github-actions bot reviewed Mar 10, 2025

View reviewed changes

imashkanov force-pushed the feature/EPMRPP-test-plan-controller-to-migrate-tms branch from 29fe3ec to efda028 Compare March 10, 2025 13:57

github-actions bot reviewed Mar 10, 2025

View reviewed changes

imashkanov force-pushed the feature/EPMRPP-test-plan-controller-to-migrate-tms branch from efda028 to e3ae333 Compare March 10, 2025 14:00

github-actions bot reviewed Mar 10, 2025

View reviewed changes

imashkanov force-pushed the feature/EPMRPP-test-plan-controller-to-migrate-tms branch from 0c9c193 to c51e29f Compare March 10, 2025 14:20

github-actions bot reviewed Mar 10, 2025

View reviewed changes

imashkanov marked this pull request as ready for review March 10, 2025 17:51

imashkanov requested a review from pbortnik as a code owner March 10, 2025 17:51

imashkanov changed the title ~~Feature/epmrpp test plan controller to migrate tms~~ test plan controller to migrate tms Mar 10, 2025

Implementation of tms_test_plan CRUD

9ec3010

imashkanov force-pushed the feature/EPMRPP-test-plan-controller-to-migrate-tms branch from c51e29f to 9ec3010 Compare March 11, 2025 07:58

github-actions bot reviewed Mar 11, 2025

View reviewed changes


		import static com.epam.ta.reportportal.auth.permissions.Permissions.ASSIGNED_TO_PROJECT;

		@RestController


		import java.util.List;

		public interface TmsTestPlanAttributeService {


		public interface TmsTestPlanAttributeService {

		void createTestPlanAttributes(TmsTestPlan tmsTestPlan, List<TmsTestPlanAttributeRQ> attributes);


		void createTestPlanAttributes(TmsTestPlan tmsTestPlan, List<TmsTestPlanAttributeRQ> attributes);

		void updateTestPlanAttributes(TmsTestPlan existingTestPlan,


		private final TmsTestPlanService tmsTestPlanService;

		@PostMapping


		public abstract Set<TmsMilestone> convertToTmsMilestones(List<Long> milestoneIds);

		@Mapping(target = "id", source = "tmsMilestoneId")

test plan controller to migrate tms #2191

Are you sure you want to change the base?

test plan controller to migrate tms #2191

Conversation

imashkanov commented Mar 5, 2025

github-actions bot left a comment

Choose a reason for hiding this comment

github-actions bot Mar 5, 2025

Choose a reason for hiding this comment

github-actions bot left a comment

Choose a reason for hiding this comment

github-actions bot Mar 7, 2025

Choose a reason for hiding this comment

github-actions bot left a comment

Choose a reason for hiding this comment

github-actions bot Mar 7, 2025

Choose a reason for hiding this comment

github-actions bot Mar 7, 2025

Choose a reason for hiding this comment

github-actions bot Mar 7, 2025

Choose a reason for hiding this comment

github-actions bot Mar 7, 2025

Choose a reason for hiding this comment

github-actions bot Mar 7, 2025

Choose a reason for hiding this comment

github-actions bot Mar 7, 2025

Choose a reason for hiding this comment

github-actions bot Mar 7, 2025

Choose a reason for hiding this comment

github-actions bot left a comment

Choose a reason for hiding this comment

github-actions bot Mar 7, 2025

Choose a reason for hiding this comment

github-actions bot Mar 7, 2025

Choose a reason for hiding this comment

github-actions bot Mar 7, 2025

Choose a reason for hiding this comment

github-actions bot Mar 7, 2025

Choose a reason for hiding this comment

github-actions bot Mar 7, 2025

Choose a reason for hiding this comment

github-actions bot Mar 7, 2025

Choose a reason for hiding this comment

github-actions bot Mar 7, 2025

Choose a reason for hiding this comment

github-actions bot Mar 7, 2025

Choose a reason for hiding this comment

github-actions bot Mar 7, 2025

Choose a reason for hiding this comment

github-actions bot Mar 7, 2025

Choose a reason for hiding this comment

github-actions bot left a comment

Choose a reason for hiding this comment

github-actions bot Mar 7, 2025

Choose a reason for hiding this comment

github-actions bot Mar 7, 2025

Choose a reason for hiding this comment

github-actions bot Mar 7, 2025

Choose a reason for hiding this comment

github-actions bot Mar 7, 2025

Choose a reason for hiding this comment

github-actions bot Mar 7, 2025

Choose a reason for hiding this comment

github-actions bot Mar 7, 2025

Choose a reason for hiding this comment

github-actions bot Mar 7, 2025

Choose a reason for hiding this comment

github-actions bot Mar 7, 2025

Choose a reason for hiding this comment

github-actions bot Mar 7, 2025

Choose a reason for hiding this comment

github-actions bot Mar 7, 2025

Choose a reason for hiding this comment

github-actions bot left a comment

Choose a reason for hiding this comment

github-actions bot Mar 9, 2025

Choose a reason for hiding this comment

github-actions bot Mar 9, 2025

Choose a reason for hiding this comment

github-actions bot Mar 9, 2025