Module:UnitTests/sandbox - Revision history

Aidan9382: sync to live, keeping current changes

2023-03-02T15:53:09Z

sync to live, keeping current changes

← Previous revision		Revision as of 15:53, 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: Sync to live, keeping current changes

2023-02-08T20:07:21Z

Sync to live, keeping current changes

@@ Line 212: / Line 212: @@
 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

Uzume: merge with more fixes from active

2023-02-08T20:06:34Z

merge with more fixes from active

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

Uzume: merge with fixes from active

2023-02-08T19:48:57Z

merge with fixes from active

← Previous revision		Revision as of 19:48, 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)
Line 212:		Line 212:
	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 224:		Line 224:
	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 232:		Line 232:
	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

Aidan9382: rename calculate_output to a better name, use a starting _ to indicate its mainly for internal use

2023-02-06T12:04:10Z

rename calculate_output to a better name, use a starting _ to indicate its mainly for internal use

← Previous revision		Revision as of 12:04, 6 February 2023
Line 44:		Line 44:
	end		end

	function UnitTester:~~calculate_output~~(text, expected, actual, options)		function UnitTester:_calculate_preprocess_result(text, expected, actual, options)
	-- Set up some variables for throughout for ease		-- Set up some variables for throughout for ease
	num_runs = num_runs + 1		num_runs = num_runs + 1
Line 104:		Line 104:
	function UnitTester:preprocess_equals(text, expected, options)		function UnitTester:preprocess_equals(text, expected, options)
	local actual = frame:preprocess(text)		local actual = frame:preprocess(text)
	self:~~calculate_output~~(text, expected, actual, options)		self:_calculate_preprocess_result(text, expected, actual, options)
	end		end

Line 116:		Line 116:
	local actual = frame:preprocess(text1)		local actual = frame:preprocess(text1)
	local expected = frame:preprocess(text2)		local expected = frame:preprocess(text2)
	self:~~calculate_output~~(text1, expected, actual, options)		self:_calculate_preprocess_result(text1, expected, actual, options)
	end		end

Aidan9382: reorder functions for reading clarity

2023-02-06T12:02:45Z

reorder functions for reading clarity

← Previous revision		Revision as of 12:02, 6 February 2023
Line 117:		Line 117:
	local expected = frame:preprocess(text2)		local expected = frame:preprocess(text2)
	self:calculate_output(text1, expected, actual, options)		self:calculate_output(text1, expected, actual, options)
		⚫	end

		⚫	function UnitTester:preprocess_equals_preprocess_many(prefix1, suffix1, prefix2, suffix2, cases, options)
		⚫	for _, case in ipairs(cases) do
		⚫	self:preprocess_equals_preprocess(prefix1 .. case[1] .. suffix1, prefix2 .. (case[2] and case[2] or case[1]) .. suffix2, options)
		⚫	end
	end		end

Line 159:		Line 165:
	"\n\|-\n"		"\n\|-\n"
	)		)
⚫	end

⚫	function UnitTester:preprocess_equals_preprocess_many(prefix1, suffix1, prefix2, suffix2, cases, options)
⚫	for _, case in ipairs(cases) do
⚫	self:preprocess_equals_preprocess(prefix1 .. case[1] .. suffix1, prefix2 .. (case[2] and case[2] or case[1]) .. suffix2, options)
⚫	end
	end		end

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

2023-01-29T06:07:45Z

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 }

Aidan9382: some notes

2023-01-28T20:47:45Z

some notes

← Previous revision		Revision as of 20:47, 28 January 2023
Line 120:		Line 120:

	function UnitTester:preprocess_equals_compare(live, sandbox, expected, options)		function UnitTester:preprocess_equals_compare(live, sandbox, expected, options)
			--TODO: Standardise and simplyify - also make table headers more compatible
			--Currently, if we use any other option to compare at literally any point, we can't use this, and vice versa
			--This is because the change in the header setup is far too awkward to handle right now
			--As for how to fix this... dunno, needs some thinking. We could probably assume
			--that a _compare run and another run won't be used in the exact same function at least,
			--and sort the headers per function somehow instead of just having either one header or another for every test
	local live_text = frame:preprocess(live)		local live_text = frame:preprocess(live)
	local sandbox_text = frame:preprocess(sandbox)		local sandbox_text = frame:preprocess(sandbox)

