Tests: Try to fix test that fails on Travis, but not on my setup

tests/Browser/Components/Toolbarmenu.php

@@ -48,7 +48,7 @@ class Toolbarmenu extends BaseComponent
     /**
      * Assert toolbar menu state
      */
-    public function assertMenuState($browser, $active, $disabled)
+    public function assertMenuState($browser, $active, $disabled = [], $missing = [])
     {
         // On phone the menu is invisible, open it
         if ($browser->isPhone()) {
@@ -77,6 +77,10 @@ class Toolbarmenu extends BaseComponent
             }
         }
 
+        foreach ($missing as $option) {
+            $browser->assertMissing("a.{$option}");
+        }
+
         $this->closeMenu($browser);
     }
 

tests/Browser/Mail/List.php

@@ -2,6 +2,8 @@
 
 namespace Tests\Browser\Mail;
 
+use Tests\Browser\Components\Toolbarmenu;
+
 class MailList extends \Tests\Browser\TestCase
 {
     protected function setUp()
@@ -69,22 +71,20 @@ class MailList extends \Tests\Browser\TestCase
             }
             else { // phone
                 // On phones list options are in the toolbar menu
-                $browser->click('.toolbar-menu-button');
+                $browser->with(new Toolbarmenu(), function ($browser) {
-
+                    $active  = ['select', 'options'];
-                $browser->with('#toolbar-menu', function ($browser) {
+                    $missing = [];
-                    $browser->assertVisible('a.select:not(.disabled)');
-                    $browser->assertVisible('a.options:not(.disabled)');
-
                     $imap = \bootstrap::get_storage();
+
                     if ($imap->get_threading()) {
-                        $browser->assertVisible('a.threads:not(.disabled)');
+                        $active[] = 'threads';
                     }
                     else {
-                        $browser->assertMissing('a.threads');
+                        $missing[] = 'threads';
                     }
-                });
 
-                $browser->closeToolbarMenu();
+                    $browser->assertMenuState($active, [], $missing);
+                });
             }
         });
     }
@@ -96,8 +96,9 @@ class MailList extends \Tests\Browser\TestCase
     {
         $this->browse(function ($browser) {
             if ($browser->isPhone()) {
-                $browser->click('.toolbar-menu-button');
+                $browser->with(new Toolbarmenu(), function ($browser) {
-                $browser->click('#toolbar-menu a.select');
+                    $browser->clickMenuItem('select');
+                });
             }
             else if ($browser->isTablet()) {
                 $browser->click('.toolbar-list-button');