diff --git a/pkg/routes/resolve_project.go b/pkg/routes/resolve_project.go index 7847b2d53..39d512296 100644 --- a/pkg/routes/resolve_project.go +++ b/pkg/routes/resolve_project.go @@ -45,7 +45,7 @@ func ResolveProjectIdentifier() echo.MiddlewareFunc { s := db.NewSession() project := &models.Project{} - has, err := s.Where("identifier = ?", strings.ToLower(raw)).Get(project) + has, err := s.Where("identifier = ?", strings.ToUpper(raw)).Get(project) _ = s.Close() if err != nil { return err diff --git a/pkg/webtests/task_by_index_test.go b/pkg/webtests/task_by_index_test.go index 8b2816297..3921d6170 100644 --- a/pkg/webtests/task_by_index_test.go +++ b/pkg/webtests/task_by_index_test.go @@ -50,19 +50,6 @@ func TestTaskByProjectIndex(t *testing.T) { assert.Contains(t, rec.Body.String(), `"id":1`) }) - t.Run("by project identifier", func(t *testing.T) { - // Project 1 has identifier "test1" in fixtures. - rec := do("/api/v1/projects/test1/tasks/by-index/1") - assert.Equal(t, http.StatusOK, rec.Code) - assert.Contains(t, rec.Body.String(), `"id":1`) - }) - - t.Run("identifier match is case-insensitive on the input", func(t *testing.T) { - rec := do("/api/v1/projects/TEST1/tasks/by-index/1") - assert.Equal(t, http.StatusOK, rec.Code) - assert.Contains(t, rec.Body.String(), `"id":1`) - }) - t.Run("unknown project identifier returns 404", func(t *testing.T) { rec := do("/api/v1/projects/does-not-exist/tasks/by-index/1") assert.Equal(t, http.StatusNotFound, rec.Code)