diff options
| author | Antoine GIRARD <sapk@users.noreply.github.com> | 2017-11-29 00:35:23 +0100 |
|---|---|---|
| committer | Lauris BH <lauris@nix.lv> | 2017-11-29 01:35:23 +0200 |
| commit | 4035ab05fa2d2c8ec95d346fea91cab8211dab17 (patch) | |
| tree | 623d09ee6ab41eba7307cb3be7ce176458455dab /integrations | |
| parent | c80d147fa911c2aa711bc3f310d7bef11681230b (diff) | |
| download | gitea-4035ab05fa2d2c8ec95d346fea91cab8211dab17.tar.gz gitea-4035ab05fa2d2c8ec95d346fea91cab8211dab17.zip | |
Fix git lfs path (#3016)
* add suffix .git
* Remove Content-Type on GET methods
* Add migrations to force creation before any further potential migration
Diffstat (limited to 'integrations')
| -rw-r--r-- | integrations/api_repo_lfs_locks_test.go | 23 |
1 files changed, 10 insertions, 13 deletions
diff --git a/integrations/api_repo_lfs_locks_test.go b/integrations/api_repo_lfs_locks_test.go index 9fd796b6c5..11b017e87a 100644 --- a/integrations/api_repo_lfs_locks_test.go +++ b/integrations/api_repo_lfs_locks_test.go @@ -23,13 +23,13 @@ func TestAPILFSLocksNotStarted(t *testing.T) { user := models.AssertExistsAndLoadBean(t, &models.User{ID: 2}).(*models.User) repo := models.AssertExistsAndLoadBean(t, &models.Repository{ID: 1}).(*models.Repository) - req := NewRequestf(t, "GET", "/%s/%s/info/lfs/locks", user.Name, repo.Name) + req := NewRequestf(t, "GET", "/%s/%s.git/info/lfs/locks", user.Name, repo.Name) MakeRequest(t, req, http.StatusNotFound) - req = NewRequestf(t, "POST", "/%s/%s/info/lfs/locks", user.Name, repo.Name) + req = NewRequestf(t, "POST", "/%s/%s.git/info/lfs/locks", user.Name, repo.Name) MakeRequest(t, req, http.StatusNotFound) - req = NewRequestf(t, "GET", "/%s/%s/info/lfs/locks/verify", user.Name, repo.Name) + req = NewRequestf(t, "GET", "/%s/%s.git/info/lfs/locks/verify", user.Name, repo.Name) MakeRequest(t, req, http.StatusNotFound) - req = NewRequestf(t, "GET", "/%s/%s/info/lfs/locks/10/unlock", user.Name, repo.Name) + req = NewRequestf(t, "GET", "/%s/%s.git/info/lfs/locks/10/unlock", user.Name, repo.Name) MakeRequest(t, req, http.StatusNotFound) } @@ -39,9 +39,8 @@ func TestAPILFSLocksNotLogin(t *testing.T) { user := models.AssertExistsAndLoadBean(t, &models.User{ID: 2}).(*models.User) repo := models.AssertExistsAndLoadBean(t, &models.Repository{ID: 1}).(*models.Repository) - req := NewRequestf(t, "GET", "/%s/%s/info/lfs/locks", user.Name, repo.Name) + req := NewRequestf(t, "GET", "/%s/%s.git/info/lfs/locks", user.Name, repo.Name) req.Header.Set("Accept", "application/vnd.git-lfs+json") - req.Header.Set("Content-Type", "application/vnd.git-lfs+json") resp := MakeRequest(t, req, http.StatusForbidden) var lfsLockError api.LFSLockError DecodeJSON(t, resp, &lfsLockError) @@ -102,7 +101,7 @@ func TestAPILFSLocksLogged(t *testing.T) { //create locks for _, test := range tests { session := loginUser(t, test.user.Name) - req := NewRequestWithJSON(t, "POST", fmt.Sprintf("/%s/info/lfs/locks", test.repo.FullName()), map[string]string{"path": test.path}) + req := NewRequestWithJSON(t, "POST", fmt.Sprintf("/%s.git/info/lfs/locks", test.repo.FullName()), map[string]string{"path": test.path}) req.Header.Set("Accept", "application/vnd.git-lfs+json") req.Header.Set("Content-Type", "application/vnd.git-lfs+json") session.MakeRequest(t, req, test.httpResult) @@ -116,9 +115,8 @@ func TestAPILFSLocksLogged(t *testing.T) { //check creation for _, test := range resultsTests { session := loginUser(t, test.user.Name) - req := NewRequestf(t, "GET", "/%s/info/lfs/locks", test.repo.FullName()) + req := NewRequestf(t, "GET", "/%s.git/info/lfs/locks", test.repo.FullName()) req.Header.Set("Accept", "application/vnd.git-lfs+json") - req.Header.Set("Content-Type", "application/vnd.git-lfs+json") resp := session.MakeRequest(t, req, http.StatusOK) var lfsLocks api.LFSLockList DecodeJSON(t, resp, &lfsLocks) @@ -128,7 +126,7 @@ func TestAPILFSLocksLogged(t *testing.T) { assert.WithinDuration(t, test.locksTimes[i], lock.LockedAt, 1*time.Second) } - req = NewRequestWithJSON(t, "POST", fmt.Sprintf("/%s/info/lfs/locks/verify", test.repo.FullName()), map[string]string{}) + req = NewRequestWithJSON(t, "POST", fmt.Sprintf("/%s.git/info/lfs/locks/verify", test.repo.FullName()), map[string]string{}) req.Header.Set("Accept", "application/vnd.git-lfs+json") req.Header.Set("Content-Type", "application/vnd.git-lfs+json") resp = session.MakeRequest(t, req, http.StatusOK) @@ -152,7 +150,7 @@ func TestAPILFSLocksLogged(t *testing.T) { //remove all locks for _, test := range deleteTests { session := loginUser(t, test.user.Name) - req := NewRequestWithJSON(t, "POST", fmt.Sprintf("/%s/info/lfs/locks/%s/unlock", test.repo.FullName(), test.lockID), map[string]string{}) + req := NewRequestWithJSON(t, "POST", fmt.Sprintf("/%s.git/info/lfs/locks/%s/unlock", test.repo.FullName(), test.lockID), map[string]string{}) req.Header.Set("Accept", "application/vnd.git-lfs+json") req.Header.Set("Content-Type", "application/vnd.git-lfs+json") resp := session.MakeRequest(t, req, http.StatusOK) @@ -165,9 +163,8 @@ func TestAPILFSLocksLogged(t *testing.T) { // check that we don't have any lock for _, test := range resultsTests { session := loginUser(t, test.user.Name) - req := NewRequestf(t, "GET", "/%s/info/lfs/locks", test.repo.FullName()) + req := NewRequestf(t, "GET", "/%s.git/info/lfs/locks", test.repo.FullName()) req.Header.Set("Accept", "application/vnd.git-lfs+json") - req.Header.Set("Content-Type", "application/vnd.git-lfs+json") resp := session.MakeRequest(t, req, http.StatusOK) var lfsLocks api.LFSLockList DecodeJSON(t, resp, &lfsLocks) |