Inject Configuraiton object into PatchSetReviewer Change-Id: I8853a578068e107fdfeb488d0ec599a780cb9744
diff --git a/src/main/java/com/googlesource/gerrit/plugins/chatgpt/PatchSetReviewer.java b/src/main/java/com/googlesource/gerrit/plugins/chatgpt/PatchSetReviewer.java index b2f54de..d4baa7e 100644 --- a/src/main/java/com/googlesource/gerrit/plugins/chatgpt/PatchSetReviewer.java +++ b/src/main/java/com/googlesource/gerrit/plugins/chatgpt/PatchSetReviewer.java
@@ -30,9 +30,9 @@ private static final String SPLIT_REVIEW_MSG = "Too many changes. Please consider splitting into patches smaller " + "than %s lines for review."; + private final Configuration config; private final GerritClient gerritClient; - private Configuration config; @Getter private IChatGptClient chatGptClient; private GerritCommentRange gerritCommentRange; @@ -41,12 +41,12 @@ private List<Integer> reviewScores; @Inject - PatchSetReviewer(GerritClient gerritClient) { + PatchSetReviewer(GerritClient gerritClient, Configuration config) { + this.config = config; this.gerritClient = gerritClient; } - public void review(Configuration config, GerritChange change) throws Exception { - this.config = config; + public void review(GerritChange change) throws Exception { reviewBatches = new ArrayList<>(); reviewScores = new ArrayList<>(); commentProperties = gerritClient.getClientData(change).getCommentProperties();
diff --git a/src/main/java/com/googlesource/gerrit/plugins/chatgpt/listener/EventHandlerTask.java b/src/main/java/com/googlesource/gerrit/plugins/chatgpt/listener/EventHandlerTask.java index ff05b16..796e77f 100644 --- a/src/main/java/com/googlesource/gerrit/plugins/chatgpt/listener/EventHandlerTask.java +++ b/src/main/java/com/googlesource/gerrit/plugins/chatgpt/listener/EventHandlerTask.java
@@ -96,7 +96,7 @@ try { log.info("Processing change: {}", change.getFullChangeId()); - reviewer.review(config, change); + reviewer.review(change); log.info("Finished processing change: {}", change.getFullChangeId()); } catch (Exception e) { log.error("Error while processing change: {}", change.getFullChangeId(), e);
diff --git a/src/test/java/com/googlesource/gerrit/plugins/chatgpt/ChatGptReviewTestBase.java b/src/test/java/com/googlesource/gerrit/plugins/chatgpt/ChatGptReviewTestBase.java index ddba64a..59d96de 100644 --- a/src/test/java/com/googlesource/gerrit/plugins/chatgpt/ChatGptReviewTestBase.java +++ b/src/test/java/com/googlesource/gerrit/plugins/chatgpt/ChatGptReviewTestBase.java
@@ -232,7 +232,7 @@ private void initTest () throws NoSuchProjectException { gerritClient = new GerritClient(); - patchSetReviewer = new PatchSetReviewer(gerritClient); + patchSetReviewer = new PatchSetReviewer(gerritClient, config); mockConfigCreator = mock(ConfigCreator.class); }