Revert "Use t.Parallel() in most tests"

This reverts commit a9da3f0.

Author: foxcpp

global_limits_test.go

@@ -11,8 +11,6 @@ import (
 )
 
 func TestGlobalMaxUses(t *testing.T) {
-	t.Parallel()
-
 	conf := filedrop.Default
 	conf.Limits.MaxUses = 2
 	serv := initServ(conf)
@@ -38,8 +36,6 @@ func TestGlobalMaxUses(t *testing.T) {
 }
 
 func TestGlobalMaxFileSize(t *testing.T) {
-	t.Parallel()
-
 	conf := filedrop.Default
 	conf.Limits.MaxFileSize = uint(len(file) - 20)
 	serv := initServ(conf)
@@ -64,8 +60,6 @@ func TestGlobalMaxFileSize(t *testing.T) {
 }
 
 func TestGlobalMaxStoreTime(t *testing.T) {
-	t.Parallel()
-
 	conf := filedrop.Default
 	conf.Limits.MaxStoreSecs = 3
 	serv := initServ(conf)

perfile_limits_test.go

@@ -10,8 +10,6 @@ import (
 )
 
 func TestPerFileMaxUses(t *testing.T) {
-	t.Parallel()
-
 	conf := filedrop.Default
 	conf.Limits.MaxUses = 2
 	serv := initServ(conf)
@@ -51,8 +49,6 @@ func TestPerFileMaxUses(t *testing.T) {
 }
 
 func TestPerFileStoreTime(t *testing.T) {
-	t.Parallel()
-
 	conf := filedrop.Default
 	conf.Limits.MaxStoreSecs = 5
 	serv := initServ(conf)
@@ -101,8 +97,6 @@ func TestPerFileStoreTime(t *testing.T) {
 }
 
 func TestUnboundedFileLimits(t *testing.T) {
-	t.Parallel()
-
 	// Setting no limits in file should allow us to set them to any value in arguments.
 	conf := filedrop.Default
 	serv := initServ(conf)

server_test.go

@@ -17,8 +17,6 @@ import (
 )
 
 func TestBasicSubmit(t *testing.T) {
-	t.Parallel()
-
 	serv := initServ(filedrop.Default)
 	ts := httptest.NewServer(serv)
 	defer cleanServ(serv)
@@ -41,8 +39,6 @@ func TestBasicSubmit(t *testing.T) {
 }
 
 func TestFakeFilename(t *testing.T) {
-	t.Parallel()
-
 	conf := filedrop.Default
 	serv := initServ(conf)
 	ts := httptest.NewServer(serv)
@@ -62,8 +58,6 @@ func TestFakeFilename(t *testing.T) {
 }
 
 func TestNonExistent(t *testing.T) {
-	t.Parallel()
-
 	// Non-existent file should correctly return 404 code.
 	serv := initServ(filedrop.Default)
 	ts := httptest.NewServer(serv)
@@ -97,8 +91,6 @@ func TestNonExistent(t *testing.T) {
 }
 
 func TestContentTypePreserved(t *testing.T) {
-	t.Parallel()
-
 	serv := initServ(filedrop.Default)
 	ts := httptest.NewServer(serv)
 	defer cleanServ(serv)
@@ -134,8 +126,6 @@ func TestContentTypePreserved(t *testing.T) {
 }
 
 func TestNoContentType(t *testing.T) {
-	t.Parallel()
-
 	serv := initServ(filedrop.Default)
 	ts := httptest.NewServer(serv)
 	defer cleanServ(serv)
@@ -166,8 +156,6 @@ func TestNoContentType(t *testing.T) {
 }
 
 func TestHTTPSDownstream(t *testing.T) {
-	t.Parallel()
-
 	serv := initServ(filedrop.Default)
 	ts := httptest.NewServer(serv)
 	defer cleanServ(serv)
@@ -255,8 +243,6 @@ func testWithPrefix(t *testing.T, ts *httptest.Server, c *http.Client, prefix st
 }
 
 func TestPrefixAgnostic(t *testing.T) {
-	t.Parallel()
-
 	// Server should be able to handle requests independently
 	// from full URL.
 	serv := initServ(filedrop.Default)
@@ -271,9 +257,6 @@ func TestPrefixAgnostic(t *testing.T) {
 }
 
 func TestCleanup(t *testing.T) {
-	// FIXME: For some reason this test breaks when parallelized.
-	//t.Parallel()
-
 	conf := filedrop.Default
 	conf.CleanupIntervalSecs = 1
 	serv := initServ(conf)