Merge pull request #9 from wajox/feature/improve-test-coverage

Feature/improve test coverage
wajox · Jul 26, 2022 · c7f0bf2 · c7f0bf2
2 parents a4aeee6 + a638874
commit c7f0bf2
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 0 deletions.
diff --git a/README.md b/README.md
@@ -1,5 +1,6 @@
 # Mongol
 
+[![License: MIT](https://img.shields.io/badge/License-MIT-yellow.svg)](https://opensource.org/licenses/MIT)
 [![Go Reference](https://pkg.go.dev/badge/github.com/wajox/mongol.svg)](https://pkg.go.dev/github.com/wajox/mongol)
 [![codecov](https://codecov.io/gh/wajox/mongol/branch/master/graph/badge.svg?token=MFEF13319U)](https://codecov.io/gh/wajox/mongol)
 ![Go workflow](https://github.com/wajox/mongol/actions/workflows/go.yml/badge.svg)

diff --git a/base_collection_test.go b/base_collection_test.go
@@ -308,6 +308,23 @@ var _ = Describe("BaseCollection", func() {
 						})
 					})
 
+					Context("with wrong record ID", func() {
+						It("should not update the model", func() {
+
+							curTime := time.Now().UTC().Add(time.Hour * 1)
+
+							timecop.Freeze(curTime)
+							defer timecop.Return()
+
+							newTitle := "New title " + uuid.New().String()
+							m.Title = newTitle
+
+							_, updateErr := storage.ReplaceOneByID(context.TODO(), "123", m)
+
+							Expect(updateErr).To(Equal(ErrInvalidObjectID))
+						})
+					})
+
 					It("should update the model", func() {
 						emptyModel := &ExampleModel{}