diff --git a/tests/external_node_update.test b/tests/external_node_update.test
index 21f919e..c78ddd9 100644
--- a/tests/external_node_update.test
+++ b/tests/external_node_update.test
@@ -35,7 +35,7 @@ class WorkbenchModerationExternalNodeUpdateTestCase extends WorkbenchModerationT
   /**
    * {@inheritdoc}
    */
-  function setUp($modules = array()) {
+  public function setUp($modules = array()) {
     // Enable a test module that will publish and unpublish nodes for us.
     parent::setUp(array_merge($modules, array('workbench_moderation_test')));
 
@@ -45,7 +45,7 @@ class WorkbenchModerationExternalNodeUpdateTestCase extends WorkbenchModerationT
   /**
    * Tests if nodes can be moderated by third party modules.
    */
-  function testNodeSave() {
+  public function testNodeSave() {
     // Create a brand new unpublished node programmatically.
     $settings = array(
       'title' => $this->randomName(),
@@ -112,17 +112,21 @@ class WorkbenchModerationExternalNodeUpdateTestCase extends WorkbenchModerationT
    *   An associative array containing expected moderation status values.
    * @param string $status
    *   Which status to assert. Can be either 'current' or 'published'.
-   * @param $message
+   * @param string $message
    *   The message to display along with the assertion.
    *
-   * @return
+   * @return bool
    *   TRUE if the assertion succeeded, FALSE otherwise.
    */
   public function assertModerationStatus(array $expected, $status = 'current', $message = '') {
     $record = $this->getModerationRecord($status);
     $success = TRUE;
     foreach ($expected as $key => $value) {
-      $success |= $this->assertEqual($value, $record[$key], format_string('Found value %value for %key, expected %expected.', array('%key' => $key, '%value' => $record[$key], '%expected' => $value)));
+      $success |= $this->assertEqual($value, $record[$key], format_string('Found value %value for %key, expected %expected.', array(
+        '%key' => $key,
+        '%value' => $record[$key],
+        '%expected' => $value,
+      )));
     }
 
     return $this->assertTrue($success, $message);
@@ -131,10 +135,10 @@ class WorkbenchModerationExternalNodeUpdateTestCase extends WorkbenchModerationT
   /**
    * Checks if the node is not marked as 'published' in the node history table.
    *
-   * @param $message
+   * @param string $message
    *   The message to display along with the assertion.
    *
-   * @return
+   * @return bool
    *   TRUE if the assertion succeeded, FALSE otherwise.
    */
   public function assertNoPublishedRecord($message = '') {
@@ -150,7 +154,8 @@ class WorkbenchModerationExternalNodeUpdateTestCase extends WorkbenchModerationT
    * @param string $message
    *   The message to display along with the assertion.
    *
-   * @return type
+   * @return bool
+   *   TRUE if the assertion succeeded, FALSE otherwise.
    */
   public function assertPublicationState($expected, $message = '') {
     return $this->assertEqual($expected, $this->node->status, $message);