remove all occurrences of the "equals" and "same" function in the unit tests`

Reported by:	mikesherov	Owned by:
Priority:	low	Milestone:	1.7.1
Component:	misc	Version:	1.7
Keywords:		Cc:
Blocked by:		Blocking:

Description

in testrunner.js:

	window.equals = window.equal;
	window.same = window.deepEqual;

it's time this aliases went bye bye. 1.8?

Attachments (0)

Change History (11)

Changed November 06, 2011 03:33AM UTC by rwaldron comment:1

component:	unfiled → misc
keywords:	→ 1.8-discuss
milestone:	None → 1.8
priority:	undecided → low
status:	new → open
type:	bug → enhancement

+1 and Yay!

Changed November 06, 2011 04:03AM UTC by mikesherov comment:2

Here's the pull request: https://github.com/jquery/jquery/pull/580

Changed November 06, 2011 01:56PM UTC by dmethvin comment:3

Since there's a PR we should either land this ASAP or close the PR since it's going to get awfully stale.

Changed November 06, 2011 02:21PM UTC by mikesherov comment:4

Yeah. I guess 1.7.1 then? I tested in FF and Chrome from a hosted copy (so isLocal was false and all tests ran). No reason this shouldnt work in other browsers.

Changed November 06, 2011 08:30PM UTC by Mike Sherov comment:5

resolution:	→ fixed
status:	open → closed

Fix #10691. Remove all instances of equals() and same(), as these are deprecated in QUnit.

Changeset: f35ba5e699c15a1987baa389dbd4fc99f7a7eb03

Changed November 06, 2011 10:05PM UTC by dmethvin comment:6

milestone:	1.8 → 1.7.1

Changed November 09, 2011 09:39AM UTC by gnarf comment:7

Thanks Mike - someone has needed to do this for quite a while now... Glad you took the initiative!

Changed November 09, 2011 01:14PM UTC by rwaldron comment:8

;)

Changed November 09, 2011 02:05PM UTC by mikesherov comment:9

Rick deserves the credit here. A midnight IRC convo with him caused the change. All I did was find and replace.

Changed November 09, 2011 03:42PM UTC by rwaldron comment:10

Mike deserves the credit for actually doing the dirty work that no one else wanted to do ahaha

Changed December 10, 2011 07:17PM UTC by dmethvin comment:11

keywords:	1.8-discuss

Side navigation

#10691 closed enhancement (fixed)