@quenty/binder 14.35.0 → 14.36.0

package/CHANGELOG.md

@@ -3,6 +3,12 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+# [14.36.0](https://github.com/Quenty/NevermoreEngine/compare/@quenty/binder@14.35.0...@quenty/binder@14.36.0) (2026-05-29)
+
+### Bug Fixes
+
+- Binder provider tests are self contained and safer ([9c90efb](https://github.com/Quenty/NevermoreEngine/commit/9c90efbe9972f88fb16f948937b4e0c18f45a531))
+
 # [14.35.0](https://github.com/Quenty/NevermoreEngine/compare/@quenty/binder@14.34.2...@quenty/binder@14.35.0) (2026-05-18)
 
 **Note:** Version bump only for package @quenty/binder

package/deploy.nevermore.json

@@ -0,0 +1,10 @@
+{
+  "targets": {
+    "test": {
+      "universeId": 9716264427,
+      "placeId": 91449389174744,
+      "project": "test/default.project.json",
+      "scriptTemplate": "test/scripts/Server/ServerMain.server.lua"
+    }
+  }
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@quenty/binder",
-  "version": "14.35.0",
+  "version": "14.36.0",
   "description": "Utility object to Bind a class to Roblox object, and associated helper methods",
   "keywords": [
     "Roblox",
@@ -29,10 +29,10 @@
   ],
   "dependencies": {
     "@quenty/baseobject": "10.13.0",
-    "@quenty/brio": "14.29.2",
+    "@quenty/brio": "14.30.0",
     "@quenty/canceltoken": "11.19.0",
-    "@quenty/instanceutils": "13.29.2",
-    "@quenty/linkutils": "13.29.2",
+    "@quenty/instanceutils": "13.30.0",
+    "@quenty/linkutils": "13.30.0",
     "@quenty/loader": "10.11.0",
     "@quenty/maid": "3.9.0",
     "@quenty/nevermore-test-runner": "1.4.0",
@@ -41,11 +41,11 @@
     "@quenty/servicebag": "11.18.0",
     "@quenty/signal": "7.13.0",
     "@quenty/table": "3.9.2",
-    "@quenty/valueobject": "13.30.2",
+    "@quenty/valueobject": "13.31.0",
     "@quentystudios/jest-lua": "3.10.0-quenty.2"
   },
   "publishConfig": {
     "access": "public"
   },
-  "gitHead": "5cba15660c2856169fd0362e1eeeca4c260029c2"
+  "gitHead": "f4a374a0a294ee8900aa5cb68ab138b0acf3e0ae"
 }

package/src/Shared/BinderProvider.lua

@@ -203,12 +203,22 @@ end
 function BinderProvider:Destroy()
 	self._destroyed = true
 
-	for _, item in self._binders do
-		rawset(self, item:GetTag(), nil)
+	local binders = rawget(self, "_binders")
+	rawset(self, "_binders", nil)
+
+	if binders then
+		for _, item in binders do
+			rawset(self, item:GetTag(), nil)
+		end
 	end
 
-	self._maid:DoCleaning()
-	self._binders = nil
+	local maid = rawget(self, "_maid")
+	rawset(self, "_maid", nil)
+
+	if maid then
+		maid:DoCleaning()
+		maid = nil
+	end
 end
 
 return BinderProvider

package/src/Shared/BinderProvider.spec.lua

@@ -16,33 +16,44 @@ local expect = Jest.Globals.expect
 local it = Jest.Globals.it
 
 describe("BinderProvider.new()", function()
-	local provider
-	local initialized = false
+	it("should construct a provider", function()
+		local provider = BinderProvider.new("BinderServiceName", function(_self) end)
+		expect(provider).toEqual(expect.any("table"))
+		provider:Destroy()
+	end)
 
-	it("should execute immediately", function()
-		provider = BinderProvider.new("BinderServiceName", function(self, arg)
+	it("should initialize and call the init callback", function()
+		local initialized = false
+
+		local provider = BinderProvider.new("BinderServiceName", function(_self, arg)
 			initialized = true
 			assert(arg == 12345, "Bad arg")
-
-			self:Add(Binder.new("Test", function()
-				return { Destroy = function() end }
-			end))
 		end)
 
-		expect(provider).toEqual(expect.any("table"))
-	end)
-
-	it("should initialize", function()
 		expect(initialized).toEqual(false)
 		provider:Init(12345)
 		expect(initialized).toEqual(true)
+
+		provider:Destroy()
 	end)
 
-	it("should contain the binder", function()
+	it("should contain the binder after init", function()
+		local binder
+
+		local provider = BinderProvider.new("BinderServiceName", function(self)
+			binder = Binder.new("Test", function()
+				return { Destroy = function() end }
+			end)
+			self:Add(binder)
+		end)
+
+		provider:Init()
+
 		expect(provider.Test).toEqual(expect.any("table"))
-	end)
 
-	if provider then
 		provider:Destroy()
-	end
+		if binder then
+			binder:Destroy()
+		end
+	end)
 end)

package/test/default.project.json

@@ -0,0 +1,25 @@
+{
+  "name": "BinderTest",
+  "globIgnorePaths": [
+    "**/.package-lock.json",
+    "**/.pnpm",
+    "**/.pnpm-workspace-state-v1.json",
+    "**/.modules.yaml",
+    "**/.ignored",
+    "**/.ignored_*"
+  ],
+  "tree": {
+    "$className": "DataModel",
+    "ServerScriptService": {
+      "$properties": {
+        "LoadStringEnabled": true
+      },
+      "binder": {
+        "$path": ".."
+      },
+      "Script": {
+        "$path": "scripts/Server"
+      }
+    }
+  }
+}

package/test/scripts/Server/ServerMain.server.lua

@@ -0,0 +1,13 @@
+--[[
+	@class ServerMain
+]]
+local ServerScriptService = game:GetService("ServerScriptService")
+
+local root = ServerScriptService.binder
+local loader = root:FindFirstChild("LoaderUtils", true).Parent
+local require = require(loader).bootstrapGame(root)
+
+local NevermoreTestRunnerUtils = require("NevermoreTestRunnerUtils")
+if NevermoreTestRunnerUtils.runTestsIfNeededAsync(root) then
+	return
+end