Skip to content
This repository was archived by the owner on Sep 5, 2024. It is now read-only.

Commit 857d3b8

Browse files
crisbetoThomasBurleson
authored andcommitted
chore(panel): fix failing unit test (#9566)
When running the panel tests separately from all the others, the one about disabling scrolling was failing. This wasn't happening when running it together with all the other components, because some other component (not sure exactly which one, but it seemed like either dialog or autocomplete) wasn't cleaning up the DOM properly and was leaving a scroll mask lying around. This change removes the DOM lookup for the scroll mask, because it looks like it got disabled in PR #8901, however the test didn't get updated.
1 parent a3b3e7b commit 857d3b8

File tree

1 file changed

+1
-5
lines changed

1 file changed

+1
-5
lines changed

src/components/panel/panel.spec.js

+1-5
Original file line numberDiff line numberDiff line change
@@ -675,14 +675,10 @@ describe('$mdPanel', function() {
675675
spyOn($mdUtil, 'disableScrollAround').and.callThrough();
676676

677677
openPanel(config);
678-
679678
expect(PANEL_EL).toExist();
680-
expect(SCROLL_MASK_CLASS).toExist();
681-
679+
expect(SCROLL_MASK_CLASS).not.toExist();
682680
closePanel();
683681

684-
var scrollMaskEl = $rootEl[0].querySelector(SCROLL_MASK_CLASS);
685-
expect(scrollMaskEl).not.toExist();
686682
expect($mdUtil.disableScrollAround).toHaveBeenCalled();
687683
});
688684

0 commit comments

Comments
 (0)