Fix flaky BuildBuildPageTest (#3621)

`BuildBuildPageTest` has failed ~4% of the time across recent CI runs.
While I haven't been able to reproduce the failure locally myself, the
failure pattern seems to be correlated with clicking on the the
collapsible subproject headers. I suspect this is due to the click
location not always clicking on the summary element. This changes the
behavior to explicitly click on the summary, thus opening the expandable
component.

Author: williamjallen

tests/Browser/Pages/BuildBuildPageTest.php

@@ -479,7 +479,7 @@ public function testShowsListOfSubProjects(): void
                 ->waitForText($this->subproject2->name)
                 ->assertDontSee($buildError1->stdoutput)
                 ->assertDontSee($buildError2->stdoutput)
-                ->click('@collapse-' . $this->subproject1->id)
+                ->click('@collapse-' . $this->subproject1->id . ' summary')
                 ->assertSee($buildError1->stdoutput)
                 ->assertDontSee($buildError2->stdoutput)
             ;
@@ -779,7 +779,7 @@ public function testShowsNewAndFixedWarningsBySubproject(): void
         ): void {
             $browser->visit("/builds/{$current_parent_build->id}/build")
                 ->waitForText($this->subproject1->name)
-                ->click('@collapse-' . $this->subproject1->id)
+                ->click('@collapse-' . $this->subproject1->id . ' summary')
                 ->with('@collapse-' . $this->subproject1->id, function (Browser $browser) use (
                     $buildError1,
                     $buildError2,
@@ -793,7 +793,7 @@ public function testShowsNewAndFixedWarningsBySubproject(): void
                         ->assertSee($buildError4->stdoutput)
                     ;
                 })
-                ->click('@collapse-' . $this->subproject2->id)
+                ->click('@collapse-' . $this->subproject2->id . ' summary')
                 ->with('@collapse-' . $this->subproject2->id, function (Browser $browser) use (
                     $buildError1,
                     $buildError2,
@@ -811,7 +811,7 @@ public function testShowsNewAndFixedWarningsBySubproject(): void
 
             $browser->visit("/builds/{$current_parent_build->id}/build?onlydeltap")
                 ->waitForText($this->subproject1->name)
-                ->click('@collapse-' . $this->subproject1->id)
+                ->click('@collapse-' . $this->subproject1->id . ' summary')
                 ->with('@collapse-' . $this->subproject1->id, function (Browser $browser) use (
                     $buildError1,
                     $buildError2,
@@ -825,7 +825,7 @@ public function testShowsNewAndFixedWarningsBySubproject(): void
                         ->assertSee($buildError4->stdoutput)
                     ;
                 })
-                ->click('@collapse-' . $this->subproject2->id)
+                ->click('@collapse-' . $this->subproject2->id . ' summary')
                 ->with('@collapse-' . $this->subproject2->id, function (Browser $browser) use (
                     $buildError1,
                     $buildError2,
@@ -843,7 +843,7 @@ public function testShowsNewAndFixedWarningsBySubproject(): void
 
             $browser->visit("/builds/{$current_parent_build->id}/build?onlydeltan")
                 ->waitForText($this->subproject1->name)
-                ->click('@collapse-' . $this->subproject1->id)
+                ->click('@collapse-' . $this->subproject1->id . ' summary')
                 ->with('@collapse-' . $this->subproject1->id, function (Browser $browser) use (
                     $buildError1,
                     $buildError2,
@@ -857,7 +857,7 @@ public function testShowsNewAndFixedWarningsBySubproject(): void
                         ->assertDontSee($buildError4->stdoutput)
                     ;
                 })
-                ->click('@collapse-' . $this->subproject2->id)
+                ->click('@collapse-' . $this->subproject2->id . ' summary')
                 ->with('@collapse-' . $this->subproject2->id, function (Browser $browser): void {
                     $browser->waitForText('No fixed errors or warnings for this build.');
                 })