For mozilla-mobile#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.
Mugurell · Jul 8, 2021 · e15ec20 · e15ec20
1 parent fca33ab
commit e15ec20
Showing 1 changed file with 0 additions and 11 deletions.
diff --git a/...r/src/test/java/mozilla/components/browser/toolbar/behavior/BrowserToolbarBehaviorTest.kt b/...r/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))