move package to root and move tests to _test package

victorguarana · May 3, 2024 · 477ce2a · 477ce2a
1 parent 8adc9dd
commit 477ce2a
Show file tree

Hide file tree

Showing 10 changed files with 187 additions and 198 deletions.
diff --git a/gomongo/collection.go → collection.go b/gomongo/collection.go → collection.go
diff --git a/gomongo/collection_test.go → collection_test.go b/gomongo/collection_test.go → collection_test.go
diff --git a/gomongo/connection.go → connection.go b/gomongo/connection.go → connection.go
diff --git a/gomongo/database.go → database.go b/gomongo/database.go → database.go
diff --git a/database_test.go b/database_test.go
@@ -0,0 +1,101 @@
+package gomongo_test
+
+import (
+	"context"
+	"time"
+
+	"github.com/testcontainers/testcontainers-go/modules/mongodb"
+	"github.com/victorguarana/gomongo"
+
+	. "github.com/onsi/ginkgo/v2"
+	. "github.com/onsi/gomega"
+)
+
+var _ = Describe("NewDatabase", Ordered, func() {
+	var (
+		mongodbContainer    *mongodb.MongoDBContainer
+		mongodbContainerURI string
+		connectionSettings  gomongo.ConnectionSettings
+	)
+
+	BeforeAll(func() {
+		mongodbContainer, mongodbContainerURI = runMongoContainer(context.Background())
+	})
+
+	Context("when connection settings param is invalid", func() {
+		Context("when URI is empty", func() {
+			It("returns ErrInvalidSettings", func() {
+				connectionSettings := gomongo.ConnectionSettings{
+					DatabaseName: "test",
+				}
+
+				receivedDatabase, receivedErr := gomongo.NewDatabase(context.Background(), connectionSettings)
+				Expect(receivedErr).To(MatchError(gomongo.ErrInvalidSettings))
+				Expect(receivedErr).To(MatchError(ContainSubstring("URI can not be empty")))
+				Expect(receivedDatabase).To(Equal(gomongo.Database{}))
+			})
+		})
+
+		Context("when DatabaseName is empty", func() {
+			It("returns ErrInvalidSettings", func() {
+				connectionSettings := gomongo.ConnectionSettings{
+					URI: mongodbContainerURI,
+				}
+
+				receivedDatabase, receivedErr := gomongo.NewDatabase(context.Background(), connectionSettings)
+				Expect(receivedErr).To(MatchError(gomongo.ErrInvalidSettings))
+				Expect(receivedErr).To(MatchError(ContainSubstring("Database Name can not be empty")))
+				Expect(receivedDatabase).To(Equal(gomongo.Database{}))
+			})
+		})
+	})
+
+	Context("when connection settings param is valid", func() {
+		Context("when mongo is up", func() {
+			Context("when all settings are filled", func() {
+				It("returns nil", func() {
+					connectionSettings := gomongo.ConnectionSettings{
+						URI:               mongodbContainerURI,
+						DatabaseName:      "test",
+						ConnectionTimeout: 10 * time.Second,
+					}
+
+					receivedDatabase, receivedErr := gomongo.NewDatabase(context.Background(), connectionSettings)
+					Expect(receivedErr).NotTo(HaveOccurred())
+					Expect(receivedDatabase).NotTo(Equal(gomongo.Database{}))
+				})
+			})
+
+			Context("when timeout is empty", func() {
+				It("returns nil", func() {
+					connectionSettings := gomongo.ConnectionSettings{
+						URI:          mongodbContainerURI,
+						DatabaseName: "test",
+					}
+
+					receivedDatabase, receivedErr := gomongo.NewDatabase(context.Background(), connectionSettings)
+					Expect(receivedErr).NotTo(HaveOccurred())
+					Expect(receivedDatabase).NotTo(Equal(gomongo.Database{}))
+				})
+			})
+		})
+
+		Context("when mongo is down", func() {
+			BeforeAll(func() {
+				terminateMongoContainer(mongodbContainer, context.Background())
+			})
+
+			It("returns error", func() {
+				connectionSettings = gomongo.ConnectionSettings{
+					URI:               mongodbContainerURI,
+					DatabaseName:      "test",
+					ConnectionTimeout: 1 * time.Second,
+				}
+
+				receivedDatabase, receivedErr := gomongo.NewDatabase(context.Background(), connectionSettings)
+				Expect(receivedErr).To(MatchError(gomongo.ErrGomongoCanNotConnect))
+				Expect(receivedDatabase).To(Equal(gomongo.Database{}))
+			})
+		})
+	})
+})
diff --git a/gomongo/connection_test.go b/gomongo/connection_test.go
diff --git a/gomongo/database_test.go b/gomongo/database_test.go
diff --git a/gomongo/gomongo_suite_test.go → gomongo_suite_test.go b/gomongo/gomongo_suite_test.go → gomongo_suite_test.go
@@ -1,4 +1,4 @@
-package gomongo
+package gomongo_test
 
 import (
 	"testing"

diff --git a/gomongo/mongodb_operations.go → mongodb_operations.go b/gomongo/mongodb_operations.go → mongodb_operations.go
diff --git a/gomongo/testcontainer_helper_test.go → testcontainer_helper_test.go b/gomongo/testcontainer_helper_test.go → testcontainer_helper_test.go
@@ -1,4 +1,4 @@
-package gomongo
+package gomongo_test
 
 import (
 	"context"