datasift / storyplayer

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

Doc Comments +3 added lines, -3 removed lines

@@ -171,7 +171,7 @@ 
      * what should we use to validate a host adapter that is being added
      * to one of the hosts in this group?
      *
-     * @return \Storyplayer\TestEnvironments\HostAdapterValidator
+     * @return \Storyplayer\SPv3\TestEnvironments\HostAdapterValidator
      */
     public function getHostAdapterValidator()
     {
@@ -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 +4 added lines, -1 removed lines

@@ -74,6 +74,9 @@ 
      */
     protected $storySettings;
 
+    /**
+     * @param string $hostId
+     */
     public function __construct(TestEnvironment_GroupDefinition $parentGroup, $hostId)
     {
         $this->setParentGroup($parentGroup);
@@ -494,7 +497,7 @@ 
     /**
      * get access to the adapter for the group that we belong to
      *
-     * @return \Storyplayer\TestEnvironments\GroupAdapter
+     * @return \Storyplayer\SPv3\TestEnvironments\GroupAdapter
      */
     public function getGroupAdapter()
     {

src/php/Storyplayer/SPv3/TestEnvironments/Dsbuild/Adapter.php

Indentation +71 added lines, -71 removed lines

@@ -61,75 +61,75 @@
 
 class Dsbuild_Adapter implements ProvisioningAdapter
 {
-	public function __construct()
-	{
-		$this->setExecutePath(getcwd() . DIRECTORY_SEPARATOR . "dsbuild.sh");
-	}
-
-	// ==================================================================
-	//
-	// Support for executing the script
-	//
-	// ------------------------------------------------------------------
-
-	/**
-	 * where is the script that we are going to execute?
-	 *
-	 * @var string
-	 */
-	protected $executePath;
-
-	/**
-	 * which folder are we executing things in?
-	 *
-	 * @return string
-	 */
-	public function getExecuteDir()
-	{
-		return dirname($this->executePath);
-	}
-
-	/**
-	 * where is the script that we are going to execute?
-	 *
-	 * @return string
-	 */
-	public function getExecutePath()
-	{
-		return $this->executePath;
-	}
-
-	/**
-	 * tell me which script to execute
-	 *
-	 * @param string $path
-	 *        path to the dsbuild script
-	 */
-	public function setExecutePath($path)
-	{
-		$this->executePath = $path;
-
-		return $this;
-	}
-
-	// ==================================================================
-	//
-	// SPv3.0-style config support
-	//
-	// ------------------------------------------------------------------
-
-	public function getAsConfig()
-	{
-		// our return value
-		$retval = new BaseObject;
-
-		// this is who we are
-		$retval->engine = "dsbuild";
-
-		// this is what needs running
-		$retval->execute = $this->getExecutePath();
-
-		// all done
-		return $retval;
-	}
+    public function __construct()
+    {
+        $this->setExecutePath(getcwd() . DIRECTORY_SEPARATOR . "dsbuild.sh");
+    }
+
+    // ==================================================================
+    //
+    // Support for executing the script
+    //
+    // ------------------------------------------------------------------
+
+    /**
+     * where is the script that we are going to execute?
+     *
+     * @var string
+     */
+    protected $executePath;
+
+    /**
+     * which folder are we executing things in?
+     *
+     * @return string
+     */
+    public function getExecuteDir()
+    {
+        return dirname($this->executePath);
+    }
+
+    /**
+     * where is the script that we are going to execute?
+     *
+     * @return string
+     */
+    public function getExecutePath()
+    {
+        return $this->executePath;
+    }
+
+    /**
+     * tell me which script to execute
+     *
+     * @param string $path
+     *        path to the dsbuild script
+     */
+    public function setExecutePath($path)
+    {
+        $this->executePath = $path;
+
+        return $this;
+    }
+
+    // ==================================================================
+    //
+    // SPv3.0-style config support
+    //
+    // ------------------------------------------------------------------
+
+    public function getAsConfig()
+    {
+        // our return value
+        $retval = new BaseObject;
+
+        // this is who we are
+        $retval->engine = "dsbuild";
+
+        // this is what needs running
+        $retval->execute = $this->getExecutePath();
+
+        // all done
+        return $retval;
+    }
 }

src/php/Storyplayer/SPv3/TestEnvironments/OsAdapterValidator.php

Indentation +8 added lines, -8 removed lines

@@ -59,12 +59,12 @@
 
 interface OsAdapterValidator
 {
-	/**
-	 * validate an operating system adapter
-	 *
-	 * @param  OsAdapter $osAdapter
-	 *         the adapter to validate
-	 * @return void
-	 */
-	public function validate(OsAdapter $osAdapter);
+    /**
+     * validate an operating system adapter
+     *
+     * @param  OsAdapter $osAdapter
+     *         the adapter to validate
+     * @return void
+     */
+    public function validate(OsAdapter $osAdapter);
 }

src/php/Storyplayer/SPv3/TestEnvironments/Vagrant/E4xx/NoVagrantFile.php

Indentation +5 added lines, -5 removed lines

@@ -60,9 +60,9 @@
  */
 class Vagrant_E4xx_NoVagrantFile extends Vagrant_E4xx_Exception
 {
-	public function __construct($candidates)
-	{
-		$msg = "Cannot find your Vagrantfile. Expected to find it in one of these folders: " . implode(", ", $candidates);
-		parent::__construct(400, $msg, $msg);
-	}
+    public function __construct($candidates)
+    {
+        $msg = "Cannot find your Vagrantfile. Expected to find it in one of these folders: " . implode(", ", $candidates);
+        parent::__construct(400, $msg, $msg);
+    }
 }

src/php/Storyplayer/SPv3/TestEnvironments/Vagrant/GroupAdapter.php

Indentation +63 added lines, -63 removed lines

@@ -59,48 +59,48 @@ 
 
 class Vagrant_GroupAdapter implements GroupAdapter
 {
-	public function __construct()
-	{
-		$this->determineBaseFolder();
-	}
+    public function __construct()
+    {
+        $this->determineBaseFolder();
+    }
 
-	// ==================================================================
-	//
-	// Base folder support goes here
-	//
-	// ------------------------------------------------------------------
+    // ==================================================================
+    //
+    // Base folder support goes here
+    //
+    // ------------------------------------------------------------------
 
-	protected $baseFolder;
+    protected $baseFolder;
 
-	/**
-	 * automagically work out where our test environment's files and
-	 * such like are
-	 *
-	 * @return void
-	 */
-	protected function determineBaseFolder()
-	{
-		// where should we be looking?
-		//
-		// first match wins!
-		$candidates = [
-			dirname(debug_backtrace()[1]['file']),
-			getcwd()
-		];
+    /**
+     * automagically work out where our test environment's files and
+     * such like are
+     *
+     * @return void
+     */
+    protected function determineBaseFolder()
+    {
+        // where should we be looking?
+        //
+        // first match wins!
+        $candidates = [
+            dirname(debug_backtrace()[1]['file']),
+            getcwd()
+        ];
 
-		foreach ($candidates as $folder) {
-			if (file_exists($folder . '/Vagrantfile')) {
-				$this->baseFolder = str_replace(getcwd(), '.', $folder);
+        foreach ($candidates as $folder) {
+            if (file_exists($folder . '/Vagrantfile')) {
+                $this->baseFolder = str_replace(getcwd(), '.', $folder);
 
-				// all done
-				return;
-			}
-		}
+                // all done
+                return;
+            }
+        }
 
-		// if we get here, then we do not know where the Vagrantfile
-		// is, and it is time to bail
-		throw new Vagrant_E4xx_NoVagrantFile($candidates);
-	}
+        // if we get here, then we do not know where the Vagrantfile
+        // is, and it is time to bail
+        throw new Vagrant_E4xx_NoVagrantFile($candidates);
+    }
 
     /**
      * which folder should SPv3 be in when interacting with this group
@@ -110,33 +110,33 @@ 
      */
     public function getBaseFolder()
     {
-    	return $this->baseFolder;
+        return $this->baseFolder;
     }
 
-	// ==================================================================
-	//
-	// Host support goes here
-	//
-	// ------------------------------------------------------------------
+    // ==================================================================
+    //
+    // Host support goes here
+    //
+    // ------------------------------------------------------------------
 
-	/**
-	 * how do we validate any host adapters used by hosts in this group?
-	 *
-	 * @return HostAdapterValidator
-	 */
-	public function getHostAdapterValidator()
-	{
-		return new Vagrant_HostAdapterValidator($this);
-	}
+    /**
+     * how do we validate any host adapters used by hosts in this group?
+     *
+     * @return HostAdapterValidator
+     */
+    public function getHostAdapterValidator()
+    {
+        return new Vagrant_HostAdapterValidator($this);
+    }
 
-	// ==================================================================
-	//
-	// Stuff to support SPv3.0-style internals goes here
-	//
-	// Everything below here is technical debt, and the plan is to
-	// gradually phase it all out over several SPv3 releases
-	//
-	// ------------------------------------------------------------------
+    // ==================================================================
+    //
+    // Stuff to support SPv3.0-style internals goes here
+    //
+    // Everything below here is technical debt, and the plan is to
+    // gradually phase it all out over several SPv3 releases
+    //
+    // ------------------------------------------------------------------
 
     /**
      * what type of group are we?
@@ -146,8 +146,8 @@ 
      *
      * @return string
      */
-	public function getType()
-	{
-		return "LocalVagrantVms";
-	}
+    public function getType()
+    {
+        return "LocalVagrantVms";
+    }
 }

src/php/Storyplayer/SPv3/TestEnvironments/Vagrant/HostAdapterValidator.php

Indentation +15 added lines, -15 removed lines

@@ -59,20 +59,20 @@
 
 class Vagrant_HostAdapterValidator implements HostAdapterValidator
 {
-	/**
-	 * check to make sure that a host adapter plugin is compatible with
-	 * the group adapter
-	 *
-	 * @param  HostAdapter $hostAdapter
-	 *         the adapter to validate
-	 * @return boolean
-	 */
-	public function validate(HostAdapter $hostAdapter)
-	{
-		if (!$hostAdapter instanceof Vagrant_HostAdapter) {
-			return false;
-		}
+    /**
+     * check to make sure that a host adapter plugin is compatible with
+     * the group adapter
+     *
+     * @param  HostAdapter $hostAdapter
+     *         the adapter to validate
+     * @return boolean
+     */
+    public function validate(HostAdapter $hostAdapter)
+    {
+        if (!$hostAdapter instanceof Vagrant_HostAdapter) {
+            return false;
+        }
 
-		return true;
-	}
+        return true;
+    }
 }

src/php/Storyplayer/SPv3/TestEnvironments/Vagrant/VirtualboxHostAdapter.php

Indentation +14 added lines, -14 removed lines

@@ -62,18 +62,18 @@
 
 class Vagrant_VirtualboxHostAdapter extends Vagrant_HostAdapter
 {
-	/**
-	 * create an empty host definition for this kind of host
-	 *
-	 * @param  TestEnvironment_GroupDefinition $groupDef
-	 *         the group that we belong to
-	 * @param  string $hostId
-	 *         the alias for this host
-	 * @return object
-	 *         a host definition to populate
-	 */
-	public function newHostDefinition(TestEnvironment_GroupDefinition $groupDef, $hostId)
-	{
-		return new Vagrant_VirtualboxHostDefinition($groupDef, $hostId, $this);
-	}
+    /**
+     * create an empty host definition for this kind of host
+     *
+     * @param  TestEnvironment_GroupDefinition $groupDef
+     *         the group that we belong to
+     * @param  string $hostId
+     *         the alias for this host
+     * @return object
+     *         a host definition to populate
+     */
+    public function newHostDefinition(TestEnvironment_GroupDefinition $groupDef, $hostId)
+    {
+        return new Vagrant_VirtualboxHostDefinition($groupDef, $hostId, $this);
+    }
 }

src/php/Storyplayer/SPv3/TestEnvironments/HostAdapter.php

Indentation +11 added lines, -11 removed lines

@@ -61,15 +61,15 @@
 
 interface HostAdapter
 {
-	/**
-	 * create an empty host definition for this kind of host
-	 *
-	 * @param  TestEnvironment_GroupDefinition $groupDef
-	 *         the group that we belong to
-	 * @param  string $hostId
-	 *         the alias for this host
-	 * @return object
-	 *         a host definition to populate
-	 */
-	public function newHostDefinition(TestEnvironment_GroupDefinition $groupDef, $hostId);
+    /**
+     * create an empty host definition for this kind of host
+     *
+     * @param  TestEnvironment_GroupDefinition $groupDef
+     *         the group that we belong to
+     * @param  string $hostId
+     *         the alias for this host
+     * @return object
+     *         a host definition to populate
+     */
+    public function newHostDefinition(TestEnvironment_GroupDefinition $groupDef, $hostId);
 }