From db0cc46edb69c13d006cf80bcff5db4ee3fbc199 Mon Sep 17 00:00:00 2001 From: Krinkle Date: Wed, 7 Mar 2012 02:43:46 +0000 Subject: [PATCH] [jquery.textSelection.test] disable caretTest per bug 34820 * Right now it's causing testswarm clients to have to this test 3 times, and if until bug 34820 is fixed there is no point in clogging up testswarm with it, making all IE columns uselessly red and making it harder to detect an actual regression --- .../qunit/suites/resources/jquery/jquery.textSelection.test.js | 3 +++ 1 file changed, 3 insertions(+) diff --git a/tests/qunit/suites/resources/jquery/jquery.textSelection.test.js b/tests/qunit/suites/resources/jquery/jquery.textSelection.test.js index 1b2f30240e..e01f217a04 100644 --- a/tests/qunit/suites/resources/jquery/jquery.textSelection.test.js +++ b/tests/qunit/suites/resources/jquery/jquery.textSelection.test.js @@ -253,6 +253,8 @@ var caretTest = function(options) { var caretSample = "Some big text that we like to work with. Nothing fancy... you know what I mean?"; +/* + // @broken: Disabled per bug 34820 caretTest({ description: 'getCaretPosition with original/empty selection - bug 31847 with IE 6/7/8', text: caretSample, @@ -260,6 +262,7 @@ caretTest({ end: [0, caretSample.length], // Other browsers default it to the beginning (0), so check both. mode: 'get' }); +*/ caretTest({ description: 'set/getCaretPosition with forced empty selection', -- 2.20.1