From 3d8058a9a4de8fbb70b89a84ebb6913ecbfdb56a Mon Sep 17 00:00:00 2001 From: Nathan Sobo Date: Fri, 11 Dec 2015 13:23:25 -0700 Subject: [PATCH] Remove failing redundant assertions MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The expectNoStateUpdate is sufficient to test the issue in question. Now that most flags are removed, we can’t count on this part of the state not being updated when we request a state recompilation. --- spec/text-editor-presenter-spec.coffee | 8 -------- 1 file changed, 8 deletions(-) diff --git a/spec/text-editor-presenter-spec.coffee b/spec/text-editor-presenter-spec.coffee index effa579b1..f20439d9c 100644 --- a/spec/text-editor-presenter-spec.coffee +++ b/spec/text-editor-presenter-spec.coffee @@ -136,14 +136,6 @@ describe "TextEditorPresenter", -> # clearing additional rows won't trigger a state update expectNoStateUpdate presenter, -> presenter.clearScreenRowsToMeasure() - expect(stateFn(presenter).tiles[0]).toBeDefined() - expect(stateFn(presenter).tiles[2]).toBeDefined() - expect(stateFn(presenter).tiles[4]).toBeDefined() - expect(stateFn(presenter).tiles[6]).toBeDefined() - expect(stateFn(presenter).tiles[8]).toBeUndefined() - expect(stateFn(presenter).tiles[10]).toBeDefined() - expect(stateFn(presenter).tiles[12]).toBeDefined() - # when another change triggers a state update we remove useless lines expectStateUpdate presenter, -> presenter.setScrollTop(1)