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);
}
