Migrate PHPUnit syntax

Author: joelambert

tests/DocumentParsers/MarkdownDocumentParserTest.php

@@ -36,7 +36,7 @@ public function indents_are_not_treated_as_code_blocks()
 
         $outputDoc = $parser->parse($doc);
 
-        $this->assertNotContains('<pre><code>', $outputDoc->content());
-        $this->assertNotContains('</code></pre>', $outputDoc->content());
+        $this->assertStringNotContainsString('<pre><code>', $outputDoc->content());
+        $this->assertStringNotContainsString('</code></pre>', $outputDoc->content());
     }
 }

tests/FileSystemDocumentProviderTest.php

@@ -174,12 +174,10 @@ public function can_get_document_that_is_numerically_ordered()
         $this->assertSame('01-building.md', $pattern2->content());
     }
 
-    /**
-     * @test
-     * @expectedException Rareloop\Primer\Exceptions\DocumentNotFoundException
-     */
+    /** @test */
     public function getDocument_throws_exception_if_document_does_not_exist()
     {
+        $this->expectException(\Rareloop\Primer\Exceptions\DocumentNotFoundException::class);
         vfsStream::setup();
         $root = vfsStream::create([
             'foo' => [
@@ -202,12 +200,11 @@ public function allDocumentIds_does_not_fall_over_if_no_paths_are_provided()
         $this->assertSame([], $dataProvider->allDocumentIds());
     }
 
-    /**
-     * @test
-     * @expectedException Rareloop\Primer\Exceptions\DocumentNotFoundException
-     */
+    /** @test */
     public function getDocument_does_not_fall_over_if_no_paths_are_provided()
     {
+        $this->expectException(\Rareloop\Primer\Exceptions\DocumentNotFoundException::class);
+
         $dataProvider = new FileSystemDocumentProvider([], 'md');
 
         $dataProvider->getDocument('not/found');

tests/FileSystemPatternProviderTest.php

@@ -109,12 +109,11 @@ public function can_get_template_contents()
         $this->assertSame('<footer>Hello World</footer>', $dataProvider->getPatternTemplate('components/misc/footer'));
     }
 
-    /**
-     * @test
-     * @expectedException Rareloop\Primer\Exceptions\PatternNotFoundException
-     */
+    /** @test */
     public function can_not_get_template_contents_if_partial_id_provided()
     {
+        $this->expectException(\Rareloop\Primer\Exceptions\PatternNotFoundException::class);
+
         vfsStream::setup();
         $root = vfsStream::create([
             'foo' => [
@@ -215,12 +214,11 @@ public function can_check_if_state_data_exists_for_a_pattern_as_php()
         $this->assertFalse($dataProvider->patternHasState('components/misc/header', 'unexpected'));
     }
 
-    /**
-     * @test
-     * @expectedException Rareloop\Primer\Exceptions\PatternNotFoundException
-     */
+    /** @test */
     public function patternHasState_throws_exception_if_pattern_does_not_exist()
     {
+        $this->expectException(\Rareloop\Primer\Exceptions\PatternNotFoundException::class);
+
         vfsStream::setup();
         $root = vfsStream::create([
             'foo' => [
@@ -535,12 +533,11 @@ public function getPatternStateData_returns_empty_array_when_no_data_is_availabl
         $this->assertSame([], $data);
     }
 
-    /**
-     * @test
-     * @expectedException Rareloop\Primer\Exceptions\PatternNotFoundException
-     */
+    /** @test */
     public function getPatternStateData_throws_exception_when_pattern_is_invalid()
     {
+        $this->expectException(\Rareloop\Primer\Exceptions\PatternNotFoundException::class);
+
         vfsStream::setup();
         $root = vfsStream::create([
             'foo' => [
@@ -592,12 +589,11 @@ public function getPatternStateData_does_not_fall_over_if_no_paths_are_provided(
         $this->assertSame([], $dataProvider->getPatternStateData('not/found', 'state'));
     }
 
-    /**
-     * @test
-     * @expectedException Rareloop\Primer\Exceptions\PatternNotFoundException
-     */
+    /** @test */
     public function getPattern_does_not_fall_over_if_no_paths_are_provided()
     {
+        $this->expectException(\Rareloop\Primer\Exceptions\PatternNotFoundException::class);
+
         $dataProvider = new FileSystemPatternProvider([], 'md');
 
         $dataProvider->getPattern('not/found');

tests/MenuTest.php

@@ -56,12 +56,11 @@ public function can_test_if_a_section_exists()
         $this->assertFalse($menu->hasSection('templates'));
     }
 
-    /**
-     * @test
-     * @expectedException Rareloop\Primer\Exceptions\SectionNotFoundException
-     */
+    /** @test */
     public function getSection_throws_an_exception_if_section_is_not_found()
     {
+        $this->expectException(\Rareloop\Primer\Exceptions\SectionNotFoundException::class);
+
         $menu = new Menu();
         $menu->getSection('patterns');
     }

tests/PrimerTest.php

@@ -355,12 +355,11 @@ public function correct_patterns_are_rendered_when_one_name_is_contained_in_anot
         $this->assertSame('<p>Testing123</p>', $primer->renderPatterns('components/misc/header'));
     }
 
-    /**
-     * @test
-     * @expectedException Rareloop\Primer\Exceptions\PatternNotFoundException
-     */
+    /** @test */
     public function renderPatterns_throws_exception_if_no_matching_patterns_are_found()
     {
+        $this->expectException(\Rareloop\Primer\Exceptions\PatternNotFoundException::class);
+
         $patternIds = [
             'components/misc/header',
             'components/misc/footer',

tests/TreeTest.php

@@ -343,12 +343,11 @@ public function setCurrent_ignores_leading_and_trailing_slashes()
         ], $tree->toArray());
     }
 
-    /**
-     * @test
-     * @expectedException Rareloop\Primer\Exceptions\TreeNodeNotFoundException
-     */
+    /** @test  */
     public function setCurrent_throws_exception_when_node_not_found()
     {
+        $this->expectException(\Rareloop\Primer\Exceptions\TreeNodeNotFoundException::class);
+
         $tree = new Tree([
             'components/misc/footer',
             'components/misc/header',