Aidan9382: make highlighting an #invoke: based option

2023-01-28T16:39:48Z

make highlighting an #invoke: based option

← Previous revision		Revision as of 16:39, 28 January 2023
Line 3:		Line 3:
	local UnitTester = {}		local UnitTester = {}

	local frame, tick, cross		local frame, tick, cross, should_highlight
	local result_table_header = "{\|class=\"wikitable\"\n\|+ %s\n! !! Text !! Expected !! Actual"		local result_table_header = "{\|class=\"wikitable\"\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\"\n\|+ %s\n! !! Test !! Live !! Sandbox !! Expected"
Line 78:		Line 78:
	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, compared_actual)) or ''
	-- Local copies of tick/cross to allow for highlighting		-- Local copies of tick/cross to allow for highlighting
	local highlight = (not success and 'style="background:#fc0;" ') or ''		local highlight = (should_highlight and not success and 'style="background:#fc0;" ') or ''
	result_table:insert( -- Start output		result_table:insert( -- Start output
	'\| ', highlight, 'rowspan=2\|', success and tick or cross, -- Tick/Cross (2 rows)		'\| ', highlight, 'rowspan=2\|', success and tick or cross, -- Tick/Cross (2 rows)
Line 91:		Line 91:
	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, 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 = (not success and 'style="background:#fc0;"\|') or ''		local highlight = (should_highlight and not success and 'style="background:#fc0;"\|') or ''
	result_table:insert( -- Start output		result_table:insert( -- Start output
	'\| ', highlight, success and tick or cross, -- Tick/Cross		'\| ', highlight, success and tick or cross, -- Tick/Cross
Line 144:		Line 144:
	' \n\| ',		' \n\| ',
	mw.text.nowiki(live),		mw.text.nowiki(live),
	highlight_live and ' \n\|style="background: #fc0;"\| ' or ' \n\| ',		should_highlight and highlight_live and ' \n\|style="background: #fc0;"\| ' or ' \n\| ',
	formatting(live_text),		formatting(live_text),
	highlight_sandbox and ' \n\|style="background: #fc0;"\| ' or ' \n\| ',		should_highlight and highlight_sandbox and ' \n\|style="background: #fc0;"\| ' or ' \n\| ',
	formatting(sandbox_text),		formatting(sandbox_text),
	' \n\| ',		' \n\| ',
Line 292:		Line 292:
	if frame.args['live_sandbox'] then		if frame.args['live_sandbox'] then
	table_header = result_table_live_sandbox_header		table_header = result_table_live_sandbox_header
			end
			if frame.args.highlight then
			should_highlight = true
	end		end

Aidan9382: fix get_format stuff from previous tests

2023-01-28T14:56:47Z

fix get_format stuff from previous tests

← Previous revision		Revision as of 14:56, 28 January 2023
Line 88:		Line 88:
	)		)
	else		else
			-- 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, 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
Line 138:		Line 139:
	end		end
	end		end
	local formatting = ~~get_format~~(options)		local formatting = (options and options.nowiki and mw.text.nowiki) or return_varargs
	local differs_at = self.differs_at and (' \n\| ' .. first_difference(expected, live_text) or first_difference(expected, sandbox_text)) or ''		local differs_at = self.differs_at and (' \n\| ' .. first_difference(expected, live_text) or first_difference(expected, sandbox_text)) or ''
	result_table:insert(		result_table:insert(
Line 176:		Line 177:
	num_failures = num_failures + 1		num_failures = num_failures + 1
	end		end
	local formatting = ~~get_format~~(options)		local formatting = (options and options.nowiki and mw.text.nowiki) or return_varargs
	local differs_at = self.differs_at and (' \n\| ' .. first_difference(expected, actual)) or ''		local differs_at = self.differs_at and (' \n\| ' .. first_difference(expected, actual)) or ''
	local display = options and options.display or return_varargs		local display = options and options.display or return_varargs
Line 247:		Line 248:
	num_failures = num_failures + 1		num_failures = num_failures + 1
	end		end
	local formatting = ~~get_format~~(options)		local formatting = (options and options.nowiki and mw.text.nowiki) or return_varargs
	local actual_str = val_to_str(actual)		local actual_str = val_to_str(actual)
	local expected_str = val_to_str(expected)		local expected_str = val_to_str(expected)