feat(mr list): Support `--reviewer=any/none

Author: claytonrcarter

cmd/mr_list.go

@@ -146,7 +146,11 @@ func mrList(args []string) ([]*gitlab.MergeRequest, error) {
 		mrMilestone = milestone.Title
 	}
 
-	if mrReviewer != "" {
+	if mrReviewer == "any" {
+		mrReviewerID = gitlab.ReviewerID(gitlab.UserIDAny)
+	} else if mrReviewer == "none" {
+		mrReviewerID = gitlab.ReviewerID(gitlab.UserIDNone)
+	} else if mrReviewer != "" {
 		reviewerID := getUserID(mrReviewer)
 		if reviewerID == nil {
 			log.Fatalf("%s user not found\n", mrReviewer)
@@ -247,7 +251,7 @@ func init() {
 	listCmd.Flags().BoolVar(&mrConflicts, "conflicts", false, "list only MRs that cannot be merged")
 	listCmd.Flags().BoolVarP(&mrExactMatch, "exact-match", "x", false, "match on the exact (case-insensitive) search terms")
 	listCmd.Flags().StringVar(
-		&mrReviewer, "reviewer", "", "list only MRs with reviewer set to $username")
+		&mrReviewer, "reviewer", "", "list only MRs with reviewer set to $username/any/none")
 
 	mrCmd.AddCommand(listCmd)
 	carapace.Gen(listCmd).FlagCompletion(carapace.ActionMap{

cmd/mr_list_test.go

@@ -360,3 +360,54 @@ func Test_mrListApproverNone(t *testing.T) {
 	require.NotContains(t, mrs, "!1 Test MR for lab list")
 	require.Contains(t, mrs, "!3 for testings filtering with labels and lists")
 }
+
+func Test_mrListReviewer(t *testing.T) {
+	t.Parallel()
+	repo := copyTestRepo(t)
+	cmd := exec.Command(labBinaryPath, "mr", "list", "--reviewer=lab-testing")
+	cmd.Dir = repo
+
+	b, err := cmd.CombinedOutput()
+	if err != nil {
+		t.Fatal(err)
+	}
+
+	mrs := strings.Split(string(b), "\n")
+	t.Log(mrs)
+	require.NotContains(t, mrs, "!1 Test MR for lab list")
+	require.Contains(t, mrs, "!329 MR for assign and review commands")
+}
+
+func Test_mrListReviewerAny(t *testing.T) {
+	t.Parallel()
+	repo := copyTestRepo(t)
+	cmd := exec.Command(labBinaryPath, "mr", "list", "--reviewer=any")
+	cmd.Dir = repo
+
+	b, err := cmd.CombinedOutput()
+	if err != nil {
+		t.Fatal(err)
+	}
+
+	mrs := strings.Split(string(b), "\n")
+	t.Log(mrs)
+	require.NotContains(t, mrs, "!1 Test MR for lab list")
+	require.Contains(t, mrs, "!329 MR for assign and review commands")
+}
+
+func Test_mrListReviewerNone(t *testing.T) {
+	t.Parallel()
+	repo := copyTestRepo(t)
+	cmd := exec.Command(labBinaryPath, "mr", "list", "--reviewer=none")
+	cmd.Dir = repo
+
+	b, err := cmd.CombinedOutput()
+	if err != nil {
+		t.Fatal(err)
+	}
+
+	mrs := strings.Split(string(b), "\n")
+	t.Log(mrs)
+	require.Contains(t, mrs, "!1 Test MR for lab list")
+	require.NotContains(t, mrs, "!329 MR for assign and review commands")
+}