Module:Redirect hatnote/testcases - Revision history

Tamzin: Reverted edits by Tamzin (talk) to last version by Nihiltres.

2022-05-09T09:19:20Z

Reverted edits by Tamzin (talk) to last version by Nihiltres.

Show changes

Tamzin: change for version I'm testing; displays same to reader, just different HTML

2022-05-09T08:43:59Z

change for version I'm testing; displays same to reader, just different HTML

Show changes

Nihiltres: Add test for not tracking RfD'd pages and made some fixes to others

2021-04-14T18:40:39Z

Add test for not tracking RfD'd pages and made some fixes to others

@@ Line 283: / Line 283: @@
 function suite:testTrackingCategoryNotRedirect()
 	local currentTitle = suite.makeFakeTitleObject('Example')
-	local redirectTitle = suite.makeFakeTitleObject('Example 2', {exists = true, isRedirect = false})
+	local redirectTitle = suite.makeFakeTitleObject('Example 2', {exists = true, isRedirect = false, getContent = function ()
 	self:assertContainsTrackingCategory(d.invalidCat, _redirect({redirectTitle.prefixedText}, nil, nil, currentTitle, redirectTitle))
 end

Nihiltres: Updated test case to match template's error message

2021-03-26T02:01:06Z

Updated test case to match template's error message

← Previous revision		Revision as of 02:01, 26 March 2021
Line 23:		Line 23:
	function suite:testBlank()		function suite:testBlank()
	local result = suite.runRedirect({},{})		local result = suite.runRedirect({},{})
	self:assertStringContains('no redirect ~~specified~~', result, true)		self:assertStringContains('missing redirect parameter', result, true)
	self:assertStringContains('Template:Redirect#Errors', result, true)		self:assertStringContains('Template:Redirect#Errors', result, true)
	end		end

Nihiltres: A few more tweaks missed in the previous edit

2016-05-05T14:52:24Z

A few more tweaks missed in the previous edit

Nihiltres: Updated to match argument/formatting differences expected from sandbox version of module

2016-05-05T14:41:32Z

Updated to match argument/formatting differences expected from sandbox version of module

← Previous revision		Revision as of 14:41, 5 May 2016
Line 98:		Line 98:
	function suite:testTwoAnds()		function suite:testTwoAnds()
	local result = suite.runRedirect({'REDIRECT', 'USE1', 'PAGE1', 'and', 'PAGE2', 'and', 'PAGE3'},{1})		local result = suite.runRedirect({'REDIRECT', 'USE1', 'PAGE1', 'and', 'PAGE2', 'and', 'PAGE3'},{1})
	self:assertStringContains('"REDIRECT" redirects here. For USE1, see [[:PAGE1]], [[:PAGE2]] and [[:PAGE3]].', result, true)		self:assertStringContains('"REDIRECT" redirects here. For USE1, see [[:PAGE1]], [[:PAGE2]], and [[:PAGE3]].', result, true)
	end		end

	function suite:testThreeAnds()		function suite:testThreeAnds()
	local result = suite.runRedirect({'REDIRECT', 'USE1', 'PAGE1', 'and', 'PAGE2', 'and', 'PAGE3', 'and', 'PAGE4'},{1})		local result = suite.runRedirect({'REDIRECT', 'USE1', 'PAGE1', 'and', 'PAGE2', 'and', 'PAGE3', 'and', 'PAGE4'},{1})
	self:assertStringContains('"REDIRECT" redirects here. For USE1, see [[:PAGE1]], [[:PAGE2]], [[:PAGE3]] and [[:PAGE4]].', result, true)		self:assertStringContains('"REDIRECT" redirects here. For USE1, see [[:PAGE1]], [[:PAGE2]], [[:PAGE3]], and [[:PAGE4]].', result, true)
	end		end

	function suite:testTwoAndsUse2()		function suite:testTwoAndsUse2()
	local result = suite.runRedirect({'REDIRECT', 'USE1', 'PAGE1', 'and', 'PAGE2', 'and', 'PAGE3', 'USE2', 'PAGE4'},{1})		local result = suite.runRedirect({'REDIRECT', 'USE1', 'PAGE1', 'and', 'PAGE2', 'and', 'PAGE3', 'USE2', 'PAGE4'},{1})
	self:assertStringContains('"REDIRECT" redirects here. For USE1, see [[:PAGE1]], [[:PAGE2]] and [[:PAGE3]]. For USE2, see [[:PAGE4]].', result, true)		self:assertStringContains('"REDIRECT" redirects here. For USE1, see [[:PAGE1]], [[:PAGE2]], and [[:PAGE3]]. For USE2, see [[:PAGE4]].', result, true)
	end		end

Line 193:		Line 193:

	function suite:testLuaRedirectOnly()		function suite:testLuaRedirectOnly()
	local result = _redirect('REDIRECT')		local result = _redirect({'REDIRECT'})
	self:assertStringContains('"REDIRECT" redirects here. For other uses, see [[:REDIRECT (disambiguation)]].', result, true)		self:assertStringContains('"REDIRECT" redirects here. For other uses, see [[:REDIRECT (disambiguation)]].', result, true)
	end		end
Line 199:		Line 199:
	function suite:testLuaPage1()		function suite:testLuaPage1()
	local result = _redirect(		local result = _redirect(
	'REDIRECT',		{'REDIRECT', 'USE1', 'PAGE1'},
	{		1
	{use = 'USE1', pages = {'PAGE1'}}
	}
	)		)
	self:assertStringContains('"REDIRECT" redirects here. For USE1, see [[:PAGE1]].', result, true)		self:assertStringContains('"REDIRECT" redirects here. For USE1, see [[:PAGE1]].', result, true)
Line 209:		Line 207:
	function suite:testLuaPage3()		function suite:testLuaPage3()
	local result = _redirect(		local result = _redirect(
		⚫	{'REDIRECT', 'USE1', 'PAGE1', 'USE2', 'PAGE2', 'USE3', 'PAGE3'},
	'REDIRECT',
	{		1
	{use = 'USE1', pages = {'PAGE1'}},
	{use = 'USE2', pages = {'PAGE2'}},
	{use = 'USE3', pages = {'PAGE3'}}
	}
	)		)
	self:assertStringContains('"REDIRECT" redirects here. For USE1, see [[:PAGE1]]. For USE2, see [[:PAGE2]]. For USE3, see [[:PAGE3]].', result, true)		self:assertStringContains('"REDIRECT" redirects here. For USE1, see [[:PAGE1]]. For USE2, see [[:PAGE2]]. For USE3, see [[:PAGE3]].', result, true)
Line 221:		Line 215:
	function suite:testLuaMultiplePages()		function suite:testLuaMultiplePages()
	local result = _redirect(		local result = _redirect(
			{'REDIRECT', 'USE1', 'PAGE1', 'and', 'PAGE2', 'and', 'PAGE3', 'USE2', 'PAGE4', 'and', 'PAGE5'},
	'REDIRECT',
	{		1
⚫	{~~use =~~ 'USE1', ~~pages = {~~'PAGE1', 'PAGE2', 'PAGE3'}},
	{use = 'USE2', pages = {'PAGE4', 'PAGE5'}},
	}
	)		)
	self:assertStringContains('"REDIRECT" redirects here. For USE1, see [[:PAGE1]], [[:PAGE2]] and [[:PAGE3]]. For USE2, see [[:PAGE4]] and [[:PAGE5]].', result, true)		self:assertStringContains('"REDIRECT" redirects here. For USE1, see [[:PAGE1]], [[:PAGE2]] and [[:PAGE3]]. For USE2, see [[:PAGE4]] and [[:PAGE5]].', result, true)
Line 231:		Line 222:

	function suite:testLuaSelfref()		function suite:testLuaSelfref()
	local result = _redirect('REDIRECT', ~~nil~~, {selfref = true})		local result = _redirect({'REDIRECT'}, 1, {selfref = true})
	self:assertStringContains('class=".-selfref.-"', result, false)		self:assertStringContains('class=".-selfref.-"', result, false)
	end		end
Line 269:		Line 260:
	function suite:testTrackingCategoryNonMainspace()		function suite:testTrackingCategoryNonMainspace()
	local currentTitle = suite.makeFakeTitleObject('Wikipedia:Namespaces')		local currentTitle = suite.makeFakeTitleObject('Wikipedia:Namespaces')
	suite:assertNotContainsTrackingCategory(_redirect('Example', nil, nil, currentTitle))		suite:assertNotContainsTrackingCategory(_redirect({'Example'}, nil, nil, currentTitle))
	end		end

Line 275:		Line 266:
	local currentTitle = suite.makeFakeTitleObject('Example')		local currentTitle = suite.makeFakeTitleObject('Example')
	for _, s in ipairs{'REDIRECT', 'REDIRECT1', 'REDIRECT1678', 'TERM'} do		for _, s in ipairs{'REDIRECT', 'REDIRECT1', 'REDIRECT1678', 'TERM'} do
	self:assertNotContainsTrackingCategory(_redirect(s, nil, nil, currentTitle))		self:assertNotContainsTrackingCategory(_redirect({s}, nil, nil, currentTitle))
	end		end
	end		end

Nihiltres: Updated tests to reflect removal of '1' → 'other uses' behaviour. Deleted suite:testUse1Is1() and suite:testUse2Is1() and changed '1' to 'other uses' in suite:testOtherUsesIsNotFinal()

2016-04-17T16:48:06Z

Updated tests to reflect removal of '1' → 'other uses' behaviour. Deleted suite:testUse1Is1() and suite:testUse2Is1() and changed '1' to 'other uses' in suite:testOtherUsesIsNotFinal()

← Previous revision		Revision as of 16:48, 17 April 2016
Line 157:		Line 157:
	local result = suite.runRedirect({'REDIRECT', 'USE1', 'PAGE1', '', 'PAGE2'},{1})		local result = suite.runRedirect({'REDIRECT', 'USE1', 'PAGE1', '', 'PAGE2'},{1})
	self:assertStringContains('"REDIRECT" redirects here. For USE1, see [[:PAGE1]].', result, true)		self:assertStringContains('"REDIRECT" redirects here. For USE1, see [[:PAGE1]].', result, true)
	end

	function suite:testUse2Is1()
	local result = suite.runRedirect({'REDIRECT', 'USE1', 'PAGE1', '1', 'PAGE2'},{1})
	self:assertStringContains('"REDIRECT" redirects here. For USE1, see [[:PAGE1]]. For other uses, see [[:PAGE2]].', result, true)
	end

	function suite:testUse1Is1()
	local result = suite.runRedirect({'REDIRECT', '1', 'PAGE1', 'USE2', 'PAGE2'},{1})
	self:assertStringContains('"REDIRECT" redirects here. For 1, see [[:PAGE1]]. For USE2, see [[:PAGE2]].', result, true)
	end		end

	function suite:testOtherUsesIsNotFinal()		function suite:testOtherUsesIsNotFinal()
	local result = suite.runRedirect({'REDIRECT', 'USE1', 'PAGE1', '1', 'PAGE2', 'USE3', 'PAGE3'},{1})		local result = suite.runRedirect({'REDIRECT', 'USE1', 'PAGE1', 'other uses', 'PAGE2', 'USE3', 'PAGE3'},{1})
	self:assertStringContains('"REDIRECT" redirects here. For USE1, see [[:PAGE1]]. For other uses, see [[:PAGE2]]. For USE3, see [[:PAGE3]].', result, true)		self:assertStringContains('"REDIRECT" redirects here. For USE1, see [[:PAGE1]]. For other uses, see [[:PAGE2]]. For USE3, see [[:PAGE3]].', result, true)
	end		end

Nihiltres: Updated suite:testPage1Skipped() and suite:testPage2Skipped() with new expected behaviour. Not happy with these, since repeated defaulting to the same thing is ugly, but it is expected

2016-04-14T14:29:58Z

Updated suite:testPage1Skipped() and suite:testPage2Skipped() with new expected behaviour. Not happy with these, since repeated defaulting to the same thing is ugly, but it *is* expected

← Previous revision		Revision as of 14:29, 14 April 2016
Line 122:		Line 122:
	function suite:testPage1Skipped()		function suite:testPage1Skipped()
	local result = suite.runRedirect({'REDIRECT', 'USE1', '', 'USE2'},{1})		local result = suite.runRedirect({'REDIRECT', 'USE1', '', 'USE2'},{1})
	self:assertStringContains('"REDIRECT" redirects here. For USE1, see [[:REDIRECT (disambiguation)]]. For USE2, see [[:~~USE2~~ (disambiguation)]].', result, true)		self:assertStringContains('"REDIRECT" redirects here. For USE1, see [[:REDIRECT (disambiguation)]]. For USE2, see [[:REDIRECT (disambiguation)]].', result, true)
	end		end

Line 132:		Line 132:
	function suite:testPage2Skipped()		function suite:testPage2Skipped()
	local result = suite.runRedirect({'REDIRECT', 'USE1', 'PAGE1', 'USE2', '', 'USE3'},{1})		local result = suite.runRedirect({'REDIRECT', 'USE1', 'PAGE1', 'USE2', '', 'USE3'},{1})
	self:assertStringContains('"REDIRECT" redirects here. For USE1, see [[:PAGE1]]. For USE2, see [[:~~USE2~~ (disambiguation)]]. For USE3, see [[:~~USE3~~ (disambiguation)]].', result, true)		self:assertStringContains('"REDIRECT" redirects here. For USE1, see [[:PAGE1]]. For USE2, see [[:REDIRECT (disambiguation)]]. For USE3, see [[:REDIRECT (disambiguation)]].', result, true)
	end		end

Nihiltres: Updated suite:testUse2(), suite:testUse3(), suite:testUse4(), and suite:testMiddlePageSkipped() with new expected behaviour

2016-04-14T14:23:21Z

Updated suite:testUse2(), suite:testUse3(), suite:testUse4(), and suite:testMiddlePageSkipped() with new expected behaviour

← Previous revision		Revision as of 14:23, 14 April 2016
Line 44:		Line 44:
	function suite:testUse2()		function suite:testUse2()
	local result = suite.runRedirect({'REDIRECT', 'USE1', 'PAGE1', 'USE2'},{1})		local result = suite.runRedirect({'REDIRECT', 'USE1', 'PAGE1', 'USE2'},{1})
	self:assertStringContains('"REDIRECT" redirects here. For USE1, see [[:PAGE1]]. For USE2, see [[:~~USE2~~ (disambiguation)]].', result, true)		self:assertStringContains('"REDIRECT" redirects here. For USE1, see [[:PAGE1]]. For USE2, see [[:REDIRECT (disambiguation)]].', result, true)
	end		end

Line 54:		Line 54:
	function suite:testUse3()		function suite:testUse3()
	local result = suite.runRedirect({'REDIRECT', 'USE1', 'PAGE1', 'USE2', 'PAGE2', 'USE3'},{1})		local result = suite.runRedirect({'REDIRECT', 'USE1', 'PAGE1', 'USE2', 'PAGE2', 'USE3'},{1})
	self:assertStringContains('"REDIRECT" redirects here. For USE1, see [[:PAGE1]]. For USE2, see [[:PAGE2]]. For USE3, see [[:~~USE3~~ (disambiguation)]].', result, true)		self:assertStringContains('"REDIRECT" redirects here. For USE1, see [[:PAGE1]]. For USE2, see [[:PAGE2]]. For USE3, see [[:REDIRECT (disambiguation)]].', result, true)
	end		end

Line 64:		Line 64:
	function suite:testUse4()		function suite:testUse4()
	local result = suite.runRedirect({'REDIRECT', 'USE1', 'PAGE1', 'USE2', 'PAGE2', 'USE3', 'PAGE3', 'USE4'},{1})		local result = suite.runRedirect({'REDIRECT', 'USE1', 'PAGE1', 'USE2', 'PAGE2', 'USE3', 'PAGE3', 'USE4'},{1})
	self:assertStringContains('"REDIRECT" redirects here. For USE1, see [[:PAGE1]]. For USE2, see [[:PAGE2]]. For USE3, see [[:PAGE3]]. For USE4, see [[:~~USE4~~ (disambiguation)]].', result, true)		self:assertStringContains('"REDIRECT" redirects here. For USE1, see [[:PAGE1]]. For USE2, see [[:PAGE2]]. For USE3, see [[:PAGE3]]. For USE4, see [[:REDIRECT (disambiguation)]].', result, true)
	end		end

Line 147:		Line 147:
	function suite:testMiddlePageSkipped()		function suite:testMiddlePageSkipped()
	local result = suite.runRedirect({'REDIRECT', 'USE1', 'PAGE1', 'USE2', '', 'USE3', 'PAGE3'},{1})		local result = suite.runRedirect({'REDIRECT', 'USE1', 'PAGE1', 'USE2', '', 'USE3', 'PAGE3'},{1})
	self:assertStringContains('"REDIRECT" redirects here. For USE1, see [[:PAGE1]]. For USE2, see [[:~~USE2~~ (disambiguation)]]. For USE3, see [[:PAGE3]].', result, true)		self:assertStringContains('"REDIRECT" redirects here. For USE1, see [[:PAGE1]]. For USE2, see [[:REDIRECT (disambiguation)]]. For USE3, see [[:PAGE3]].', result, true)
	end		end

Nihiltres: Cleaned up previous test and applied implicit argument explicitly to applicable tests

2016-03-25T21:36:56Z

Cleaned up previous test and applied implicit argument explicitly to applicable tests

Show changes

← Previous revision		Revision as of 14:52, 5 May 2016
Line 218:		Line 218:
	1		1
	)		)
	self:assertStringContains('"REDIRECT" redirects here. For USE1, see [[:PAGE1]], [[:PAGE2]] and [[:PAGE3]]. For USE2, see [[:PAGE4]] and [[:PAGE5]].', result, true)		self:assertStringContains('"REDIRECT" redirects here. For USE1, see [[:PAGE1]], [[:PAGE2]], and [[:PAGE3]]. For USE2, see [[:PAGE4]] and [[:PAGE5]].', result, true)
	end		end

