[chore] The Big Middleware and API Refactor (tm) (#1250)

* interim commit: start refactoring middlewares into package under router

* another interim commit, this is becoming a big job

* another fucking massive interim commit

* refactor bookmarks to new style

* ambassador, wiz zeze commits you are spoiling uz

* she compiles, we're getting there

* we're just normal men; we're just innocent men

* apiutil

* whoopsie

* i'm glad noone reads commit msgs haha :blob_sweat:

* use that weirdo go-bytesize library for maxMultipartMemory

* fix media module paths

1 files changed, 0 insertions, 123 deletions

diff --git a/internal/api/client/account/statuses_test.go b/internal/api/client/account/statuses_test.go
deleted file mode 100644
index 1f935896c..000000000
--- a/internal/api/client/account/statuses_test.go
+++ /dev/null
@@ -1,123 +0,0 @@
-/*
-   GoToSocial
-   Copyright (C) 2021-2022 GoToSocial Authors admin@gotosocial.org
-
-   This program is free software: you can redistribute it and/or modify
-   it under the terms of the GNU Affero General Public License as published by
-   the Free Software Foundation, either version 3 of the License, or
-   (at your option) any later version.
-
-   This program is distributed in the hope that it will be useful,
-   but WITHOUT ANY WARRANTY; without even the implied warranty of
-   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-   GNU Affero General Public License for more details.
-
-   You should have received a copy of the GNU Affero General Public License
-   along with this program.  If not, see <http://www.gnu.org/licenses/>.
-*/
-
-package account_test
-
-import (
-	"encoding/json"
-	"fmt"
-	"io/ioutil"
-	"net/http"
-	"net/http/httptest"
-	"testing"
-
-	"github.com/gin-gonic/gin"
-	"github.com/stretchr/testify/assert"
-	"github.com/stretchr/testify/suite"
-	"github.com/superseriousbusiness/gotosocial/internal/api/client/account"
-	apimodel "github.com/superseriousbusiness/gotosocial/internal/api/model"
-)
-
-type AccountStatusesTestSuite struct {
-	AccountStandardTestSuite
-}
-
-func (suite *AccountStatusesTestSuite) TestGetStatusesPublicOnly() {
-	// set up the request
-	// we're getting statuses of admin
-	targetAccount := suite.testAccounts["admin_account"]
-	recorder := httptest.NewRecorder()
-	ctx := suite.newContext(recorder, http.MethodGet, nil, fmt.Sprintf("/api/v1/accounts/%s/statuses?limit=20&only_media=false&only_public=true", targetAccount.ID), "")
-	ctx.Params = gin.Params{
-		gin.Param{
-			Key:   account.IDKey,
-			Value: targetAccount.ID,
-		},
-	}
-
-	// call the handler
-	suite.accountModule.AccountStatusesGETHandler(ctx)
-
-	// 1. we should have OK because our request was valid
-	suite.Equal(http.StatusOK, recorder.Code)
-
-	// 2. we should have no error message in the result body
-	result := recorder.Result()
-	defer result.Body.Close()
-
-	// check the response
-	b, err := ioutil.ReadAll(result.Body)
-	assert.NoError(suite.T(), err)
-
-	// unmarshal the returned statuses
-	apimodelStatuses := []*apimodel.Status{}
-	err = json.Unmarshal(b, &apimodelStatuses)
-	suite.NoError(err)
-	suite.NotEmpty(apimodelStatuses)
-
-	for _, s := range apimodelStatuses {
-		suite.Equal(apimodel.VisibilityPublic, s.Visibility)
-	}
-
-	suite.Equal(`<http://localhost:8080/api/v1/accounts/01F8MH17FWEB39HZJ76B6VXSKF/statuses?limit=20&max_id=01F8MH75CBF9JFX4ZAD54N0W0R&exclude_replies=false&exclude_reblogs=false&pinned_only=false&only_media=false&only_public=true>; rel="next", <http://localhost:8080/api/v1/accounts/01F8MH17FWEB39HZJ76B6VXSKF/statuses?limit=20&min_id=01G36SF3V6Y6V5BF9P4R7PQG7G&exclude_replies=false&exclude_reblogs=false&pinned_only=false&only_media=false&only_public=true>; rel="prev"`, result.Header.Get("link"))
-}
-
-func (suite *AccountStatusesTestSuite) TestGetStatusesPublicOnlyMediaOnly() {
-	// set up the request
-	// we're getting statuses of admin
-	targetAccount := suite.testAccounts["admin_account"]
-	recorder := httptest.NewRecorder()
-	ctx := suite.newContext(recorder, http.MethodGet, nil, fmt.Sprintf("/api/v1/accounts/%s/statuses?limit=20&only_media=true&only_public=true", targetAccount.ID), "")
-	ctx.Params = gin.Params{
-		gin.Param{
-			Key:   account.IDKey,
-			Value: targetAccount.ID,
-		},
-	}
-
-	// call the handler
-	suite.accountModule.AccountStatusesGETHandler(ctx)
-
-	// 1. we should have OK because our request was valid
-	suite.Equal(http.StatusOK, recorder.Code)
-
-	// 2. we should have no error message in the result body
-	result := recorder.Result()
-	defer result.Body.Close()
-
-	// check the response
-	b, err := ioutil.ReadAll(result.Body)
-	assert.NoError(suite.T(), err)
-
-	// unmarshal the returned statuses
-	apimodelStatuses := []*apimodel.Status{}
-	err = json.Unmarshal(b, &apimodelStatuses)
-	suite.NoError(err)
-	suite.NotEmpty(apimodelStatuses)
-
-	for _, s := range apimodelStatuses {
-		suite.NotEmpty(s.MediaAttachments)
-		suite.Equal(apimodel.VisibilityPublic, s.Visibility)
-	}
-
-	suite.Equal(`<http://localhost:8080/api/v1/accounts/01F8MH17FWEB39HZJ76B6VXSKF/statuses?limit=20&max_id=01F8MH75CBF9JFX4ZAD54N0W0R&exclude_replies=false&exclude_reblogs=false&pinned_only=false&only_media=true&only_public=true>; rel="next", <http://localhost:8080/api/v1/accounts/01F8MH17FWEB39HZJ76B6VXSKF/statuses?limit=20&min_id=01F8MH75CBF9JFX4ZAD54N0W0R&exclude_replies=false&exclude_reblogs=false&pinned_only=false&only_media=true&only_public=true>; rel="prev"`, result.Header.Get("link"))
-}
-
-func TestAccountStatusesTestSuite(t *testing.T) {
-	suite.Run(t, new(AccountStatusesTestSuite))
-}