Bug 159591 – LayoutTests jquery/attributes.html, core.html, event.html, manipulation.html, and traversing.html are flaky

• Status: RESOLVED
• Resolution: DUPLICATE
• Component: New Bugs

[Ryan Haddad]

Regressions: Unexpected text-only failures
  jquery/attributes.html
  jquery/core.html
  jquery/event.html
  jquery/manipulation.html
  jquery/traversing.html

https://build.webkit.org/results/Apple%20El%20Capitan%20Release%20WK2%20(Tests)/r203006%20(7494)/results.html
https://build.webkit.org/builders/Apple%20Yosemite%20Release%20WK2%20%28Tests%29/builds/15964

--- /Volumes/Data/slave/elcapitan-release-tests-wk2/build/layout-test-results/jquery/core-expected.txt
+++ /Volumes/Data/slave/elcapitan-release-tests-wk2/build/layout-test-results/jquery/core-actual.txt
@@ -1,2 +1,51 @@
-ALERT: 667 tests of 667 passed, 0 failed.
+CONSOLE MESSAGE: line 900: Test map() died, exception and test follows
+CONSOLE MESSAGE: line 901: RangeError: Maximum call stack size exceeded.
+CONSOLE MESSAGE: line 902: function () {
+	expect(2);//expect(6);
 
+	same(
+		jQuery("#ap").map(function(){
+			return jQuery(this).find("a").get();
+		}).get(),
+		q("google", "groups", "anchor1", "mark"),
+		"Array Map"
+	);
+
+	same(
+		jQuery("#ap > a").map(function(){
+			return this.parentNode;
+		}).get(),
+		q("ap","ap","ap"),
+		"Single Map"
+	);
+
+	return;//these haven't been accepted yet
+
+	//for #2616
+	var keys = jQuery.map( {a:1,b:2}, function( v, k ){
+		return k;
+	}, [ ] );
+
+	equals( keys.join(""), "ab", "Map the keys from a hash to an array" );
+
+	var values = jQuery.map( {a:1,b:2}, function( v, k ){
+		return v;
+	}, [ ] );
+
+	equals( values.join(""), "12", "Map the values from a hash to an array" );
+
+	var scripts = document.getElementsByTagName("script");
+	var mapped = jQuery.map( scripts, function( v, k ){
+		return v;
+	}, {length:0} );
+
+	equals( mapped.length, scripts.length, "Map an array(-like) to a hash" );
+
+	var flat = jQuery.map( Array(4), function( v, k ){
+		return k % 2 ? k : [k,k,k];//try mixing array and regular returns
+	});
+
+	equals( flat.join(""), "00012223", "try the new flatten technique(#2616)" );
+}
+ALERT: 666 tests of 667 passed, 1 failed.
+

[Ryan Haddad]

These tests have become flaky failures on Mac. They all fail together.

[Alexey Proskuryakov]

When did this regress, was it <http://trac.webkit.org/r203067>?

Mark, would you be available to take a look?

[Mark Lam]

(In reply to comment #2)
> When did this regress, was it <http://trac.webkit.org/r203067>?
> 
> Mark, would you be available to take a look?

https://build.webkit.org/results/Apple%20El%20Capitan%20Release%20WK2%20(Tests)/r203006%20(7494)/results.html is run on r203006.  Hence, this cannot be due to r203067.

[Alexey Proskuryakov]

Indeed.

*** This bug has been marked as a duplicate of bug 159616 ***