chore(test): move string unit tests to the dedicated file (#5826)

garden-io · Mar 19, 2024 · 289d220 · 289d220
1 parent d9ef26b
commit 289d220
Show file tree

Hide file tree

Showing 2 changed files with 22 additions and 22 deletions.
diff --git a/core/test/unit/src/util/string.ts b/core/test/unit/src/util/string.ts
@@ -7,7 +7,28 @@
  */
 
 import { expect } from "chai"
-import { tailString, stripQuotes } from "../../../../src/util/string.js"
+import { splitFirst, splitLast, stripQuotes, tailString } from "../../../../src/util/string.js"
+import { describe } from "mocha"
+
+describe("splitFirst", () => {
+  it("should split string on first occurrence of given delimiter", () => {
+    expect(splitFirst("foo:bar:boo", ":")).to.eql(["foo", "bar:boo"])
+  })
+
+  it("should return the whole string as first element when no delimiter is found in string", () => {
+    expect(splitFirst("foo", ":")).to.eql(["foo", ""])
+  })
+})
+
+describe("splitLast", () => {
+  it("should split string on last occurrence of given delimiter", () => {
+    expect(splitLast("foo:bar:boo", ":")).to.eql(["foo:bar", "boo"])
+  })
+
+  it("should return the whole string as last element when no delimiter is found in string", () => {
+    expect(splitLast("foo", ":")).to.eql(["", "foo"])
+  })
+})
 
 describe("tailString", () => {
   it("should return string unchanged if it's shorter than maxLength", () => {

diff --git a/core/test/unit/src/util/util.ts b/core/test/unit/src/util/util.ts
@@ -19,7 +19,6 @@ import {
   isValidDateInstance,
 } from "../../../../src/util/util.js"
 import { expectError } from "../../../helpers.js"
-import { splitLast, splitFirst } from "../../../../src/util/string.js"
 import { getRootLogger } from "../../../../src/logger/logger.js"
 import { dedent } from "../../../../src/util/string.js"
 import { safeDumpYaml } from "../../../../src/util/serialization.js"
@@ -169,26 +168,6 @@ describe("util", () => {
     })
   })
 
-  describe("splitFirst", () => {
-    it("should split string on first occurrence of given delimiter", () => {
-      expect(splitFirst("foo:bar:boo", ":")).to.eql(["foo", "bar:boo"])
-    })
-
-    it("should return the whole string as first element when no delimiter is found in string", () => {
-      expect(splitFirst("foo", ":")).to.eql(["foo", ""])
-    })
-  })
-
-  describe("splitLast", () => {
-    it("should split string on last occurrence of given delimiter", () => {
-      expect(splitLast("foo:bar:boo", ":")).to.eql(["foo:bar", "boo"])
-    })
-
-    it("should return the whole string as last element when no delimiter is found in string", () => {
-      expect(splitLast("foo", ":")).to.eql(["", "foo"])
-    })
-  })
-
   describe("relationshipClasses", () => {
     it("should correctly partition related items", () => {
       const items = ["a", "b", "c", "d", "e", "f", "g", "ab", "bc", "cd", "de", "fg"]