Improved coverage for \PhpOffice\PhpWord\Element\Section (#665).

Author: Roman Syroeshko

src/PhpWord/Element/AbstractContainer.php

@@ -158,6 +158,8 @@ protected function addElement($elementName)
      * Get all elements
      *
      * @return array
+     *
+     * @codeCoverageIgnore
      */
     public function getElements()
     {

src/PhpWord/Element/Section.php

@@ -78,6 +78,8 @@ public function setStyle($style = null)
      * Get section style
      *
      * @return \PhpOffice\PhpWord\Style\Section
+     *
+     * @codeCoverageIgnore
      */
     public function getStyle()
     {
@@ -92,8 +94,6 @@ public function getStyle()
      * @param string $type
      *
      * @return Header
-     *
-     * @since 0.10.0
      */
     public function addHeader($type = Header::AUTO)
     {
@@ -108,8 +108,6 @@ public function addHeader($type = Header::AUTO)
      * @param string $type
      *
      * @return Footer
-     *
-     * @since 0.10.0
      */
     public function addFooter($type = Header::AUTO)
     {
@@ -120,6 +118,8 @@ public function addFooter($type = Header::AUTO)
      * Get header elements
      *
      * @return Header[]
+     *
+     * @codeCoverageIgnore
      */
     public function getHeaders()
     {
@@ -130,6 +130,8 @@ public function getHeaders()
      * Get footer elements
      *
      * @return Footer[]
+     *
+     * @codeCoverageIgnore
      */
     public function getFooters()
     {

tests/PhpWord/Element/SectionTest.php

@@ -17,55 +17,18 @@
 
 namespace PhpOffice\PhpWord\Element;
 
-use PhpOffice\PhpWord\Exception\Exception;
 use PhpOffice\PhpWord\PhpWord;
 use PhpOffice\PhpWord\Style;
 
 /**
- * Test class for PhpOffice\PhpWord\Element\Section
- *
+ * @covers \PhpOffice\PhpWord\Element\Section
+ * @coversDefaultClass \PhpOffice\PhpWord\Element\Section
  * @runTestsInSeparateProcesses
  */
 class SectionTest extends \PHPUnit_Framework_TestCase
 {
     /**
-     * Get style
-     */
-    public function testGetStyle()
-    {
-        $oSection = new Section(0);
-        $this->assertAttributeEquals($oSection->getStyle(), 'style', $oSection);
-    }
-
-    /**
-     * Get elements
-     */
-    public function testGetElements()
-    {
-        $oSection = new Section(0);
-        $this->assertAttributeEquals($oSection->getElements(), 'elements', $oSection);
-    }
-
-    /**
-     * Get footer
-     */
-    public function testGetFooters()
-    {
-        $oSection = new Section(0);
-        $this->assertAttributeEquals($oSection->getFooters(), 'footers', $oSection);
-    }
-
-    /**
-     * Get headers
-     */
-    public function testGetHeaders()
-    {
-        $oSection = new Section(0);
-        $this->assertAttributeEquals($oSection->getHeaders(), 'headers', $oSection);
-    }
-
-    /**
-     * Set settings
+     * @covers ::setStyle
      */
     public function testSetStyle()
     {
@@ -76,13 +39,12 @@ public function testSetStyle()
     }
 
     /**
-     * Add elements
+     * @coversNothing
      */
     public function testAddElements()
     {
         $objectSource = __DIR__ . '/../_files/documents/reader.docx';
         $imageSource = __DIR__ . '/../_files/images/PhpWord.png';
-        // $imageUrl = 'http://php.net//images/logos/php-med-trans-light.gif';
 
         $section = new Section(0);
         $section->setPhpWord(new PhpWord());
@@ -124,8 +86,7 @@ public function testAddElements()
     }
 
     /**
-     * Test add object exception
-     *
+     * @coversNothing
      * @expectedException \PhpOffice\PhpWord\Exception\InvalidObjectException
      */
     public function testAddObjectException()
@@ -137,6 +98,8 @@ public function testAddObjectException()
 
     /**
      * Add title with predefined style
+     *
+     * @coversNothing
      */
     public function testAddTitleWithStyle()
     {
@@ -150,7 +113,9 @@ public function testAddTitleWithStyle()
     }
 
     /**
-     * Add header footer
+     * @covers ::addHeader
+     * @covers ::addFooter
+     * @covers ::hasDifferentFirstPage
      */
     public function testAddHeaderFooter()
     {
@@ -165,7 +130,8 @@ public function testAddHeaderFooter()
     }
 
     /**
-     * Add header has different first page
+     * @covers ::addHeader
+     * @covers ::hasDifferentFirstPage
      */
     public function testHasDifferentFirstPage()
     {
@@ -176,9 +142,8 @@ public function testHasDifferentFirstPage()
     }
 
     /**
-     * Add header exception
-     *
-     * @expectedException Exception
+     * @covers ::addHeader
+     * @expectedException \Exception
      * @expectedExceptionMessage Invalid header/footer type.
      */
     public function testAddHeaderException()

tests/PhpWord/TemplateProcessorTest.php

@@ -142,7 +142,7 @@ final public function testXslStyleSheetCanNotBeAppliedOnFailureOfLoadingXmlFromT
     }
 
     /**
-     * @civers ::setValue
+     * @covers ::setValue
      * @covers ::cloneRow
      * @covers ::saveAs
      * @test