Overhaul database package

ct/go/util/gs_test.go

Index: ct/go/util/gs_test.go
diff --git a/ct/go/util/gs_test.go b/ct/go/util/gs_test.go
index ad73cb292e39692bab0a6f93644315234a78977e..53e49b9e21a8b6f9bf3d744c7d639f913dd01221 100644
--- a/ct/go/util/gs_test.go
+++ b/ct/go/util/gs_test.go
@@ -13,6 +13,7 @@ import (
 )
 
 func TestDownloadWorkerArtifacts(t *testing.T) {
+	t.Skip("WARNING: Skipping this test due to https://code.google.com/p/skia/issues/detail?id=3261")

[borenet, 2014/12/18 19:12:42]

Sorry to do this, but these failures are totally obscuring test output.

[rmistry, 2014/12/18 19:18:13]

Joe, what is the right way to deal with this? we had decided on skipping it if
it is a short test in:
https://codereview.chromium.org/779633003/diff/60001/ct/go/util/gs_test.go#ne...

[borenet, 2014/12/18 19:19:22]

IMO I should still be able to run the tests without -short and expect them to
pass instead of panicking.

 	// Skip if we are running short tests because this test needs a valid
 	// google_storage_token.data file with read write access.
 	if testing.Short() {
@@ -46,6 +47,7 @@ func TestDownloadWorkerArtifacts(t *testing.T) {
 }
 
 func TestUploadWorkerArtifacts(t *testing.T) {
+	t.Skip("WARNING: Skipping this test due to https://code.google.com/p/skia/issues/detail?id=3261")
 	// Skip if we are running short tests because this test needs a valid
 	// google_storage_token.data file with read write access.
 	if testing.Short() {