datasift / storyplayer

src/php/DataSift/Storyplayer/ConfigLib/ActiveConfig.php

Doc Comments +6 added lines

@@ -105,11 +105,17 @@
         $activeConfig->roles = $rolesTable->$testEnvName;
 	}
 
+	/**
+	 * @param Injectables $injectables
+	 */
 	public function mergeStoryplayerConfig($injectables, $spConf)
 	{
 		$this->mergeData('storyplayer', $spConf);
 	}
 
+	/**
+	 * @param Injectables $injectables
+	 */
 	public function mergeSystemUnderTestConfig($injectables, SystemUnderTestConfig $sutConfig = null)
 	{
         // do we have a system under test?

src/php/DataSift/Storyplayer/DefinitionLib/TestEnvironment/GroupDefinition.php

Doc Comments +2 added lines, -2 removed lines

@@ -220,7 +220,7 @@ 
     /**
      * how should we provision this environment?
      *
-     * @return array<ProvisioningAdapters>
+     * @return ProvisioningAdapter[]
      */
     public function getProvisioningAdapters()
     {
@@ -265,7 +265,7 @@ 
     /**
      * what hosts exist in this group?
      *
-     * @return array<TestEnvironment_HostDefinition>
+     * @return TestEnvironmentHost[]
      */
     public function getHosts()
     {

src/php/DataSift/Storyplayer/DefinitionLib/TestEnvironment/HostDefinition.php

Doc Comments +3 added lines

@@ -74,6 +74,9 @@
      */
     protected $storySettings;
 
+    /**
+     * @param string $hostId
+     */
     public function __construct(TestEnvironment_GroupDefinition $parentGroup, $hostId)
     {
         $this->setParentGroup($parentGroup);

src/php/DataSift/Storyplayer/functions.php

Doc Comments +2 added lines, -2 removed lines

@@ -53,7 +53,7 @@ 
  * this function avoids reset()ing the array, so it will not mess with
  * any iteration that you may currently be part-way through
  *
- * @param  array $arrayToSearch
+ * @param  Storyplayer\TestEnvironments\ProvisioningAdapter[] $arrayToSearch
  *         the array to get the first element of
  * @return mixed
  *         the first element of $array, or NULL if the array is empty
@@ -95,7 +95,7 @@ 
 /**
  * Create a new test environment object
  *
- * @return TestEnvironment
+ * @return TestEnvironment_Definition
  *         the test environment object to use in the script
  */
 function newTestEnvironment()

src/php/DataSift/Storyplayer/Reports/TapReport.php

Doc Comments -1 removed lines

@@ -177,7 +177,6 @@
     }
 
     /**
-     * @param integer $level
      * @param string $msg
      * @return void
      */

src/php/Prose/BaseRuntimeTable.php

Doc Comments +1 added lines, -1 removed lines

@@ -59,7 +59,7 @@
      * __construct
      *
      * @param StoryTeller $st The StoryTeller object
-     * @param array $args Any arguments to be used in this Prose module
+     * @param string[] $args Any arguments to be used in this Prose module
      *
      * @return parent::__construct
      */

src/php/Prose/FromEc2Instance.php

Doc Comments +3 added lines

@@ -99,6 +99,9 @@
         return array();
     }
 
+    /**
+     * @return string
+     */
     public function getPublicDnsName()
     {
         // make sure we have a host to work with

src/php/Prose/FromPDOStatement.php

Doc Comments +4 added lines

@@ -62,6 +62,10 @@
  */
 class FromPDOStatement extends Prose
 {
+    /**
+     * @param \DataSift\Storyplayer\PlayerLib\StoryTeller $st
+     * @param PDOStatement[] $args
+     */
     public function __construct($st, $args)
     {
         // call our parent first

src/php/Prose/FromTestEnvironment.php

Doc Comments +4 added lines, -1 removed lines

@@ -125,6 +125,9 @@ 
         return $value;
     }
 
+    /**
+     * @param string $setting
+     */
     public function getModuleSetting($setting)
     {
         // what are we doing?
@@ -154,7 +157,7 @@ 
      * We've standardised on 'getConfig()' as the documented name for this
      * functionality across all three of the config-related modules.
      *
-     * @return object
+     * @return string
      */
     public function getAllSettings()
     {