Line 272:		Line 272:
	function suite:testTrackingCategoryInvalidTitle()		function suite:testTrackingCategoryInvalidTitle()
	local currentTitle = suite.makeFakeTitleObject('Example')		local currentTitle = suite.makeFakeTitleObject('Example')
	self:assertContainsTrackingCategory(d.missingCat, _redirect('Exam<>ple 2', nil, nil, currentTitle))		self:assertContainsTrackingCategory(d.missingCat, _redirect({'Exam<>ple 2'}, nil, nil, currentTitle))
	end		end

Line 278:		Line 278:
	local currentTitle = suite.makeFakeTitleObject('Example')		local currentTitle = suite.makeFakeTitleObject('Example')
	local redirectTitle = suite.makeFakeTitleObject('Example 2', {exists = false, isRedirect = false})		local redirectTitle = suite.makeFakeTitleObject('Example 2', {exists = false, isRedirect = false})
	self:assertContainsTrackingCategory(d.missingCat, _redirect(redirectTitle.prefixedText, nil, nil, currentTitle, redirectTitle))		self:assertContainsTrackingCategory(d.missingCat, _redirect({redirectTitle.prefixedText}, nil, nil, currentTitle, redirectTitle))
	end		end

Line 284:		Line 284:
	local currentTitle = suite.makeFakeTitleObject('Example')		local currentTitle = suite.makeFakeTitleObject('Example')
	local redirectTitle = suite.makeFakeTitleObject('Example 2', {exists = true, isRedirect = false})		local redirectTitle = suite.makeFakeTitleObject('Example 2', {exists = true, isRedirect = false})
	self:assertContainsTrackingCategory(d.invalidCat, _redirect(redirectTitle.prefixedText, nil, nil, currentTitle, redirectTitle))		self:assertContainsTrackingCategory(d.invalidCat, _redirect({redirectTitle.prefixedText}, nil, nil, currentTitle, redirectTitle))
	end		end

