Switched Tests to use Jest : Purge TestEz -> Anoint Jest (#64)

Author: RoFlection Bot

bin/spec.lua

@@ -1,23 +1,16 @@
 local Root = game:GetService("ReplicatedStorage")
-
 local Packages = Root.Packages
 
-local JestGlobals = require(Packages.JestGlobals)
+local runCLI = require(Packages.JestCore).runCLI
 
-local TestEZ = JestGlobals.TestEZ
+local status, result = runCLI(Root, {
+	verbose = _G.verbose == "true",
+	ci = _G.CI == "true",
+	updateSnapshot = _G.UPDATESNAPSHOT == "true",
+}, { Packages.DomTestingLibrary }):awaitStatus()
 
--- Run all tests, collect results, and report to stdout.
-TestEZ.TestBootstrap:run(
-	{ Packages.DomTestingLibrary },
-	TestEZ.Reporters.pipe({
-		TestEZ.Reporters.JestDefaultReporter,
-		TestEZ.Reporters.JestSummaryReporter,
-	}),
-	{
-		extraEnvironment = JestGlobals.testEnv,
-	}
-)
--- ROBLOX TODO: after converting jest-runner this should be included there
-JestGlobals.runtime:teardown()
+if status == "Rejected" then
+	print(result)
+end
 
 return nil

foreman.toml

@@ -1,4 +1,4 @@
 [tools]
 rotrieve = { source = "roblox/rotriever", version = "0.5.4" }
-selene = { source = "Kampfkarren/selene", version = "0.18.2" }
+selene = { source = "Kampfkarren/selene", version = "0.20.0" }
 stylua = { source = "JohnnyMorganz/StyLua", version = "=0.13.1" }

rotriever.toml

@@ -11,7 +11,8 @@ LuauPolyfill = "github.com/roblox/luau-polyfill@0.3.4"
 LuauRegExp = "github.com/Roblox/luau-regexp@0.2.0"
 Chalk = "github.com/roblox/chalk-lua@0.1.3"
 Promise = "github.com/evaera/roblox-lua-promise@4.0.0"
+JestCore = { git = "https://github.com/roblox/jest-roblox.git", rev="master", package="JestCore" }
 JestGetType = { git = "https://github.com/roblox/jest-roblox.git", rev="master", package="JestGetType"}
 PrettyFormat = { git = "https://github.com/roblox/jest-roblox.git", rev="master", package="PrettyFormat"}
 RobloxShared = { git = "https://github.com/roblox/jest-roblox.git", rev="master", package="RobloxShared"}
-JestGlobals = { git = "https://github.com/roblox/jest-roblox.git", rev="master" }
+JestGlobals = { git = "https://github.com/roblox/jest-roblox.git", rev="master" }

selene.toml

@@ -1,4 +1,4 @@
-std = "roblox+testez"
+std = "roblox"
 
 [config]
 empty_if = { comments_count = true }

src/__tests__/config.spec.lua

@@ -1,57 +1,62 @@
 -- ROBLOX upstream: https://github.com/testing-library/dom-testing-library/blob/v8.14.0/src/__tests__/config.js
-return function()
-	local Packages = script.Parent.Parent.Parent
 
-	local JestGlobals = require(Packages.JestGlobals)
-	local jestExpect = JestGlobals.expect
+local Packages = script.Parent.Parent.Parent
 
-	local configModule = require(script.Parent.Parent.config)
-	local configure = configModule.configure
-	local getConfig = configModule.getConfig
+local JestGlobals = require(Packages.JestGlobals)
+local expect = JestGlobals.expect
+local test = JestGlobals.test
+local describe = JestGlobals.describe
+local beforeEach = JestGlobals.beforeEach
+local afterEach = JestGlobals.afterEach
 
-	describe("configuration API", function()
-		local originalConfig
-		beforeEach(function()
-			-- Grab the existing configuration so we can restore
-			-- it at the end of the test
-			configure(function(existingConfig)
-				originalConfig = existingConfig -- Don't change the existing config
-				return {}
-			end)
+local configModule = require(script.Parent.Parent.config)
+local configure = configModule.configure
+local getConfig = configModule.getConfig
+
+describe("configuration API", function()
+	local originalConfig
+	beforeEach(function()
+		-- Grab the existing configuration so we can restore
+		-- it at the end of the test
+		configure(function(existingConfig)
+			originalConfig = existingConfig -- Don't change the existing config
+			return {}
 		end)
-		afterEach(function()
-			configure(originalConfig)
+	end)
+	afterEach(function()
+		configure(originalConfig)
+	end)
+	beforeEach(function()
+		configure({ other = 123 })
+	end)
+	describe("getConfig", function()
+		test("returns existing configuration", function()
+			local conf = getConfig()
+			expect(conf.testIdAttribute).toEqual("data-testid")
 		end)
-		beforeEach(function()
-			configure({ other = 123 })
+	end)
+	describe("configure", function()
+		test("merges a delta rather than replacing the whole config", function()
+			local conf = getConfig()
+			expect(conf).toMatchObject({ testIdAttribute = "data-testid" })
 		end)
-		describe("getConfig", function()
-			it("returns existing configuration", function()
-				local conf = getConfig()
-				jestExpect(conf.testIdAttribute).toEqual("data-testid")
-			end)
+		test("overrides existing values", function()
+			configure({ testIdAttribute = "new-id" })
+			local conf = getConfig()
+			expect(conf.testIdAttribute).toEqual("new-id")
 		end)
-		describe("configure", function()
-			it("merges a delta rather than replacing the whole config", function()
-				local conf = getConfig()
-				jestExpect(conf).toMatchObject({ testIdAttribute = "data-testid" })
-			end)
-			it("overrides existing values", function()
-				configure({ testIdAttribute = "new-id" })
-				local conf = getConfig()
-				jestExpect(conf.testIdAttribute).toEqual("new-id")
-			end)
-			it("passes existing config out to config function", function()
-				-- Create a new config key based on the value of an existing one
-				configure(function(existingConfig)
-					return {
-						testIdAttribute = ("%s-derived"):format(tostring(existingConfig.testIdAttribute)),
-					}
-				end)
-				local conf = getConfig() -- The new value should be there, and existing values should be
-				-- untouched
-				jestExpect(conf).toMatchObject({ testIdAttribute = "data-testid-derived" })
+		test("passes existing config out to config function", function()
+			-- Create a new config key based on the value of an existing one
+			configure(function(existingConfig)
+				return {
+					testIdAttribute = ("%s-derived"):format(tostring(existingConfig.testIdAttribute)),
+				}
 			end)
+			local conf = getConfig() -- The new value should be there, and existing values should be
+			-- untouched
+			expect(conf).toMatchObject({ testIdAttribute = "data-testid-derived" })
 		end)
 	end)
-end
+end)
+
+return {}