Adding scope list check and test. (#574)

internal/servers/controller/handlers/scopes/scope_service.go

@@ -478,6 +478,9 @@ func validateDeleteRequest(req *pbs.DeleteScopeRequest) error {
 
 func validateListRequest(req *pbs.ListScopesRequest) error {
 	badFields := map[string]string{}
+	if req.GetScopeId() != scope.Global.String() && !handlers.ValidId(scope.Org.Prefix(), req.GetScopeId()) {
+		badFields["scope_id"] = "Must be 'global' or a valid org scope id when listing."
+	}
 	if len(badFields) > 0 {
 		return handlers.InvalidArgumentErrorf("Error in provided request.", badFields)
 	}

internal/servers/controller/handlers/scopes/scope_service_test.go

@@ -198,6 +198,12 @@ func TestList(t *testing.T) {
 			req:     &pbs.ListScopesRequest{ScopeId: oNoProjects.GetPublicId()},
 			res:     &pbs.ListScopesResponse{},
 		},
+		{
+			name:    "Cant List Project Scopes",
+			scopeId: p1.GetPublicId(),
+			req:     &pbs.ListScopesRequest{ScopeId: p1.GetPublicId()},
+			err: handlers.ApiErrorWithCode(codes.InvalidArgument),
+		},
 	}
 	for _, tc := range cases {
 		t.Run(tc.name, func(t *testing.T) {