From e15ec2074e58f22a6602e45b872751e56862cd24 Mon Sep 17 00:00:00 2001 From: Mugurell Date: Thu, 8 Jul 2021 13:52:09 +0300 Subject: [PATCH] For #10555 - BrowserToolbarBehaviorTest cleanup This commented method seems to be a leftover from the previous refactoring. "behavior.forceExpand(..)" now calls "expandWithAnimation" for which we already have a test at line 450. --- .../toolbar/behavior/BrowserToolbarBehaviorTest.kt | 11 ----------- 1 file changed, 11 deletions(-) diff --git a/components/browser/toolbar/src/test/java/mozilla/components/browser/toolbar/behavior/BrowserToolbarBehaviorTest.kt b/components/browser/toolbar/src/test/java/mozilla/components/browser/toolbar/behavior/BrowserToolbarBehaviorTest.kt index 36c72a6d47f..9a66f600d3c 100644 --- a/components/browser/toolbar/src/test/java/mozilla/components/browser/toolbar/behavior/BrowserToolbarBehaviorTest.kt +++ b/components/browser/toolbar/src/test/java/mozilla/components/browser/toolbar/behavior/BrowserToolbarBehaviorTest.kt @@ -493,17 +493,6 @@ class BrowserToolbarBehaviorTest { assertEquals(Gravity.TOP or Gravity.CENTER_HORIZONTAL, layoutParams.gravity) } - @Test - fun `Behavior will animateSnap UP when forceExpand is called`() { - // val behavior = spy(BrowserToolbarBehavior(testContext, null, ToolbarPosition.BOTTOM)) - // doReturn(true).`when`(behavior).shouldScroll - // val toolbar: BrowserToolbar = mock() - // - // behavior.forceExpand(toolbar) - // - // verify(behavior).animateSnap(toolbar, SnapDirection.UP) - } - @Test fun `Behavior will forceExpand when scrolling up and !shouldScroll if the touch was handled in the browser`() { val behavior = spy(BrowserToolbarBehavior(testContext, null, ToolbarPosition.BOTTOM))