chore: Improve UX by providing more information

api/http/handlerfuncs.go

@@ -36,7 +36,8 @@ func rootHandler(rw http.ResponseWriter, req *http.Request) {
 		req.Context(),
 		rw,
 		simpleDataResponse(
-			"response", "Welcome to the DefraDB HTTP API. Use /graphql to send queries to the database",
+			"response", "Welcome to the DefraDB HTTP API. Use /graphql to send queries to the database."+
+				" Read the documentation at https://docs.source.network/.",
 		),
 		http.StatusOK,
 	)

api/http/handlerfuncs_test.go

@@ -67,7 +67,7 @@ func TestRootHandler(t *testing.T) {
 	})
 	switch v := resp.Data.(type) {
 	case map[string]any:
-		assert.Equal(t, "Welcome to the DefraDB HTTP API. Use /graphql to send queries to the database", v["response"])
+		assert.Equal(t, "Welcome to the DefraDB HTTP API. Use /graphql to send queries to the database. Read the documentation at https://docs.source.network/.", v["response"])
 	default:
 		t.Fatalf("data should be of type map[string]any but got %T", resp.Data)
 	}

api/http/logger.go

@@ -72,11 +72,11 @@ func loggerMiddleware(next http.Handler) http.Handler {
 				lrw.statusCode,
 			),
 			logging.NewKV(
-				"Length",
+				"LengthBytes",
 				lrw.contentLength,
 			),
 			logging.NewKV(
-				"Elapsed",
+				"ElapsedSeconds",
 				elapsed,
 			),
 		)

api/http/logger_test.go

@@ -100,7 +100,7 @@ func TestLoggerKeyValueOutput(t *testing.T) {
 	assert.Equal(t, "GET", kv["Method"])
 	assert.Equal(t, "/ping", kv["Path"])
 	assert.Equal(t, float64(200), kv["Status"])
-	assert.Equal(t, float64(28), kv["Length"])
+	assert.Equal(t, float64(28), kv["LengthBytes"])
 }
 
 func readLog(path string) (map[string]any, error) {

db/collection.go

@@ -32,7 +32,7 @@ import (
 )
 
 var (
-	ErrDocumentAlreadyExists = errors.New("A document with the given key already exists")
+	ErrDocumentAlreadyExists = errors.New("A document with the given dockey already exists")
 	ErrUnknownCRDTArgument   = errors.New("Invalid CRDT arguments")
 	ErrUnknownCRDT           = errors.New("")
 )

tests/integration/mutation/simple/create/simple_test.go

@@ -57,7 +57,7 @@ func TestMutationCreateSimpleDoesNotCreateDocGivenDuplicate(t *testing.T) {
 				}`,
 			},
 		},
-		ExpectedError: "A document with the given key already exists",
+		ExpectedError: "A document with the given dockey already exists",
 	}
 
 	simpleTests.ExecuteTestCase(t, test)