Module:UnitTests - Revision history

Gonnym: add

tags to tests to allow the ToC to appear

2025-03-03T11:39:52Z

add <h2> tags to tests to allow the ToC to appear

← Previous revision		Revision as of 11:39, 3 March 2025
Line 317:		Line 317:
	-- Add results to the results table.		-- Add results to the results table.
	for _, value in ipairs(self_sorted) do		for _, value in ipairs(self_sorted) do
			result_table:insert_format("<h2>%s</h2>\n", value)
	result_table:insert_format(table_header .. "\n\|-\n", value)		result_table:insert_format(table_header .. "\n\|-\n", value)
	self[value](self)		self[value](self)

Aidan9382: expected_actual should be based off of the expected, not the actual. This was basically seeing if actual == actual (which is always gonna be true). I've also adjusted the variable name to be more accurate

2023-05-25T17:03:26Z

expected_actual should be based off of the expected, not the actual. This was basically seeing if actual == actual (which is always gonna be true). I've also adjusted the variable name to be more accurate

Pppery: Undid revision 1156849067 by BrandonXLF (talk) Seems to cause all tests, even those that should fail, to pass

2023-05-25T13:02:34Z

Undid revision 1156849067 by BrandonXLF (talk) Seems to cause all tests, even those that should fail, to pass

BrandonXLF: Replace all strip markers

2023-05-24T23:10:10Z

Replace all strip markers

Aidan9382: force first_difference to turn inputs into string for :equals(_deep)

2023-03-02T13:44:37Z

force first_difference to turn inputs into string for :equals(_deep)

