Coding standards

tests/FillPdfMergeTestCase.test

@@ -12,7 +12,7 @@ class FillPdfMergeTestCase extends ImageFieldTestCase {
   protected $nonPrivilegedUser;
 
   /**
-   *
+   * {@inheritdoc}
    */
   public static function getInfo() {
     // Note: getInfo() strings are not translated with t().
@@ -24,7 +24,7 @@ class FillPdfMergeTestCase extends ImageFieldTestCase {
   }
 
   /**
-   *
+   * {@inheritdoc}
    */
   public function setUp() {
     parent::setUp();
@@ -56,6 +56,7 @@ class FillPdfMergeTestCase extends ImageFieldTestCase {
   }
 
   /**
+   * Test PDF merging.
    *
    * @throws \FieldException
    */
@@ -292,7 +293,12 @@ class FillPdfMergeTestCase extends ImageFieldTestCase {
       }
     }
     $expected_filename = 'Token_Test_term_title.pdf';
-    $this->assertEqual($expected_filename, $filename, t('Filename of generated file (@actual) matches specified entity token pattern (@expected). Current page URL: @url', array('@actual' => $filename, '@expected' => $expected_filename, '@url' => $where_are_we)));
+    $args = array(
+      '@actual' => $filename,
+      '@expected' => $expected_filename,
+      '@url' => $where_are_we,
+    );
+    $this->assertEqual($expected_filename, $filename, t('Filename of generated file (@actual) matches specified entity token pattern (@expected). Current page URL: @url', $args));
 
     // Test that node access via entity access works.
     // Make a basic page.
@@ -517,7 +523,7 @@ class FillPdfMergeTestCase extends ImageFieldTestCase {
   }
 
   /**
-   *
+   * Tests PDF merging using a local service.
    */
   public function testLocalServicePdfMerging() {
     $this->configureLocalServiceBackend();
@@ -553,7 +559,7 @@ class FillPdfMergeTestCase extends ImageFieldTestCase {
   }
 
   /**
-   *
+   * Tests PDF merging.
    */
   protected function backendTest() {
     $this->uploadTestPdf();

tests/FillPdfTestHelper.test

 <?php
 
 /**
- *
+ * Helper functions for FillPDF testing.
  */
 trait FillPdfTestHelper {