Skip to content

Commit 17200a5

Browse files
committed
fix: integ tests failures due to scroll track marker refactor
1 parent b612cf9 commit 17200a5

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

test/spec/FindReplace-integ-test.js

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -126,7 +126,9 @@ define(function (require, exports, module) {
126126
}
127127

128128
// Verify number of tickmarks equals number of highlights
129-
var tickmarks = tw$(".tickmark-track .tickmark", myEditor.getRootElement());
129+
var tickmarks = testWindow.brackets.test.ScrollTrackMarkers._getTickmarks(myEditor);
130+
// we cannot compare the dom nodes like below as the algorithm will merge overlapping scrolltrackers.
131+
// var tickmarks = tw$(".tickmark-track .tickmark:not(.tickmark-current)", myEditor.getRootElement());
130132
expect(tickmarks.length).toEql(selections.length);
131133

132134
// Verify that editor UI doesn't have extra ranges left highlighted from earlier
@@ -395,7 +397,7 @@ define(function (require, exports, module) {
395397
enterSearchText("foo");
396398
expectHighlightedMatches(fooExpectedMatches);
397399

398-
var marks = testWindow.brackets.test.ScrollTrackMarkers._getTickmarks();
400+
var marks = testWindow.brackets.test.ScrollTrackMarkers._getTickmarks(myEditor);
399401
expect(marks.length).toEql(fooExpectedMatches.length);
400402

401403
marks.forEach(function (mark, index) {

0 commit comments

Comments
 (0)