Eliminate unnecessary code in GoCloud test stubs

See merge request gitlab-org/gitlab-workhorse!590

This should eliminate a race condition that only occurred in the test
where multiple Goroutines attempt to write to the same value when
opening a bucket.

[ci skip]

Remove an in-memory buffer for LSIF transformation

Closes #284

See merge request gitlab-org/gitlab-workhorse!586

Support either local or remote upload but not both NO CHANGELOG

See merge request gitlab-org/gitlab-workhorse!574

Verify changelog file presence in CI NO CHANGELOG

Closes #273

See merge request gitlab-org/gitlab-workhorse!580

Fix race condition in httprs test

See merge request gitlab-org/gitlab-workhorse!584

Silence errors when Azure objects have aleady been deleted

See merge request gitlab-org/gitlab-workhorse!585

Fix objectstore.uploader.uploadError race

See merge request gitlab-org/gitlab-workhorse!583

In an normal upload flow, GitLab Rails will move a file to its final
destination and remove the temporary file. Workhorse issues a DELETE a
request to ensure this is cleaned up. Previously this was generating log
noise with every upload. Now only log an error message if there were
some error other than a 404.

The `So(server, ShouldNotBeNil)` call in the httprs test attempts to
print a string representation of the `server`, but this is racy because
the reconnection state is initialized in a
Goroutine. `httptest.NewServer()` always returns a valid value, so we
can drop this test.

Relates to https://gitlab.com/gitlab-org/gitlab-workhorse/-/issues/285

Tests: block until grpc server has stopped

Closes #281

See merge request gitlab-org/gitlab-workhorse!576

[ci skip]

Add additional logging for dynamic img resizing

See merge request gitlab-org/gitlab-workhorse!562

Fix semantical merge conflict (double import)

Closes #282

See merge request gitlab-org/gitlab-workhorse!577

Remove ProcessReferences flag

See merge request gitlab-org/gitlab-workhorse!543

Use testify/require in top level tests

See merge request gitlab-org/gitlab-workhorse!570

RAM usage of parsing LSIF references are now back to normalcy and
feature works.

This removes the code in workhorse for determining whether parsing
references is enabled or not. It'll now always be enabled.

Make more use of testify/require

See merge request gitlab-org/gitlab-workhorse!573

Use switch statement instead of long if-else

See merge request gitlab-org/gitlab-workhorse!572

Stop using require.New and assert.New

See merge request gitlab-org/gitlab-workhorse!567

Use more require in internal/{upload,upstream}

See merge request gitlab-org/gitlab-workhorse!571

Tests: consistent helper function names

See merge request gitlab-org/gitlab-workhorse!566