fixup! Add in_depends-on operator
* Remove unused var in InDependsOnOperator
* Remove unwanted query in test_independson_operator.sh
Change-Id: I4e938ce7b96b77e0a4361ed4d8a9e282c59887bd
diff --git a/src/main/java/com/googlesource/gerrit/plugins/depends/on/InDependsOnOperator.java b/src/main/java/com/googlesource/gerrit/plugins/depends/on/InDependsOnOperator.java
index c89d241..e5566f0 100644
--- a/src/main/java/com/googlesource/gerrit/plugins/depends/on/InDependsOnOperator.java
+++ b/src/main/java/com/googlesource/gerrit/plugins/depends/on/InDependsOnOperator.java
@@ -24,7 +24,6 @@
import com.google.gerrit.server.query.change.ChangeQueryBuilder.ChangeOperatorFactory;
import com.google.inject.Inject;
import com.google.inject.Singleton;
-import java.util.HashSet;
import java.util.Set;
import java.util.stream.Collectors;
import org.slf4j.Logger;
@@ -39,7 +38,6 @@
public InDependsOnPredicate(String value) {
super(InDependsOnOperator.FIELD, value);
- Change.Id change = Change.Id.tryParse(value).get();
dependentChanges =
changeMessageStore.load(Change.Id.tryParse(value).get()).stream()
.map(d -> d.id())
diff --git a/test/test_independson_operator.sh b/test/test_independson_operator.sh
index 858be00..ac9ce3a 100755
--- a/test/test_independson_operator.sh
+++ b/test/test_independson_operator.sh
@@ -126,7 +126,7 @@
CHANGE=$(create_change "$SRC_REF_BRANCH" "$FILE_A") || \
die "Failed to create change on project: $PROJECT branch: $SRC_REF_BRANCH"
gssh gerrit review --message \'"Depends-on: $DEPENDENT_CHANGE"\' "$CHANGE",1
-EXPECTED="$(query "$DEPENDENT_CHANGE" | jq --raw-output '.number')"
+EXPECTED=$DEPENDENT_CHANGE
ACTUAL="$(query "independson:$CHANGE" | jq --raw-output '.number')"
result_out "independson operator" "$EXPECTED" "$ACTUAL"
exit $RESULT