Line 293:		Line 293:
	end})		end})
	local targetTitle = suite.makeFakeTitleObject('Example 3')		local targetTitle = suite.makeFakeTitleObject('Example 3')
	self:assertContainsTrackingCategory(d.invalidCat, _redirect(redirectTitle.prefixedText, nil, nil, currentTitle, redirectTitle, targetTitle))		self:assertContainsTrackingCategory(d.invalidCat, _redirect({redirectTitle.prefixedText}, nil, nil, currentTitle, redirectTitle, targetTitle))
	end		end

Line 302:		Line 302:
	end})		end})
	local targetTitle = suite.makeFakeTitleObject('Example')		local targetTitle = suite.makeFakeTitleObject('Example')
	self:assertNotContainsTrackingCategory(_redirect(redirectTitle.prefixedText, nil, nil, currentTitle, redirectTitle, targetTitle))		self:assertNotContainsTrackingCategory(_redirect({redirectTitle.prefixedText}, nil, nil, currentTitle, redirectTitle, targetTitle))
	end		end

Module:Redirect hatnote/testcases - Revision history

Tamzin: Reverted edits by Tamzin (talk) to last version by Nihiltres.

Tamzin: change for version I'm testing; displays same to reader, just different HTML

Nihiltres: Add test for not tracking RfD'd pages and made some fixes to others

Nihiltres: Updated test case to match template's error message

Nihiltres: A few more tweaks missed in the previous edit

Nihiltres: Updated to match argument/formatting differences expected from sandbox version of module

Nihiltres: Updated tests to reflect removal of '1' → 'other uses' behaviour. Deleted suite:testUse1Is1() and suite:testUse2Is1() and changed '1' to 'other uses' in suite:testOtherUsesIsNotFinal()

Nihiltres: Updated suite:testPage1Skipped() and suite:testPage2Skipped() with new expected behaviour. Not happy with these, since repeated defaulting to the same thing is ugly, but it *is* expected

Nihiltres: Updated suite:testUse2(), suite:testUse3(), suite:testUse4(), and suite:testMiddlePageSkipped() with new expected behaviour

Nihiltres: Cleaned up previous test and applied implicit argument explicitly to applicable tests

Nihiltres: Updated suite:testPage1Skipped() and suite:testPage2Skipped() with new expected behaviour. Not happy with these, since repeated defaulting to the same thing is ugly, but it is expected