← Previous revision		Revision as of 13:44, 2 March 2023
Line 32:		Line 32:

	local function first_difference(s1, s2)		local function first_difference(s1, s2)
			s1, s2 = tostring(s1), tostring(s2)
	if s1 == s2 then return '' end		if s1 == s2 then return '' end
	local max = math.min(#s1, #s2)		local max = math.min(#s1, #s2)

Aidan9382: re-format val_to_str's structure to avoid awkward definition ordering of functions

2023-02-08T20:05:21Z

re-format val_to_str's structure to avoid awkward definition ordering of functions

@@ Line 206: / Line 206: @@
 end
-local val_to_str
+local function val_to_str(obj)
-        return k
+        else
     end
         end
-        v = mw.ustring.gsub(v, '\n', '\\n')
+        for k, v in ipairs(obj) do
-            return "'" .. v .. "'"
+            checked[k] = true
         end
-        return '"' .. mw.ustring.gsub(v, '"', '\\"' ) .. '"'
+        for k, v in pairs(obj) do
     else
-        return type(v) == 'table' and table_to_str(v) or tostring(v)
+        return tostring(obj)
     end
 end

Aidan9382: Temporarily re-arrange function order to account for localisation as to avoid unreferenced locals error (these depend on eachother in weird ways, will look at sorting them)

2023-02-08T19:48:41Z

Temporarily re-arrange function order to account for localisation as to avoid unreferenced locals error (these depend on eachother in weird ways, will look at sorting them)

@@ Line 206: / Line 206: @@
 end
-local function val_to_str(v)
+local val_to_str
 local function table_key_to_str(k)
@@ Line 238: / Line 228: @@
     end
     return '{' .. table.concat(result, ',') .. '}'
 end

Uzume: local functions

2023-02-08T19:44:16Z

local functions

← Previous revision		Revision as of 19:44, 8 February 2023
Line 206:		Line 206:
	end		end

	function val_to_str(v)		local function val_to_str(v)
	if type(v) == 'string' then		if type(v) == 'string' then
	v = mw.ustring.gsub(v, '\n', '\\n')		v = mw.ustring.gsub(v, '\n', '\\n')
Line 218:		Line 218:
	end		end

	function table_key_to_str(k)		local function table_key_to_str(k)
	if type(k) == 'string' and mw.ustring.match(k, '^[_%a][_%a%d]*$') then		if type(k) == 'string' and mw.ustring.match(k, '^[_%a][_%a%d]*$') then
	return k		return k
Line 226:		Line 226:
	end		end

	function table_to_str(tbl)		local function table_to_str(tbl)
	local result, done = {}, {}		local result, done = {}, {}
	for k, v in ipairs(tbl) do		for k, v in ipairs(tbl) do

Uzume: fix "first_difference" local; causes failures testing modules that use "strict"

2023-02-08T19:40:42Z

fix "first_difference" local; causes failures testing modules that use "strict"

← Previous revision		Revision as of 19:40, 8 February 2023
Line 31:		Line 31:
	local num_runs = 0		local num_runs = 0

	function first_difference(s1, s2)		local function first_difference(s1, s2)
	if s1 == s2 then return '' end		if s1 == s2 then return '' end
	local max = math.min(#s1, #s2)		local max = math.min(#s1, #s2)

Aidan9382: implement a custom class into the table headers for script ease

2023-01-29T06:07:51Z

implement a custom class into the table headers for script ease

← Previous revision		Revision as of 06:07, 29 January 2023
Line 4:		Line 4:

	local frame, tick, cross, should_highlight		local frame, tick, cross, should_highlight
	local result_table_header = "{\|class=\"wikitable\"\n\|+ %s\n! !! Text !! Expected !! Actual"		local result_table_header = "{\|class=\"wikitable unit-tests-result\"\n\|+ %s\n! !! Text !! Expected !! Actual"
	local result_table_live_sandbox_header = "{\|class=\"wikitable\"\n\|+ %s\n! !! Test !! Live !! Sandbox !! Expected"		local result_table_live_sandbox_header = "{\|class=\"wikitable unit-tests-result\"\n\|+ %s\n! !! Test !! Live !! Sandbox !! Expected"

	local result_table = { n = 0 }		local result_table = { n = 0 }

← Previous revision		Revision as of 17:03, 25 May 2023
Line 51:		Line 51:

	-- Fix any stripmarkers if asked to do so to prevent incorrect fails		-- Fix any stripmarkers if asked to do so to prevent incorrect fails
			local compared_expected = expected
	local compared_actual = actual		local compared_actual = actual
	if options.templatestyles then		if options.templatestyles then
	local pattern = '(\127[^\127]UNIQ%-%-templatestyles%-)(%x+)(%-QINU[^\127]\127)'		local pattern = '(\127[^\127]UNIQ%-%-templatestyles%-)(%x+)(%-QINU[^\127]\127)'
	local _, expected_stripmarker_id = ~~expected~~:match(pattern) -- when module rendering has templatestyles strip markers, use ID from expected to prevent false test fail		local _, expected_stripmarker_id = compared_expected:match(pattern) -- when module rendering has templatestyles strip markers, use ID from expected to prevent false test fail
	if expected_stripmarker_id then		if expected_stripmarker_id then
	compared_actual = compared_actual:gsub(pattern, '%1' .. expected_stripmarker_id .. '%3') -- replace actual id with expected id; ignore second capture in pattern		compared_actual = compared_actual:gsub(pattern, '%1' .. expected_stripmarker_id .. '%3') -- replace actual id with expected id; ignore second capture in pattern
			compared_expected = compared_expected:gsub(pattern, '%1' .. expected_stripmarker_id .. '%3') -- account for other strip markers
	end		end
	end		end
	if options.stripmarker then		if options.stripmarker then
	local pattern = '(\127[^\127]UNIQ%-%-%l+%-)(%x+)(%-%-?QINU[^\127]\127)'		local pattern = '(\127[^\127]UNIQ%-%-%l+%-)(%x+)(%-%-?QINU[^\127]\127)'
	local _, expected_stripmarker_id = ~~expected~~:match(pattern)		local _, expected_stripmarker_id = compared_expected:match(pattern)
	if expected_stripmarker_id then		if expected_stripmarker_id then
	compared_actual = compared_actual:gsub(pattern, '%1' .. expected_stripmarker_id .. '%3')		compared_actual = compared_actual:gsub(pattern, '%1' .. expected_stripmarker_id .. '%3')
			compared_expected = compared_expected:gsub(pattern, '%1' .. expected_stripmarker_id .. '%3')
	end		end
	end		end

	-- Perform the comparison		-- Perform the comparison
	local success = compared_actual == ~~expected~~		local success = compared_actual == compared_expected
	if not success then		if not success then
	num_failures = num_failures + 1		num_failures = num_failures + 1
Line 77:		Line 80:
	-- We need 2 rows available for the expected and actual columns		-- We need 2 rows available for the expected and actual columns
	-- Top one is parsed, bottom is unparsed		-- Top one is parsed, bottom is unparsed
	local differs_at = self.differs_at and (' \n\| rowspan=2\|' .. first_difference(~~expected~~, compared_actual)) or ''		local differs_at = self.differs_at and (' \n\| rowspan=2\|' .. first_difference(compared_expected, compared_actual)) or ''
	-- Local copies of tick/cross to allow for highlighting		-- Local copies of tick/cross to allow for highlighting
	local highlight = (should_highlight and not success and 'style="background:#fc0;" ') or ''		local highlight = (should_highlight and not success and 'style="background:#fc0;" ') or ''
Line 90:		Line 93:
	else		else
	-- Display normally with whichever option was preferred (nowiki/parsed)		-- Display normally with whichever option was preferred (nowiki/parsed)
	local differs_at = self.differs_at and (' \n\| ' .. first_difference(~~expected~~, compared_actual)) or ''		local differs_at = self.differs_at and (' \n\| ' .. first_difference(compared_expected, compared_actual)) or ''
	local formatting = options.nowiki and mw.text.nowiki or return_varargs		local formatting = options.nowiki and mw.text.nowiki or return_varargs
	local highlight = (should_highlight and not success and 'style="background:#fc0;"\|') or ''		local highlight = (should_highlight and not success and 'style="background:#fc0;"\|') or ''

← Previous revision		Revision as of 13:02, 25 May 2023
Line 51:		Line 51:

	-- Fix any stripmarkers if asked to do so to prevent incorrect fails		-- Fix any stripmarkers if asked to do so to prevent incorrect fails
	local expected_actual = actual
	local compared_actual = actual		local compared_actual = actual
	if options.templatestyles then		if options.templatestyles then
	local pattern = '(\127[^\127]UNIQ%-%-templatestyles%-)(%x+)(%-QINU[^\127]\127)'		local pattern = '(\127[^\127]UNIQ%-%-templatestyles%-)(%x+)(%-QINU[^\127]\127)'
	local _, expected_stripmarker_id = ~~expected_actual~~:match(pattern) -- when module rendering has templatestyles strip markers, use ID from expected to prevent false test fail		local _, expected_stripmarker_id = expected:match(pattern) -- when module rendering has templatestyles strip markers, use ID from expected to prevent false test fail
	if expected_stripmarker_id then		if expected_stripmarker_id then
	compared_actual = compared_actual:gsub(pattern, '%1' .. expected_stripmarker_id .. '%3') -- replace actual id with expected id; ignore second capture in pattern		compared_actual = compared_actual:gsub(pattern, '%1' .. expected_stripmarker_id .. '%3') -- replace actual id with expected id; ignore second capture in pattern
	expected_actual = expected_actual:gsub(pattern, '%1' .. expected_stripmarker_id .. '%3') -- account for other strip markers
	end		end
	end		end
	if options.stripmarker then		if options.stripmarker then
	local pattern = '(\127[^\127]UNIQ%-%-%l+%-)(%x+)(%-%-?QINU[^\127]\127)'		local pattern = '(\127[^\127]UNIQ%-%-%l+%-)(%x+)(%-%-?QINU[^\127]\127)'
	local _, expected_stripmarker_id = ~~expected_actual~~:match(pattern)		local _, expected_stripmarker_id = expected:match(pattern)
	if expected_stripmarker_id then		if expected_stripmarker_id then
	compared_actual = compared_actual:gsub(pattern, '%1' .. expected_stripmarker_id .. '%3')		compared_actual = compared_actual:gsub(pattern, '%1' .. expected_stripmarker_id .. '%3')
	expected_actual = expected_actual:gsub(pattern, '%1' .. expected_stripmarker_id .. '%3')
	end		end
	end		end

	-- Perform the comparison		-- Perform the comparison
	local success = compared_actual == ~~expected_actual~~		local success = compared_actual == expected
	if not success then		if not success then
	num_failures = num_failures + 1		num_failures = num_failures + 1
Line 80:		Line 77:
	-- We need 2 rows available for the expected and actual columns		-- We need 2 rows available for the expected and actual columns
	-- Top one is parsed, bottom is unparsed		-- Top one is parsed, bottom is unparsed
	local differs_at = self.differs_at and (' \n\| rowspan=2\|' .. first_difference(~~expected_actual~~, compared_actual)) or ''		local differs_at = self.differs_at and (' \n\| rowspan=2\|' .. first_difference(expected, compared_actual)) or ''
	-- Local copies of tick/cross to allow for highlighting		-- Local copies of tick/cross to allow for highlighting
	local highlight = (should_highlight and not success and 'style="background:#fc0;" ') or ''		local highlight = (should_highlight and not success and 'style="background:#fc0;" ') or ''
Line 93:		Line 90:
	else		else
	-- Display normally with whichever option was preferred (nowiki/parsed)		-- Display normally with whichever option was preferred (nowiki/parsed)
	local differs_at = self.differs_at and (' \n\| ' .. first_difference(~~expected_actual~~, compared_actual)) or ''		local differs_at = self.differs_at and (' \n\| ' .. first_difference(expected, compared_actual)) or ''
	local formatting = options.nowiki and mw.text.nowiki or return_varargs		local formatting = options.nowiki and mw.text.nowiki or return_varargs
	local highlight = (should_highlight and not success and 'style="background:#fc0;"\|') or ''		local highlight = (should_highlight and not success and 'style="background:#fc0;"\|') or ''

← Previous revision		Revision as of 23:10, 24 May 2023
Line 51:		Line 51:

	-- Fix any stripmarkers if asked to do so to prevent incorrect fails		-- Fix any stripmarkers if asked to do so to prevent incorrect fails
			local expected_actual = actual
	local compared_actual = actual		local compared_actual = actual
	if options.templatestyles then		if options.templatestyles then
	local pattern = '(\127[^\127]UNIQ%-%-templatestyles%-)(%x+)(%-QINU[^\127]\127)'		local pattern = '(\127[^\127]UNIQ%-%-templatestyles%-)(%x+)(%-QINU[^\127]\127)'
	local _, expected_stripmarker_id = ~~expected~~:match(pattern) -- when module rendering has templatestyles strip markers, use ID from expected to prevent false test fail		local _, expected_stripmarker_id = expected_actual:match(pattern) -- when module rendering has templatestyles strip markers, use ID from expected to prevent false test fail
	if expected_stripmarker_id then		if expected_stripmarker_id then
	compared_actual = compared_actual:gsub(pattern, '%1' .. expected_stripmarker_id .. '%3') -- replace actual id with expected id; ignore second capture in pattern		compared_actual = compared_actual:gsub(pattern, '%1' .. expected_stripmarker_id .. '%3') -- replace actual id with expected id; ignore second capture in pattern
			expected_actual = expected_actual:gsub(pattern, '%1' .. expected_stripmarker_id .. '%3') -- account for other strip markers
	end		end
	end		end
	if options.stripmarker then		if options.stripmarker then
	local pattern = '(\127[^\127]UNIQ%-%-%l+%-)(%x+)(%-%-?QINU[^\127]\127)'		local pattern = '(\127[^\127]UNIQ%-%-%l+%-)(%x+)(%-%-?QINU[^\127]\127)'
	local _, expected_stripmarker_id = ~~expected~~:match(pattern)		local _, expected_stripmarker_id = expected_actual:match(pattern)
	if expected_stripmarker_id then		if expected_stripmarker_id then
	compared_actual = compared_actual:gsub(pattern, '%1' .. expected_stripmarker_id .. '%3')		compared_actual = compared_actual:gsub(pattern, '%1' .. expected_stripmarker_id .. '%3')
			expected_actual = expected_actual:gsub(pattern, '%1' .. expected_stripmarker_id .. '%3')
	end		end
	end		end

	-- Perform the comparison		-- Perform the comparison
	local success = compared_actual == ~~expected~~		local success = compared_actual == expected_actual
	if not success then		if not success then
	num_failures = num_failures + 1		num_failures = num_failures + 1
Line 77:		Line 80:
	-- We need 2 rows available for the expected and actual columns		-- We need 2 rows available for the expected and actual columns
	-- Top one is parsed, bottom is unparsed		-- Top one is parsed, bottom is unparsed
	local differs_at = self.differs_at and (' \n\| rowspan=2\|' .. first_difference(~~expected~~, compared_actual)) or ''		local differs_at = self.differs_at and (' \n\| rowspan=2\|' .. first_difference(expected_actual, compared_actual)) or ''
	-- Local copies of tick/cross to allow for highlighting		-- Local copies of tick/cross to allow for highlighting
	local highlight = (should_highlight and not success and 'style="background:#fc0;" ') or ''		local highlight = (should_highlight and not success and 'style="background:#fc0;" ') or ''
Line 90:		Line 93:
	else		else
	-- Display normally with whichever option was preferred (nowiki/parsed)		-- Display normally with whichever option was preferred (nowiki/parsed)
	local differs_at = self.differs_at and (' \n\| ' .. first_difference(~~expected~~, compared_actual)) or ''		local differs_at = self.differs_at and (' \n\| ' .. first_difference(expected_actual, compared_actual)) or ''
	local formatting = options.nowiki and mw.text.nowiki or return_varargs		local formatting = options.nowiki and mw.text.nowiki or return_varargs
	local highlight = (should_highlight and not success and 'style="background:#fc0;"\|') or ''		local highlight = (should_highlight and not success and 'style="background:#fc0;"\|') or ''