Refactor test to use fluent assertions

emmadesilva · emmadesilva · commit 282d1feb4aa6 · 2024-03-14T21:37:32.000+01:00
diff --git a/packages/framework/tests/Unit/Views/NavigationLinkViewTest.php b/packages/framework/tests/Unit/Views/NavigationLinkViewTest.php
@@ -68,8 +68,10 @@ public function testComponentDoesNotHaveCurrentAttributesWhenCurrentRouteDoesNot
 
     public function testComponentIsCurrentWhenCurrentRouteMatches()
     {
-        $this->mockRoute(Routes::get('index'));
-        $this->assertStringContainsString('current', $this->render(NavItem::forRoute(Routes::get('index'), 'Home')));
+        $this->mockCurrentPage('foo');
+        $this->testView()
+            ->assertSee('current')
+            ->assertAttributeIs('aria-current', 'page');
     }
 
     public function testComponentHasAriaCurrentWhenCurrentRouteMatches()

Original file line number	Diff line number	Diff line change
`@@ -68,8 +68,10 @@ public function testComponentDoesNotHaveCurrentAttributesWhenCurrentRouteDoesNot`
`68`	`68`
`69`	`69`	`public function testComponentIsCurrentWhenCurrentRouteMatches()`
`70`	`70`	`{`
`71`		`- $this->mockRoute(Routes::get('index'));`
`72`		`- $this->assertStringContainsString('current', $this->render(NavItem::forRoute(Routes::get('index'), 'Home')));`
	`71`	`+ $this->mockCurrentPage('foo');`
	`72`	`+ $this->testView()`
	`73`	`+ ->assertSee('current')`
	`74`	`+ ->assertAttributeIs('aria-current', 'page');`
`73`	`75`	`}`
`74`	`76`
`75`	`77`	`public function testComponentHasAriaCurrentWhenCurrentRouteMatches()`