gravityview / GravityView

vendor/myclabs/php-enum/src/Enum.php

Indentation +265 added lines, -265 removed lines

@@ -21,298 +21,298 @@
  */
 abstract class Enum implements \JsonSerializable
 {
-    /**
-     * Enum value
-     *
-     * @var mixed
-     * @psalm-var T
-     */
-    protected $value;
+	/**
+	 * Enum value
+	 *
+	 * @var mixed
+	 * @psalm-var T
+	 */
+	protected $value;
 
-    /**
-     * Enum key, the constant name
-     *
-     * @var string
-     */
-    private $key;
+	/**
+	 * Enum key, the constant name
+	 *
+	 * @var string
+	 */
+	private $key;
 
-    /**
-     * Store existing constants in a static cache per object.
-     *
-     *
-     * @var array
-     * @psalm-var array<class-string, array<string, mixed>>
-     */
-    protected static $cache = [];
+	/**
+	 * Store existing constants in a static cache per object.
+	 *
+	 *
+	 * @var array
+	 * @psalm-var array<class-string, array<string, mixed>>
+	 */
+	protected static $cache = [];
 
-    /**
-     * Cache of instances of the Enum class
-     *
-     * @var array
-     * @psalm-var array<class-string, array<string, static>>
-     */
-    protected static $instances = [];
+	/**
+	 * Cache of instances of the Enum class
+	 *
+	 * @var array
+	 * @psalm-var array<class-string, array<string, static>>
+	 */
+	protected static $instances = [];
 
-    /**
-     * Creates a new value of some type
-     *
-     * @psalm-pure
-     * @param mixed $value
-     *
-     * @psalm-param T $value
-     * @throws \UnexpectedValueException if incompatible type is given.
-     */
-    public function __construct($value)
-    {
-        if ($value instanceof static) {
-           /** @psalm-var T */
-            $value = $value->getValue();
-        }
+	/**
+	 * Creates a new value of some type
+	 *
+	 * @psalm-pure
+	 * @param mixed $value
+	 *
+	 * @psalm-param T $value
+	 * @throws \UnexpectedValueException if incompatible type is given.
+	 */
+	public function __construct($value)
+	{
+		if ($value instanceof static) {
+		   /** @psalm-var T */
+			$value = $value->getValue();
+		}
 
-        /** @psalm-suppress ImplicitToStringCast assertValidValueReturningKey returns always a string but psalm has currently an issue here */
-        $this->key = static::assertValidValueReturningKey($value);
+		/** @psalm-suppress ImplicitToStringCast assertValidValueReturningKey returns always a string but psalm has currently an issue here */
+		$this->key = static::assertValidValueReturningKey($value);
 
-        /** @psalm-var T */
-        $this->value = $value;
-    }
+		/** @psalm-var T */
+		$this->value = $value;
+	}
 
-    /**
-     * This method exists only for the compatibility reason when deserializing a previously serialized version
-     * that didn't had the key property
-     */
-    public function __wakeup()
-    {
-        /** @psalm-suppress DocblockTypeContradiction key can be null when deserializing an enum without the key */
-        if ($this->key === null) {
-            /**
-             * @psalm-suppress InaccessibleProperty key is not readonly as marked by psalm
-             * @psalm-suppress PossiblyFalsePropertyAssignmentValue deserializing a case that was removed
-             */
-            $this->key = static::search($this->value);
-        }
-    }
+	/**
+	 * This method exists only for the compatibility reason when deserializing a previously serialized version
+	 * that didn't had the key property
+	 */
+	public function __wakeup()
+	{
+		/** @psalm-suppress DocblockTypeContradiction key can be null when deserializing an enum without the key */
+		if ($this->key === null) {
+			/**
+			 * @psalm-suppress InaccessibleProperty key is not readonly as marked by psalm
+			 * @psalm-suppress PossiblyFalsePropertyAssignmentValue deserializing a case that was removed
+			 */
+			$this->key = static::search($this->value);
+		}
+	}
 
-    /**
-     * @param mixed $value
-     * @return static
-     */
-    public static function from($value): self
-    {
-        $key = static::assertValidValueReturningKey($value);
+	/**
+	 * @param mixed $value
+	 * @return static
+	 */
+	public static function from($value): self
+	{
+		$key = static::assertValidValueReturningKey($value);
 
-        return self::__callStatic($key, []);
-    }
+		return self::__callStatic($key, []);
+	}
 
-    /**
-     * @psalm-pure
-     * @return mixed
-     * @psalm-return T
-     */
-    public function getValue()
-    {
-        return $this->value;
-    }
+	/**
+	 * @psalm-pure
+	 * @return mixed
+	 * @psalm-return T
+	 */
+	public function getValue()
+	{
+		return $this->value;
+	}
 
-    /**
-     * Returns the enum key (i.e. the constant name).
-     *
-     * @psalm-pure
-     * @return string
-     */
-    public function getKey()
-    {
-        return $this->key;
-    }
+	/**
+	 * Returns the enum key (i.e. the constant name).
+	 *
+	 * @psalm-pure
+	 * @return string
+	 */
+	public function getKey()
+	{
+		return $this->key;
+	}
 
-    /**
-     * @psalm-pure
-     * @psalm-suppress InvalidCast
-     * @return string
-     */
-    public function __toString()
-    {
-        return (string)$this->value;
-    }
+	/**
+	 * @psalm-pure
+	 * @psalm-suppress InvalidCast
+	 * @return string
+	 */
+	public function __toString()
+	{
+		return (string)$this->value;
+	}
 
-    /**
-     * Determines if Enum should be considered equal with the variable passed as a parameter.
-     * Returns false if an argument is an object of different class or not an object.
-     *
-     * This method is final, for more information read https://github.com/myclabs/php-enum/issues/4
-     *
-     * @psalm-pure
-     * @psalm-param mixed $variable
-     * @return bool
-     */
-    final public function equals($variable = null): bool
-    {
-        return $variable instanceof self
-            && $this->getValue() === $variable->getValue()
-            && static::class === \get_class($variable);
-    }
+	/**
+	 * Determines if Enum should be considered equal with the variable passed as a parameter.
+	 * Returns false if an argument is an object of different class or not an object.
+	 *
+	 * This method is final, for more information read https://github.com/myclabs/php-enum/issues/4
+	 *
+	 * @psalm-pure
+	 * @psalm-param mixed $variable
+	 * @return bool
+	 */
+	final public function equals($variable = null): bool
+	{
+		return $variable instanceof self
+			&& $this->getValue() === $variable->getValue()
+			&& static::class === \get_class($variable);
+	}
 
-    /**
-     * Returns the names (keys) of all constants in the Enum class
-     *
-     * @psalm-pure
-     * @psalm-return list<string>
-     * @return array
-     */
-    public static function keys()
-    {
-        return \array_keys(static::toArray());
-    }
+	/**
+	 * Returns the names (keys) of all constants in the Enum class
+	 *
+	 * @psalm-pure
+	 * @psalm-return list<string>
+	 * @return array
+	 */
+	public static function keys()
+	{
+		return \array_keys(static::toArray());
+	}
 
-    /**
-     * Returns instances of the Enum class of all Enum constants
-     *
-     * @psalm-pure
-     * @psalm-return array<string, static>
-     * @return static[] Constant name in key, Enum instance in value
-     */
-    public static function values()
-    {
-        $values = array();
+	/**
+	 * Returns instances of the Enum class of all Enum constants
+	 *
+	 * @psalm-pure
+	 * @psalm-return array<string, static>
+	 * @return static[] Constant name in key, Enum instance in value
+	 */
+	public static function values()
+	{
+		$values = array();
 
-        /** @psalm-var T $value */
-        foreach (static::toArray() as $key => $value) {
-            $values[$key] = new static($value);
-        }
+		/** @psalm-var T $value */
+		foreach (static::toArray() as $key => $value) {
+			$values[$key] = new static($value);
+		}
 
-        return $values;
-    }
+		return $values;
+	}
 
-    /**
-     * Returns all possible values as an array
-     *
-     * @psalm-pure
-     * @psalm-suppress ImpureStaticProperty
-     *
-     * @psalm-return array<string, mixed>
-     * @return array Constant name in key, constant value in value
-     */
-    public static function toArray()
-    {
-        $class = static::class;
+	/**
+	 * Returns all possible values as an array
+	 *
+	 * @psalm-pure
+	 * @psalm-suppress ImpureStaticProperty
+	 *
+	 * @psalm-return array<string, mixed>
+	 * @return array Constant name in key, constant value in value
+	 */
+	public static function toArray()
+	{
+		$class = static::class;
 
-        if (!isset(static::$cache[$class])) {
-            /** @psalm-suppress ImpureMethodCall this reflection API usage has no side-effects here */
-            $reflection            = new \ReflectionClass($class);
-            /** @psalm-suppress ImpureMethodCall this reflection API usage has no side-effects here */
-            static::$cache[$class] = $reflection->getConstants();
-        }
+		if (!isset(static::$cache[$class])) {
+			/** @psalm-suppress ImpureMethodCall this reflection API usage has no side-effects here */
+			$reflection            = new \ReflectionClass($class);
+			/** @psalm-suppress ImpureMethodCall this reflection API usage has no side-effects here */
+			static::$cache[$class] = $reflection->getConstants();
+		}
 
-        return static::$cache[$class];
-    }
+		return static::$cache[$class];
+	}
 
-    /**
-     * Check if is valid enum value
-     *
-     * @param $value
-     * @psalm-param mixed $value
-     * @psalm-pure
-     * @psalm-assert-if-true T $value
-     * @return bool
-     */
-    public static function isValid($value)
-    {
-        return \in_array($value, static::toArray(), true);
-    }
+	/**
+	 * Check if is valid enum value
+	 *
+	 * @param $value
+	 * @psalm-param mixed $value
+	 * @psalm-pure
+	 * @psalm-assert-if-true T $value
+	 * @return bool
+	 */
+	public static function isValid($value)
+	{
+		return \in_array($value, static::toArray(), true);
+	}
 
-    /**
-     * Asserts valid enum value
-     *
-     * @psalm-pure
-     * @psalm-assert T $value
-     * @param mixed $value
-     */
-    public static function assertValidValue($value): void
-    {
-        self::assertValidValueReturningKey($value);
-    }
+	/**
+	 * Asserts valid enum value
+	 *
+	 * @psalm-pure
+	 * @psalm-assert T $value
+	 * @param mixed $value
+	 */
+	public static function assertValidValue($value): void
+	{
+		self::assertValidValueReturningKey($value);
+	}
 
-    /**
-     * Asserts valid enum value
-     *
-     * @psalm-pure
-     * @psalm-assert T $value
-     * @param mixed $value
-     * @return string
-     */
-    private static function assertValidValueReturningKey($value): string
-    {
-        if (false === ($key = static::search($value))) {
-            throw new \UnexpectedValueException("Value '$value' is not part of the enum " . static::class);
-        }
+	/**
+	 * Asserts valid enum value
+	 *
+	 * @psalm-pure
+	 * @psalm-assert T $value
+	 * @param mixed $value
+	 * @return string
+	 */
+	private static function assertValidValueReturningKey($value): string
+	{
+		if (false === ($key = static::search($value))) {
+			throw new \UnexpectedValueException("Value '$value' is not part of the enum " . static::class);
+		}
 
-        return $key;
-    }
+		return $key;
+	}
 
-    /**
-     * Check if is valid enum key
-     *
-     * @param $key
-     * @psalm-param string $key
-     * @psalm-pure
-     * @return bool
-     */
-    public static function isValidKey($key)
-    {
-        $array = static::toArray();
+	/**
+	 * Check if is valid enum key
+	 *
+	 * @param $key
+	 * @psalm-param string $key
+	 * @psalm-pure
+	 * @return bool
+	 */
+	public static function isValidKey($key)
+	{
+		$array = static::toArray();
 
-        return isset($array[$key]) || \array_key_exists($key, $array);
-    }
+		return isset($array[$key]) || \array_key_exists($key, $array);
+	}
 
-    /**
-     * Return key for value
-     *
-     * @param mixed $value
-     *
-     * @psalm-param mixed $value
-     * @psalm-pure
-     * @return string|false
-     */
-    public static function search($value)
-    {
-        return \array_search($value, static::toArray(), true);
-    }
+	/**
+	 * Return key for value
+	 *
+	 * @param mixed $value
+	 *
+	 * @psalm-param mixed $value
+	 * @psalm-pure
+	 * @return string|false
+	 */
+	public static function search($value)
+	{
+		return \array_search($value, static::toArray(), true);
+	}
 
-    /**
-     * Returns a value when called statically like so: MyEnum::SOME_VALUE() given SOME_VALUE is a class constant
-     *
-     * @param string $name
-     * @param array  $arguments
-     *
-     * @return static
-     * @throws \BadMethodCallException
-     *
-     * @psalm-pure
-     */
-    public static function __callStatic($name, $arguments)
-    {
-        $class = static::class;
-        if (!isset(self::$instances[$class][$name])) {
-            $array = static::toArray();
-            if (!isset($array[$name]) && !\array_key_exists($name, $array)) {
-                $message = "No static method or enum constant '$name' in class " . static::class;
-                throw new \BadMethodCallException($message);
-            }
-            return self::$instances[$class][$name] = new static($array[$name]);
-        }
-        return clone self::$instances[$class][$name];
-    }
+	/**
+	 * Returns a value when called statically like so: MyEnum::SOME_VALUE() given SOME_VALUE is a class constant
+	 *
+	 * @param string $name
+	 * @param array  $arguments
+	 *
+	 * @return static
+	 * @throws \BadMethodCallException
+	 *
+	 * @psalm-pure
+	 */
+	public static function __callStatic($name, $arguments)
+	{
+		$class = static::class;
+		if (!isset(self::$instances[$class][$name])) {
+			$array = static::toArray();
+			if (!isset($array[$name]) && !\array_key_exists($name, $array)) {
+				$message = "No static method or enum constant '$name' in class " . static::class;
+				throw new \BadMethodCallException($message);
+			}
+			return self::$instances[$class][$name] = new static($array[$name]);
+		}
+		return clone self::$instances[$class][$name];
+	}
 
-    /**
-     * Specify data which should be serialized to JSON. This method returns data that can be serialized by json_encode()
-     * natively.
-     *
-     * @return mixed
-     * @link http://php.net/manual/en/jsonserializable.jsonserialize.php
-     * @psalm-pure
-     */
-    #[\ReturnTypeWillChange]
-    public function jsonSerialize()
-    {
-        return $this->getValue();
-    }
+	/**
+	 * Specify data which should be serialized to JSON. This method returns data that can be serialized by json_encode()
+	 * natively.
+	 *
+	 * @return mixed
+	 * @link http://php.net/manual/en/jsonserializable.jsonserialize.php
+	 * @psalm-pure
+	 */
+	#[\ReturnTypeWillChange]
+	public function jsonSerialize()
+	{
+		return $this->getValue();
+	}
 }

Spacing +36 added lines, -36 removed lines

@@ -43,7 +43,7 @@ 
      * @var array
      * @psalm-var array<class-string, array<string, mixed>>
      */
-    protected static $cache = [];
+    protected static $cache = [ ];
 
     /**
      * Cache of instances of the Enum class
@@ -51,7 +51,7 @@ 
      * @var array
      * @psalm-var array<class-string, array<string, static>>
      */
-    protected static $instances = [];
+    protected static $instances = [ ];
 
     /**
      * Creates a new value of some type
@@ -62,15 +62,15 @@ 
      * @psalm-param T $value
      * @throws \UnexpectedValueException if incompatible type is given.
      */
-    public function __construct($value)
+    public function __construct( $value )
     {
-        if ($value instanceof static) {
+        if ( $value instanceof static ) {
            /** @psalm-var T */
             $value = $value->getValue();
         }
 
         /** @psalm-suppress ImplicitToStringCast assertValidValueReturningKey returns always a string but psalm has currently an issue here */
-        $this->key = static::assertValidValueReturningKey($value);
+        $this->key = static::assertValidValueReturningKey( $value );
 
         /** @psalm-var T */
         $this->value = $value;
@@ -83,12 +83,12 @@ 
     public function __wakeup()
     {
         /** @psalm-suppress DocblockTypeContradiction key can be null when deserializing an enum without the key */
-        if ($this->key === null) {
+        if ( $this->key === null ) {
             /**
              * @psalm-suppress InaccessibleProperty key is not readonly as marked by psalm
              * @psalm-suppress PossiblyFalsePropertyAssignmentValue deserializing a case that was removed
              */
-            $this->key = static::search($this->value);
+            $this->key = static::search( $this->value );
         }
     }
 
@@ -96,11 +96,11 @@ 
      * @param mixed $value
      * @return static
      */
-    public static function from($value): self
+    public static function from( $value ): self
     {
-        $key = static::assertValidValueReturningKey($value);
+        $key = static::assertValidValueReturningKey( $value );
 
-        return self::__callStatic($key, []);
+        return self::__callStatic( $key, [ ] );
     }
 
     /**
@@ -144,11 +144,11 @@ 
      * @psalm-param mixed $variable
      * @return bool
      */
-    final public function equals($variable = null): bool
+    final public function equals( $variable = null ): bool
     {
         return $variable instanceof self
             && $this->getValue() === $variable->getValue()
-            && static::class === \get_class($variable);
+            && static::class === \get_class( $variable );
     }
 
     /**
@@ -160,7 +160,7 @@ 
      */
     public static function keys()
     {
-        return \array_keys(static::toArray());
+        return \array_keys( static::toArray() );
     }
 
     /**
@@ -175,8 +175,8 @@ 
         $values = array();
 
         /** @psalm-var T $value */
-        foreach (static::toArray() as $key => $value) {
-            $values[$key] = new static($value);
+        foreach ( static::toArray() as $key => $value ) {
+            $values[ $key ] = new static( $value );
         }
 
         return $values;
@@ -195,14 +195,14 @@ 
     {
         $class = static::class;
 
-        if (!isset(static::$cache[$class])) {
+        if ( ! isset( static::$cache[ $class ] ) ) {
             /** @psalm-suppress ImpureMethodCall this reflection API usage has no side-effects here */
-            $reflection            = new \ReflectionClass($class);
+            $reflection            = new \ReflectionClass( $class );
             /** @psalm-suppress ImpureMethodCall this reflection API usage has no side-effects here */
-            static::$cache[$class] = $reflection->getConstants();
+            static::$cache[ $class ] = $reflection->getConstants();
         }
 
-        return static::$cache[$class];
+        return static::$cache[ $class ];
     }
 
     /**
@@ -214,9 +214,9 @@ 
      * @psalm-assert-if-true T $value
      * @return bool
      */
-    public static function isValid($value)
+    public static function isValid( $value )
     {
-        return \in_array($value, static::toArray(), true);
+        return \in_array( $value, static::toArray(), true );
     }
 
     /**
@@ -226,9 +226,9 @@ 
      * @psalm-assert T $value
      * @param mixed $value
      */
-    public static function assertValidValue($value): void
+    public static function assertValidValue( $value ): void
     {
-        self::assertValidValueReturningKey($value);
+        self::assertValidValueReturningKey( $value );
     }
 
     /**
@@ -239,10 +239,10 @@ 
      * @param mixed $value
      * @return string
      */
-    private static function assertValidValueReturningKey($value): string
+    private static function assertValidValueReturningKey( $value ): string
     {
-        if (false === ($key = static::search($value))) {
-            throw new \UnexpectedValueException("Value '$value' is not part of the enum " . static::class);
+        if ( false === ( $key = static::search( $value ) ) ) {
+            throw new \UnexpectedValueException( "Value '$value' is not part of the enum " . static::class );
         }
 
         return $key;
@@ -256,11 +256,11 @@ 
      * @psalm-pure
      * @return bool
      */
-    public static function isValidKey($key)
+    public static function isValidKey( $key )
     {
         $array = static::toArray();
 
-        return isset($array[$key]) || \array_key_exists($key, $array);
+        return isset( $array[ $key ] ) || \array_key_exists( $key, $array );
     }
 
     /**
@@ -272,9 +272,9 @@ 
      * @psalm-pure
      * @return string|false
      */
-    public static function search($value)
+    public static function search( $value )
     {
-        return \array_search($value, static::toArray(), true);
+        return \array_search( $value, static::toArray(), true );
     }
 
     /**
@@ -288,18 +288,18 @@ 
      *
      * @psalm-pure
      */
-    public static function __callStatic($name, $arguments)
+    public static function __callStatic( $name, $arguments )
     {
         $class = static::class;
-        if (!isset(self::$instances[$class][$name])) {
+        if ( ! isset( self::$instances[ $class ][ $name ] ) ) {
             $array = static::toArray();
-            if (!isset($array[$name]) && !\array_key_exists($name, $array)) {
+            if ( ! isset( $array[ $name ] ) && ! \array_key_exists( $name, $array ) ) {
                 $message = "No static method or enum constant '$name' in class " . static::class;
-                throw new \BadMethodCallException($message);
+                throw new \BadMethodCallException( $message );
             }
-            return self::$instances[$class][$name] = new static($array[$name]);
+            return self::$instances[ $class ][ $name ] = new static( $array[ $name ] );
         }
-        return clone self::$instances[$class][$name];
+        return clone self::$instances[ $class ][ $name ];
     }
 
     /**

Braces +14 added lines, -28 removed lines

@@ -19,8 +19,7 @@ 
  * @psalm-immutable
  * @psalm-consistent-constructor
  */
-abstract class Enum implements \JsonSerializable
-{
+abstract class Enum implements \JsonSerializable {
     /**
      * Enum value
      *
@@ -62,8 +61,7 @@ 
      * @psalm-param T $value
      * @throws \UnexpectedValueException if incompatible type is given.
      */
-    public function __construct($value)
-    {
+    public function __construct($value) {
         if ($value instanceof static) {
            /** @psalm-var T */
             $value = $value->getValue();
@@ -80,8 +78,7 @@ 
      * This method exists only for the compatibility reason when deserializing a previously serialized version
      * that didn't had the key property
      */
-    public function __wakeup()
-    {
+    public function __wakeup() {
         /** @psalm-suppress DocblockTypeContradiction key can be null when deserializing an enum without the key */
         if ($this->key === null) {
             /**
@@ -108,8 +105,7 @@ 
      * @return mixed
      * @psalm-return T
      */
-    public function getValue()
-    {
+    public function getValue() {
         return $this->value;
     }
 
@@ -119,8 +115,7 @@ 
      * @psalm-pure
      * @return string
      */
-    public function getKey()
-    {
+    public function getKey() {
         return $this->key;
     }
 
@@ -129,8 +124,7 @@ 
      * @psalm-suppress InvalidCast
      * @return string
      */
-    public function __toString()
-    {
+    public function __toString() {
         return (string)$this->value;
     }
 
@@ -158,8 +152,7 @@ 
      * @psalm-return list<string>
      * @return array
      */
-    public static function keys()
-    {
+    public static function keys() {
         return \array_keys(static::toArray());
     }
 
@@ -170,8 +163,7 @@ 
      * @psalm-return array<string, static>
      * @return static[] Constant name in key, Enum instance in value
      */
-    public static function values()
-    {
+    public static function values() {
         $values = array();
 
         /** @psalm-var T $value */
@@ -191,8 +183,7 @@ 
      * @psalm-return array<string, mixed>
      * @return array Constant name in key, constant value in value
      */
-    public static function toArray()
-    {
+    public static function toArray() {
         $class = static::class;
 
         if (!isset(static::$cache[$class])) {
@@ -214,8 +205,7 @@ 
      * @psalm-assert-if-true T $value
      * @return bool
      */
-    public static function isValid($value)
-    {
+    public static function isValid($value) {
         return \in_array($value, static::toArray(), true);
     }
 
@@ -256,8 +246,7 @@ 
      * @psalm-pure
      * @return bool
      */
-    public static function isValidKey($key)
-    {
+    public static function isValidKey($key) {
         $array = static::toArray();
 
         return isset($array[$key]) || \array_key_exists($key, $array);
@@ -272,8 +261,7 @@ 
      * @psalm-pure
      * @return string|false
      */
-    public static function search($value)
-    {
+    public static function search($value) {
         return \array_search($value, static::toArray(), true);
     }
 
@@ -288,8 +276,7 @@ 
      *
      * @psalm-pure
      */
-    public static function __callStatic($name, $arguments)
-    {
+    public static function __callStatic($name, $arguments) {
         $class = static::class;
         if (!isset(self::$instances[$class][$name])) {
             $array = static::toArray();
@@ -311,8 +298,7 @@ 
      * @psalm-pure
      */
     #[\ReturnTypeWillChange]
-    public function jsonSerialize()
-    {
+    public function jsonSerialize() {
         return $this->getValue();
     }
 }

vendor/brianhenryie/strauss/tests/Unit/Composer/Extra/StraussConfigTest.php

Indentation +189 added lines, -189 removed lines

@@ -14,13 +14,13 @@ 
 class StraussConfigTest extends TestCase
 {
 
-    /**
-     * With a full (at time of writing) config, test the getters.
-     */
-    public function testGetters()
-    {
+	/**
+	 * With a full (at time of writing) config, test the getters.
+	 */
+	public function testGetters()
+	{
 
-        $composerExtraStraussJson = <<<'EOD'
+		$composerExtraStraussJson = <<<'EOD'
 {
   "name": "brianhenryie/strauss-config-test",
   "require": {
@@ -50,38 +50,38 @@ 
 }
 EOD;
 
-        $tmpfname = tempnam(sys_get_temp_dir(), 'strauss-test-');
-        file_put_contents($tmpfname, $composerExtraStraussJson);
+		$tmpfname = tempnam(sys_get_temp_dir(), 'strauss-test-');
+		file_put_contents($tmpfname, $composerExtraStraussJson);
 
-        $composer = Factory::create(new NullIO(), $tmpfname);
+		$composer = Factory::create(new NullIO(), $tmpfname);
 
-        $sut = new StraussConfig($composer);
+		$sut = new StraussConfig($composer);
 
-        $this->assertContains('pimple/pimple', $sut->getPackages());
+		$this->assertContains('pimple/pimple', $sut->getPackages());
 
-        $this->assertEquals('target_directory' . DIRECTORY_SEPARATOR, $sut->getTargetDirectory());
+		$this->assertEquals('target_directory' . DIRECTORY_SEPARATOR, $sut->getTargetDirectory());
 
-        $this->assertEquals("BrianHenryIE\\Strauss", $sut->getNamespacePrefix());
+		$this->assertEquals("BrianHenryIE\\Strauss", $sut->getNamespacePrefix());
 
-        $this->assertEquals('BrianHenryIE_Strauss_', $sut->getClassmapPrefix());
+		$this->assertEquals('BrianHenryIE_Strauss_', $sut->getClassmapPrefix());
 
-        // @see https://github.com/BrianHenryIE/strauss/issues/14
-        $this->assertContains('/^psr.*$/', $sut->getExcludeFilePatternsFromPrefixing());
+		// @see https://github.com/BrianHenryIE/strauss/issues/14
+		$this->assertContains('/^psr.*$/', $sut->getExcludeFilePatternsFromPrefixing());
 
-        $this->assertArrayHasKey('clancats/container', $sut->getOverrideAutoload());
+		$this->assertArrayHasKey('clancats/container', $sut->getOverrideAutoload());
 
-        $this->assertFalse($sut->isDeleteVendorFiles());
-    }
+		$this->assertFalse($sut->isDeleteVendorFiles());
+	}
 
-    /**
-     * Test how it handles an extra key.
-     *
-     * Turns out it just ignores it... good!
-     */
-    public function testExtraKey()
-    {
+	/**
+	 * Test how it handles an extra key.
+	 *
+	 * Turns out it just ignores it... good!
+	 */
+	public function testExtraKey()
+	{
 
-        $composerExtraStraussJson = <<<'EOD'
+		$composerExtraStraussJson = <<<'EOD'
 {
   "name": "brianhenryie/strauss-config-test",
   "require": {
@@ -112,31 +112,31 @@ 
 }
 EOD;
 
-        $tmpfname = tempnam(sys_get_temp_dir(), 'strauss-test-');
-        file_put_contents($tmpfname, $composerExtraStraussJson);
+		$tmpfname = tempnam(sys_get_temp_dir(), 'strauss-test-');
+		file_put_contents($tmpfname, $composerExtraStraussJson);
 
-        $composer = Factory::create(new NullIO(), $tmpfname);
+		$composer = Factory::create(new NullIO(), $tmpfname);
 
-        $exception = null;
+		$exception = null;
 
-        try {
-            $sut = new StraussConfig($composer);
-        } catch (\Exception $e) {
-            $exception = $e;
-        }
+		try {
+			$sut = new StraussConfig($composer);
+		} catch (\Exception $e) {
+			$exception = $e;
+		}
 
-        $this->assertNull($exception);
-    }
+		$this->assertNull($exception);
+	}
 
-    /**
-     * straussconfig-test-3.json has no target_dir key.
-     *
-     * If no target_dir is specified, used "strauss/"
-     */
-    public function testDefaultTargetDir()
-    {
+	/**
+	 * straussconfig-test-3.json has no target_dir key.
+	 *
+	 * If no target_dir is specified, used "strauss/"
+	 */
+	public function testDefaultTargetDir()
+	{
 
-        $composerExtraStraussJson = <<<'EOD'
+		$composerExtraStraussJson = <<<'EOD'
 {
   "name": "brianhenryie/strauss-config-test",
   "require": {
@@ -163,23 +163,23 @@ 
 }
 EOD;
 
-        $tmpfname = tempnam(sys_get_temp_dir(), 'strauss-test-');
-        file_put_contents($tmpfname, $composerExtraStraussJson);
+		$tmpfname = tempnam(sys_get_temp_dir(), 'strauss-test-');
+		file_put_contents($tmpfname, $composerExtraStraussJson);
 
-        $composer = Factory::create(new NullIO(), $tmpfname);
+		$composer = Factory::create(new NullIO(), $tmpfname);
 
-        $sut = new StraussConfig($composer);
+		$sut = new StraussConfig($composer);
 
-        $this->assertEquals('strauss'. DIRECTORY_SEPARATOR, $sut->getTargetDirectory());
-    }
+		$this->assertEquals('strauss'. DIRECTORY_SEPARATOR, $sut->getTargetDirectory());
+	}
 
-    /**
-     * When the namespace prefix isn't provided, use the PSR-4 autoload key name.
-     */
-    public function testDefaultNamespacePrefixFromAutoloaderPsr4()
-    {
+	/**
+	 * When the namespace prefix isn't provided, use the PSR-4 autoload key name.
+	 */
+	public function testDefaultNamespacePrefixFromAutoloaderPsr4()
+	{
 
-        $composerExtraStraussJson = <<<'EOD'
+		$composerExtraStraussJson = <<<'EOD'
 {
   "name": "brianhenryie/strauss-config-test",
   "require": {
@@ -194,23 +194,23 @@ 
 }
 EOD;
 
-        $tmpfname = tempnam(sys_get_temp_dir(), 'strauss-test-');
-        file_put_contents($tmpfname, $composerExtraStraussJson);
+		$tmpfname = tempnam(sys_get_temp_dir(), 'strauss-test-');
+		file_put_contents($tmpfname, $composerExtraStraussJson);
 
-        $composer = Factory::create(new NullIO(), $tmpfname);
+		$composer = Factory::create(new NullIO(), $tmpfname);
 
-        $sut = new StraussConfig($composer);
+		$sut = new StraussConfig($composer);
 
-        $this->assertEquals("BrianHenryIE\\Strauss", $sut->getNamespacePrefix());
-    }
+		$this->assertEquals("BrianHenryIE\\Strauss", $sut->getNamespacePrefix());
+	}
 
-    /**
-     * When the namespace prefix isn't provided, use the PSR-0 autoload key name.
-     */
-    public function testDefaultNamespacePrefixFromAutoloaderPsr0()
-    {
+	/**
+	 * When the namespace prefix isn't provided, use the PSR-0 autoload key name.
+	 */
+	public function testDefaultNamespacePrefixFromAutoloaderPsr0()
+	{
 
-        $composerExtraStraussJson = <<<'EOD'
+		$composerExtraStraussJson = <<<'EOD'
 {
   "name": "brianhenryie/strauss-config-test",
   "require": {
@@ -224,25 +224,25 @@ 
   }
 }
 EOD;
-        $tmpfname = tempnam(sys_get_temp_dir(), 'strauss-test-');
-        file_put_contents($tmpfname, $composerExtraStraussJson);
+		$tmpfname = tempnam(sys_get_temp_dir(), 'strauss-test-');
+		file_put_contents($tmpfname, $composerExtraStraussJson);
 
-        $composer = Factory::create(new NullIO(), $tmpfname);
+		$composer = Factory::create(new NullIO(), $tmpfname);
 
-        $sut = new StraussConfig($composer);
+		$sut = new StraussConfig($composer);
 
-        $this->assertEquals("BrianHenryIE\\Strauss", $sut->getNamespacePrefix());
-    }
+		$this->assertEquals("BrianHenryIE\\Strauss", $sut->getNamespacePrefix());
+	}
 
-    /**
-     * When the namespace prefix isn't provided, and there's no PSR-0 or PSR-4 autoloader to figure it from...
-     *
-     * brianhenryie/strauss-config-test
-     */
-    public function testDefaultNamespacePrefixWithNoAutoloader()
-    {
+	/**
+	 * When the namespace prefix isn't provided, and there's no PSR-0 or PSR-4 autoloader to figure it from...
+	 *
+	 * brianhenryie/strauss-config-test
+	 */
+	public function testDefaultNamespacePrefixWithNoAutoloader()
+	{
 
-        $composerExtraStraussJson = <<<'EOD'
+		$composerExtraStraussJson = <<<'EOD'
 {
   "name": "brianhenryie/strauss-config-test",
   "require": {
@@ -251,23 +251,23 @@ 
 }
 EOD;
 
-        $tmpfname = tempnam(sys_get_temp_dir(), 'strauss-test-');
-        file_put_contents($tmpfname, $composerExtraStraussJson);
+		$tmpfname = tempnam(sys_get_temp_dir(), 'strauss-test-');
+		file_put_contents($tmpfname, $composerExtraStraussJson);
 
-        $composer = Factory::create(new NullIO(), $tmpfname);
+		$composer = Factory::create(new NullIO(), $tmpfname);
 
-        $sut = new StraussConfig($composer);
+		$sut = new StraussConfig($composer);
 
-        $this->assertEquals("Brianhenryie\\Strauss_Config_Test", $sut->getNamespacePrefix());
-    }
+		$this->assertEquals("Brianhenryie\\Strauss_Config_Test", $sut->getNamespacePrefix());
+	}
 
-    /**
-     * When the classmap prefix isn't provided, use the PSR-4 autoload key name.
-     */
-    public function testDefaultClassmapPrefixFromAutoloaderPsr4()
-    {
+	/**
+	 * When the classmap prefix isn't provided, use the PSR-4 autoload key name.
+	 */
+	public function testDefaultClassmapPrefixFromAutoloaderPsr4()
+	{
 
-        $composerExtraStraussJson = <<<'EOD'
+		$composerExtraStraussJson = <<<'EOD'
 {
   "name": "brianhenryie/strauss-config-test",
   "require": {
@@ -282,23 +282,23 @@ 
 }
 EOD;
 
-        $tmpfname = tempnam(sys_get_temp_dir(), 'strauss-test-');
-        file_put_contents($tmpfname, $composerExtraStraussJson);
+		$tmpfname = tempnam(sys_get_temp_dir(), 'strauss-test-');
+		file_put_contents($tmpfname, $composerExtraStraussJson);
 
-        $composer = Factory::create(new NullIO(), $tmpfname);
+		$composer = Factory::create(new NullIO(), $tmpfname);
 
-        $sut = new StraussConfig($composer);
+		$sut = new StraussConfig($composer);
 
-        $this->assertEquals("BrianHenryIE_Strauss_", $sut->getClassmapPrefix());
-    }
+		$this->assertEquals("BrianHenryIE_Strauss_", $sut->getClassmapPrefix());
+	}
 
-    /**
-     * When the classmap prefix isn't provided, use the PSR-0 autoload key name.
-     */
-    public function testDefaultClassmapPrefixFromAutoloaderPsr0()
-    {
+	/**
+	 * When the classmap prefix isn't provided, use the PSR-0 autoload key name.
+	 */
+	public function testDefaultClassmapPrefixFromAutoloaderPsr0()
+	{
 
-        $composerExtraStraussJson = <<<'EOD'
+		$composerExtraStraussJson = <<<'EOD'
 {
   "name": "brianhenryie/strauss-config-test",
   "require": {
@@ -312,26 +312,26 @@ 
   }
 }
 EOD;
-        $tmpfname = tempnam(sys_get_temp_dir(), 'strauss-test-');
-        file_put_contents($tmpfname, $composerExtraStraussJson);
+		$tmpfname = tempnam(sys_get_temp_dir(), 'strauss-test-');
+		file_put_contents($tmpfname, $composerExtraStraussJson);
 
-        $composer = Factory::create(new NullIO(), $tmpfname);
+		$composer = Factory::create(new NullIO(), $tmpfname);
 
 
-        $sut = new StraussConfig($composer);
+		$sut = new StraussConfig($composer);
 
-        $this->assertEquals("BrianHenryIE_Strauss_", $sut->getClassmapPrefix());
-    }
+		$this->assertEquals("BrianHenryIE_Strauss_", $sut->getClassmapPrefix());
+	}
 
-    /**
-     * When the classmap prefix isn't provided, and there's no PSR-0 or PSR-4 autoloader to figure it from...
-     *
-     * brianhenryie/strauss-config-test
-     */
-    public function testDefaultClassmapPrefixWithNoAutoloader()
-    {
+	/**
+	 * When the classmap prefix isn't provided, and there's no PSR-0 or PSR-4 autoloader to figure it from...
+	 *
+	 * brianhenryie/strauss-config-test
+	 */
+	public function testDefaultClassmapPrefixWithNoAutoloader()
+	{
 
-        $composerExtraStraussJson = <<<'EOD'
+		$composerExtraStraussJson = <<<'EOD'
 {
   "name": "brianhenryie/strauss-config-test",
   "require": {
@@ -340,23 +340,23 @@ 
 
 }
 EOD;
-        $tmpfname = tempnam(sys_get_temp_dir(), 'strauss-test-');
-        file_put_contents($tmpfname, $composerExtraStraussJson);
+		$tmpfname = tempnam(sys_get_temp_dir(), 'strauss-test-');
+		file_put_contents($tmpfname, $composerExtraStraussJson);
 
-        $composer = Factory::create(new NullIO(), $tmpfname);
+		$composer = Factory::create(new NullIO(), $tmpfname);
 
-        $sut = new StraussConfig($composer);
+		$sut = new StraussConfig($composer);
 
-        $this->assertEquals("Brianhenryie_Strauss_Config_Test", $sut->getClassmapPrefix());
-    }
+		$this->assertEquals("Brianhenryie_Strauss_Config_Test", $sut->getClassmapPrefix());
+	}
 
-    /**
-     * When Strauss config has packages specified, obviously use them.
-     */
-    public function testGetPackagesFromConfig()
-    {
+	/**
+	 * When Strauss config has packages specified, obviously use them.
+	 */
+	public function testGetPackagesFromConfig()
+	{
 
-        $composerExtraStraussJson = <<<'EOD'
+		$composerExtraStraussJson = <<<'EOD'
 {
   "name": "brianhenryie/strauss-config-test",
   "require": {
@@ -386,23 +386,23 @@ 
 }
 
 EOD;
-        $tmpfname = tempnam(sys_get_temp_dir(), 'strauss-test-');
-        file_put_contents($tmpfname, $composerExtraStraussJson);
+		$tmpfname = tempnam(sys_get_temp_dir(), 'strauss-test-');
+		file_put_contents($tmpfname, $composerExtraStraussJson);
 
-        $composer = Factory::create(new NullIO(), $tmpfname);
+		$composer = Factory::create(new NullIO(), $tmpfname);
 
-        $sut = new StraussConfig($composer);
+		$sut = new StraussConfig($composer);
 
-        $this->assertContains('pimple/pimple', $sut->getPackages());
-    }
+		$this->assertContains('pimple/pimple', $sut->getPackages());
+	}
 
-    /**
-     * When Strauss config has no packages specified, use composer.json's require list.
-     */
-    public function testGetPackagesNoConfig()
-    {
+	/**
+	 * When Strauss config has no packages specified, use composer.json's require list.
+	 */
+	public function testGetPackagesNoConfig()
+	{
 
-        $composerExtraStraussJson = <<<'EOD'
+		$composerExtraStraussJson = <<<'EOD'
 {
   "name": "brianhenryie/strauss-config-test",
   "require": {
@@ -428,23 +428,23 @@ 
   }
 }
 EOD;
-        $tmpfname = tempnam(sys_get_temp_dir(), 'strauss-test-');
-        file_put_contents($tmpfname, $composerExtraStraussJson);
+		$tmpfname = tempnam(sys_get_temp_dir(), 'strauss-test-');
+		file_put_contents($tmpfname, $composerExtraStraussJson);
 
-        $composer = Factory::create(new NullIO(), $tmpfname);
+		$composer = Factory::create(new NullIO(), $tmpfname);
 
-        $sut = new StraussConfig($composer);
+		$sut = new StraussConfig($composer);
 
-        $this->assertContains('league/container', $sut->getPackages());
-    }
+		$this->assertContains('league/container', $sut->getPackages());
+	}
 
-    /**
-     * For backwards compatibility, if a Mozart config is present, use it.
-     */
-    public function testMapMozartConfig()
-    {
+	/**
+	 * For backwards compatibility, if a Mozart config is present, use it.
+	 */
+	public function testMapMozartConfig()
+	{
 
-        $composerExtraStraussJson = <<<'EOD'
+		$composerExtraStraussJson = <<<'EOD'
 {
   "extra": {
     "mozart": {
@@ -469,41 +469,41 @@ 
   }
 }
 EOD;
-        $tmpfname = tempnam(sys_get_temp_dir(), 'strauss-test-');
-        file_put_contents($tmpfname, $composerExtraStraussJson);
+		$tmpfname = tempnam(sys_get_temp_dir(), 'strauss-test-');
+		file_put_contents($tmpfname, $composerExtraStraussJson);
 
-        $composer = Factory::create(new NullIO(), $tmpfname);
+		$composer = Factory::create(new NullIO(), $tmpfname);
 
-        $sut = new StraussConfig($composer);
+		$sut = new StraussConfig($composer);
 
-        $this->assertContains('pimple/pimple', $sut->getPackages());
+		$this->assertContains('pimple/pimple', $sut->getPackages());
 
-        $this->assertEquals('dep_directory' . DIRECTORY_SEPARATOR, $sut->getTargetDirectory());
+		$this->assertEquals('dep_directory' . DIRECTORY_SEPARATOR, $sut->getTargetDirectory());
 
-        $this->assertEquals("My_Mozart_Config", $sut->getNamespacePrefix());
+		$this->assertEquals("My_Mozart_Config", $sut->getNamespacePrefix());
 
-        $this->assertEquals('My_Mozart_Config_', $sut->getClassmapPrefix());
+		$this->assertEquals('My_Mozart_Config_', $sut->getClassmapPrefix());
 
-        $this->assertContains('psr/container', $sut->getExcludePackagesFromPrefixing());
+		$this->assertContains('psr/container', $sut->getExcludePackagesFromPrefixing());
 
-        $this->assertArrayHasKey('clancats/container', $sut->getOverrideAutoload());
+		$this->assertArrayHasKey('clancats/container', $sut->getOverrideAutoload());
 
-        // Mozart default was true.
-        $this->assertTrue($sut->isDeleteVendorFiles());
-    }
+		// Mozart default was true.
+		$this->assertTrue($sut->isDeleteVendorFiles());
+	}
+
+	/**
+	 * Since sometimes the namespace we're prefixing will already have a leading backslash, sometimes
+	 * the namespace_prefix will want its slash at the beginning, sometimes at the end.
+	 *
+	 * @see Prefixer::replaceNamespace()
+	 *
+	 * @covers \BrianHenryIE\Strauss\Composer\Extra\StraussConfig::getNamespacePrefix
+	 */
+	public function testNamespacePrefixHasNoSlash()
+	{
 
-    /**
-     * Since sometimes the namespace we're prefixing will already have a leading backslash, sometimes
-     * the namespace_prefix will want its slash at the beginning, sometimes at the end.
-     *
-     * @see Prefixer::replaceNamespace()
-     *
-     * @covers \BrianHenryIE\Strauss\Composer\Extra\StraussConfig::getNamespacePrefix
-     */
-    public function testNamespacePrefixHasNoSlash()
-    {
-
-        $composerExtraStraussJson = <<<'EOD'
+		$composerExtraStraussJson = <<<'EOD'
 {
   "extra": {
     "mozart": {
@@ -512,13 +512,13 @@ 
   }
 }
 EOD;
-        $tmpfname = tempnam(sys_get_temp_dir(), 'strauss-test-');
-        file_put_contents($tmpfname, $composerExtraStraussJson);
+		$tmpfname = tempnam(sys_get_temp_dir(), 'strauss-test-');
+		file_put_contents($tmpfname, $composerExtraStraussJson);
 
-        $composer = Factory::create(new NullIO(), $tmpfname);
+		$composer = Factory::create(new NullIO(), $tmpfname);
 
-        $sut = new StraussConfig($composer);
+		$sut = new StraussConfig($composer);
 
-        $this->assertEquals("My_Mozart_Config", $sut->getNamespacePrefix());
-    }
+		$this->assertEquals("My_Mozart_Config", $sut->getNamespacePrefix());
+	}
 }

Spacing +78 added lines, -78 removed lines

@@ -50,27 +50,27 @@ 
 }
 EOD;
 
-        $tmpfname = tempnam(sys_get_temp_dir(), 'strauss-test-');
-        file_put_contents($tmpfname, $composerExtraStraussJson);
+        $tmpfname = tempnam( sys_get_temp_dir(), 'strauss-test-' );
+        file_put_contents( $tmpfname, $composerExtraStraussJson );
 
-        $composer = Factory::create(new NullIO(), $tmpfname);
+        $composer = Factory::create( new NullIO(), $tmpfname );
 
-        $sut = new StraussConfig($composer);
+        $sut = new StraussConfig( $composer );
 
-        $this->assertContains('pimple/pimple', $sut->getPackages());
+        $this->assertContains( 'pimple/pimple', $sut->getPackages() );
 
-        $this->assertEquals('target_directory' . DIRECTORY_SEPARATOR, $sut->getTargetDirectory());
+        $this->assertEquals( 'target_directory' . DIRECTORY_SEPARATOR, $sut->getTargetDirectory() );
 
-        $this->assertEquals("BrianHenryIE\\Strauss", $sut->getNamespacePrefix());
+        $this->assertEquals( "BrianHenryIE\\Strauss", $sut->getNamespacePrefix() );
 
-        $this->assertEquals('BrianHenryIE_Strauss_', $sut->getClassmapPrefix());
+        $this->assertEquals( 'BrianHenryIE_Strauss_', $sut->getClassmapPrefix() );
 
         // @see https://github.com/BrianHenryIE/strauss/issues/14
-        $this->assertContains('/^psr.*$/', $sut->getExcludeFilePatternsFromPrefixing());
+        $this->assertContains( '/^psr.*$/', $sut->getExcludeFilePatternsFromPrefixing() );
 
-        $this->assertArrayHasKey('clancats/container', $sut->getOverrideAutoload());
+        $this->assertArrayHasKey( 'clancats/container', $sut->getOverrideAutoload() );
 
-        $this->assertFalse($sut->isDeleteVendorFiles());
+        $this->assertFalse( $sut->isDeleteVendorFiles() );
     }
 
     /**
@@ -112,20 +112,20 @@ 
 }
 EOD;
 
-        $tmpfname = tempnam(sys_get_temp_dir(), 'strauss-test-');
-        file_put_contents($tmpfname, $composerExtraStraussJson);
+        $tmpfname = tempnam( sys_get_temp_dir(), 'strauss-test-' );
+        file_put_contents( $tmpfname, $composerExtraStraussJson );
 
-        $composer = Factory::create(new NullIO(), $tmpfname);
+        $composer = Factory::create( new NullIO(), $tmpfname );
 
         $exception = null;
 
         try {
-            $sut = new StraussConfig($composer);
-        } catch (\Exception $e) {
+            $sut = new StraussConfig( $composer );
+        } catch ( \Exception $e ) {
             $exception = $e;
         }
 
-        $this->assertNull($exception);
+        $this->assertNull( $exception );
     }
 
     /**
@@ -163,14 +163,14 @@ 
 }
 EOD;
 
-        $tmpfname = tempnam(sys_get_temp_dir(), 'strauss-test-');
-        file_put_contents($tmpfname, $composerExtraStraussJson);
+        $tmpfname = tempnam( sys_get_temp_dir(), 'strauss-test-' );
+        file_put_contents( $tmpfname, $composerExtraStraussJson );
 
-        $composer = Factory::create(new NullIO(), $tmpfname);
+        $composer = Factory::create( new NullIO(), $tmpfname );
 
-        $sut = new StraussConfig($composer);
+        $sut = new StraussConfig( $composer );
 
-        $this->assertEquals('strauss'. DIRECTORY_SEPARATOR, $sut->getTargetDirectory());
+        $this->assertEquals( 'strauss' . DIRECTORY_SEPARATOR, $sut->getTargetDirectory() );
     }
 
     /**
@@ -194,14 +194,14 @@ 
 }
 EOD;
 
-        $tmpfname = tempnam(sys_get_temp_dir(), 'strauss-test-');
-        file_put_contents($tmpfname, $composerExtraStraussJson);
+        $tmpfname = tempnam( sys_get_temp_dir(), 'strauss-test-' );
+        file_put_contents( $tmpfname, $composerExtraStraussJson );
 
-        $composer = Factory::create(new NullIO(), $tmpfname);
+        $composer = Factory::create( new NullIO(), $tmpfname );
 
-        $sut = new StraussConfig($composer);
+        $sut = new StraussConfig( $composer );
 
-        $this->assertEquals("BrianHenryIE\\Strauss", $sut->getNamespacePrefix());
+        $this->assertEquals( "BrianHenryIE\\Strauss", $sut->getNamespacePrefix() );
     }
 
     /**
@@ -224,14 +224,14 @@ 
   }
 }
 EOD;
-        $tmpfname = tempnam(sys_get_temp_dir(), 'strauss-test-');
-        file_put_contents($tmpfname, $composerExtraStraussJson);
+        $tmpfname = tempnam( sys_get_temp_dir(), 'strauss-test-' );
+        file_put_contents( $tmpfname, $composerExtraStraussJson );
 
-        $composer = Factory::create(new NullIO(), $tmpfname);
+        $composer = Factory::create( new NullIO(), $tmpfname );
 
-        $sut = new StraussConfig($composer);
+        $sut = new StraussConfig( $composer );
 
-        $this->assertEquals("BrianHenryIE\\Strauss", $sut->getNamespacePrefix());
+        $this->assertEquals( "BrianHenryIE\\Strauss", $sut->getNamespacePrefix() );
     }
 
     /**
@@ -251,14 +251,14 @@ 
 }
 EOD;
 
-        $tmpfname = tempnam(sys_get_temp_dir(), 'strauss-test-');
-        file_put_contents($tmpfname, $composerExtraStraussJson);
+        $tmpfname = tempnam( sys_get_temp_dir(), 'strauss-test-' );
+        file_put_contents( $tmpfname, $composerExtraStraussJson );
 
-        $composer = Factory::create(new NullIO(), $tmpfname);
+        $composer = Factory::create( new NullIO(), $tmpfname );
 
-        $sut = new StraussConfig($composer);
+        $sut = new StraussConfig( $composer );
 
-        $this->assertEquals("Brianhenryie\\Strauss_Config_Test", $sut->getNamespacePrefix());
+        $this->assertEquals( "Brianhenryie\\Strauss_Config_Test", $sut->getNamespacePrefix() );
     }
 
     /**
@@ -282,14 +282,14 @@ 
 }
 EOD;
 
-        $tmpfname = tempnam(sys_get_temp_dir(), 'strauss-test-');
-        file_put_contents($tmpfname, $composerExtraStraussJson);
+        $tmpfname = tempnam( sys_get_temp_dir(), 'strauss-test-' );
+        file_put_contents( $tmpfname, $composerExtraStraussJson );
 
-        $composer = Factory::create(new NullIO(), $tmpfname);
+        $composer = Factory::create( new NullIO(), $tmpfname );
 
-        $sut = new StraussConfig($composer);
+        $sut = new StraussConfig( $composer );
 
-        $this->assertEquals("BrianHenryIE_Strauss_", $sut->getClassmapPrefix());
+        $this->assertEquals( "BrianHenryIE_Strauss_", $sut->getClassmapPrefix() );
     }
 
     /**
@@ -312,15 +312,15 @@ 
   }
 }
 EOD;
-        $tmpfname = tempnam(sys_get_temp_dir(), 'strauss-test-');
-        file_put_contents($tmpfname, $composerExtraStraussJson);
+        $tmpfname = tempnam( sys_get_temp_dir(), 'strauss-test-' );
+        file_put_contents( $tmpfname, $composerExtraStraussJson );
 
-        $composer = Factory::create(new NullIO(), $tmpfname);
+        $composer = Factory::create( new NullIO(), $tmpfname );
 
 
-        $sut = new StraussConfig($composer);
+        $sut = new StraussConfig( $composer );
 
-        $this->assertEquals("BrianHenryIE_Strauss_", $sut->getClassmapPrefix());
+        $this->assertEquals( "BrianHenryIE_Strauss_", $sut->getClassmapPrefix() );
     }
 
     /**
@@ -340,14 +340,14 @@ 
 
 }
 EOD;
-        $tmpfname = tempnam(sys_get_temp_dir(), 'strauss-test-');
-        file_put_contents($tmpfname, $composerExtraStraussJson);
+        $tmpfname = tempnam( sys_get_temp_dir(), 'strauss-test-' );
+        file_put_contents( $tmpfname, $composerExtraStraussJson );
 
-        $composer = Factory::create(new NullIO(), $tmpfname);
+        $composer = Factory::create( new NullIO(), $tmpfname );
 
-        $sut = new StraussConfig($composer);
+        $sut = new StraussConfig( $composer );
 
-        $this->assertEquals("Brianhenryie_Strauss_Config_Test", $sut->getClassmapPrefix());
+        $this->assertEquals( "Brianhenryie_Strauss_Config_Test", $sut->getClassmapPrefix() );
     }
 
     /**
@@ -386,14 +386,14 @@ 
 }
 
 EOD;
-        $tmpfname = tempnam(sys_get_temp_dir(), 'strauss-test-');
-        file_put_contents($tmpfname, $composerExtraStraussJson);
+        $tmpfname = tempnam( sys_get_temp_dir(), 'strauss-test-' );
+        file_put_contents( $tmpfname, $composerExtraStraussJson );
 
-        $composer = Factory::create(new NullIO(), $tmpfname);
+        $composer = Factory::create( new NullIO(), $tmpfname );
 
-        $sut = new StraussConfig($composer);
+        $sut = new StraussConfig( $composer );
 
-        $this->assertContains('pimple/pimple', $sut->getPackages());
+        $this->assertContains( 'pimple/pimple', $sut->getPackages() );
     }
 
     /**
@@ -428,14 +428,14 @@ 
   }
 }
 EOD;
-        $tmpfname = tempnam(sys_get_temp_dir(), 'strauss-test-');
-        file_put_contents($tmpfname, $composerExtraStraussJson);
+        $tmpfname = tempnam( sys_get_temp_dir(), 'strauss-test-' );
+        file_put_contents( $tmpfname, $composerExtraStraussJson );
 
-        $composer = Factory::create(new NullIO(), $tmpfname);
+        $composer = Factory::create( new NullIO(), $tmpfname );
 
-        $sut = new StraussConfig($composer);
+        $sut = new StraussConfig( $composer );
 
-        $this->assertContains('league/container', $sut->getPackages());
+        $this->assertContains( 'league/container', $sut->getPackages() );
     }
 
     /**
@@ -469,27 +469,27 @@ 
   }
 }
 EOD;
-        $tmpfname = tempnam(sys_get_temp_dir(), 'strauss-test-');
-        file_put_contents($tmpfname, $composerExtraStraussJson);
+        $tmpfname = tempnam( sys_get_temp_dir(), 'strauss-test-' );
+        file_put_contents( $tmpfname, $composerExtraStraussJson );
 
-        $composer = Factory::create(new NullIO(), $tmpfname);
+        $composer = Factory::create( new NullIO(), $tmpfname );
 
-        $sut = new StraussConfig($composer);
+        $sut = new StraussConfig( $composer );
 
-        $this->assertContains('pimple/pimple', $sut->getPackages());
+        $this->assertContains( 'pimple/pimple', $sut->getPackages() );
 
-        $this->assertEquals('dep_directory' . DIRECTORY_SEPARATOR, $sut->getTargetDirectory());
+        $this->assertEquals( 'dep_directory' . DIRECTORY_SEPARATOR, $sut->getTargetDirectory() );
 
-        $this->assertEquals("My_Mozart_Config", $sut->getNamespacePrefix());
+        $this->assertEquals( "My_Mozart_Config", $sut->getNamespacePrefix() );
 
-        $this->assertEquals('My_Mozart_Config_', $sut->getClassmapPrefix());
+        $this->assertEquals( 'My_Mozart_Config_', $sut->getClassmapPrefix() );
 
-        $this->assertContains('psr/container', $sut->getExcludePackagesFromPrefixing());
+        $this->assertContains( 'psr/container', $sut->getExcludePackagesFromPrefixing() );
 
-        $this->assertArrayHasKey('clancats/container', $sut->getOverrideAutoload());
+        $this->assertArrayHasKey( 'clancats/container', $sut->getOverrideAutoload() );
 
         // Mozart default was true.
-        $this->assertTrue($sut->isDeleteVendorFiles());
+        $this->assertTrue( $sut->isDeleteVendorFiles() );
     }
 
     /**
@@ -512,13 +512,13 @@ 
   }
 }
 EOD;
-        $tmpfname = tempnam(sys_get_temp_dir(), 'strauss-test-');
-        file_put_contents($tmpfname, $composerExtraStraussJson);
+        $tmpfname = tempnam( sys_get_temp_dir(), 'strauss-test-' );
+        file_put_contents( $tmpfname, $composerExtraStraussJson );
 
-        $composer = Factory::create(new NullIO(), $tmpfname);
+        $composer = Factory::create( new NullIO(), $tmpfname );
 
-        $sut = new StraussConfig($composer);
+        $sut = new StraussConfig( $composer );
 
-        $this->assertEquals("My_Mozart_Config", $sut->getNamespacePrefix());
+        $this->assertEquals( "My_Mozart_Config", $sut->getNamespacePrefix() );
     }
 }

Braces +14 added lines, -28 removed lines

@@ -11,14 +11,12 @@ 
 use Composer\IO\NullIO;
 use PHPUnit\Framework\TestCase;
 
-class StraussConfigTest extends TestCase
-{
+class StraussConfigTest extends TestCase {
 
     /**
      * With a full (at time of writing) config, test the getters.
      */
-    public function testGetters()
-    {
+    public function testGetters() {
 
         $composerExtraStraussJson = <<<'EOD'
 {
@@ -78,8 +76,7 @@ 
      *
      * Turns out it just ignores it... good!
      */
-    public function testExtraKey()
-    {
+    public function testExtraKey() {
 
         $composerExtraStraussJson = <<<'EOD'
 {
@@ -133,8 +130,7 @@ 
      *
      * If no target_dir is specified, used "strauss/"
      */
-    public function testDefaultTargetDir()
-    {
+    public function testDefaultTargetDir() {
 
         $composerExtraStraussJson = <<<'EOD'
 {
@@ -176,8 +172,7 @@ 
     /**
      * When the namespace prefix isn't provided, use the PSR-4 autoload key name.
      */
-    public function testDefaultNamespacePrefixFromAutoloaderPsr4()
-    {
+    public function testDefaultNamespacePrefixFromAutoloaderPsr4() {
 
         $composerExtraStraussJson = <<<'EOD'
 {
@@ -207,8 +202,7 @@ 
     /**
      * When the namespace prefix isn't provided, use the PSR-0 autoload key name.
      */
-    public function testDefaultNamespacePrefixFromAutoloaderPsr0()
-    {
+    public function testDefaultNamespacePrefixFromAutoloaderPsr0() {
 
         $composerExtraStraussJson = <<<'EOD'
 {
@@ -239,8 +233,7 @@ 
      *
      * brianhenryie/strauss-config-test
      */
-    public function testDefaultNamespacePrefixWithNoAutoloader()
-    {
+    public function testDefaultNamespacePrefixWithNoAutoloader() {
 
         $composerExtraStraussJson = <<<'EOD'
 {
@@ -264,8 +257,7 @@ 
     /**
      * When the classmap prefix isn't provided, use the PSR-4 autoload key name.
      */
-    public function testDefaultClassmapPrefixFromAutoloaderPsr4()
-    {
+    public function testDefaultClassmapPrefixFromAutoloaderPsr4() {
 
         $composerExtraStraussJson = <<<'EOD'
 {
@@ -295,8 +287,7 @@ 
     /**
      * When the classmap prefix isn't provided, use the PSR-0 autoload key name.
      */
-    public function testDefaultClassmapPrefixFromAutoloaderPsr0()
-    {
+    public function testDefaultClassmapPrefixFromAutoloaderPsr0() {
 
         $composerExtraStraussJson = <<<'EOD'
 {
@@ -328,8 +319,7 @@ 
      *
      * brianhenryie/strauss-config-test
      */
-    public function testDefaultClassmapPrefixWithNoAutoloader()
-    {
+    public function testDefaultClassmapPrefixWithNoAutoloader() {
 
         $composerExtraStraussJson = <<<'EOD'
 {
@@ -353,8 +343,7 @@ 
     /**
      * When Strauss config has packages specified, obviously use them.
      */
-    public function testGetPackagesFromConfig()
-    {
+    public function testGetPackagesFromConfig() {
 
         $composerExtraStraussJson = <<<'EOD'
 {
@@ -399,8 +388,7 @@ 
     /**
      * When Strauss config has no packages specified, use composer.json's require list.
      */
-    public function testGetPackagesNoConfig()
-    {
+    public function testGetPackagesNoConfig() {
 
         $composerExtraStraussJson = <<<'EOD'
 {
@@ -441,8 +429,7 @@ 
     /**
      * For backwards compatibility, if a Mozart config is present, use it.
      */
-    public function testMapMozartConfig()
-    {
+    public function testMapMozartConfig() {
 
         $composerExtraStraussJson = <<<'EOD'
 {
@@ -500,8 +487,7 @@ 
      *
      * @covers \BrianHenryIE\Strauss\Composer\Extra\StraussConfig::getNamespacePrefix
      */
-    public function testNamespacePrefixHasNoSlash()
-    {
+    public function testNamespacePrefixHasNoSlash() {
 
         $composerExtraStraussJson = <<<'EOD'
 {

Upper-Lower-Casing +13 added lines, -13 removed lines

@@ -48,7 +48,7 @@ 
     }
   }
 }
-EOD;
+eod;
 
         $tmpfname = tempnam(sys_get_temp_dir(), 'strauss-test-');
         file_put_contents($tmpfname, $composerExtraStraussJson);
@@ -110,7 +110,7 @@ 
     }
   }
 }
-EOD;
+eod;
 
         $tmpfname = tempnam(sys_get_temp_dir(), 'strauss-test-');
         file_put_contents($tmpfname, $composerExtraStraussJson);
@@ -161,7 +161,7 @@ 
     }
   }
 }
-EOD;
+eod;
 
         $tmpfname = tempnam(sys_get_temp_dir(), 'strauss-test-');
         file_put_contents($tmpfname, $composerExtraStraussJson);
@@ -192,7 +192,7 @@ 
     }
   }
 }
-EOD;
+eod;
 
         $tmpfname = tempnam(sys_get_temp_dir(), 'strauss-test-');
         file_put_contents($tmpfname, $composerExtraStraussJson);
@@ -223,7 +223,7 @@ 
     }
   }
 }
-EOD;
+eod;
         $tmpfname = tempnam(sys_get_temp_dir(), 'strauss-test-');
         file_put_contents($tmpfname, $composerExtraStraussJson);
 
@@ -249,7 +249,7 @@ 
     "league/container": "*"
   }
 }
-EOD;
+eod;
 
         $tmpfname = tempnam(sys_get_temp_dir(), 'strauss-test-');
         file_put_contents($tmpfname, $composerExtraStraussJson);
@@ -280,7 +280,7 @@ 
     }
   }
 }
-EOD;
+eod;
 
         $tmpfname = tempnam(sys_get_temp_dir(), 'strauss-test-');
         file_put_contents($tmpfname, $composerExtraStraussJson);
@@ -311,7 +311,7 @@ 
     }
   }
 }
-EOD;
+eod;
         $tmpfname = tempnam(sys_get_temp_dir(), 'strauss-test-');
         file_put_contents($tmpfname, $composerExtraStraussJson);
 
@@ -339,7 +339,7 @@ 
   }
 
 }
-EOD;
+eod;
         $tmpfname = tempnam(sys_get_temp_dir(), 'strauss-test-');
         file_put_contents($tmpfname, $composerExtraStraussJson);
 
@@ -385,7 +385,7 @@ 
   }
 }
 
-EOD;
+eod;
         $tmpfname = tempnam(sys_get_temp_dir(), 'strauss-test-');
         file_put_contents($tmpfname, $composerExtraStraussJson);
 
@@ -427,7 +427,7 @@ 
     }
   }
 }
-EOD;
+eod;
         $tmpfname = tempnam(sys_get_temp_dir(), 'strauss-test-');
         file_put_contents($tmpfname, $composerExtraStraussJson);
 
@@ -468,7 +468,7 @@ 
     }
   }
 }
-EOD;
+eod;
         $tmpfname = tempnam(sys_get_temp_dir(), 'strauss-test-');
         file_put_contents($tmpfname, $composerExtraStraussJson);
 
@@ -511,7 +511,7 @@ 
     }
   }
 }
-EOD;
+eod;
         $tmpfname = tempnam(sys_get_temp_dir(), 'strauss-test-');
         file_put_contents($tmpfname, $composerExtraStraussJson);
 

vendor/brianhenryie/strauss/tests/Unit/Composer/ComposerPackageTest.php

Indentation +88 added lines, -88 removed lines

@@ -8,134 +8,134 @@
 class ComposerPackageTest extends TestCase
 {
 
-    /**
-     * A simple test to check the getters all work.
-     */
-    public function testParseJson()
-    {
+	/**
+	 * A simple test to check the getters all work.
+	 */
+	public function testParseJson()
+	{
 
-        $testFile = __DIR__ . '/composerpackage-test-libmergepdf.json';
+		$testFile = __DIR__ . '/composerpackage-test-libmergepdf.json';
 
-        $composer = new ComposerPackage($testFile);
+		$composer = new ComposerPackage($testFile);
 
-        $this->assertEquals('iio/libmergepdf', $composer->getName());
+		$this->assertEquals('iio/libmergepdf', $composer->getName());
 
-        $this->assertIsArray($composer->getAutoload());
+		$this->assertIsArray($composer->getAutoload());
 
-        $this->assertIsArray($composer->getRequiresNames());
-    }
+		$this->assertIsArray($composer->getRequiresNames());
+	}
 
-    /**
-     * Test the dependencies' names are returned.
-     */
-    public function testGetRequiresNames()
-    {
+	/**
+	 * Test the dependencies' names are returned.
+	 */
+	public function testGetRequiresNames()
+	{
 
-        $testFile = __DIR__ . '/composerpackage-test-libmergepdf.json';
+		$testFile = __DIR__ . '/composerpackage-test-libmergepdf.json';
 
-        $composer = new ComposerPackage($testFile);
+		$composer = new ComposerPackage($testFile);
 
-        $requiresNames = $composer->getRequiresNames();
+		$requiresNames = $composer->getRequiresNames();
 
-        $this->assertContains('tecnickcom/tcpdf', $requiresNames);
-        $this->assertContains('setasign/fpdi', $requiresNames);
-    }
+		$this->assertContains('tecnickcom/tcpdf', $requiresNames);
+		$this->assertContains('setasign/fpdi', $requiresNames);
+	}
 
-    /**
-     * Test PHP and ext- are not returned, since we won't be dealing with them.
-     */
-    public function testGetRequiresNamesDoesNotContain()
-    {
+	/**
+	 * Test PHP and ext- are not returned, since we won't be dealing with them.
+	 */
+	public function testGetRequiresNamesDoesNotContain()
+	{
 
-        $testFile = __DIR__ . '/composerpackage-test-easypost-php.json';
+		$testFile = __DIR__ . '/composerpackage-test-easypost-php.json';
 
-        $composer = new ComposerPackage($testFile);
+		$composer = new ComposerPackage($testFile);
 
-        $requiresNames = $composer->getRequiresNames();
+		$requiresNames = $composer->getRequiresNames();
 
-        $this->assertNotContains('ext-curl', $requiresNames);
-        $this->assertNotContains('php', $requiresNames);
-    }
+		$this->assertNotContains('ext-curl', $requiresNames);
+		$this->assertNotContains('php', $requiresNames);
+	}
 
 
-    /**
-     *
-     */
-    public function testAutoloadPsr0()
-    {
+	/**
+	 *
+	 */
+	public function testAutoloadPsr0()
+	{
 
-        $testFile = __DIR__ . '/composerpackage-test-easypost-php.json';
+		$testFile = __DIR__ . '/composerpackage-test-easypost-php.json';
 
-        $composer = new ComposerPackage($testFile);
+		$composer = new ComposerPackage($testFile);
 
-        $autoload = $composer->getAutoload();
+		$autoload = $composer->getAutoload();
 
-        $this->assertArrayHasKey('psr-0', $autoload);
+		$this->assertArrayHasKey('psr-0', $autoload);
 
-        $this->assertIsArray($autoload['psr-0']);
-    }
+		$this->assertIsArray($autoload['psr-0']);
+	}
 
-    /**
-     *
-     */
-    public function testAutoloadPsr4()
-    {
+	/**
+	 *
+	 */
+	public function testAutoloadPsr4()
+	{
 
-        $testFile = __DIR__ . '/composerpackage-test-libmergepdf.json';
+		$testFile = __DIR__ . '/composerpackage-test-libmergepdf.json';
 
-        $composer = new ComposerPackage($testFile);
+		$composer = new ComposerPackage($testFile);
 
-        $autoload = $composer->getAutoload();
+		$autoload = $composer->getAutoload();
 
-        $this->assertArrayHasKey('psr-4', $autoload);
+		$this->assertArrayHasKey('psr-4', $autoload);
 
-        $this->assertIsArray($autoload['psr-4']);
-    }
+		$this->assertIsArray($autoload['psr-4']);
+	}
 
-    /**
-     *
-     */
-    public function testAutoloadClassmap()
-    {
+	/**
+	 *
+	 */
+	public function testAutoloadClassmap()
+	{
 
-        $testFile = __DIR__ . '/composerpackage-test-libmergepdf.json';
+		$testFile = __DIR__ . '/composerpackage-test-libmergepdf.json';
 
-        $composer = new ComposerPackage($testFile);
+		$composer = new ComposerPackage($testFile);
 
-        $autoload = $composer->getAutoload();
+		$autoload = $composer->getAutoload();
 
-        $this->assertArrayHasKey('classmap', $autoload);
+		$this->assertArrayHasKey('classmap', $autoload);
 
-        $this->assertIsArray($autoload['classmap']);
-    }
+		$this->assertIsArray($autoload['classmap']);
+	}
 
-    /**
-     *
-     */
-    public function testAutoloadFiles()
-    {
+	/**
+	 *
+	 */
+	public function testAutoloadFiles()
+	{
 
-        $testFile = __DIR__ . '/composerpackage-test-php-di.json';
+		$testFile = __DIR__ . '/composerpackage-test-php-di.json';
 
-        $composer = new ComposerPackage($testFile);
+		$composer = new ComposerPackage($testFile);
 
-        $autoload = $composer->getAutoload();
+		$autoload = $composer->getAutoload();
 
-        $this->assertArrayHasKey('files', $autoload);
+		$this->assertArrayHasKey('files', $autoload);
 
-        $this->assertIsArray($autoload['files']);
-    }
+		$this->assertIsArray($autoload['files']);
+	}
 
-    public function testOverrideAutoload()
-    {
-        $this->markTestIncomplete();
-    }
+	public function testOverrideAutoload()
+	{
+		$this->markTestIncomplete();
+	}
 
-    /**
-     * When composer.json is not where it was specified, what error message (via Exception) should be returned?
-     */
-    public function testMissingComposer()
-    {
-        $this->markTestIncomplete();
-    }
+	/**
+	 * When composer.json is not where it was specified, what error message (via Exception) should be returned?
+	 */
+	public function testMissingComposer()
+	{
+		$this->markTestIncomplete();
+	}
 }

Spacing +22 added lines, -22 removed lines

@@ -16,13 +16,13 @@ 
 
         $testFile = __DIR__ . '/composerpackage-test-libmergepdf.json';
 
-        $composer = new ComposerPackage($testFile);
+        $composer = new ComposerPackage( $testFile );
 
-        $this->assertEquals('iio/libmergepdf', $composer->getName());
+        $this->assertEquals( 'iio/libmergepdf', $composer->getName() );
 
-        $this->assertIsArray($composer->getAutoload());
+        $this->assertIsArray( $composer->getAutoload() );
 
-        $this->assertIsArray($composer->getRequiresNames());
+        $this->assertIsArray( $composer->getRequiresNames() );
     }
 
     /**
@@ -33,12 +33,12 @@ 
 
         $testFile = __DIR__ . '/composerpackage-test-libmergepdf.json';
 
-        $composer = new ComposerPackage($testFile);
+        $composer = new ComposerPackage( $testFile );
 
         $requiresNames = $composer->getRequiresNames();
 
-        $this->assertContains('tecnickcom/tcpdf', $requiresNames);
-        $this->assertContains('setasign/fpdi', $requiresNames);
+        $this->assertContains( 'tecnickcom/tcpdf', $requiresNames );
+        $this->assertContains( 'setasign/fpdi', $requiresNames );
     }
 
     /**
@@ -49,12 +49,12 @@ 
 
         $testFile = __DIR__ . '/composerpackage-test-easypost-php.json';
 
-        $composer = new ComposerPackage($testFile);
+        $composer = new ComposerPackage( $testFile );
 
         $requiresNames = $composer->getRequiresNames();
 
-        $this->assertNotContains('ext-curl', $requiresNames);
-        $this->assertNotContains('php', $requiresNames);
+        $this->assertNotContains( 'ext-curl', $requiresNames );
+        $this->assertNotContains( 'php', $requiresNames );
     }
 
 
@@ -66,13 +66,13 @@ 
 
         $testFile = __DIR__ . '/composerpackage-test-easypost-php.json';
 
-        $composer = new ComposerPackage($testFile);
+        $composer = new ComposerPackage( $testFile );
 
         $autoload = $composer->getAutoload();
 
-        $this->assertArrayHasKey('psr-0', $autoload);
+        $this->assertArrayHasKey( 'psr-0', $autoload );
 
-        $this->assertIsArray($autoload['psr-0']);
+        $this->assertIsArray( $autoload[ 'psr-0' ] );
     }
 
     /**
@@ -83,13 +83,13 @@ 
 
         $testFile = __DIR__ . '/composerpackage-test-libmergepdf.json';
 
-        $composer = new ComposerPackage($testFile);
+        $composer = new ComposerPackage( $testFile );
 
         $autoload = $composer->getAutoload();
 
-        $this->assertArrayHasKey('psr-4', $autoload);
+        $this->assertArrayHasKey( 'psr-4', $autoload );
 
-        $this->assertIsArray($autoload['psr-4']);
+        $this->assertIsArray( $autoload[ 'psr-4' ] );
     }
 
     /**
@@ -100,13 +100,13 @@ 
 
         $testFile = __DIR__ . '/composerpackage-test-libmergepdf.json';
 
-        $composer = new ComposerPackage($testFile);
+        $composer = new ComposerPackage( $testFile );
 
         $autoload = $composer->getAutoload();
 
-        $this->assertArrayHasKey('classmap', $autoload);
+        $this->assertArrayHasKey( 'classmap', $autoload );
 
-        $this->assertIsArray($autoload['classmap']);
+        $this->assertIsArray( $autoload[ 'classmap' ] );
     }
 
     /**
@@ -117,13 +117,13 @@ 
 
         $testFile = __DIR__ . '/composerpackage-test-php-di.json';
 
-        $composer = new ComposerPackage($testFile);
+        $composer = new ComposerPackage( $testFile );
 
         $autoload = $composer->getAutoload();
 
-        $this->assertArrayHasKey('files', $autoload);
+        $this->assertArrayHasKey( 'files', $autoload );
 
-        $this->assertIsArray($autoload['files']);
+        $this->assertIsArray( $autoload[ 'files' ] );
     }
 
     public function testOverrideAutoload()

Braces +10 added lines, -20 removed lines

@@ -5,14 +5,12 @@ 
 use BrianHenryIE\Strauss\Composer\ComposerPackage;
 use PHPUnit\Framework\TestCase;
 
-class ComposerPackageTest extends TestCase
-{
+class ComposerPackageTest extends TestCase {
 
     /**
      * A simple test to check the getters all work.
      */
-    public function testParseJson()
-    {
+    public function testParseJson() {
 
         $testFile = __DIR__ . '/composerpackage-test-libmergepdf.json';
 
@@ -28,8 +26,7 @@ 
     /**
      * Test the dependencies' names are returned.
      */
-    public function testGetRequiresNames()
-    {
+    public function testGetRequiresNames() {
 
         $testFile = __DIR__ . '/composerpackage-test-libmergepdf.json';
 
@@ -44,8 +41,7 @@ 
     /**
      * Test PHP and ext- are not returned, since we won't be dealing with them.
      */
-    public function testGetRequiresNamesDoesNotContain()
-    {
+    public function testGetRequiresNamesDoesNotContain() {
 
         $testFile = __DIR__ . '/composerpackage-test-easypost-php.json';
 
@@ -61,8 +57,7 @@ 
     /**
      *
      */
-    public function testAutoloadPsr0()
-    {
+    public function testAutoloadPsr0() {
 
         $testFile = __DIR__ . '/composerpackage-test-easypost-php.json';
 
@@ -78,8 +73,7 @@ 
     /**
      *
      */
-    public function testAutoloadPsr4()
-    {
+    public function testAutoloadPsr4() {
 
         $testFile = __DIR__ . '/composerpackage-test-libmergepdf.json';
 
@@ -95,8 +89,7 @@ 
     /**
      *
      */
-    public function testAutoloadClassmap()
-    {
+    public function testAutoloadClassmap() {
 
         $testFile = __DIR__ . '/composerpackage-test-libmergepdf.json';
 
@@ -112,8 +105,7 @@ 
     /**
      *
      */
-    public function testAutoloadFiles()
-    {
+    public function testAutoloadFiles() {
 
         $testFile = __DIR__ . '/composerpackage-test-php-di.json';
 
@@ -126,16 +118,14 @@ 
         $this->assertIsArray($autoload['files']);
     }
 
-    public function testOverrideAutoload()
-    {
+    public function testOverrideAutoload() {
         $this->markTestIncomplete();
     }
 
     /**
      * When composer.json is not where it was specified, what error message (via Exception) should be returned?
      */
-    public function testMissingComposer()
-    {
+    public function testMissingComposer() {
         $this->markTestIncomplete();
     }
 }

vendor/brianhenryie/strauss/tests/Unit/Composer/ProjectComposerPackageTest.php

Indentation +10 added lines, -10 removed lines

@@ -9,18 +9,18 @@
 class ProjectComposerPackageTest extends TestCase
 {
 
-    /**
-     * A simple test to check the getters all work.
-     */
-    public function testParseJson()
-    {
+	/**
+	 * A simple test to check the getters all work.
+	 */
+	public function testParseJson()
+	{
 
-        $testFile = __DIR__ . '/projectcomposerpackage-test-1.json';
+		$testFile = __DIR__ . '/projectcomposerpackage-test-1.json';
 
-        $composer = new ProjectComposerPackage($testFile);
+		$composer = new ProjectComposerPackage($testFile);
 
-        $config = $composer->getStraussConfig();
+		$config = $composer->getStraussConfig();
 
-        $this->assertInstanceOf(StraussConfig::class, $config);
-    }
+		$this->assertInstanceOf(StraussConfig::class, $config);
+	}
 }

Spacing +2 added lines, -2 removed lines

@@ -17,10 +17,10 @@
 
         $testFile = __DIR__ . '/projectcomposerpackage-test-1.json';
 
-        $composer = new ProjectComposerPackage($testFile);
+        $composer = new ProjectComposerPackage( $testFile );
 
         $config = $composer->getStraussConfig();
 
-        $this->assertInstanceOf(StraussConfig::class, $config);
+        $this->assertInstanceOf( StraussConfig::class, $config );
     }
 }

Braces +2 added lines, -4 removed lines

@@ -6,14 +6,12 @@
 use BrianHenryIE\Strauss\Composer\ProjectComposerPackage;
 use PHPUnit\Framework\TestCase;
 
-class ProjectComposerPackageTest extends TestCase
-{
+class ProjectComposerPackageTest extends TestCase {
 
     /**
      * A simple test to check the getters all work.
      */
-    public function testParseJson()
-    {
+    public function testParseJson() {
 
         $testFile = __DIR__ . '/projectcomposerpackage-test-1.json';
 

vendor/brianhenryie/strauss/tests/Unit/ChangeEnumeratorTest.php

Indentation +234 added lines, -234 removed lines

@@ -12,19 +12,19 @@ 
 class ChangeEnumeratorTest extends TestCase
 {
 
-    // PREG_BACKTRACK_LIMIT_ERROR
+	// PREG_BACKTRACK_LIMIT_ERROR
 
-    // Single implied global namespace.
-    // Single named namespace.
-    // Single explicit global namespace.
-    // Multiple namespaces.
+	// Single implied global namespace.
+	// Single named namespace.
+	// Single explicit global namespace.
+	// Multiple namespaces.
 
 
 
-    public function testSingleNamespace()
-    {
+	public function testSingleNamespace()
+	{
 
-        $validPhp = <<<'EOD'
+		$validPhp = <<<'EOD'
 <?php
 namespace MyNamespace;
 
@@ -32,22 +32,22 @@ 
 }
 EOD;
 
-        $config = $this->createMock(StraussConfig::class);
-        $config->method('getNamespacePrefix')->willReturn('Prefix');
-        $sut = new ChangeEnumerator($config);
+		$config = $this->createMock(StraussConfig::class);
+		$config->method('getNamespacePrefix')->willReturn('Prefix');
+		$sut = new ChangeEnumerator($config);
 
-        $sut->find($validPhp);
+		$sut->find($validPhp);
 
-        $this->assertArrayHasKey('MyNamespace', $sut->getDiscoveredNamespaceReplacements());
-        $this->assertContains('Prefix\MyNamespace', $sut->getDiscoveredNamespaceReplacements());
+		$this->assertArrayHasKey('MyNamespace', $sut->getDiscoveredNamespaceReplacements());
+		$this->assertContains('Prefix\MyNamespace', $sut->getDiscoveredNamespaceReplacements());
 
-        $this->assertNotContains('MyClass', $sut->getDiscoveredClasses());
-    }
+		$this->assertNotContains('MyClass', $sut->getDiscoveredClasses());
+	}
 
-    public function testGlobalNamespace()
-    {
+	public function testGlobalNamespace()
+	{
 
-        $validPhp = <<<'EOD'
+		$validPhp = <<<'EOD'
 <?php
 namespace {
     class MyClass {
@@ -55,21 +55,21 @@ 
 }
 EOD;
 
-        $config = $this->createMock(StraussConfig::class);
-        $sut = new ChangeEnumerator($config);
+		$config = $this->createMock(StraussConfig::class);
+		$sut = new ChangeEnumerator($config);
 
-        $sut->find($validPhp);
+		$sut->find($validPhp);
 
-        $this->assertContains('MyClass', $sut->getDiscoveredClasses());
-    }
+		$this->assertContains('MyClass', $sut->getDiscoveredClasses());
+	}
 
-    /**
-     *
-     */
-    public function testMultipleNamespace()
-    {
+	/**
+	 *
+	 */
+	public function testMultipleNamespace()
+	{
 
-        $validPhp = <<<'EOD'
+		$validPhp = <<<'EOD'
 <?php
 namespace MyNamespace {
 }
@@ -79,24 +79,24 @@ 
 }
 EOD;
 
-        $config = $this->createMock(StraussConfig::class);
-        $sut = new ChangeEnumerator($config);
+		$config = $this->createMock(StraussConfig::class);
+		$sut = new ChangeEnumerator($config);
 
-        $sut->find($validPhp);
+		$sut->find($validPhp);
 
-        $this->assertContains('\MyNamespace', $sut->getDiscoveredNamespaceReplacements());
+		$this->assertContains('\MyNamespace', $sut->getDiscoveredNamespaceReplacements());
 
-        $this->assertContains('MyClass', $sut->getDiscoveredClasses());
-    }
+		$this->assertContains('MyClass', $sut->getDiscoveredClasses());
+	}
 
 
-    /**
-     *
-     */
-    public function testMultipleNamespaceGlobalFirst()
-    {
+	/**
+	 *
+	 */
+	public function testMultipleNamespaceGlobalFirst()
+	{
 
-        $validPhp = <<<'EOD'
+		$validPhp = <<<'EOD'
 <?php
 
 namespace {
@@ -109,25 +109,25 @@ 
 }
 EOD;
 
-        $config = $this->createMock(StraussConfig::class);
-        $sut = new ChangeEnumerator($config);
+		$config = $this->createMock(StraussConfig::class);
+		$sut = new ChangeEnumerator($config);
 
-        $sut->find($validPhp);
+		$sut->find($validPhp);
 
-        $this->assertContains('\MyNamespace', $sut->getDiscoveredNamespaceReplacements());
+		$this->assertContains('\MyNamespace', $sut->getDiscoveredNamespaceReplacements());
 
-        $this->assertContains('MyClass', $sut->getDiscoveredClasses());
-        $this->assertNotContains('MyOtherClass', $sut->getDiscoveredClasses());
-    }
+		$this->assertContains('MyClass', $sut->getDiscoveredClasses());
+		$this->assertNotContains('MyOtherClass', $sut->getDiscoveredClasses());
+	}
 
 
-    /**
-     *
-     */
-    public function testMultipleClasses()
-    {
+	/**
+	 *
+	 */
+	public function testMultipleClasses()
+	{
 
-        $validPhp = <<<'EOD'
+		$validPhp = <<<'EOD'
 <?php
 class MyClass {
 }
@@ -136,43 +136,43 @@ 
 }
 EOD;
 
-        $config = $this->createMock(StraussConfig::class);
-        $sut = new ChangeEnumerator($config);
+		$config = $this->createMock(StraussConfig::class);
+		$sut = new ChangeEnumerator($config);
 
-        $sut->find($validPhp);
+		$sut->find($validPhp);
 
-        $this->assertContains('MyClass', $sut->getDiscoveredClasses());
-        $this->assertContains('MyOtherClass', $sut->getDiscoveredClasses());
-    }
+		$this->assertContains('MyClass', $sut->getDiscoveredClasses());
+		$this->assertContains('MyOtherClass', $sut->getDiscoveredClasses());
+	}
 
-    /**
-     *
-     * @author BrianHenryIE
-     */
-    public function test_it_does_not_treat_comments_as_classes()
-    {
-        $contents = "
+	/**
+	 *
+	 * @author BrianHenryIE
+	 */
+	public function test_it_does_not_treat_comments_as_classes()
+	{
+		$contents = "
     	// A class as good as any.
     	class Whatever {
     	
     	}
     	";
 
-        $config = $this->createMock(StraussConfig::class);
-        $changeEnumerator = new ChangeEnumerator($config);
-        $changeEnumerator->find($contents);
-
-        $this->assertNotContains('as', $changeEnumerator->getDiscoveredClasses());
-        $this->assertContains('Whatever', $changeEnumerator->getDiscoveredClasses());
-    }
-
-    /**
-     *
-     * @author BrianHenryIE
-     */
-    public function test_it_does_not_treat_multiline_comments_as_classes()
-    {
-        $contents = "
+		$config = $this->createMock(StraussConfig::class);
+		$changeEnumerator = new ChangeEnumerator($config);
+		$changeEnumerator->find($contents);
+
+		$this->assertNotContains('as', $changeEnumerator->getDiscoveredClasses());
+		$this->assertContains('Whatever', $changeEnumerator->getDiscoveredClasses());
+	}
+
+	/**
+	 *
+	 * @author BrianHenryIE
+	 */
+	public function test_it_does_not_treat_multiline_comments_as_classes()
+	{
+		$contents = "
     	 /**
     	  * A class as good as any; class as.
     	  */
@@ -180,24 +180,24 @@ 
     	}
     	";
 
-        $config = $this->createMock(StraussConfig::class);
-        $changeEnumerator = new ChangeEnumerator($config);
-        $changeEnumerator->find($contents);
-
-        $this->assertNotContains('as', $changeEnumerator->getDiscoveredClasses());
-        $this->assertContains('Whatever', $changeEnumerator->getDiscoveredClasses());
-    }
-
-    /**
-     * This worked without adding the expected regex:
-     *
-     * // \s*\\/?\\*{2,}[^\n]* |                        # Skip multiline comment bodies
-     *
-     * @author BrianHenryIE
-     */
-    public function test_it_does_not_treat_multiline_comments_opening_line_as_classes()
-    {
-        $contents = "
+		$config = $this->createMock(StraussConfig::class);
+		$changeEnumerator = new ChangeEnumerator($config);
+		$changeEnumerator->find($contents);
+
+		$this->assertNotContains('as', $changeEnumerator->getDiscoveredClasses());
+		$this->assertContains('Whatever', $changeEnumerator->getDiscoveredClasses());
+	}
+
+	/**
+	 * This worked without adding the expected regex:
+	 *
+	 * // \s*\\/?\\*{2,}[^\n]* |                        # Skip multiline comment bodies
+	 *
+	 * @author BrianHenryIE
+	 */
+	public function test_it_does_not_treat_multiline_comments_opening_line_as_classes()
+	{
+		$contents = "
     	 /** A class as good as any; class as.
     	  *
     	  */
@@ -205,110 +205,110 @@ 
     	}
     	";
 
-        $config = $this->createMock(StraussConfig::class);
-        $changeEnumerator = new ChangeEnumerator($config);
-        $changeEnumerator->find($contents);
+		$config = $this->createMock(StraussConfig::class);
+		$changeEnumerator = new ChangeEnumerator($config);
+		$changeEnumerator->find($contents);
 
-        $this->assertNotContains('as', $changeEnumerator->getDiscoveredClasses());
-        $this->assertContains('Whatever', $changeEnumerator->getDiscoveredClasses());
-    }
+		$this->assertNotContains('as', $changeEnumerator->getDiscoveredClasses());
+		$this->assertContains('Whatever', $changeEnumerator->getDiscoveredClasses());
+	}
 
 
-    /**
-     *
-     * @author BrianHenryIE
-     */
-    public function test_it_does_not_treat_multiline_comments_on_one_line_as_classes()
-    {
-        $contents = "
+	/**
+	 *
+	 * @author BrianHenryIE
+	 */
+	public function test_it_does_not_treat_multiline_comments_on_one_line_as_classes()
+	{
+		$contents = "
     	 /** A class as good as any; class as. */ class Whatever_Trevor {
     	}
     	";
 
-        $config = $this->createMock(StraussConfig::class);
-        $changeEnumerator = new ChangeEnumerator($config);
-        $changeEnumerator->find($contents);
-
-        $this->assertNotContains('as', $changeEnumerator->getDiscoveredClasses());
-        $this->assertContains('Whatever_Trevor', $changeEnumerator->getDiscoveredClasses());
-    }
-
-    /**
-     * If someone were to put a semicolon in the comment it would mess with the previous fix.
-     *
-     * @author BrianHenryIE
-     *
-     * @test
-     */
-    public function test_it_does_not_treat_comments_with_semicolons_as_classes()
-    {
-        $contents = "
+		$config = $this->createMock(StraussConfig::class);
+		$changeEnumerator = new ChangeEnumerator($config);
+		$changeEnumerator->find($contents);
+
+		$this->assertNotContains('as', $changeEnumerator->getDiscoveredClasses());
+		$this->assertContains('Whatever_Trevor', $changeEnumerator->getDiscoveredClasses());
+	}
+
+	/**
+	 * If someone were to put a semicolon in the comment it would mess with the previous fix.
+	 *
+	 * @author BrianHenryIE
+	 *
+	 * @test
+	 */
+	public function test_it_does_not_treat_comments_with_semicolons_as_classes()
+	{
+		$contents = "
     	// A class as good as any; class as versatile as any.
     	class Whatever_Ever {
     	
     	}
     	";
 
-        $config = $this->createMock(StraussConfig::class);
-        $changeEnumerator = new ChangeEnumerator($config);
-        $changeEnumerator->find($contents);
+		$config = $this->createMock(StraussConfig::class);
+		$changeEnumerator = new ChangeEnumerator($config);
+		$changeEnumerator->find($contents);
 
-        $this->assertNotContains('as', $changeEnumerator->getDiscoveredClasses());
-        $this->assertContains('Whatever_Ever', $changeEnumerator->getDiscoveredClasses());
-    }
+		$this->assertNotContains('as', $changeEnumerator->getDiscoveredClasses());
+		$this->assertContains('Whatever_Ever', $changeEnumerator->getDiscoveredClasses());
+	}
 
-    /**
-     * @author BrianHenryIE
-     */
-    public function test_it_parses_classes_after_semicolon()
-    {
+	/**
+	 * @author BrianHenryIE
+	 */
+	public function test_it_parses_classes_after_semicolon()
+	{
 
-        $contents = "
+		$contents = "
 	    myvar = 123; class Pear { };
 	    ";
 
-        $config = $this->createMock(StraussConfig::class);
-        $changeEnumerator = new ChangeEnumerator($config);
-        $changeEnumerator->find($contents);
+		$config = $this->createMock(StraussConfig::class);
+		$changeEnumerator = new ChangeEnumerator($config);
+		$changeEnumerator->find($contents);
 
-        $this->assertContains('Pear', $changeEnumerator->getDiscoveredClasses());
-    }
+		$this->assertContains('Pear', $changeEnumerator->getDiscoveredClasses());
+	}
 
 
-    /**
-     * @author BrianHenryIE
-     */
-    public function test_it_parses_classes_followed_by_comment()
-    {
+	/**
+	 * @author BrianHenryIE
+	 */
+	public function test_it_parses_classes_followed_by_comment()
+	{
 
-        $contents = <<<'EOD'
+		$contents = <<<'EOD'
 	class WP_Dependency_Installer {
 		/**
 		 *
 		 */
 EOD;
 
-        $config = $this->createMock(StraussConfig::class);
-        $changeEnumerator = new ChangeEnumerator($config);
-        $changeEnumerator->find($contents);
+		$config = $this->createMock(StraussConfig::class);
+		$changeEnumerator = new ChangeEnumerator($config);
+		$changeEnumerator->find($contents);
 
-        $this->assertContains('WP_Dependency_Installer', $changeEnumerator->getDiscoveredClasses());
-    }
+		$this->assertContains('WP_Dependency_Installer', $changeEnumerator->getDiscoveredClasses());
+	}
 
 
-    /**
-     * It's possible to have multiple namespaces inside one file.
-     *
-     * To have two classes in one file, one in a namespace and the other not, the global namespace needs to be explicit.
-     *
-     * @author BrianHenryIE
-     *
-     * @test
-     */
-    public function it_does_not_replace_inside_named_namespace_but_does_inside_explicit_global_namespace_a(): void
-    {
+	/**
+	 * It's possible to have multiple namespaces inside one file.
+	 *
+	 * To have two classes in one file, one in a namespace and the other not, the global namespace needs to be explicit.
+	 *
+	 * @author BrianHenryIE
+	 *
+	 * @test
+	 */
+	public function it_does_not_replace_inside_named_namespace_but_does_inside_explicit_global_namespace_a(): void
+	{
 
-        $contents = "
+		$contents = "
 		namespace My_Project {
 			class A_Class { }
 		}
@@ -317,85 +317,85 @@ 
 		}
 		";
 
-        $config = $this->createMock(StraussConfig::class);
-        $changeEnumerator = new ChangeEnumerator($config);
-        $changeEnumerator->find($contents);
+		$config = $this->createMock(StraussConfig::class);
+		$changeEnumerator = new ChangeEnumerator($config);
+		$changeEnumerator->find($contents);
 
-        $this->assertNotContains('A_Class', $changeEnumerator->getDiscoveredClasses());
-        $this->assertContains('B_Class', $changeEnumerator->getDiscoveredClasses());
-    }
+		$this->assertNotContains('A_Class', $changeEnumerator->getDiscoveredClasses());
+		$this->assertContains('B_Class', $changeEnumerator->getDiscoveredClasses());
+	}
 
-    public function testExcludePackagesFromPrefix()
-    {
+	public function testExcludePackagesFromPrefix()
+	{
 
-        $config = $this->createMock(StraussConfig::class);
-        $config->method('getExcludePackagesFromPrefixing')->willReturn(
-            array('brianhenryie/pdfhelpers')
-        );
+		$config = $this->createMock(StraussConfig::class);
+		$config->method('getExcludePackagesFromPrefixing')->willReturn(
+			array('brianhenryie/pdfhelpers')
+		);
 
-        $dir = '';
-        $composerPackage = $this->createMock(ComposerPackage::class);
-        $composerPackage->method('getName')->willReturn('brianhenryie/pdfhelpers');
-        $relativeFilepaths = array( 'irrelevent' => $composerPackage);
+		$dir = '';
+		$composerPackage = $this->createMock(ComposerPackage::class);
+		$composerPackage->method('getName')->willReturn('brianhenryie/pdfhelpers');
+		$relativeFilepaths = array( 'irrelevent' => $composerPackage);
 
-        $changeEnumerator = new ChangeEnumerator($config);
-        $changeEnumerator->findInFiles($dir, $relativeFilepaths);
+		$changeEnumerator = new ChangeEnumerator($config);
+		$changeEnumerator->findInFiles($dir, $relativeFilepaths);
 
-        $this->assertEmpty($changeEnumerator->getDiscoveredNamespaceReplacements());
-    }
+		$this->assertEmpty($changeEnumerator->getDiscoveredNamespaceReplacements());
+	}
 
 
-    public function testExcludeFilePatternsFromPrefix()
-    {
-        $config = $this->createMock(StraussConfig::class);
-        $config->method('getExcludeFilePatternsFromPrefixing')->willReturn(
-            array('/to/')
-        );
+	public function testExcludeFilePatternsFromPrefix()
+	{
+		$config = $this->createMock(StraussConfig::class);
+		$config->method('getExcludeFilePatternsFromPrefixing')->willReturn(
+			array('/to/')
+		);
 
-        $dir = '';
-        $composerPackage = $this->createMock(ComposerPackage::class);
-        $composerPackage->method('getName')->willReturn('brianhenryie/pdfhelpers');
-        $relativeFilepaths = array( 'path/to/file' => $composerPackage);
+		$dir = '';
+		$composerPackage = $this->createMock(ComposerPackage::class);
+		$composerPackage->method('getName')->willReturn('brianhenryie/pdfhelpers');
+		$relativeFilepaths = array( 'path/to/file' => $composerPackage);
 
-        $changeEnumerator = new ChangeEnumerator($config);
-        $changeEnumerator->findInFiles($dir, $relativeFilepaths);
+		$changeEnumerator = new ChangeEnumerator($config);
+		$changeEnumerator->findInFiles($dir, $relativeFilepaths);
 
-        $this->assertEmpty($changeEnumerator->getDiscoveredNamespaceReplacements());
-    }
+		$this->assertEmpty($changeEnumerator->getDiscoveredNamespaceReplacements());
+	}
 
-    /**
-     * Test custom replacements
-     */
-    public function testNamespaceReplacementPatterns()
-    {
+	/**
+	 * Test custom replacements
+	 */
+	public function testNamespaceReplacementPatterns()
+	{
 
-        $contents = "
+		$contents = "
 		namespace BrianHenryIE\PdfHelpers {
 			class A_Class { }
 		}
 		";
 
-        $config = $this->createMock(StraussConfig::class);
-        $config->method('getNamespacePrefix')->willReturn('BrianHenryIE\Prefix');
-        $config->method('getNamespaceReplacementPatterns')->willReturn(
-            array('/BrianHenryIE\\\\(PdfHelpers)/'=>'BrianHenryIE\\Prefix\\\\$1')
-        );
+		$config = $this->createMock(StraussConfig::class);
+		$config->method('getNamespacePrefix')->willReturn('BrianHenryIE\Prefix');
+		$config->method('getNamespaceReplacementPatterns')->willReturn(
+			array('/BrianHenryIE\\\\(PdfHelpers)/'=>'BrianHenryIE\\Prefix\\\\$1')
+		);
 
-        $changeEnumerator = new ChangeEnumerator($config);
-        $changeEnumerator->find($contents);
+		$changeEnumerator = new ChangeEnumerator($config);
+		$changeEnumerator->find($contents);
 
-        $this->assertArrayHasKey('BrianHenryIE\PdfHelpers', $changeEnumerator->getDiscoveredNamespaceReplacements());
-        $this->assertContains('BrianHenryIE\Prefix\PdfHelpers', $changeEnumerator->getDiscoveredNamespaceReplacements());
-        $this->assertNotContains('BrianHenryIE\Prefix\BrianHenryIE\PdfHelpers', $changeEnumerator->getDiscoveredNamespaceReplacements());
-    }
+		$this->assertArrayHasKey('BrianHenryIE\PdfHelpers', $changeEnumerator->getDiscoveredNamespaceReplacements());
+		$this->assertContains('BrianHenryIE\Prefix\PdfHelpers', $changeEnumerator->getDiscoveredNamespaceReplacements());
+		$this->assertNotContains('BrianHenryIE\Prefix\BrianHenryIE\PdfHelpers', $changeEnumerator->getDiscoveredNamespaceReplacements());
+	}
 
-    /**
-     * @see https://github.com/BrianHenryIE/strauss/issues/19
-     */
-    public function testPhraseClassObjectIsNotMistaken()
-    {
+	/**
+	 * @see https://github.com/BrianHenryIE/strauss/issues/19
+	 */
+	public function testPhraseClassObjectIsNotMistaken()
+	{
 
-        $contents = <<<'EOD'
+		$contents = <<<'EOD'
 <?php
 
 class TCPDF_STATIC
@@ -419,10 +419,10 @@ 
 }
 EOD;
 
-        $config = $this->createMock(StraussConfig::class);
-        $changeEnumerator = new ChangeEnumerator($config);
-        $changeEnumerator->find($contents);
+		$config = $this->createMock(StraussConfig::class);
+		$changeEnumerator = new ChangeEnumerator($config);
+		$changeEnumerator->find($contents);
 
-        $this->assertNotContains('object', $changeEnumerator->getDiscoveredClasses());
-    }
+		$this->assertNotContains('object', $changeEnumerator->getDiscoveredClasses());
+	}
 }

Spacing +96 added lines, -96 removed lines

@@ -32,16 +32,16 @@ 
 }
 EOD;
 
-        $config = $this->createMock(StraussConfig::class);
-        $config->method('getNamespacePrefix')->willReturn('Prefix');
-        $sut = new ChangeEnumerator($config);
+        $config = $this->createMock( StraussConfig::class );
+        $config->method( 'getNamespacePrefix' )->willReturn( 'Prefix' );
+        $sut = new ChangeEnumerator( $config );
 
-        $sut->find($validPhp);
+        $sut->find( $validPhp );
 
-        $this->assertArrayHasKey('MyNamespace', $sut->getDiscoveredNamespaceReplacements());
-        $this->assertContains('Prefix\MyNamespace', $sut->getDiscoveredNamespaceReplacements());
+        $this->assertArrayHasKey( 'MyNamespace', $sut->getDiscoveredNamespaceReplacements() );
+        $this->assertContains( 'Prefix\MyNamespace', $sut->getDiscoveredNamespaceReplacements() );
 
-        $this->assertNotContains('MyClass', $sut->getDiscoveredClasses());
+        $this->assertNotContains( 'MyClass', $sut->getDiscoveredClasses() );
     }
 
     public function testGlobalNamespace()
@@ -55,12 +55,12 @@ 
 }
 EOD;
 
-        $config = $this->createMock(StraussConfig::class);
-        $sut = new ChangeEnumerator($config);
+        $config = $this->createMock( StraussConfig::class );
+        $sut = new ChangeEnumerator( $config );
 
-        $sut->find($validPhp);
+        $sut->find( $validPhp );
 
-        $this->assertContains('MyClass', $sut->getDiscoveredClasses());
+        $this->assertContains( 'MyClass', $sut->getDiscoveredClasses() );
     }
 
     /**
@@ -79,14 +79,14 @@ 
 }
 EOD;
 
-        $config = $this->createMock(StraussConfig::class);
-        $sut = new ChangeEnumerator($config);
+        $config = $this->createMock( StraussConfig::class );
+        $sut = new ChangeEnumerator( $config );
 
-        $sut->find($validPhp);
+        $sut->find( $validPhp );
 
-        $this->assertContains('\MyNamespace', $sut->getDiscoveredNamespaceReplacements());
+        $this->assertContains( '\MyNamespace', $sut->getDiscoveredNamespaceReplacements() );
 
-        $this->assertContains('MyClass', $sut->getDiscoveredClasses());
+        $this->assertContains( 'MyClass', $sut->getDiscoveredClasses() );
     }
 
 
@@ -109,15 +109,15 @@ 
 }
 EOD;
 
-        $config = $this->createMock(StraussConfig::class);
-        $sut = new ChangeEnumerator($config);
+        $config = $this->createMock( StraussConfig::class );
+        $sut = new ChangeEnumerator( $config );
 
-        $sut->find($validPhp);
+        $sut->find( $validPhp );
 
-        $this->assertContains('\MyNamespace', $sut->getDiscoveredNamespaceReplacements());
+        $this->assertContains( '\MyNamespace', $sut->getDiscoveredNamespaceReplacements() );
 
-        $this->assertContains('MyClass', $sut->getDiscoveredClasses());
-        $this->assertNotContains('MyOtherClass', $sut->getDiscoveredClasses());
+        $this->assertContains( 'MyClass', $sut->getDiscoveredClasses() );
+        $this->assertNotContains( 'MyOtherClass', $sut->getDiscoveredClasses() );
     }
 
 
@@ -136,13 +136,13 @@ 
 }
 EOD;
 
-        $config = $this->createMock(StraussConfig::class);
-        $sut = new ChangeEnumerator($config);
+        $config = $this->createMock( StraussConfig::class );
+        $sut = new ChangeEnumerator( $config );
 
-        $sut->find($validPhp);
+        $sut->find( $validPhp );
 
-        $this->assertContains('MyClass', $sut->getDiscoveredClasses());
-        $this->assertContains('MyOtherClass', $sut->getDiscoveredClasses());
+        $this->assertContains( 'MyClass', $sut->getDiscoveredClasses() );
+        $this->assertContains( 'MyOtherClass', $sut->getDiscoveredClasses() );
     }
 
     /**
@@ -158,12 +158,12 @@ 
     	}
     	";
 
-        $config = $this->createMock(StraussConfig::class);
-        $changeEnumerator = new ChangeEnumerator($config);
-        $changeEnumerator->find($contents);
+        $config = $this->createMock( StraussConfig::class );
+        $changeEnumerator = new ChangeEnumerator( $config );
+        $changeEnumerator->find( $contents );
 
-        $this->assertNotContains('as', $changeEnumerator->getDiscoveredClasses());
-        $this->assertContains('Whatever', $changeEnumerator->getDiscoveredClasses());
+        $this->assertNotContains( 'as', $changeEnumerator->getDiscoveredClasses() );
+        $this->assertContains( 'Whatever', $changeEnumerator->getDiscoveredClasses() );
     }
 
     /**
@@ -180,12 +180,12 @@ 
     	}
     	";
 
-        $config = $this->createMock(StraussConfig::class);
-        $changeEnumerator = new ChangeEnumerator($config);
-        $changeEnumerator->find($contents);
+        $config = $this->createMock( StraussConfig::class );
+        $changeEnumerator = new ChangeEnumerator( $config );
+        $changeEnumerator->find( $contents );
 
-        $this->assertNotContains('as', $changeEnumerator->getDiscoveredClasses());
-        $this->assertContains('Whatever', $changeEnumerator->getDiscoveredClasses());
+        $this->assertNotContains( 'as', $changeEnumerator->getDiscoveredClasses() );
+        $this->assertContains( 'Whatever', $changeEnumerator->getDiscoveredClasses() );
     }
 
     /**
@@ -205,12 +205,12 @@ 
     	}
     	";
 
-        $config = $this->createMock(StraussConfig::class);
-        $changeEnumerator = new ChangeEnumerator($config);
-        $changeEnumerator->find($contents);
+        $config = $this->createMock( StraussConfig::class );
+        $changeEnumerator = new ChangeEnumerator( $config );
+        $changeEnumerator->find( $contents );
 
-        $this->assertNotContains('as', $changeEnumerator->getDiscoveredClasses());
-        $this->assertContains('Whatever', $changeEnumerator->getDiscoveredClasses());
+        $this->assertNotContains( 'as', $changeEnumerator->getDiscoveredClasses() );
+        $this->assertContains( 'Whatever', $changeEnumerator->getDiscoveredClasses() );
     }
 
 
@@ -225,12 +225,12 @@ 
     	}
     	";
 
-        $config = $this->createMock(StraussConfig::class);
-        $changeEnumerator = new ChangeEnumerator($config);
-        $changeEnumerator->find($contents);
+        $config = $this->createMock( StraussConfig::class );
+        $changeEnumerator = new ChangeEnumerator( $config );
+        $changeEnumerator->find( $contents );
 
-        $this->assertNotContains('as', $changeEnumerator->getDiscoveredClasses());
-        $this->assertContains('Whatever_Trevor', $changeEnumerator->getDiscoveredClasses());
+        $this->assertNotContains( 'as', $changeEnumerator->getDiscoveredClasses() );
+        $this->assertContains( 'Whatever_Trevor', $changeEnumerator->getDiscoveredClasses() );
     }
 
     /**
@@ -249,12 +249,12 @@ 
     	}
     	";
 
-        $config = $this->createMock(StraussConfig::class);
-        $changeEnumerator = new ChangeEnumerator($config);
-        $changeEnumerator->find($contents);
+        $config = $this->createMock( StraussConfig::class );
+        $changeEnumerator = new ChangeEnumerator( $config );
+        $changeEnumerator->find( $contents );
 
-        $this->assertNotContains('as', $changeEnumerator->getDiscoveredClasses());
-        $this->assertContains('Whatever_Ever', $changeEnumerator->getDiscoveredClasses());
+        $this->assertNotContains( 'as', $changeEnumerator->getDiscoveredClasses() );
+        $this->assertContains( 'Whatever_Ever', $changeEnumerator->getDiscoveredClasses() );
     }
 
     /**
@@ -267,11 +267,11 @@ 
 	    myvar = 123; class Pear { };
 	    ";
 
-        $config = $this->createMock(StraussConfig::class);
-        $changeEnumerator = new ChangeEnumerator($config);
-        $changeEnumerator->find($contents);
+        $config = $this->createMock( StraussConfig::class );
+        $changeEnumerator = new ChangeEnumerator( $config );
+        $changeEnumerator->find( $contents );
 
-        $this->assertContains('Pear', $changeEnumerator->getDiscoveredClasses());
+        $this->assertContains( 'Pear', $changeEnumerator->getDiscoveredClasses() );
     }
 
 
@@ -288,11 +288,11 @@ 
 		 */
 EOD;
 
-        $config = $this->createMock(StraussConfig::class);
-        $changeEnumerator = new ChangeEnumerator($config);
-        $changeEnumerator->find($contents);
+        $config = $this->createMock( StraussConfig::class );
+        $changeEnumerator = new ChangeEnumerator( $config );
+        $changeEnumerator->find( $contents );
 
-        $this->assertContains('WP_Dependency_Installer', $changeEnumerator->getDiscoveredClasses());
+        $this->assertContains( 'WP_Dependency_Installer', $changeEnumerator->getDiscoveredClasses() );
     }
 
 
@@ -317,50 +317,50 @@ 
 		}
 		";
 
-        $config = $this->createMock(StraussConfig::class);
-        $changeEnumerator = new ChangeEnumerator($config);
-        $changeEnumerator->find($contents);
+        $config = $this->createMock( StraussConfig::class );
+        $changeEnumerator = new ChangeEnumerator( $config );
+        $changeEnumerator->find( $contents );
 
-        $this->assertNotContains('A_Class', $changeEnumerator->getDiscoveredClasses());
-        $this->assertContains('B_Class', $changeEnumerator->getDiscoveredClasses());
+        $this->assertNotContains( 'A_Class', $changeEnumerator->getDiscoveredClasses() );
+        $this->assertContains( 'B_Class', $changeEnumerator->getDiscoveredClasses() );
     }
 
     public function testExcludePackagesFromPrefix()
     {
 
-        $config = $this->createMock(StraussConfig::class);
-        $config->method('getExcludePackagesFromPrefixing')->willReturn(
-            array('brianhenryie/pdfhelpers')
+        $config = $this->createMock( StraussConfig::class );
+        $config->method( 'getExcludePackagesFromPrefixing' )->willReturn(
+            array( 'brianhenryie/pdfhelpers' )
         );
 
         $dir = '';
-        $composerPackage = $this->createMock(ComposerPackage::class);
-        $composerPackage->method('getName')->willReturn('brianhenryie/pdfhelpers');
-        $relativeFilepaths = array( 'irrelevent' => $composerPackage);
+        $composerPackage = $this->createMock( ComposerPackage::class );
+        $composerPackage->method( 'getName' )->willReturn( 'brianhenryie/pdfhelpers' );
+        $relativeFilepaths = array( 'irrelevent' => $composerPackage );
 
-        $changeEnumerator = new ChangeEnumerator($config);
-        $changeEnumerator->findInFiles($dir, $relativeFilepaths);
+        $changeEnumerator = new ChangeEnumerator( $config );
+        $changeEnumerator->findInFiles( $dir, $relativeFilepaths );
 
-        $this->assertEmpty($changeEnumerator->getDiscoveredNamespaceReplacements());
+        $this->assertEmpty( $changeEnumerator->getDiscoveredNamespaceReplacements() );
     }
 
 
     public function testExcludeFilePatternsFromPrefix()
     {
-        $config = $this->createMock(StraussConfig::class);
-        $config->method('getExcludeFilePatternsFromPrefixing')->willReturn(
-            array('/to/')
+        $config = $this->createMock( StraussConfig::class );
+        $config->method( 'getExcludeFilePatternsFromPrefixing' )->willReturn(
+            array( '/to/' )
         );
 
         $dir = '';
-        $composerPackage = $this->createMock(ComposerPackage::class);
-        $composerPackage->method('getName')->willReturn('brianhenryie/pdfhelpers');
-        $relativeFilepaths = array( 'path/to/file' => $composerPackage);
+        $composerPackage = $this->createMock( ComposerPackage::class );
+        $composerPackage->method( 'getName' )->willReturn( 'brianhenryie/pdfhelpers' );
+        $relativeFilepaths = array( 'path/to/file' => $composerPackage );
 
-        $changeEnumerator = new ChangeEnumerator($config);
-        $changeEnumerator->findInFiles($dir, $relativeFilepaths);
+        $changeEnumerator = new ChangeEnumerator( $config );
+        $changeEnumerator->findInFiles( $dir, $relativeFilepaths );
 
-        $this->assertEmpty($changeEnumerator->getDiscoveredNamespaceReplacements());
+        $this->assertEmpty( $changeEnumerator->getDiscoveredNamespaceReplacements() );
     }
 
     /**
@@ -375,18 +375,18 @@ 
 		}
 		";
 
-        $config = $this->createMock(StraussConfig::class);
-        $config->method('getNamespacePrefix')->willReturn('BrianHenryIE\Prefix');
-        $config->method('getNamespaceReplacementPatterns')->willReturn(
-            array('/BrianHenryIE\\\\(PdfHelpers)/'=>'BrianHenryIE\\Prefix\\\\$1')
+        $config = $this->createMock( StraussConfig::class );
+        $config->method( 'getNamespacePrefix' )->willReturn( 'BrianHenryIE\Prefix' );
+        $config->method( 'getNamespaceReplacementPatterns' )->willReturn(
+            array( '/BrianHenryIE\\\\(PdfHelpers)/'=>'BrianHenryIE\\Prefix\\\\$1' )
         );
 
-        $changeEnumerator = new ChangeEnumerator($config);
-        $changeEnumerator->find($contents);
+        $changeEnumerator = new ChangeEnumerator( $config );
+        $changeEnumerator->find( $contents );
 
-        $this->assertArrayHasKey('BrianHenryIE\PdfHelpers', $changeEnumerator->getDiscoveredNamespaceReplacements());
-        $this->assertContains('BrianHenryIE\Prefix\PdfHelpers', $changeEnumerator->getDiscoveredNamespaceReplacements());
-        $this->assertNotContains('BrianHenryIE\Prefix\BrianHenryIE\PdfHelpers', $changeEnumerator->getDiscoveredNamespaceReplacements());
+        $this->assertArrayHasKey( 'BrianHenryIE\PdfHelpers', $changeEnumerator->getDiscoveredNamespaceReplacements() );
+        $this->assertContains( 'BrianHenryIE\Prefix\PdfHelpers', $changeEnumerator->getDiscoveredNamespaceReplacements() );
+        $this->assertNotContains( 'BrianHenryIE\Prefix\BrianHenryIE\PdfHelpers', $changeEnumerator->getDiscoveredNamespaceReplacements() );
     }
 
     /**
@@ -419,10 +419,10 @@ 
 }
 EOD;
 
-        $config = $this->createMock(StraussConfig::class);
-        $changeEnumerator = new ChangeEnumerator($config);
-        $changeEnumerator->find($contents);
+        $config = $this->createMock( StraussConfig::class );
+        $changeEnumerator = new ChangeEnumerator( $config );
+        $changeEnumerator->find( $contents );
 
-        $this->assertNotContains('object', $changeEnumerator->getDiscoveredClasses());
+        $this->assertNotContains( 'object', $changeEnumerator->getDiscoveredClasses() );
     }
 }

Braces +17 added lines, -34 removed lines

@@ -9,8 +9,7 @@ 
 use Composer\Composer;
 use PHPUnit\Framework\TestCase;
 
-class ChangeEnumeratorTest extends TestCase
-{
+class ChangeEnumeratorTest extends TestCase {
 
     // PREG_BACKTRACK_LIMIT_ERROR
 
@@ -21,8 +20,7 @@ 
 
 
 
-    public function testSingleNamespace()
-    {
+    public function testSingleNamespace() {
 
         $validPhp = <<<'EOD'
 <?php
@@ -44,8 +42,7 @@ 
         $this->assertNotContains('MyClass', $sut->getDiscoveredClasses());
     }
 
-    public function testGlobalNamespace()
-    {
+    public function testGlobalNamespace() {
 
         $validPhp = <<<'EOD'
 <?php
@@ -66,8 +63,7 @@ 
     /**
      *
      */
-    public function testMultipleNamespace()
-    {
+    public function testMultipleNamespace() {
 
         $validPhp = <<<'EOD'
 <?php
@@ -93,8 +89,7 @@ 
     /**
      *
      */
-    public function testMultipleNamespaceGlobalFirst()
-    {
+    public function testMultipleNamespaceGlobalFirst() {
 
         $validPhp = <<<'EOD'
 <?php
@@ -124,8 +119,7 @@ 
     /**
      *
      */
-    public function testMultipleClasses()
-    {
+    public function testMultipleClasses() {
 
         $validPhp = <<<'EOD'
 <?php
@@ -149,8 +143,7 @@ 
      *
      * @author BrianHenryIE
      */
-    public function test_it_does_not_treat_comments_as_classes()
-    {
+    public function test_it_does_not_treat_comments_as_classes() {
         $contents = "
     	// A class as good as any.
     	class Whatever {
@@ -170,8 +163,7 @@ 
      *
      * @author BrianHenryIE
      */
-    public function test_it_does_not_treat_multiline_comments_as_classes()
-    {
+    public function test_it_does_not_treat_multiline_comments_as_classes() {
         $contents = "
     	 /**
     	  * A class as good as any; class as.
@@ -195,8 +187,7 @@ 
      *
      * @author BrianHenryIE
      */
-    public function test_it_does_not_treat_multiline_comments_opening_line_as_classes()
-    {
+    public function test_it_does_not_treat_multiline_comments_opening_line_as_classes() {
         $contents = "
     	 /** A class as good as any; class as.
     	  *
@@ -218,8 +209,7 @@ 
      *
      * @author BrianHenryIE
      */
-    public function test_it_does_not_treat_multiline_comments_on_one_line_as_classes()
-    {
+    public function test_it_does_not_treat_multiline_comments_on_one_line_as_classes() {
         $contents = "
     	 /** A class as good as any; class as. */ class Whatever_Trevor {
     	}
@@ -240,8 +230,7 @@ 
      *
      * @test
      */
-    public function test_it_does_not_treat_comments_with_semicolons_as_classes()
-    {
+    public function test_it_does_not_treat_comments_with_semicolons_as_classes() {
         $contents = "
     	// A class as good as any; class as versatile as any.
     	class Whatever_Ever {
@@ -260,8 +249,7 @@ 
     /**
      * @author BrianHenryIE
      */
-    public function test_it_parses_classes_after_semicolon()
-    {
+    public function test_it_parses_classes_after_semicolon() {
 
         $contents = "
 	    myvar = 123; class Pear { };
@@ -278,8 +266,7 @@ 
     /**
      * @author BrianHenryIE
      */
-    public function test_it_parses_classes_followed_by_comment()
-    {
+    public function test_it_parses_classes_followed_by_comment() {
 
         $contents = <<<'EOD'
 	class WP_Dependency_Installer {
@@ -325,8 +312,7 @@ 
         $this->assertContains('B_Class', $changeEnumerator->getDiscoveredClasses());
     }
 
-    public function testExcludePackagesFromPrefix()
-    {
+    public function testExcludePackagesFromPrefix() {
 
         $config = $this->createMock(StraussConfig::class);
         $config->method('getExcludePackagesFromPrefixing')->willReturn(
@@ -345,8 +331,7 @@ 
     }
 
 
-    public function testExcludeFilePatternsFromPrefix()
-    {
+    public function testExcludeFilePatternsFromPrefix() {
         $config = $this->createMock(StraussConfig::class);
         $config->method('getExcludeFilePatternsFromPrefixing')->willReturn(
             array('/to/')
@@ -366,8 +351,7 @@ 
     /**
      * Test custom replacements
      */
-    public function testNamespaceReplacementPatterns()
-    {
+    public function testNamespaceReplacementPatterns() {
 
         $contents = "
 		namespace BrianHenryIE\PdfHelpers {
@@ -392,8 +376,7 @@ 
     /**
      * @see https://github.com/BrianHenryIE/strauss/issues/19
      */
-    public function testPhraseClassObjectIsNotMistaken()
-    {
+    public function testPhraseClassObjectIsNotMistaken() {
 
         $contents = <<<'EOD'
 <?php

Upper-Lower-Casing +7 added lines, -7 removed lines

@@ -30,7 +30,7 @@ 
 
 class MyClass {
 }
-EOD;
+eod;
 
         $config = $this->createMock(StraussConfig::class);
         $config->method('getNamespacePrefix')->willReturn('Prefix');
@@ -53,7 +53,7 @@ 
     class MyClass {
     }
 }
-EOD;
+eod;
 
         $config = $this->createMock(StraussConfig::class);
         $sut = new ChangeEnumerator($config);
@@ -77,7 +77,7 @@ 
     class MyClass {
     }
 }
-EOD;
+eod;
 
         $config = $this->createMock(StraussConfig::class);
         $sut = new ChangeEnumerator($config);
@@ -107,7 +107,7 @@ 
     class MyOtherClass {
     }
 }
-EOD;
+eod;
 
         $config = $this->createMock(StraussConfig::class);
         $sut = new ChangeEnumerator($config);
@@ -134,7 +134,7 @@ 
 class MyOtherClass {
 
 }
-EOD;
+eod;
 
         $config = $this->createMock(StraussConfig::class);
         $sut = new ChangeEnumerator($config);
@@ -286,7 +286,7 @@ 
 		/**
 		 *
 		 */
-EOD;
+eod;
 
         $config = $this->createMock(StraussConfig::class);
         $changeEnumerator = new ChangeEnumerator($config);
@@ -417,7 +417,7 @@ 
         return @clone($object);
     }
 }
-EOD;
+eod;
 
         $config = $this->createMock(StraussConfig::class);
         $changeEnumerator = new ChangeEnumerator($config);

vendor/brianhenryie/strauss/tests/Unit/PrefixerTest.php

Indentation +474 added lines, -474 removed lines

@@ -23,13 +23,13 @@ 
 class PrefixerTest extends TestCase
 {
 
-    protected StraussConfig $config;
+	protected StraussConfig $config;
 
-    protected function aaasetUp(): void
-    {
-        parent::setUp();
+	protected function aaasetUp(): void
+	{
+		parent::setUp();
 
-        $composerJson = <<<'EOD'
+		$composerJson = <<<'EOD'
 {
 "name": "brianhenryie/strauss-replacer-test",
 "extra": {
@@ -38,18 +38,18 @@ 
 }
 EOD;
 
-        $composerConfig = new Config(false);
-        $composerConfig->merge(json_decode($composerJson, true));
-        $composer = new Composer();
-        $composer->setConfig($composerConfig);
+		$composerConfig = new Config(false);
+		$composerConfig->merge(json_decode($composerJson, true));
+		$composer = new Composer();
+		$composer->setConfig($composerConfig);
 
-        $this->config = new StraussConfig($composer);
-    }
+		$this->config = new StraussConfig($composer);
+	}
 
-    public function testNamespaceReplacer()
-    {
+	public function testNamespaceReplacer()
+	{
 
-        $contents = <<<'EOD'
+		$contents = <<<'EOD'
 <?php
 /*
  * Copyright 2010 Google Inc.
@@ -122,25 +122,25 @@ 
   }
 }
 EOD;
-        $config = $this->createMock(StraussConfig::class);
+		$config = $this->createMock(StraussConfig::class);
 
-        $replacer = new Prefixer($config, __DIR__);
+		$replacer = new Prefixer($config, __DIR__);
 
-        $original = "Google\\Http\\Batch";
-        $replacement = "BrianHenryIE\\Strauss\\Google\\Http\\Batch";
+		$original = "Google\\Http\\Batch";
+		$replacement = "BrianHenryIE\\Strauss\\Google\\Http\\Batch";
 
-        $result = $replacer->replaceNamespace($contents, $original, $replacement);
+		$result = $replacer->replaceNamespace($contents, $original, $replacement);
 
-        $expected = "use BrianHenryIE\\Strauss\\Google\\Http\\Batch;";
+		$expected = "use BrianHenryIE\\Strauss\\Google\\Http\\Batch;";
 
-        $this->assertStringContainsString($expected, $result);
-    }
+		$this->assertStringContainsString($expected, $result);
+	}
 
 
-    public function testClassnameReplacer()
-    {
+	public function testClassnameReplacer()
+	{
 
-        $contents = <<<'EOD'
+		$contents = <<<'EOD'
 <?php
 /*******************************************************************************
 * FPDF                                                                         *
@@ -161,263 +161,263 @@ 
 }
 EOD;
 
-        $config = $this->createMock(StraussConfig::class);
+		$config = $this->createMock(StraussConfig::class);
 
-        $replacer = new Prefixer($config, __DIR__);
+		$replacer = new Prefixer($config, __DIR__);
 
-        $original = "FPDF";
-        $classnamePrefix = "BrianHenryIE_Strauss_";
+		$original = "FPDF";
+		$classnamePrefix = "BrianHenryIE_Strauss_";
 
-        $result = $replacer->replaceClassname($contents, $original, $classnamePrefix);
+		$result = $replacer->replaceClassname($contents, $original, $classnamePrefix);
 
-        $expected = "class BrianHenryIE_Strauss_FPDF";
+		$expected = "class BrianHenryIE_Strauss_FPDF";
 
-        $this->assertStringContainsString($expected, $result);
-    }
+		$this->assertStringContainsString($expected, $result);
+	}
 
-    /**
-     * PHP 7.4 typed parameters were being prefixed.
-     */
-    public function testTypeFunctionParameter()
-    {
-        $this->markTestIncomplete();
-    }
+	/**
+	 * PHP 7.4 typed parameters were being prefixed.
+	 */
+	public function testTypeFunctionParameter()
+	{
+		$this->markTestIncomplete();
+	}
 
-    /**
-     * @author CoenJacobs
-     */
-    public function test_it_replaces_class_declarations(): void
-    {
-        $contents = 'class Hello_World {';
-        $originalClassname = 'Hello_World';
-        $classnamePrefix = 'Mozart_';
+	/**
+	 * @author CoenJacobs
+	 */
+	public function test_it_replaces_class_declarations(): void
+	{
+		$contents = 'class Hello_World {';
+		$originalClassname = 'Hello_World';
+		$classnamePrefix = 'Mozart_';
 
-        $config = $this->createMock(StraussConfig::class);
+		$config = $this->createMock(StraussConfig::class);
 
-        $replacer = new Prefixer($config, __DIR__);
+		$replacer = new Prefixer($config, __DIR__);
 
-        $result = $replacer->replaceClassname($contents, $originalClassname, $classnamePrefix);
+		$result = $replacer->replaceClassname($contents, $originalClassname, $classnamePrefix);
 
-        $this->assertEquals('class Mozart_Hello_World {', $result);
-    }
+		$this->assertEquals('class Mozart_Hello_World {', $result);
+	}
 
-    /**
-     * @author CoenJacobs
-     */
-    public function test_it_replaces_abstract_class_declarations(): void
-    {
-        $contents = 'abstract class Hello_World {';
+	/**
+	 * @author CoenJacobs
+	 */
+	public function test_it_replaces_abstract_class_declarations(): void
+	{
+		$contents = 'abstract class Hello_World {';
 
-        $originalClassname = 'Hello_World';
-        $classnamePrefix = 'Mozart_';
+		$originalClassname = 'Hello_World';
+		$classnamePrefix = 'Mozart_';
 
-        $config = $this->createMock(StraussConfig::class);
+		$config = $this->createMock(StraussConfig::class);
 
-        $replacer = new Prefixer($config, __DIR__);
+		$replacer = new Prefixer($config, __DIR__);
 
-        $result = $replacer->replaceClassname($contents, $originalClassname, $classnamePrefix);
+		$result = $replacer->replaceClassname($contents, $originalClassname, $classnamePrefix);
 
-        $this->assertEquals('abstract class Mozart_Hello_World {', $result);
-    }
+		$this->assertEquals('abstract class Mozart_Hello_World {', $result);
+	}
 
-    /**
-     * @author CoenJacobs
-     */
-    public function test_it_replaces_interface_class_declarations(): void
-    {
-        $contents = 'interface Hello_World {';
+	/**
+	 * @author CoenJacobs
+	 */
+	public function test_it_replaces_interface_class_declarations(): void
+	{
+		$contents = 'interface Hello_World {';
 
-        $originalClassname = 'Hello_World';
-        $classnamePrefix = 'Mozart_';
+		$originalClassname = 'Hello_World';
+		$classnamePrefix = 'Mozart_';
 
-        $config = $this->createMock(StraussConfig::class);
+		$config = $this->createMock(StraussConfig::class);
 
-        $replacer = new Prefixer($config, __DIR__);
+		$replacer = new Prefixer($config, __DIR__);
 
-        $result = $replacer->replaceClassname($contents, $originalClassname, $classnamePrefix);
+		$result = $replacer->replaceClassname($contents, $originalClassname, $classnamePrefix);
 
-        $this->assertEquals('interface Mozart_Hello_World {', $result);
-    }
+		$this->assertEquals('interface Mozart_Hello_World {', $result);
+	}
 
-    /**
-     * @author CoenJacobs
-     */
-    public function test_it_replaces_class_declarations_that_extend_other_classes(): void
-    {
-        $contents = 'class Hello_World extends Bye_World {';
+	/**
+	 * @author CoenJacobs
+	 */
+	public function test_it_replaces_class_declarations_that_extend_other_classes(): void
+	{
+		$contents = 'class Hello_World extends Bye_World {';
 
-        $originalClassname = 'Hello_World';
-        $classnamePrefix = 'Mozart_';
+		$originalClassname = 'Hello_World';
+		$classnamePrefix = 'Mozart_';
 
-        $config = $this->createMock(StraussConfig::class);
+		$config = $this->createMock(StraussConfig::class);
 
-        $replacer = new Prefixer($config, __DIR__);
+		$replacer = new Prefixer($config, __DIR__);
 
-        $result = $replacer->replaceClassname($contents, $originalClassname, $classnamePrefix);
+		$result = $replacer->replaceClassname($contents, $originalClassname, $classnamePrefix);
 
-        $this->assertEquals('class Mozart_Hello_World extends Bye_World {', $result);
-    }
+		$this->assertEquals('class Mozart_Hello_World extends Bye_World {', $result);
+	}
 
-    /**
-     * @author CoenJacobs
-     */
-    public function test_it_replaces_class_declarations_that_implement_interfaces(): void
-    {
-        $contents = 'class Hello_World implements Bye_World {';
+	/**
+	 * @author CoenJacobs
+	 */
+	public function test_it_replaces_class_declarations_that_implement_interfaces(): void
+	{
+		$contents = 'class Hello_World implements Bye_World {';
 
-        $originalClassname = 'Hello_World';
-        $classnamePrefix = 'Mozart_';
+		$originalClassname = 'Hello_World';
+		$classnamePrefix = 'Mozart_';
 
-        $config = $this->createMock(StraussConfig::class);
+		$config = $this->createMock(StraussConfig::class);
 
-        $replacer = new Prefixer($config, __DIR__);
+		$replacer = new Prefixer($config, __DIR__);
 
-        $result = $replacer->replaceClassname($contents, $originalClassname, $classnamePrefix);
+		$result = $replacer->replaceClassname($contents, $originalClassname, $classnamePrefix);
 
-        $this->assertEquals('class Mozart_Hello_World implements Bye_World {', $result);
-    }
+		$this->assertEquals('class Mozart_Hello_World implements Bye_World {', $result);
+	}
 
 
-    /**
-     * @author BrianHenryIE
-     */
-    public function testItReplacesNamespacesInInterface(): void
-    {
-        $contents = 'class Hello_World implements \Strauss\Bye_World {';
+	/**
+	 * @author BrianHenryIE
+	 */
+	public function testItReplacesNamespacesInInterface(): void
+	{
+		$contents = 'class Hello_World implements \Strauss\Bye_World {';
 
-        $originalNamespace = 'Strauss';
-        $replacement = 'Prefix\Strauss';
+		$originalNamespace = 'Strauss';
+		$replacement = 'Prefix\Strauss';
 
-        $config = $this->createMock(StraussConfig::class);
+		$config = $this->createMock(StraussConfig::class);
 
-        $replacer = new Prefixer($config, __DIR__);
+		$replacer = new Prefixer($config, __DIR__);
 
-        $result = $replacer->replaceNamespace($contents, $originalNamespace, $replacement);
+		$result = $replacer->replaceNamespace($contents, $originalNamespace, $replacement);
 
-        $this->assertEquals('class Hello_World implements \Prefix\Strauss\Bye_World {', $result);
-    }
+		$this->assertEquals('class Hello_World implements \Prefix\Strauss\Bye_World {', $result);
+	}
 
-    /**
-     * @author CoenJacobs
-     */
-    public function test_it_stores_replaced_class_names(): void
-    {
-        $this->markTestIncomplete('TODO Delete/move');
-
-        $contents = 'class Hello_World {';
-        $replacer = new Prefixer($config, __DIR__);
-        $replacer->setClassmapPrefix('Mozart_');
-        $replacer->replace($contents);
-        $this->assertArrayHasKey('Hello_World', $replacer->getReplacedClasses());
-    }
+	/**
+	 * @author CoenJacobs
+	 */
+	public function test_it_stores_replaced_class_names(): void
+	{
+		$this->markTestIncomplete('TODO Delete/move');
 
-    /**
-     * @author https://github.com/stephenharris
-     * @see https://github.com/coenjacobs/mozart/commit/fd7906943396c9a17110d1bfaf9d778f3b1f322a#diff-87828794e62b55ce8d7263e3ab1a918d1370e283ac750cd44e3ac61db5daee54
-     */
-    public function test_it_replaces_class_declarations_psr2(): void
-    {
-        $contents = "class Hello_World\n{";
+		$contents = 'class Hello_World {';
+		$replacer = new Prefixer($config, __DIR__);
+		$replacer->setClassmapPrefix('Mozart_');
+		$replacer->replace($contents);
+		$this->assertArrayHasKey('Hello_World', $replacer->getReplacedClasses());
+	}
 
-        $originalClassname = 'Hello_World';
-        $classnamePrefix = 'Mozart_';
+	/**
+	 * @author https://github.com/stephenharris
+	 * @see https://github.com/coenjacobs/mozart/commit/fd7906943396c9a17110d1bfaf9d778f3b1f322a#diff-87828794e62b55ce8d7263e3ab1a918d1370e283ac750cd44e3ac61db5daee54
+	 */
+	public function test_it_replaces_class_declarations_psr2(): void
+	{
+		$contents = "class Hello_World\n{";
 
-        $config = $this->createMock(StraussConfig::class);
+		$originalClassname = 'Hello_World';
+		$classnamePrefix = 'Mozart_';
 
-        $replacer = new Prefixer($config, __DIR__);
+		$config = $this->createMock(StraussConfig::class);
 
-        $result = $replacer->replaceClassname($contents, $originalClassname, $classnamePrefix);
+		$replacer = new Prefixer($config, __DIR__);
 
-        $this->assertEquals("class Mozart_Hello_World\n{", $result);
-    }
+		$result = $replacer->replaceClassname($contents, $originalClassname, $classnamePrefix);
 
-    /**
-     * @see https://github.com/coenjacobs/mozart/issues/81
-     * @author BrianHenryIE
-     *
-     */
-    public function test_it_replaces_class(): void
-    {
-        $contents = "class Hello_World {";
+		$this->assertEquals("class Mozart_Hello_World\n{", $result);
+	}
 
-        $originalClassname = 'Hello_World';
-        $classnamePrefix = 'Mozart_';
+	/**
+	 * @see https://github.com/coenjacobs/mozart/issues/81
+	 * @author BrianHenryIE
+	 *
+	 */
+	public function test_it_replaces_class(): void
+	{
+		$contents = "class Hello_World {";
 
-        $config = $this->createMock(StraussConfig::class);
+		$originalClassname = 'Hello_World';
+		$classnamePrefix = 'Mozart_';
 
-        $replacer = new Prefixer($config, __DIR__);
+		$config = $this->createMock(StraussConfig::class);
 
-        $result = $replacer->replaceClassname($contents, $originalClassname, $classnamePrefix);
+		$replacer = new Prefixer($config, __DIR__);
 
-        $this->assertEquals("class Mozart_Hello_World {", $result);
-    }
+		$result = $replacer->replaceClassname($contents, $originalClassname, $classnamePrefix);
 
+		$this->assertEquals("class Mozart_Hello_World {", $result);
+	}
 
-    /**
-     * @see ClassmapReplacerIntegrationTest::test_it_does_not_make_classname_replacement_inside_namespaced_file()
-     * @see https://github.com/coenjacobs/mozart/issues/93
-     *
-     * @author BrianHenryIE
-     *
-     * @test
-     */
-    public function it_does_not_replace_inside_namespace_multiline(): void
-    {
-        $contents = "
+
+	/**
+	 * @see ClassmapReplacerIntegrationTest::test_it_does_not_make_classname_replacement_inside_namespaced_file()
+	 * @see https://github.com/coenjacobs/mozart/issues/93
+	 *
+	 * @author BrianHenryIE
+	 *
+	 * @test
+	 */
+	public function it_does_not_replace_inside_namespace_multiline(): void
+	{
+		$contents = "
         namespace Mozart;
         class Hello_World
         ";
 
-        $originalClassname = 'Hello_World';
-        $classnamePrefix = 'Mozart_';
+		$originalClassname = 'Hello_World';
+		$classnamePrefix = 'Mozart_';
 
-        $config = $this->createMock(StraussConfig::class);
-        $config->method("getClassmapPrefix")->willReturn($classnamePrefix);
+		$config = $this->createMock(StraussConfig::class);
+		$config->method("getClassmapPrefix")->willReturn($classnamePrefix);
 
-        $replacer = new Prefixer($config, __DIR__);
+		$replacer = new Prefixer($config, __DIR__);
 
-        $result = $replacer->replaceInString([$originalClassname], [], $contents);
+		$result = $replacer->replaceInString([$originalClassname], [], $contents);
 
-        $this->assertEquals($contents, $result);
-    }
+		$this->assertEquals($contents, $result);
+	}
 
-    /**
-     * @see ClassmapReplacerIntegrationTest::test_it_does_not_make_classname_replacement_inside_namespaced_file()
-     * @see https://github.com/coenjacobs/mozart/issues/93
-     *
-     * @author BrianHenryIE
-     */
-    public function test_it_does_not_replace_inside_namespace_singleline(): void
-    {
-        $contents = "namespace Mozart; class Hello_World";
+	/**
+	 * @see ClassmapReplacerIntegrationTest::test_it_does_not_make_classname_replacement_inside_namespaced_file()
+	 * @see https://github.com/coenjacobs/mozart/issues/93
+	 *
+	 * @author BrianHenryIE
+	 */
+	public function test_it_does_not_replace_inside_namespace_singleline(): void
+	{
+		$contents = "namespace Mozart; class Hello_World";
 
-        $originalClassname = 'Hello_World';
-        $classnamePrefix = 'Mozart_';
+		$originalClassname = 'Hello_World';
+		$classnamePrefix = 'Mozart_';
 
-        $config = $this->createMock(StraussConfig::class);
+		$config = $this->createMock(StraussConfig::class);
 
-        $replacer = new Prefixer($config, __DIR__);
+		$replacer = new Prefixer($config, __DIR__);
 
-        $result = $replacer->replaceClassname($contents, $originalClassname, $classnamePrefix);
+		$result = $replacer->replaceClassname($contents, $originalClassname, $classnamePrefix);
 
-        $this->assertEquals($contents, $result);
-    }
+		$this->assertEquals($contents, $result);
+	}
 
 
 
-    /**
-     * It's possible to have multiple namespaces inside one file.
-     *
-     * To have two classes in one file, one in a namespace and the other not, the global namespace needs to be explicit.
-     *
-     * @author BrianHenryIE
-     *
-     * @test
-     */
-    public function it_does_not_replace_inside_named_namespace_but_does_inside_explicit_global_namespace_b(): void
-    {
+	/**
+	 * It's possible to have multiple namespaces inside one file.
+	 *
+	 * To have two classes in one file, one in a namespace and the other not, the global namespace needs to be explicit.
+	 *
+	 * @author BrianHenryIE
+	 *
+	 * @test
+	 */
+	public function it_does_not_replace_inside_named_namespace_but_does_inside_explicit_global_namespace_b(): void
+	{
 
-        $contents = "
+		$contents = "
 		namespace My_Project {
 			class A_Class { }
 		}
@@ -426,389 +426,389 @@ 
 		}
 		";
 
-        $classnamePrefix = 'Mozart_';
+		$classnamePrefix = 'Mozart_';
 
-        $config = $this->createMock(StraussConfig::class);
+		$config = $this->createMock(StraussConfig::class);
 
-        $replacer = new Prefixer($config, __DIR__);
+		$replacer = new Prefixer($config, __DIR__);
 
-        $result = $replacer->replaceClassname($contents, 'B_Class', $classnamePrefix);
+		$result = $replacer->replaceClassname($contents, 'B_Class', $classnamePrefix);
 
-        $this->assertStringContainsString('Mozart_B_Class', $result);
-    }
+		$this->assertStringContainsString('Mozart_B_Class', $result);
+	}
 
-    /** @test */
-    public function it_replaces_namespace_declarations(): void
-    {
-        $contents = 'namespace Test\\Test;';
+	/** @test */
+	public function it_replaces_namespace_declarations(): void
+	{
+		$contents = 'namespace Test\\Test;';
 
-        $namespace = "Test\\Test";
-        $replacement = "My\\Mozart\\Prefix\\Test\\Test";
+		$namespace = "Test\\Test";
+		$replacement = "My\\Mozart\\Prefix\\Test\\Test";
 
-        $config = $this->createMock(StraussConfig::class);
+		$config = $this->createMock(StraussConfig::class);
 
-        $replacer = new Prefixer($config, __DIR__);
+		$replacer = new Prefixer($config, __DIR__);
 
-        $result = $replacer->replaceNamespace($contents, $namespace, $replacement);
+		$result = $replacer->replaceNamespace($contents, $namespace, $replacement);
 
-        $this->assertEquals('namespace My\\Mozart\\Prefix\\Test\\Test;', $result);
-    }
+		$this->assertEquals('namespace My\\Mozart\\Prefix\\Test\\Test;', $result);
+	}
 
 
-    /**
-     * This test doesn't seem to match its name.
-     */
-    public function test_it_doesnt_replaces_namespace_inside_namespace(): void
-    {
-        $contents = "namespace Test\\Something;\n\nuse Test\\Test;";
+	/**
+	 * This test doesn't seem to match its name.
+	 */
+	public function test_it_doesnt_replaces_namespace_inside_namespace(): void
+	{
+		$contents = "namespace Test\\Something;\n\nuse Test\\Test;";
 
-        $config = $this->createMock(StraussConfig::class);
+		$config = $this->createMock(StraussConfig::class);
 
-        $replacer = new Prefixer($config, __DIR__);
-        $result = $replacer->replaceNamespace($contents, "Test\\Something", "My\\Mozart\\Prefix\\Test\\Something");
-        $result = $replacer->replaceNamespace($result, "Test\\Test", "My\\Mozart\\Prefix\\Test\\Test");
+		$replacer = new Prefixer($config, __DIR__);
+		$result = $replacer->replaceNamespace($contents, "Test\\Something", "My\\Mozart\\Prefix\\Test\\Something");
+		$result = $replacer->replaceNamespace($result, "Test\\Test", "My\\Mozart\\Prefix\\Test\\Test");
 
-        $this->assertEquals("namespace My\\Mozart\\Prefix\\Test\\Something;\n\nuse My\\Mozart\\Prefix\\Test\\Test;", $result);
-    }
+		$this->assertEquals("namespace My\\Mozart\\Prefix\\Test\\Something;\n\nuse My\\Mozart\\Prefix\\Test\\Test;", $result);
+	}
 
-    /**
-     *
-     */
-    public function test_it_does_notreplaces_partial_namespace_declarations(): void
-    {
-        $contents = 'namespace Test\\Test\\Another;';
+	/**
+	 *
+	 */
+	public function test_it_does_notreplaces_partial_namespace_declarations(): void
+	{
+		$contents = 'namespace Test\\Test\\Another;';
 
-        $namespace = "Test\\Another";
-        $prefix = "My\\Mozart\\Prefix";
+		$namespace = "Test\\Another";
+		$prefix = "My\\Mozart\\Prefix";
 
-        $config = $this->createMock(StraussConfig::class);
+		$config = $this->createMock(StraussConfig::class);
 
-        $replacer = new Prefixer($config, __DIR__);
-        $result = $replacer->replaceNamespace($contents, $namespace, $prefix);
+		$replacer = new Prefixer($config, __DIR__);
+		$result = $replacer->replaceNamespace($contents, $namespace, $prefix);
 
-        $this->assertEquals('namespace Test\\Test\\Another;', $result);
-    }
+		$this->assertEquals('namespace Test\\Test\\Another;', $result);
+	}
 
 
-    public function test_it_doesnt_prefix_already_prefixed_namespace(): void
-    {
+	public function test_it_doesnt_prefix_already_prefixed_namespace(): void
+	{
 
-        $contents = 'namespace My\\Mozart\\Prefix\\Test\\Another;';
+		$contents = 'namespace My\\Mozart\\Prefix\\Test\\Another;';
 
-        $namespace = "Test\\Another";
-        $prefix = "My\\Mozart\\Prefix";
+		$namespace = "Test\\Another";
+		$prefix = "My\\Mozart\\Prefix";
 
-        $config = $this->createMock(StraussConfig::class);
+		$config = $this->createMock(StraussConfig::class);
 
-        $replacer = new Prefixer($config, __DIR__);
-        $result = $replacer->replaceNamespace($contents, $namespace, $prefix);
+		$replacer = new Prefixer($config, __DIR__);
+		$result = $replacer->replaceNamespace($contents, $namespace, $prefix);
 
-        $this->assertEquals('namespace My\\Mozart\\Prefix\\Test\\Another;', $result);
-    }
+		$this->assertEquals('namespace My\\Mozart\\Prefix\\Test\\Another;', $result);
+	}
 
-    /**
-     * @author markjaquith
-     */
-    public function test_it_doesnt_double_replace_namespaces_that_also_exist_inside_another_namespace(): void
-    {
+	/**
+	 * @author markjaquith
+	 */
+	public function test_it_doesnt_double_replace_namespaces_that_also_exist_inside_another_namespace(): void
+	{
 
-        // This is a tricky situation. We are referencing Chicken\Egg,
-        // but Egg *also* exists as a separate top level class.
-        $contents = 'use Chicken\\Egg;';
-        $expected = 'use My\\Mozart\\Prefix\\Chicken\\Egg;';
+		// This is a tricky situation. We are referencing Chicken\Egg,
+		// but Egg *also* exists as a separate top level class.
+		$contents = 'use Chicken\\Egg;';
+		$expected = 'use My\\Mozart\\Prefix\\Chicken\\Egg;';
 
-        $config = $this->createMock(StraussConfig::class);
+		$config = $this->createMock(StraussConfig::class);
 
-        $replacer = new Prefixer($config, __DIR__);
+		$replacer = new Prefixer($config, __DIR__);
 
-        $result = $replacer->replaceNamespace($contents, 'Chicken', 'My\\Mozart\\Prefix\\Chicken');
-        $result = $replacer->replaceNamespace($result, 'Egg', 'My\\Mozart\\Prefix\\Egg');
+		$result = $replacer->replaceNamespace($contents, 'Chicken', 'My\\Mozart\\Prefix\\Chicken');
+		$result = $replacer->replaceNamespace($result, 'Egg', 'My\\Mozart\\Prefix\\Egg');
 
-        $this->assertEquals($expected, $result);
-    }
+		$this->assertEquals($expected, $result);
+	}
 
-    /**
-     * @see https://github.com/coenjacobs/mozart/issues/75
-     *
-     * @test
-     */
-    public function it_replaces_namespace_use_as_declarations(): void
-    {
-        $namespace = 'Symfony\Polyfill\Mbstring';
-        $replacement = "MBViews\Dependencies\Symfony\Polyfill\Mbstring";
+	/**
+	 * @see https://github.com/coenjacobs/mozart/issues/75
+	 *
+	 * @test
+	 */
+	public function it_replaces_namespace_use_as_declarations(): void
+	{
+		$namespace = 'Symfony\Polyfill\Mbstring';
+		$replacement = "MBViews\Dependencies\Symfony\Polyfill\Mbstring";
 
-        $contents = "use Symfony\Polyfill\Mbstring as p;";
+		$contents = "use Symfony\Polyfill\Mbstring as p;";
 
-        $config = $this->createMock(StraussConfig::class);
+		$config = $this->createMock(StraussConfig::class);
 
-        $replacer = new Prefixer($config, __DIR__);
-        $result = $replacer->replaceNamespace($contents, $namespace, $replacement);
+		$replacer = new Prefixer($config, __DIR__);
+		$result = $replacer->replaceNamespace($contents, $namespace, $replacement);
 
-        $expected = "use MBViews\Dependencies\Symfony\Polyfill\Mbstring as p;";
+		$expected = "use MBViews\Dependencies\Symfony\Polyfill\Mbstring as p;";
 
-        $this->assertEquals($expected, $result);
-    }
+		$this->assertEquals($expected, $result);
+	}
 
-    /**
-     * @author BrianHenryIE
-     */
-    public function test_it_doesnt_prefix_function_types_that_happen_to_match_the_namespace()
-    {
-        $namespace = 'Mpdf';
-        $prefix = "Mozart";
-        $contents = 'public function getServices( Mpdf $mpdf, LoggerInterface $logger, $config, )';
+	/**
+	 * @author BrianHenryIE
+	 */
+	public function test_it_doesnt_prefix_function_types_that_happen_to_match_the_namespace()
+	{
+		$namespace = 'Mpdf';
+		$prefix = "Mozart";
+		$contents = 'public function getServices( Mpdf $mpdf, LoggerInterface $logger, $config, )';
 
-        $config = $this->createMock(StraussConfig::class);
+		$config = $this->createMock(StraussConfig::class);
 
-        $replacer = new Prefixer($config, __DIR__);
-        $result = $replacer->replaceNamespace($contents, $namespace, $prefix);
+		$replacer = new Prefixer($config, __DIR__);
+		$result = $replacer->replaceNamespace($contents, $namespace, $prefix);
 
-        $expected = 'public function getServices( Mpdf $mpdf, LoggerInterface $logger, $config, )';
+		$expected = 'public function getServices( Mpdf $mpdf, LoggerInterface $logger, $config, )';
 
-        $this->assertEquals($expected, $result);
-    }
+		$this->assertEquals($expected, $result);
+	}
 
-    public function testLeadingSlashInString()
-    {
-        $originalNamespace = "Strauss\\Test";
-        $replacement = "Prefix\\Strauss\\Test";
-        $contents = '$mentionedClass = "\\Strauss\\Test\\Classname";';
+	public function testLeadingSlashInString()
+	{
+		$originalNamespace = "Strauss\\Test";
+		$replacement = "Prefix\\Strauss\\Test";
+		$contents = '$mentionedClass = "\\Strauss\\Test\\Classname";';
 
-        $config = $this->createMock(StraussConfig::class);
+		$config = $this->createMock(StraussConfig::class);
 
-        $replacer = new Prefixer($config, __DIR__);
-        $result = $replacer->replaceNamespace($contents, $originalNamespace, $replacement);
+		$replacer = new Prefixer($config, __DIR__);
+		$result = $replacer->replaceNamespace($contents, $originalNamespace, $replacement);
 
-        $expected = '$mentionedClass = "\\Prefix\\Strauss\\Test\\Classname";';
+		$expected = '$mentionedClass = "\\Prefix\\Strauss\\Test\\Classname";';
 
-        $this->assertEquals($expected, $result);
-    }
+		$this->assertEquals($expected, $result);
+	}
 
-    public function testDoubleLeadingSlashInString()
-    {
-        $originalNamespace = "Strauss\\Test";
-        $replacement = "Prefix\\Strauss\\Test";
-        $contents = '$mentionedClass = "\\\\Strauss\\\\Test\\\\Classname";';
+	public function testDoubleLeadingSlashInString()
+	{
+		$originalNamespace = "Strauss\\Test";
+		$replacement = "Prefix\\Strauss\\Test";
+		$contents = '$mentionedClass = "\\\\Strauss\\\\Test\\\\Classname";';
 
-        $config = $this->createMock(StraussConfig::class);
+		$config = $this->createMock(StraussConfig::class);
 
-        $replacer = new Prefixer($config, __DIR__);
-        $result = $replacer->replaceNamespace($contents, $originalNamespace, $replacement);
+		$replacer = new Prefixer($config, __DIR__);
+		$result = $replacer->replaceNamespace($contents, $originalNamespace, $replacement);
 
-        $expected = '$mentionedClass = "\\\\Prefix\\\\Strauss\\\\Test\\\\Classname";';
+		$expected = '$mentionedClass = "\\\\Prefix\\\\Strauss\\\\Test\\\\Classname";';
 
-        $this->assertEquals($expected, $result);
-    }
+		$this->assertEquals($expected, $result);
+	}
 
-    public function testItReplacesSlashedNamespaceInFunctionParameter()
-    {
+	public function testItReplacesSlashedNamespaceInFunctionParameter()
+	{
 
-        $originalNamespace = "net\\authorize\\api\\contract\\v1";
-        $replacement = "Prefix\\net\\authorize\\api\\contract\\v1";
-        $contents = "public function __construct(\\net\\authorize\\api\\contract\\v1\\AnetApiRequestType \$request, \$responseType)";
+		$originalNamespace = "net\\authorize\\api\\contract\\v1";
+		$replacement = "Prefix\\net\\authorize\\api\\contract\\v1";
+		$contents = "public function __construct(\\net\\authorize\\api\\contract\\v1\\AnetApiRequestType \$request, \$responseType)";
 
-        $config = $this->createMock(StraussConfig::class);
+		$config = $this->createMock(StraussConfig::class);
 
-        $replacer = new Prefixer($config, __DIR__);
-        $result = $replacer->replaceNamespace($contents, $originalNamespace, $replacement);
+		$replacer = new Prefixer($config, __DIR__);
+		$result = $replacer->replaceNamespace($contents, $originalNamespace, $replacement);
 
-        $expected = "public function __construct(\\Prefix\\net\\authorize\\api\\contract\\v1\\AnetApiRequestType \$request, \$responseType)";
+		$expected = "public function __construct(\\Prefix\\net\\authorize\\api\\contract\\v1\\AnetApiRequestType \$request, \$responseType)";
 
-        $this->assertEquals($expected, $result);
-    }
+		$this->assertEquals($expected, $result);
+	}
 
 
-    public function testItReplacesNamespaceInFunctionParameterDefaultAgumentValue()
-    {
+	public function testItReplacesNamespaceInFunctionParameterDefaultAgumentValue()
+	{
 
-        $originalNamespace = "net\\authorize\\api\constants";
-        $replacement = "Prefix\\net\\authorize\\api\constants";
-        $contents = "public function executeWithApiResponse(\$endPoint = \\net\\authorize\\api\\constants\\ANetEnvironment::CUSTOM)";
+		$originalNamespace = "net\\authorize\\api\constants";
+		$replacement = "Prefix\\net\\authorize\\api\constants";
+		$contents = "public function executeWithApiResponse(\$endPoint = \\net\\authorize\\api\\constants\\ANetEnvironment::CUSTOM)";
 
-        $config = $this->createMock(StraussConfig::class);
+		$config = $this->createMock(StraussConfig::class);
 
-        $replacer = new Prefixer($config, __DIR__);
-        $result = $replacer->replaceNamespace($contents, $originalNamespace, $replacement);
+		$replacer = new Prefixer($config, __DIR__);
+		$result = $replacer->replaceNamespace($contents, $originalNamespace, $replacement);
 
-        $expected = "public function executeWithApiResponse(\$endPoint = \\Prefix\\net\\authorize\\api\\constants\\ANetEnvironment::CUSTOM)";
+		$expected = "public function executeWithApiResponse(\$endPoint = \\Prefix\\net\\authorize\\api\\constants\\ANetEnvironment::CUSTOM)";
 
-        $this->assertEquals($expected, $result);
-    }
+		$this->assertEquals($expected, $result);
+	}
 
 
-    public function testItReplacesNamespaceConcatenatedStringConst()
-    {
+	public function testItReplacesNamespaceConcatenatedStringConst()
+	{
 
-        $originalNamespace = "net\\authorize\\api\\constants";
-        $replacement = "Prefix\\net\\authorize\\api\\constants";
-        $contents = "\$this->apiRequest->setClientId(\"sdk-php-\" . \\net\\authorize\\api\\constants\\ANetEnvironment::VERSION);";
+		$originalNamespace = "net\\authorize\\api\\constants";
+		$replacement = "Prefix\\net\\authorize\\api\\constants";
+		$contents = "\$this->apiRequest->setClientId(\"sdk-php-\" . \\net\\authorize\\api\\constants\\ANetEnvironment::VERSION);";
 
-        $config = $this->createMock(StraussConfig::class);
+		$config = $this->createMock(StraussConfig::class);
 
-        $replacer = new Prefixer($config, __DIR__);
-        $result = $replacer->replaceNamespace($contents, $originalNamespace, $replacement);
+		$replacer = new Prefixer($config, __DIR__);
+		$result = $replacer->replaceNamespace($contents, $originalNamespace, $replacement);
 
-        $expected = "\$this->apiRequest->setClientId(\"sdk-php-\" . \\Prefix\\net\\authorize\\api\\constants\\ANetEnvironment::VERSION);";
+		$expected = "\$this->apiRequest->setClientId(\"sdk-php-\" . \\Prefix\\net\\authorize\\api\\constants\\ANetEnvironment::VERSION);";
 
 
-        $this->assertEquals($expected, $result);
-    }
+		$this->assertEquals($expected, $result);
+	}
 
-    /**
-     * Another mpdf issue where the class "Mpdf" is in the namespace "Mpdf" and incorrect replacements are being made.
-     */
-    public function testClassnameNotConfusedWithNamespace()
-    {
+	/**
+	 * Another mpdf issue where the class "Mpdf" is in the namespace "Mpdf" and incorrect replacements are being made.
+	 */
+	public function testClassnameNotConfusedWithNamespace()
+	{
 
-        $contents = '$default_font_size = $mmsize * (Mpdf::SCALE);';
-        $expected = $contents;
+		$contents = '$default_font_size = $mmsize * (Mpdf::SCALE);';
+		$expected = $contents;
 
-        $config = $this->createMock(StraussConfig::class);
+		$config = $this->createMock(StraussConfig::class);
 
-        $replacer = new Prefixer($config, __DIR__);
-        $result = $replacer->replaceNamespace($contents, 'Mpdf', 'BrianHenryIE\Strauss\Mpdf');
+		$replacer = new Prefixer($config, __DIR__);
+		$result = $replacer->replaceNamespace($contents, 'Mpdf', 'BrianHenryIE\Strauss\Mpdf');
 
-        $this->assertEquals($expected, $result);
-    }
+		$this->assertEquals($expected, $result);
+	}
 
-    public function testClassExtendsNamspacedClassIsPrefixed()
-    {
+	public function testClassExtendsNamspacedClassIsPrefixed()
+	{
 
-        $contents = 'class BarcodeException extends \Mpdf\MpdfException';
-        $expected = 'class BarcodeException extends \BrianHenryIE\Strauss\Mpdf\MpdfException';
+		$contents = 'class BarcodeException extends \Mpdf\MpdfException';
+		$expected = 'class BarcodeException extends \BrianHenryIE\Strauss\Mpdf\MpdfException';
 
-        $config = $this->createMock(StraussConfig::class);
+		$config = $this->createMock(StraussConfig::class);
 
-        $replacer = new Prefixer($config, __DIR__);
-        $result = $replacer->replaceNamespace($contents, 'Mpdf', 'BrianHenryIE\Strauss\Mpdf');
+		$replacer = new Prefixer($config, __DIR__);
+		$result = $replacer->replaceNamespace($contents, 'Mpdf', 'BrianHenryIE\Strauss\Mpdf');
 
-        $this->assertEquals($expected, $result);
-    }
+		$this->assertEquals($expected, $result);
+	}
 
-    /**
-     * Prefix namespaced classnames after `new` keyword.
-     *
-     * @see https://github.com/BrianHenryIE/strauss/issues/11
-     */
-    public function testNewNamespacedClassIsPrefixed()
-    {
+	/**
+	 * Prefix namespaced classnames after `new` keyword.
+	 *
+	 * @see https://github.com/BrianHenryIE/strauss/issues/11
+	 */
+	public function testNewNamespacedClassIsPrefixed()
+	{
 
-        $contents = '$ioc->register( new \Carbon_Fields\Provider\Container_Condition_Provider() );';
-        $expected = '$ioc->register( new \BrianHenryIE\Strauss\Carbon_Fields\Provider\Container_Condition_Provider() );';
+		$contents = '$ioc->register( new \Carbon_Fields\Provider\Container_Condition_Provider() );';
+		$expected = '$ioc->register( new \BrianHenryIE\Strauss\Carbon_Fields\Provider\Container_Condition_Provider() );';
 
-        $config = $this->createMock(StraussConfig::class);
+		$config = $this->createMock(StraussConfig::class);
 
-        $replacer = new Prefixer($config, __DIR__);
-        $result = $replacer->replaceNamespace($contents, 'Carbon_Fields\Provider', 'BrianHenryIE\Strauss\Carbon_Fields\Provider');
+		$replacer = new Prefixer($config, __DIR__);
+		$result = $replacer->replaceNamespace($contents, 'Carbon_Fields\Provider', 'BrianHenryIE\Strauss\Carbon_Fields\Provider');
 
-        $this->assertEquals($expected, $result);
-    }
+		$this->assertEquals($expected, $result);
+	}
 
 
 
-    /**
-     * Prefix namespaced classnames after `static` keyword.
-     *
-     * @see https://github.com/BrianHenryIE/strauss/issues/11
-     */
-    public function testStaticNamespacedClassIsPrefixed()
-    {
+	/**
+	 * Prefix namespaced classnames after `static` keyword.
+	 *
+	 * @see https://github.com/BrianHenryIE/strauss/issues/11
+	 */
+	public function testStaticNamespacedClassIsPrefixed()
+	{
 
-        $contents = '@method static \Carbon_Fields\Container\Comment_Meta_Container';
-        $expected = '@method static \BrianHenryIE\Strauss\Carbon_Fields\Container\Comment_Meta_Container';
+		$contents = '@method static \Carbon_Fields\Container\Comment_Meta_Container';
+		$expected = '@method static \BrianHenryIE\Strauss\Carbon_Fields\Container\Comment_Meta_Container';
 
-        $config = $this->createMock(StraussConfig::class);
+		$config = $this->createMock(StraussConfig::class);
 
-        $replacer = new Prefixer($config, __DIR__);
-        $result = $replacer->replaceNamespace($contents, 'Carbon_Fields\Container', 'BrianHenryIE\Strauss\Carbon_Fields\Container');
+		$replacer = new Prefixer($config, __DIR__);
+		$result = $replacer->replaceNamespace($contents, 'Carbon_Fields\Container', 'BrianHenryIE\Strauss\Carbon_Fields\Container');
 
-        $this->assertEquals($expected, $result);
-    }
+		$this->assertEquals($expected, $result);
+	}
 
-    /**
-     * Prefix namespaced classnames after return statement.
-     *
-     * @see https://github.com/BrianHenryIE/strauss/issues/11
-     */
-    public function testReturnedNamespacedClassIsPrefixed()
-    {
+	/**
+	 * Prefix namespaced classnames after return statement.
+	 *
+	 * @see https://github.com/BrianHenryIE/strauss/issues/11
+	 */
+	public function testReturnedNamespacedClassIsPrefixed()
+	{
 
-        $contents = 'return \Carbon_Fields\Carbon_Fields::resolve';
-        $expected = 'return \BrianHenryIE\Strauss\Carbon_Fields\Carbon_Fields::resolve';
+		$contents = 'return \Carbon_Fields\Carbon_Fields::resolve';
+		$expected = 'return \BrianHenryIE\Strauss\Carbon_Fields\Carbon_Fields::resolve';
 
-        $config = $this->createMock(StraussConfig::class);
+		$config = $this->createMock(StraussConfig::class);
 
-        $replacer = new Prefixer($config, __DIR__);
-        $result = $replacer->replaceNamespace($contents, 'Carbon_Fields', 'BrianHenryIE\Strauss\Carbon_Fields');
+		$replacer = new Prefixer($config, __DIR__);
+		$result = $replacer->replaceNamespace($contents, 'Carbon_Fields', 'BrianHenryIE\Strauss\Carbon_Fields');
 
-        $this->assertEquals($expected, $result);
-    }
+		$this->assertEquals($expected, $result);
+	}
 
-    /**
-     * Prefix namespaced classnames between two tabs and colon.
-     *
-     * @see https://github.com/BrianHenryIE/strauss/issues/11
-     */
-    public function testNamespacedStaticIsPrefixed()
-    {
+	/**
+	 * Prefix namespaced classnames between two tabs and colon.
+	 *
+	 * @see https://github.com/BrianHenryIE/strauss/issues/11
+	 */
+	public function testNamespacedStaticIsPrefixed()
+	{
 
-        $contents = "		\\Carbon_Fields\\Carbon_Fields::service( 'legacy_storage' )->enable()";
-        $expected = "		\\BrianHenryIE\\Strauss\\Carbon_Fields\\Carbon_Fields::service( 'legacy_storage' )->enable()";
+		$contents = "		\\Carbon_Fields\\Carbon_Fields::service( 'legacy_storage' )->enable()";
+		$expected = "		\\BrianHenryIE\\Strauss\\Carbon_Fields\\Carbon_Fields::service( 'legacy_storage' )->enable()";
 
-        $config = $this->createMock(StraussConfig::class);
+		$config = $this->createMock(StraussConfig::class);
 
-        $replacer = new Prefixer($config, __DIR__);
-        $result = $replacer->replaceNamespace(
-            $contents,
-            'Carbon_Fields',
-            'BrianHenryIE\Strauss\Carbon_Fields'
-        );
+		$replacer = new Prefixer($config, __DIR__);
+		$result = $replacer->replaceNamespace(
+			$contents,
+			'Carbon_Fields',
+			'BrianHenryIE\Strauss\Carbon_Fields'
+		);
 
-        $this->assertEquals($expected, $result);
-    }
+		$this->assertEquals($expected, $result);
+	}
 
-    /**
-     * Sometimes the namespace in a string should be replaced, but sometimes not.
-     *
-     * @see https://github.com/BrianHenryIE/strauss/issues/15
-     */
-    public function testDoNotReplaceInStringThatIsNotCode()
-    {
-        $originalNamespace = "TrustedLogin";
-        $replacement = "Prefix\\TrustedLogin";
-        $contents = "esc_html__( 'Learn about TrustedLogin', 'trustedlogin' )";
+	/**
+	 * Sometimes the namespace in a string should be replaced, but sometimes not.
+	 *
+	 * @see https://github.com/BrianHenryIE/strauss/issues/15
+	 */
+	public function testDoNotReplaceInStringThatIsNotCode()
+	{
+		$originalNamespace = "TrustedLogin";
+		$replacement = "Prefix\\TrustedLogin";
+		$contents = "esc_html__( 'Learn about TrustedLogin', 'trustedlogin' )";
 
-        $config = $this->createMock(StraussConfig::class);
+		$config = $this->createMock(StraussConfig::class);
 
-        $replacer = new Prefixer($config, __DIR__);
-        $result = $replacer->replaceNamespace($contents, $originalNamespace, $replacement);
+		$replacer = new Prefixer($config, __DIR__);
+		$result = $replacer->replaceNamespace($contents, $originalNamespace, $replacement);
 
-        $expected = "esc_html__( 'Learn about TrustedLogin', 'trustedlogin' )";
+		$expected = "esc_html__( 'Learn about TrustedLogin', 'trustedlogin' )";
 
-        $this->assertEquals($expected, $result);
-    }
+		$this->assertEquals($expected, $result);
+	}
 
 
-    /**
-     *
-     *
-     * @see https://github.com/BrianHenryIE/strauss/issues/19
-     *
-     */
-    public function testDoNotReplaceInVariableNames()
-    {
-        $originalClassname = 'object';
-        $classnamePrefix = 'Strauss_Issue19_';
-        $contents = "public static function objclone(\$object) {";
+	/**
+	 *
+	 *
+	 * @see https://github.com/BrianHenryIE/strauss/issues/19
+	 *
+	 */
+	public function testDoNotReplaceInVariableNames()
+	{
+		$originalClassname = 'object';
+		$classnamePrefix = 'Strauss_Issue19_';
+		$contents = "public static function objclone(\$object) {";
 
-        $config = $this->createMock(StraussConfig::class);
+		$config = $this->createMock(StraussConfig::class);
 
-        $replacer = new Prefixer($config, __DIR__);
-        $result = $replacer->replaceClassname($contents, $originalClassname, $classnamePrefix);
+		$replacer = new Prefixer($config, __DIR__);
+		$result = $replacer->replaceClassname($contents, $originalClassname, $classnamePrefix);
 
-        // NOT public static function objclone($Strauss_Issue19_object) {
-        $expected = "public static function objclone(\$object) {";
+		// NOT public static function objclone($Strauss_Issue19_object) {
+		$expected = "public static function objclone(\$object) {";
 
-        $this->assertEquals($expected, $result);
-    }
+		$this->assertEquals($expected, $result);
+	}
 }

Spacing +143 added lines, -143 removed lines

@@ -38,12 +38,12 @@ 
 }
 EOD;
 
-        $composerConfig = new Config(false);
-        $composerConfig->merge(json_decode($composerJson, true));
+        $composerConfig = new Config( false );
+        $composerConfig->merge( json_decode( $composerJson, true ) );
         $composer = new Composer();
-        $composer->setConfig($composerConfig);
+        $composer->setConfig( $composerConfig );
 
-        $this->config = new StraussConfig($composer);
+        $this->config = new StraussConfig( $composer );
     }
 
     public function testNamespaceReplacer()
@@ -122,18 +122,18 @@ 
   }
 }
 EOD;
-        $config = $this->createMock(StraussConfig::class);
+        $config = $this->createMock( StraussConfig::class );
 
-        $replacer = new Prefixer($config, __DIR__);
+        $replacer = new Prefixer( $config, __DIR__ );
 
         $original = "Google\\Http\\Batch";
         $replacement = "BrianHenryIE\\Strauss\\Google\\Http\\Batch";
 
-        $result = $replacer->replaceNamespace($contents, $original, $replacement);
+        $result = $replacer->replaceNamespace( $contents, $original, $replacement );
 
         $expected = "use BrianHenryIE\\Strauss\\Google\\Http\\Batch;";
 
-        $this->assertStringContainsString($expected, $result);
+        $this->assertStringContainsString( $expected, $result );
     }
 
 
@@ -161,18 +161,18 @@ 
 }
 EOD;
 
-        $config = $this->createMock(StraussConfig::class);
+        $config = $this->createMock( StraussConfig::class );
 
-        $replacer = new Prefixer($config, __DIR__);
+        $replacer = new Prefixer( $config, __DIR__ );
 
         $original = "FPDF";
         $classnamePrefix = "BrianHenryIE_Strauss_";
 
-        $result = $replacer->replaceClassname($contents, $original, $classnamePrefix);
+        $result = $replacer->replaceClassname( $contents, $original, $classnamePrefix );
 
         $expected = "class BrianHenryIE_Strauss_FPDF";
 
-        $this->assertStringContainsString($expected, $result);
+        $this->assertStringContainsString( $expected, $result );
     }
 
     /**
@@ -192,13 +192,13 @@ 
         $originalClassname = 'Hello_World';
         $classnamePrefix = 'Mozart_';
 
-        $config = $this->createMock(StraussConfig::class);
+        $config = $this->createMock( StraussConfig::class );
 
-        $replacer = new Prefixer($config, __DIR__);
+        $replacer = new Prefixer( $config, __DIR__ );
 
-        $result = $replacer->replaceClassname($contents, $originalClassname, $classnamePrefix);
+        $result = $replacer->replaceClassname( $contents, $originalClassname, $classnamePrefix );
 
-        $this->assertEquals('class Mozart_Hello_World {', $result);
+        $this->assertEquals( 'class Mozart_Hello_World {', $result );
     }
 
     /**
@@ -211,13 +211,13 @@ 
         $originalClassname = 'Hello_World';
         $classnamePrefix = 'Mozart_';
 
-        $config = $this->createMock(StraussConfig::class);
+        $config = $this->createMock( StraussConfig::class );
 
-        $replacer = new Prefixer($config, __DIR__);
+        $replacer = new Prefixer( $config, __DIR__ );
 
-        $result = $replacer->replaceClassname($contents, $originalClassname, $classnamePrefix);
+        $result = $replacer->replaceClassname( $contents, $originalClassname, $classnamePrefix );
 
-        $this->assertEquals('abstract class Mozart_Hello_World {', $result);
+        $this->assertEquals( 'abstract class Mozart_Hello_World {', $result );
     }
 
     /**
@@ -230,13 +230,13 @@ 
         $originalClassname = 'Hello_World';
         $classnamePrefix = 'Mozart_';
 
-        $config = $this->createMock(StraussConfig::class);
+        $config = $this->createMock( StraussConfig::class );
 
-        $replacer = new Prefixer($config, __DIR__);
+        $replacer = new Prefixer( $config, __DIR__ );
 
-        $result = $replacer->replaceClassname($contents, $originalClassname, $classnamePrefix);
+        $result = $replacer->replaceClassname( $contents, $originalClassname, $classnamePrefix );
 
-        $this->assertEquals('interface Mozart_Hello_World {', $result);
+        $this->assertEquals( 'interface Mozart_Hello_World {', $result );
     }
 
     /**
@@ -249,13 +249,13 @@ 
         $originalClassname = 'Hello_World';
         $classnamePrefix = 'Mozart_';
 
-        $config = $this->createMock(StraussConfig::class);
+        $config = $this->createMock( StraussConfig::class );
 
-        $replacer = new Prefixer($config, __DIR__);
+        $replacer = new Prefixer( $config, __DIR__ );
 
-        $result = $replacer->replaceClassname($contents, $originalClassname, $classnamePrefix);
+        $result = $replacer->replaceClassname( $contents, $originalClassname, $classnamePrefix );
 
-        $this->assertEquals('class Mozart_Hello_World extends Bye_World {', $result);
+        $this->assertEquals( 'class Mozart_Hello_World extends Bye_World {', $result );
     }
 
     /**
@@ -268,13 +268,13 @@ 
         $originalClassname = 'Hello_World';
         $classnamePrefix = 'Mozart_';
 
-        $config = $this->createMock(StraussConfig::class);
+        $config = $this->createMock( StraussConfig::class );
 
-        $replacer = new Prefixer($config, __DIR__);
+        $replacer = new Prefixer( $config, __DIR__ );
 
-        $result = $replacer->replaceClassname($contents, $originalClassname, $classnamePrefix);
+        $result = $replacer->replaceClassname( $contents, $originalClassname, $classnamePrefix );
 
-        $this->assertEquals('class Mozart_Hello_World implements Bye_World {', $result);
+        $this->assertEquals( 'class Mozart_Hello_World implements Bye_World {', $result );
     }
 
 
@@ -288,13 +288,13 @@ 
         $originalNamespace = 'Strauss';
         $replacement = 'Prefix\Strauss';
 
-        $config = $this->createMock(StraussConfig::class);
+        $config = $this->createMock( StraussConfig::class );
 
-        $replacer = new Prefixer($config, __DIR__);
+        $replacer = new Prefixer( $config, __DIR__ );
 
-        $result = $replacer->replaceNamespace($contents, $originalNamespace, $replacement);
+        $result = $replacer->replaceNamespace( $contents, $originalNamespace, $replacement );
 
-        $this->assertEquals('class Hello_World implements \Prefix\Strauss\Bye_World {', $result);
+        $this->assertEquals( 'class Hello_World implements \Prefix\Strauss\Bye_World {', $result );
     }
 
     /**
@@ -302,13 +302,13 @@ 
      */
     public function test_it_stores_replaced_class_names(): void
     {
-        $this->markTestIncomplete('TODO Delete/move');
+        $this->markTestIncomplete( 'TODO Delete/move' );
 
         $contents = 'class Hello_World {';
-        $replacer = new Prefixer($config, __DIR__);
-        $replacer->setClassmapPrefix('Mozart_');
-        $replacer->replace($contents);
-        $this->assertArrayHasKey('Hello_World', $replacer->getReplacedClasses());
+        $replacer = new Prefixer( $config, __DIR__ );
+        $replacer->setClassmapPrefix( 'Mozart_' );
+        $replacer->replace( $contents );
+        $this->assertArrayHasKey( 'Hello_World', $replacer->getReplacedClasses() );
     }
 
     /**
@@ -322,13 +322,13 @@ 
         $originalClassname = 'Hello_World';
         $classnamePrefix = 'Mozart_';
 
-        $config = $this->createMock(StraussConfig::class);
+        $config = $this->createMock( StraussConfig::class );
 
-        $replacer = new Prefixer($config, __DIR__);
+        $replacer = new Prefixer( $config, __DIR__ );
 
-        $result = $replacer->replaceClassname($contents, $originalClassname, $classnamePrefix);
+        $result = $replacer->replaceClassname( $contents, $originalClassname, $classnamePrefix );
 
-        $this->assertEquals("class Mozart_Hello_World\n{", $result);
+        $this->assertEquals( "class Mozart_Hello_World\n{", $result );
     }
 
     /**
@@ -343,13 +343,13 @@ 
         $originalClassname = 'Hello_World';
         $classnamePrefix = 'Mozart_';
 
-        $config = $this->createMock(StraussConfig::class);
+        $config = $this->createMock( StraussConfig::class );
 
-        $replacer = new Prefixer($config, __DIR__);
+        $replacer = new Prefixer( $config, __DIR__ );
 
-        $result = $replacer->replaceClassname($contents, $originalClassname, $classnamePrefix);
+        $result = $replacer->replaceClassname( $contents, $originalClassname, $classnamePrefix );
 
-        $this->assertEquals("class Mozart_Hello_World {", $result);
+        $this->assertEquals( "class Mozart_Hello_World {", $result );
     }
 
 
@@ -371,14 +371,14 @@ 
         $originalClassname = 'Hello_World';
         $classnamePrefix = 'Mozart_';
 
-        $config = $this->createMock(StraussConfig::class);
-        $config->method("getClassmapPrefix")->willReturn($classnamePrefix);
+        $config = $this->createMock( StraussConfig::class );
+        $config->method( "getClassmapPrefix" )->willReturn( $classnamePrefix );
 
-        $replacer = new Prefixer($config, __DIR__);
+        $replacer = new Prefixer( $config, __DIR__ );
 
-        $result = $replacer->replaceInString([$originalClassname], [], $contents);
+        $result = $replacer->replaceInString( [ $originalClassname ], [ ], $contents );
 
-        $this->assertEquals($contents, $result);
+        $this->assertEquals( $contents, $result );
     }
 
     /**
@@ -394,13 +394,13 @@ 
         $originalClassname = 'Hello_World';
         $classnamePrefix = 'Mozart_';
 
-        $config = $this->createMock(StraussConfig::class);
+        $config = $this->createMock( StraussConfig::class );
 
-        $replacer = new Prefixer($config, __DIR__);
+        $replacer = new Prefixer( $config, __DIR__ );
 
-        $result = $replacer->replaceClassname($contents, $originalClassname, $classnamePrefix);
+        $result = $replacer->replaceClassname( $contents, $originalClassname, $classnamePrefix );
 
-        $this->assertEquals($contents, $result);
+        $this->assertEquals( $contents, $result );
     }
 
 
@@ -428,13 +428,13 @@ 
 
         $classnamePrefix = 'Mozart_';
 
-        $config = $this->createMock(StraussConfig::class);
+        $config = $this->createMock( StraussConfig::class );
 
-        $replacer = new Prefixer($config, __DIR__);
+        $replacer = new Prefixer( $config, __DIR__ );
 
-        $result = $replacer->replaceClassname($contents, 'B_Class', $classnamePrefix);
+        $result = $replacer->replaceClassname( $contents, 'B_Class', $classnamePrefix );
 
-        $this->assertStringContainsString('Mozart_B_Class', $result);
+        $this->assertStringContainsString( 'Mozart_B_Class', $result );
     }
 
     /** @test */
@@ -445,13 +445,13 @@ 
         $namespace = "Test\\Test";
         $replacement = "My\\Mozart\\Prefix\\Test\\Test";
 
-        $config = $this->createMock(StraussConfig::class);
+        $config = $this->createMock( StraussConfig::class );
 
-        $replacer = new Prefixer($config, __DIR__);
+        $replacer = new Prefixer( $config, __DIR__ );
 
-        $result = $replacer->replaceNamespace($contents, $namespace, $replacement);
+        $result = $replacer->replaceNamespace( $contents, $namespace, $replacement );
 
-        $this->assertEquals('namespace My\\Mozart\\Prefix\\Test\\Test;', $result);
+        $this->assertEquals( 'namespace My\\Mozart\\Prefix\\Test\\Test;', $result );
     }
 
 
@@ -462,13 +462,13 @@ 
     {
         $contents = "namespace Test\\Something;\n\nuse Test\\Test;";
 
-        $config = $this->createMock(StraussConfig::class);
+        $config = $this->createMock( StraussConfig::class );
 
-        $replacer = new Prefixer($config, __DIR__);
-        $result = $replacer->replaceNamespace($contents, "Test\\Something", "My\\Mozart\\Prefix\\Test\\Something");
-        $result = $replacer->replaceNamespace($result, "Test\\Test", "My\\Mozart\\Prefix\\Test\\Test");
+        $replacer = new Prefixer( $config, __DIR__ );
+        $result = $replacer->replaceNamespace( $contents, "Test\\Something", "My\\Mozart\\Prefix\\Test\\Something" );
+        $result = $replacer->replaceNamespace( $result, "Test\\Test", "My\\Mozart\\Prefix\\Test\\Test" );
 
-        $this->assertEquals("namespace My\\Mozart\\Prefix\\Test\\Something;\n\nuse My\\Mozart\\Prefix\\Test\\Test;", $result);
+        $this->assertEquals( "namespace My\\Mozart\\Prefix\\Test\\Something;\n\nuse My\\Mozart\\Prefix\\Test\\Test;", $result );
     }
 
     /**
@@ -481,12 +481,12 @@ 
         $namespace = "Test\\Another";
         $prefix = "My\\Mozart\\Prefix";
 
-        $config = $this->createMock(StraussConfig::class);
+        $config = $this->createMock( StraussConfig::class );
 
-        $replacer = new Prefixer($config, __DIR__);
-        $result = $replacer->replaceNamespace($contents, $namespace, $prefix);
+        $replacer = new Prefixer( $config, __DIR__ );
+        $result = $replacer->replaceNamespace( $contents, $namespace, $prefix );
 
-        $this->assertEquals('namespace Test\\Test\\Another;', $result);
+        $this->assertEquals( 'namespace Test\\Test\\Another;', $result );
     }
 
 
@@ -498,12 +498,12 @@ 
         $namespace = "Test\\Another";
         $prefix = "My\\Mozart\\Prefix";
 
-        $config = $this->createMock(StraussConfig::class);
+        $config = $this->createMock( StraussConfig::class );
 
-        $replacer = new Prefixer($config, __DIR__);
-        $result = $replacer->replaceNamespace($contents, $namespace, $prefix);
+        $replacer = new Prefixer( $config, __DIR__ );
+        $result = $replacer->replaceNamespace( $contents, $namespace, $prefix );
 
-        $this->assertEquals('namespace My\\Mozart\\Prefix\\Test\\Another;', $result);
+        $this->assertEquals( 'namespace My\\Mozart\\Prefix\\Test\\Another;', $result );
     }
 
     /**
@@ -517,14 +517,14 @@ 
         $contents = 'use Chicken\\Egg;';
         $expected = 'use My\\Mozart\\Prefix\\Chicken\\Egg;';
 
-        $config = $this->createMock(StraussConfig::class);
+        $config = $this->createMock( StraussConfig::class );
 
-        $replacer = new Prefixer($config, __DIR__);
+        $replacer = new Prefixer( $config, __DIR__ );
 
-        $result = $replacer->replaceNamespace($contents, 'Chicken', 'My\\Mozart\\Prefix\\Chicken');
-        $result = $replacer->replaceNamespace($result, 'Egg', 'My\\Mozart\\Prefix\\Egg');
+        $result = $replacer->replaceNamespace( $contents, 'Chicken', 'My\\Mozart\\Prefix\\Chicken' );
+        $result = $replacer->replaceNamespace( $result, 'Egg', 'My\\Mozart\\Prefix\\Egg' );
 
-        $this->assertEquals($expected, $result);
+        $this->assertEquals( $expected, $result );
     }
 
     /**
@@ -539,14 +539,14 @@ 
 
         $contents = "use Symfony\Polyfill\Mbstring as p;";
 
-        $config = $this->createMock(StraussConfig::class);
+        $config = $this->createMock( StraussConfig::class );
 
-        $replacer = new Prefixer($config, __DIR__);
-        $result = $replacer->replaceNamespace($contents, $namespace, $replacement);
+        $replacer = new Prefixer( $config, __DIR__ );
+        $result = $replacer->replaceNamespace( $contents, $namespace, $replacement );
 
         $expected = "use MBViews\Dependencies\Symfony\Polyfill\Mbstring as p;";
 
-        $this->assertEquals($expected, $result);
+        $this->assertEquals( $expected, $result );
     }
 
     /**
@@ -558,14 +558,14 @@ 
         $prefix = "Mozart";
         $contents = 'public function getServices( Mpdf $mpdf, LoggerInterface $logger, $config, )';
 
-        $config = $this->createMock(StraussConfig::class);
+        $config = $this->createMock( StraussConfig::class );
 
-        $replacer = new Prefixer($config, __DIR__);
-        $result = $replacer->replaceNamespace($contents, $namespace, $prefix);
+        $replacer = new Prefixer( $config, __DIR__ );
+        $result = $replacer->replaceNamespace( $contents, $namespace, $prefix );
 
         $expected = 'public function getServices( Mpdf $mpdf, LoggerInterface $logger, $config, )';
 
-        $this->assertEquals($expected, $result);
+        $this->assertEquals( $expected, $result );
     }
 
     public function testLeadingSlashInString()
@@ -574,14 +574,14 @@ 
         $replacement = "Prefix\\Strauss\\Test";
         $contents = '$mentionedClass = "\\Strauss\\Test\\Classname";';
 
-        $config = $this->createMock(StraussConfig::class);
+        $config = $this->createMock( StraussConfig::class );
 
-        $replacer = new Prefixer($config, __DIR__);
-        $result = $replacer->replaceNamespace($contents, $originalNamespace, $replacement);
+        $replacer = new Prefixer( $config, __DIR__ );
+        $result = $replacer->replaceNamespace( $contents, $originalNamespace, $replacement );
 
         $expected = '$mentionedClass = "\\Prefix\\Strauss\\Test\\Classname";';
 
-        $this->assertEquals($expected, $result);
+        $this->assertEquals( $expected, $result );
     }
 
     public function testDoubleLeadingSlashInString()
@@ -590,14 +590,14 @@ 
         $replacement = "Prefix\\Strauss\\Test";
         $contents = '$mentionedClass = "\\\\Strauss\\\\Test\\\\Classname";';
 
-        $config = $this->createMock(StraussConfig::class);
+        $config = $this->createMock( StraussConfig::class );
 
-        $replacer = new Prefixer($config, __DIR__);
-        $result = $replacer->replaceNamespace($contents, $originalNamespace, $replacement);
+        $replacer = new Prefixer( $config, __DIR__ );
+        $result = $replacer->replaceNamespace( $contents, $originalNamespace, $replacement );
 
         $expected = '$mentionedClass = "\\\\Prefix\\\\Strauss\\\\Test\\\\Classname";';
 
-        $this->assertEquals($expected, $result);
+        $this->assertEquals( $expected, $result );
     }
 
     public function testItReplacesSlashedNamespaceInFunctionParameter()
@@ -607,14 +607,14 @@ 
         $replacement = "Prefix\\net\\authorize\\api\\contract\\v1";
         $contents = "public function __construct(\\net\\authorize\\api\\contract\\v1\\AnetApiRequestType \$request, \$responseType)";
 
-        $config = $this->createMock(StraussConfig::class);
+        $config = $this->createMock( StraussConfig::class );
 
-        $replacer = new Prefixer($config, __DIR__);
-        $result = $replacer->replaceNamespace($contents, $originalNamespace, $replacement);
+        $replacer = new Prefixer( $config, __DIR__ );
+        $result = $replacer->replaceNamespace( $contents, $originalNamespace, $replacement );
 
         $expected = "public function __construct(\\Prefix\\net\\authorize\\api\\contract\\v1\\AnetApiRequestType \$request, \$responseType)";
 
-        $this->assertEquals($expected, $result);
+        $this->assertEquals( $expected, $result );
     }
 
 
@@ -625,14 +625,14 @@ 
         $replacement = "Prefix\\net\\authorize\\api\constants";
         $contents = "public function executeWithApiResponse(\$endPoint = \\net\\authorize\\api\\constants\\ANetEnvironment::CUSTOM)";
 
-        $config = $this->createMock(StraussConfig::class);
+        $config = $this->createMock( StraussConfig::class );
 
-        $replacer = new Prefixer($config, __DIR__);
-        $result = $replacer->replaceNamespace($contents, $originalNamespace, $replacement);
+        $replacer = new Prefixer( $config, __DIR__ );
+        $result = $replacer->replaceNamespace( $contents, $originalNamespace, $replacement );
 
         $expected = "public function executeWithApiResponse(\$endPoint = \\Prefix\\net\\authorize\\api\\constants\\ANetEnvironment::CUSTOM)";
 
-        $this->assertEquals($expected, $result);
+        $this->assertEquals( $expected, $result );
     }
 
 
@@ -643,15 +643,15 @@ 
         $replacement = "Prefix\\net\\authorize\\api\\constants";
         $contents = "\$this->apiRequest->setClientId(\"sdk-php-\" . \\net\\authorize\\api\\constants\\ANetEnvironment::VERSION);";
 
-        $config = $this->createMock(StraussConfig::class);
+        $config = $this->createMock( StraussConfig::class );
 
-        $replacer = new Prefixer($config, __DIR__);
-        $result = $replacer->replaceNamespace($contents, $originalNamespace, $replacement);
+        $replacer = new Prefixer( $config, __DIR__ );
+        $result = $replacer->replaceNamespace( $contents, $originalNamespace, $replacement );
 
         $expected = "\$this->apiRequest->setClientId(\"sdk-php-\" . \\Prefix\\net\\authorize\\api\\constants\\ANetEnvironment::VERSION);";
 
 
-        $this->assertEquals($expected, $result);
+        $this->assertEquals( $expected, $result );
     }
 
     /**
@@ -663,12 +663,12 @@ 
         $contents = '$default_font_size = $mmsize * (Mpdf::SCALE);';
         $expected = $contents;
 
-        $config = $this->createMock(StraussConfig::class);
+        $config = $this->createMock( StraussConfig::class );
 
-        $replacer = new Prefixer($config, __DIR__);
-        $result = $replacer->replaceNamespace($contents, 'Mpdf', 'BrianHenryIE\Strauss\Mpdf');
+        $replacer = new Prefixer( $config, __DIR__ );
+        $result = $replacer->replaceNamespace( $contents, 'Mpdf', 'BrianHenryIE\Strauss\Mpdf' );
 
-        $this->assertEquals($expected, $result);
+        $this->assertEquals( $expected, $result );
     }
 
     public function testClassExtendsNamspacedClassIsPrefixed()
@@ -677,12 +677,12 @@ 
         $contents = 'class BarcodeException extends \Mpdf\MpdfException';
         $expected = 'class BarcodeException extends \BrianHenryIE\Strauss\Mpdf\MpdfException';
 
-        $config = $this->createMock(StraussConfig::class);
+        $config = $this->createMock( StraussConfig::class );
 
-        $replacer = new Prefixer($config, __DIR__);
-        $result = $replacer->replaceNamespace($contents, 'Mpdf', 'BrianHenryIE\Strauss\Mpdf');
+        $replacer = new Prefixer( $config, __DIR__ );
+        $result = $replacer->replaceNamespace( $contents, 'Mpdf', 'BrianHenryIE\Strauss\Mpdf' );
 
-        $this->assertEquals($expected, $result);
+        $this->assertEquals( $expected, $result );
     }
 
     /**
@@ -696,12 +696,12 @@ 
         $contents = '$ioc->register( new \Carbon_Fields\Provider\Container_Condition_Provider() );';
         $expected = '$ioc->register( new \BrianHenryIE\Strauss\Carbon_Fields\Provider\Container_Condition_Provider() );';
 
-        $config = $this->createMock(StraussConfig::class);
+        $config = $this->createMock( StraussConfig::class );
 
-        $replacer = new Prefixer($config, __DIR__);
-        $result = $replacer->replaceNamespace($contents, 'Carbon_Fields\Provider', 'BrianHenryIE\Strauss\Carbon_Fields\Provider');
+        $replacer = new Prefixer( $config, __DIR__ );
+        $result = $replacer->replaceNamespace( $contents, 'Carbon_Fields\Provider', 'BrianHenryIE\Strauss\Carbon_Fields\Provider' );
 
-        $this->assertEquals($expected, $result);
+        $this->assertEquals( $expected, $result );
     }
 
 
@@ -717,12 +717,12 @@ 
         $contents = '@method static \Carbon_Fields\Container\Comment_Meta_Container';
         $expected = '@method static \BrianHenryIE\Strauss\Carbon_Fields\Container\Comment_Meta_Container';
 
-        $config = $this->createMock(StraussConfig::class);
+        $config = $this->createMock( StraussConfig::class );
 
-        $replacer = new Prefixer($config, __DIR__);
-        $result = $replacer->replaceNamespace($contents, 'Carbon_Fields\Container', 'BrianHenryIE\Strauss\Carbon_Fields\Container');
+        $replacer = new Prefixer( $config, __DIR__ );
+        $result = $replacer->replaceNamespace( $contents, 'Carbon_Fields\Container', 'BrianHenryIE\Strauss\Carbon_Fields\Container' );
 
-        $this->assertEquals($expected, $result);
+        $this->assertEquals( $expected, $result );
     }
 
     /**
@@ -736,12 +736,12 @@ 
         $contents = 'return \Carbon_Fields\Carbon_Fields::resolve';
         $expected = 'return \BrianHenryIE\Strauss\Carbon_Fields\Carbon_Fields::resolve';
 
-        $config = $this->createMock(StraussConfig::class);
+        $config = $this->createMock( StraussConfig::class );
 
-        $replacer = new Prefixer($config, __DIR__);
-        $result = $replacer->replaceNamespace($contents, 'Carbon_Fields', 'BrianHenryIE\Strauss\Carbon_Fields');
+        $replacer = new Prefixer( $config, __DIR__ );
+        $result = $replacer->replaceNamespace( $contents, 'Carbon_Fields', 'BrianHenryIE\Strauss\Carbon_Fields' );
 
-        $this->assertEquals($expected, $result);
+        $this->assertEquals( $expected, $result );
     }
 
     /**
@@ -755,16 +755,16 @@ 
         $contents = "		\\Carbon_Fields\\Carbon_Fields::service( 'legacy_storage' )->enable()";
         $expected = "		\\BrianHenryIE\\Strauss\\Carbon_Fields\\Carbon_Fields::service( 'legacy_storage' )->enable()";
 
-        $config = $this->createMock(StraussConfig::class);
+        $config = $this->createMock( StraussConfig::class );
 
-        $replacer = new Prefixer($config, __DIR__);
+        $replacer = new Prefixer( $config, __DIR__ );
         $result = $replacer->replaceNamespace(
             $contents,
             'Carbon_Fields',
             'BrianHenryIE\Strauss\Carbon_Fields'
         );
 
-        $this->assertEquals($expected, $result);
+        $this->assertEquals( $expected, $result );
     }
 
     /**
@@ -778,14 +778,14 @@ 
         $replacement = "Prefix\\TrustedLogin";
         $contents = "esc_html__( 'Learn about TrustedLogin', 'trustedlogin' )";
 
-        $config = $this->createMock(StraussConfig::class);
+        $config = $this->createMock( StraussConfig::class );
 
-        $replacer = new Prefixer($config, __DIR__);
-        $result = $replacer->replaceNamespace($contents, $originalNamespace, $replacement);
+        $replacer = new Prefixer( $config, __DIR__ );
+        $result = $replacer->replaceNamespace( $contents, $originalNamespace, $replacement );
 
         $expected = "esc_html__( 'Learn about TrustedLogin', 'trustedlogin' )";
 
-        $this->assertEquals($expected, $result);
+        $this->assertEquals( $expected, $result );
     }
 
 
@@ -801,14 +801,14 @@ 
         $classnamePrefix = 'Strauss_Issue19_';
         $contents = "public static function objclone(\$object) {";
 
-        $config = $this->createMock(StraussConfig::class);
+        $config = $this->createMock( StraussConfig::class );
 
-        $replacer = new Prefixer($config, __DIR__);
-        $result = $replacer->replaceClassname($contents, $originalClassname, $classnamePrefix);
+        $replacer = new Prefixer( $config, __DIR__ );
+        $result = $replacer->replaceClassname( $contents, $originalClassname, $classnamePrefix );
 
         // NOT public static function objclone($Strauss_Issue19_object) {
         $expected = "public static function objclone(\$object) {";
 
-        $this->assertEquals($expected, $result);
+        $this->assertEquals( $expected, $result );
     }
 }

Braces +18 added lines, -36 removed lines

@@ -20,8 +20,7 @@ 
  * @package BrianHenryIE\Strauss
  * @covers \BrianHenryIE\Strauss\Prefixer
  */
-class PrefixerTest extends TestCase
-{
+class PrefixerTest extends TestCase {
 
     protected StraussConfig $config;
 
@@ -46,8 +45,7 @@ 
         $this->config = new StraussConfig($composer);
     }
 
-    public function testNamespaceReplacer()
-    {
+    public function testNamespaceReplacer() {
 
         $contents = <<<'EOD'
 <?php
@@ -137,8 +135,7 @@ 
     }
 
 
-    public function testClassnameReplacer()
-    {
+    public function testClassnameReplacer() {
 
         $contents = <<<'EOD'
 <?php
@@ -178,8 +175,7 @@ 
     /**
      * PHP 7.4 typed parameters were being prefixed.
      */
-    public function testTypeFunctionParameter()
-    {
+    public function testTypeFunctionParameter() {
         $this->markTestIncomplete();
     }
 
@@ -552,8 +548,7 @@ 
     /**
      * @author BrianHenryIE
      */
-    public function test_it_doesnt_prefix_function_types_that_happen_to_match_the_namespace()
-    {
+    public function test_it_doesnt_prefix_function_types_that_happen_to_match_the_namespace() {
         $namespace = 'Mpdf';
         $prefix = "Mozart";
         $contents = 'public function getServices( Mpdf $mpdf, LoggerInterface $logger, $config, )';
@@ -568,8 +563,7 @@ 
         $this->assertEquals($expected, $result);
     }
 
-    public function testLeadingSlashInString()
-    {
+    public function testLeadingSlashInString() {
         $originalNamespace = "Strauss\\Test";
         $replacement = "Prefix\\Strauss\\Test";
         $contents = '$mentionedClass = "\\Strauss\\Test\\Classname";';
@@ -584,8 +578,7 @@ 
         $this->assertEquals($expected, $result);
     }
 
-    public function testDoubleLeadingSlashInString()
-    {
+    public function testDoubleLeadingSlashInString() {
         $originalNamespace = "Strauss\\Test";
         $replacement = "Prefix\\Strauss\\Test";
         $contents = '$mentionedClass = "\\\\Strauss\\\\Test\\\\Classname";';
@@ -600,8 +593,7 @@ 
         $this->assertEquals($expected, $result);
     }
 
-    public function testItReplacesSlashedNamespaceInFunctionParameter()
-    {
+    public function testItReplacesSlashedNamespaceInFunctionParameter() {
 
         $originalNamespace = "net\\authorize\\api\\contract\\v1";
         $replacement = "Prefix\\net\\authorize\\api\\contract\\v1";
@@ -618,8 +610,7 @@ 
     }
 
 
-    public function testItReplacesNamespaceInFunctionParameterDefaultAgumentValue()
-    {
+    public function testItReplacesNamespaceInFunctionParameterDefaultAgumentValue() {
 
         $originalNamespace = "net\\authorize\\api\constants";
         $replacement = "Prefix\\net\\authorize\\api\constants";
@@ -636,8 +627,7 @@ 
     }
 
 
-    public function testItReplacesNamespaceConcatenatedStringConst()
-    {
+    public function testItReplacesNamespaceConcatenatedStringConst() {
 
         $originalNamespace = "net\\authorize\\api\\constants";
         $replacement = "Prefix\\net\\authorize\\api\\constants";
@@ -657,8 +647,7 @@ 
     /**
      * Another mpdf issue where the class "Mpdf" is in the namespace "Mpdf" and incorrect replacements are being made.
      */
-    public function testClassnameNotConfusedWithNamespace()
-    {
+    public function testClassnameNotConfusedWithNamespace() {
 
         $contents = '$default_font_size = $mmsize * (Mpdf::SCALE);';
         $expected = $contents;
@@ -671,8 +660,7 @@ 
         $this->assertEquals($expected, $result);
     }
 
-    public function testClassExtendsNamspacedClassIsPrefixed()
-    {
+    public function testClassExtendsNamspacedClassIsPrefixed() {
 
         $contents = 'class BarcodeException extends \Mpdf\MpdfException';
         $expected = 'class BarcodeException extends \BrianHenryIE\Strauss\Mpdf\MpdfException';
@@ -690,8 +678,7 @@ 
      *
      * @see https://github.com/BrianHenryIE/strauss/issues/11
      */
-    public function testNewNamespacedClassIsPrefixed()
-    {
+    public function testNewNamespacedClassIsPrefixed() {
 
         $contents = '$ioc->register( new \Carbon_Fields\Provider\Container_Condition_Provider() );';
         $expected = '$ioc->register( new \BrianHenryIE\Strauss\Carbon_Fields\Provider\Container_Condition_Provider() );';
@@ -711,8 +698,7 @@ 
      *
      * @see https://github.com/BrianHenryIE/strauss/issues/11
      */
-    public function testStaticNamespacedClassIsPrefixed()
-    {
+    public function testStaticNamespacedClassIsPrefixed() {
 
         $contents = '@method static \Carbon_Fields\Container\Comment_Meta_Container';
         $expected = '@method static \BrianHenryIE\Strauss\Carbon_Fields\Container\Comment_Meta_Container';
@@ -730,8 +716,7 @@ 
      *
      * @see https://github.com/BrianHenryIE/strauss/issues/11
      */
-    public function testReturnedNamespacedClassIsPrefixed()
-    {
+    public function testReturnedNamespacedClassIsPrefixed() {
 
         $contents = 'return \Carbon_Fields\Carbon_Fields::resolve';
         $expected = 'return \BrianHenryIE\Strauss\Carbon_Fields\Carbon_Fields::resolve';
@@ -749,8 +734,7 @@ 
      *
      * @see https://github.com/BrianHenryIE/strauss/issues/11
      */
-    public function testNamespacedStaticIsPrefixed()
-    {
+    public function testNamespacedStaticIsPrefixed() {
 
         $contents = "		\\Carbon_Fields\\Carbon_Fields::service( 'legacy_storage' )->enable()";
         $expected = "		\\BrianHenryIE\\Strauss\\Carbon_Fields\\Carbon_Fields::service( 'legacy_storage' )->enable()";
@@ -772,8 +756,7 @@ 
      *
      * @see https://github.com/BrianHenryIE/strauss/issues/15
      */
-    public function testDoNotReplaceInStringThatIsNotCode()
-    {
+    public function testDoNotReplaceInStringThatIsNotCode() {
         $originalNamespace = "TrustedLogin";
         $replacement = "Prefix\\TrustedLogin";
         $contents = "esc_html__( 'Learn about TrustedLogin', 'trustedlogin' )";
@@ -795,8 +778,7 @@ 
      * @see https://github.com/BrianHenryIE/strauss/issues/19
      *
      */
-    public function testDoNotReplaceInVariableNames()
-    {
+    public function testDoNotReplaceInVariableNames() {
         $originalClassname = 'object';
         $classnamePrefix = 'Strauss_Issue19_';
         $contents = "public static function objclone(\$object) {";

Upper-Lower-Casing +3 added lines, -3 removed lines

@@ -36,7 +36,7 @@ 
 
 }
 }
-EOD;
+eod;
 
         $composerConfig = new Config(false);
         $composerConfig->merge(json_decode($composerJson, true));
@@ -121,7 +121,7 @@ 
     );
   }
 }
-EOD;
+eod;
         $config = $this->createMock(StraussConfig::class);
 
         $replacer = new Prefixer($config, __DIR__);
@@ -159,7 +159,7 @@ 
 protected $offsets;            // array of object offsets
 protected $buffer;             // buffer holding in-memory PDF
 }
-EOD;
+eod;
 
         $config = $this->createMock(StraussConfig::class);
 

vendor/brianhenryie/strauss/tests/Unit/LicenserTest.php

Indentation +142 added lines, -142 removed lines

@@ -22,80 +22,80 @@ 
 {
 
 
-    /**
-     * @covers ::findLicenseFiles
-     *
-     */
-    public function testFindLicenceFilesPathsAreRelative()
-    {
-        $config = $this->createStub(StraussConfig::class);
-        $workingDir = __DIR__ . DIRECTORY_SEPARATOR;
+	/**
+	 * @covers ::findLicenseFiles
+	 *
+	 */
+	public function testFindLicenceFilesPathsAreRelative()
+	{
+		$config = $this->createStub(StraussConfig::class);
+		$workingDir = __DIR__ . DIRECTORY_SEPARATOR;
 
-        $dependencies = array();
+		$dependencies = array();
 
-        $dependency = $this->createStub(ComposerPackage::class);
-        $dependency->method('getPath')->willReturn('developer-name/project-name/');
-        $dependencies[] = $dependency;
+		$dependency = $this->createStub(ComposerPackage::class);
+		$dependency->method('getPath')->willReturn('developer-name/project-name/');
+		$dependencies[] = $dependency;
 
-        $sut = new Licenser($config, $workingDir, $dependencies, 'BrianHenryIE');
+		$sut = new Licenser($config, $workingDir, $dependencies, 'BrianHenryIE');
 
-        $finder = $this->createStub(Finder::class);
+		$finder = $this->createStub(Finder::class);
 
-        $file = $this->createStub(\SplFileInfo::class);
-        $file->method('getPathname')
-            ->willReturn(__DIR__.'/vendor/developer-name/project-name/license.md');
+		$file = $this->createStub(\SplFileInfo::class);
+		$file->method('getPathname')
+			->willReturn(__DIR__.'/vendor/developer-name/project-name/license.md');
 
-        $finderArrayIterator = new ArrayIterator(array(
-            $file
-        ));
+		$finderArrayIterator = new ArrayIterator(array(
+			$file
+		));
 
-        $finder->method('getIterator')->willReturn($finderArrayIterator);
+		$finder->method('getIterator')->willReturn($finderArrayIterator);
 
-        // Make the rest fluent.
-        $callableConstraintNotGetIterator = function ($methodName) {
-            return 'getIterator' !== $methodName;
-        };
-        $finder->method(new Callback($callableConstraintNotGetIterator))->willReturn($finder);
+		// Make the rest fluent.
+		$callableConstraintNotGetIterator = function ($methodName) {
+			return 'getIterator' !== $methodName;
+		};
+		$finder->method(new Callback($callableConstraintNotGetIterator))->willReturn($finder);
 
-        $sut->findLicenseFiles($finder);
+		$sut->findLicenseFiles($finder);
 
-        $result = $sut->getDiscoveredLicenseFiles();
+		$result = $sut->getDiscoveredLicenseFiles();
 
-        // Currently contains an array entry: /Users/brianhenry/Sites/mozart/mozart/tests/Unit/developer-name/project-name/license.md
-        $this->assertStringContainsString('developer-name/project-name/license.md', $result[0]);
-    }
+		// Currently contains an array entry: /Users/brianhenry/Sites/mozart/mozart/tests/Unit/developer-name/project-name/license.md
+		$this->assertStringContainsString('developer-name/project-name/license.md', $result[0]);
+	}
 
-    /**
-     * Licence files should be found regardless of case and regardless of British/US-English spelling.
-     *
-     * @see https://www.phpliveregex.com/p/A5y
-     */
+	/**
+	 * Licence files should be found regardless of case and regardless of British/US-English spelling.
+	 *
+	 * @see https://www.phpliveregex.com/p/A5y
+	 */
 
-    // https://schibsted.com/blog/mocking-the-file-system-using-phpunit-and-vfsstream/
+	// https://schibsted.com/blog/mocking-the-file-system-using-phpunit-and-vfsstream/
 
 
-    /**
-     * @see https://github.com/AuthorizeNet/sdk-php/blob/a3e76f96f674d16e892f87c58bedb99dada4b067/lib/net/authorize/api/contract/v1/ANetApiRequestType.php
-     *
-     * @covers ::addChangeDeclarationToPhpString
-     */
-    public function testAppendHeaderCommentInformationNoHeader()
-    {
+	/**
+	 * @see https://github.com/AuthorizeNet/sdk-php/blob/a3e76f96f674d16e892f87c58bedb99dada4b067/lib/net/authorize/api/contract/v1/ANetApiRequestType.php
+	 *
+	 * @covers ::addChangeDeclarationToPhpString
+	 */
+	public function testAppendHeaderCommentInformationNoHeader()
+	{
 
-        $author = 'BrianHenryIE';
+		$author = 'BrianHenryIE';
 
-        $config = $this->createStub(StraussConfig::class);
-        $sut = new Licenser($config, __DIR__, array(), $author);
+		$config = $this->createStub(StraussConfig::class);
+		$sut = new Licenser($config, __DIR__, array(), $author);
 
-        $given = <<<'EOD'
+		$given = <<<'EOD'
 <?php
 
 namespace net\authorize\api\contract\v1;
 EOD;
 
-        //     "license": "proprietary",
+		//     "license": "proprietary",
 
-        $expected = <<<'EOD'
+		$expected = <<<'EOD'
 <?php
 /**
  * @license proprietary
@@ -107,22 +107,22 @@ 
 namespace net\authorize\api\contract\v1;
 EOD;
 
-        $actual = $sut->addChangeDeclarationToPhpString($given, '25-April-2021', 'proprietary');
+		$actual = $sut->addChangeDeclarationToPhpString($given, '25-April-2021', 'proprietary');
 
-        $this->assertEquals($expected, $actual);
-    }
+		$this->assertEquals($expected, $actual);
+	}
 
-    /**
-     * @covers ::addChangeDeclarationToPhpString
-     */
-    public function testWithLicenceAlreadyInHeader(): void
-    {
+	/**
+	 * @covers ::addChangeDeclarationToPhpString
+	 */
+	public function testWithLicenceAlreadyInHeader(): void
+	{
 
-        $config = $this->createStub(StraussConfig::class);
-        $author = 'BrianHenryIE';
-        $sut = new Licenser($config, __DIR__, array(), $author);
+		$config = $this->createStub(StraussConfig::class);
+		$author = 'BrianHenryIE';
+		$sut = new Licenser($config, __DIR__, array(), $author);
 
-        $given = <<<'EOD'
+		$given = <<<'EOD'
 <?php // phpcs:ignore WordPress.Files.FileName
 /**
  * Handles dismissing admin notices.
@@ -137,7 +137,7 @@ 
 namespace Yeah;
 EOD;
 
-        $expected = <<<'EOD'
+		$expected = <<<'EOD'
 <?php // phpcs:ignore WordPress.Files.FileName
 /**
  * Handles dismissing admin notices.
@@ -155,25 +155,25 @@ 
 namespace Yeah;
 EOD;
 
-        $actual = $sut->addChangeDeclarationToPhpString($given, '25-April-2021', 'GPL-2.0-or-later');
+		$actual = $sut->addChangeDeclarationToPhpString($given, '25-April-2021', 'GPL-2.0-or-later');
 
-        $this->assertEquals($expected, $actual);
-    }
+		$this->assertEquals($expected, $actual);
+	}
 
 
-    /**
-     * Shouldn't matter too much but y'know regexes.
-     *
-     * @covers ::addChangeDeclarationToPhpString
-     */
-    public function testWithTwoCommentsBeforeFirstCode()
-    {
+	/**
+	 * Shouldn't matter too much but y'know regexes.
+	 *
+	 * @covers ::addChangeDeclarationToPhpString
+	 */
+	public function testWithTwoCommentsBeforeFirstCode()
+	{
 
-        $config = $this->createStub(StraussConfig::class);
-        $author = 'BrianHenryIE';
-        $sut = new Licenser($config, __DIR__, array(), $author);
+		$config = $this->createStub(StraussConfig::class);
+		$author = 'BrianHenryIE';
+		$sut = new Licenser($config, __DIR__, array(), $author);
 
-        $given = <<<'EOD'
+		$given = <<<'EOD'
 <?php
 /**
  * WP Dependency Installer
@@ -194,7 +194,7 @@ 
 if ( ! defined( 'WPINC' ) ) {
 EOD;
 
-        $expected = <<<'EOD'
+		$expected = <<<'EOD'
 <?php
 /**
  * WP Dependency Installer
@@ -219,22 +219,22 @@ 
 EOD;
 
 
-        $actual = $sut->addChangeDeclarationToPhpString($given, '25-April-2021', 'MIT');
+		$actual = $sut->addChangeDeclarationToPhpString($given, '25-April-2021', 'MIT');
 
-        $this->assertEquals($expected, $actual);
-    }
+		$this->assertEquals($expected, $actual);
+	}
 
-    /**
-     * @covers ::addChangeDeclarationToPhpString
-     */
-    public function testUnusualHeaderCommentStyle()
-    {
+	/**
+	 * @covers ::addChangeDeclarationToPhpString
+	 */
+	public function testUnusualHeaderCommentStyle()
+	{
 
-        $config = $this->createStub(StraussConfig::class);
-        $author = 'BrianHenryIE';
-        $sut = new Licenser($config, __DIR__, array(), $author);
+		$config = $this->createStub(StraussConfig::class);
+		$author = 'BrianHenryIE';
+		$sut = new Licenser($config, __DIR__, array(), $author);
 
-        $given = <<<'EOD'
+		$given = <<<'EOD'
 <?php
 /*******************************************************************************
 * FPDF                                                                         *
@@ -247,7 +247,7 @@ 
 define('FPDF_VERSION','1.82');
 EOD;
 
-        $expected = <<<'EOD'
+		$expected = <<<'EOD'
 <?php
 /*******************************************************************************
 * FPDF                                                                         *
@@ -264,22 +264,22 @@ 
 define('FPDF_VERSION','1.82');
 EOD;
 
-        $actual = $sut->addChangeDeclarationToPhpString($given, '25-April-2021', 'proprietary');
+		$actual = $sut->addChangeDeclarationToPhpString($given, '25-April-2021', 'proprietary');
 
-        $this->assertEquals($expected, $actual);
-    }
+		$this->assertEquals($expected, $actual);
+	}
 
-    /**
-     * @covers ::addChangeDeclarationToPhpString
-     */
-    public function testCommentWithLicenseWord()
-    {
+	/**
+	 * @covers ::addChangeDeclarationToPhpString
+	 */
+	public function testCommentWithLicenseWord()
+	{
 
-        $config = $this->createStub(StraussConfig::class);
-        $author = 'BrianHenryIE';
-        $sut = new Licenser($config, __DIR__, array(), $author);
+		$config = $this->createStub(StraussConfig::class);
+		$author = 'BrianHenryIE';
+		$sut = new Licenser($config, __DIR__, array(), $author);
 
-        $given = <<<'EOD'
+		$given = <<<'EOD'
 <?php
 
 /**
@@ -297,7 +297,7 @@ 
 namespace Your_Domain\Assert;
 EOD;
 
-        $expected = <<<'EOD'
+		$expected = <<<'EOD'
 <?php
 
 /**
@@ -318,27 +318,27 @@ 
 namespace Your_Domain\Assert;
 EOD;
 
-        $actual = $sut->addChangeDeclarationToPhpString($given, '25-April-2021', 'MIT');
+		$actual = $sut->addChangeDeclarationToPhpString($given, '25-April-2021', 'MIT');
 
-        $this->assertEquals($expected, $actual);
-    }
+		$this->assertEquals($expected, $actual);
+	}
 
-    /**
-     * This was matching the "no header comment" regex.
-     *
-     * FOCK: The test passed. How do I debug when the test passes?! The test is passing but actual output is incorrect.
-     * @see https://www.youtube.com/watch?v=QnxpHIl5Ynw
-     *
-     * Seems files loaded are treated different to strings passed.
-     */
-    public function testIncorrectlyMatching()
-    {
+	/**
+	 * This was matching the "no header comment" regex.
+	 *
+	 * FOCK: The test passed. How do I debug when the test passes?! The test is passing but actual output is incorrect.
+	 * @see https://www.youtube.com/watch?v=QnxpHIl5Ynw
+	 *
+	 * Seems files loaded are treated different to strings passed.
+	 */
+	public function testIncorrectlyMatching()
+	{
 
-        $config = $this->createStub(StraussConfig::class);
-        $author = 'BrianHenryIE';
-        $sut = new Licenser($config, __DIR__, array(), $author);
+		$config = $this->createStub(StraussConfig::class);
+		$author = 'BrianHenryIE';
+		$sut = new Licenser($config, __DIR__, array(), $author);
 
-        $fileContents = <<<'EOD'
+		$fileContents = <<<'EOD'
 <?php
 /**
  * WP Dependency Installer
@@ -361,12 +361,12 @@ 
 }
 EOD;
 
-        // Attempt to replicate the failing test, since the contents seem the same but the input manner is different.
-        $tmpfname = tempnam(sys_get_temp_dir(), 'Strauss-' . __CLASS__ . '-' . __FUNCTION__);
-        file_put_contents($tmpfname, $fileContents);
-        $given = file_get_contents($tmpfname);
+		// Attempt to replicate the failing test, since the contents seem the same but the input manner is different.
+		$tmpfname = tempnam(sys_get_temp_dir(), 'Strauss-' . __CLASS__ . '-' . __FUNCTION__);
+		file_put_contents($tmpfname, $fileContents);
+		$given = file_get_contents($tmpfname);
 
-        $expected = <<<'EOD'
+		$expected = <<<'EOD'
 <?php
 /**
  * WP Dependency Installer
@@ -393,22 +393,22 @@ 
 EOD;
 
 
-        $actual = $sut->addChangeDeclarationToPhpString($given, '25-April-2021', 'MIT');
+		$actual = $sut->addChangeDeclarationToPhpString($given, '25-April-2021', 'MIT');
 
-        $this->assertEquals($expected, $actual);
-    }
+		$this->assertEquals($expected, $actual);
+	}
 
-    /**
-     * The licence was being inserted after every `<?php` in the file.
-     */
-    public function testLicenseDetailsOnlyInsertedOncePerFile()
-    {
+	/**
+	 * The licence was being inserted after every `<?php` in the file.
+	 */
+	public function testLicenseDetailsOnlyInsertedOncePerFile()
+	{
 
-        $config = $this->createStub(StraussConfig::class);
-        $author = 'BrianHenryIE';
-        $sut = new Licenser($config, __DIR__, array(), $author);
+		$config = $this->createStub(StraussConfig::class);
+		$author = 'BrianHenryIE';
+		$sut = new Licenser($config, __DIR__, array(), $author);
 
-        $fileContents = <<<'EOD'
+		$fileContents = <<<'EOD'
 <?php
 
 ?>
@@ -418,7 +418,7 @@ 
 ?>
 EOD;
 
-        $expected = <<<'EOD'
+		$expected = <<<'EOD'
 <?php
 /**
  * @license MIT
@@ -435,8 +435,8 @@ 
 EOD;
 
 
-        $actual = $sut->addChangeDeclarationToPhpString($fileContents, '25-April-2021', 'MIT');
+		$actual = $sut->addChangeDeclarationToPhpString($fileContents, '25-April-2021', 'MIT');
 
-        $this->assertEquals($expected, $actual);
-    }
+		$this->assertEquals($expected, $actual);
+	}
 }

Spacing +47 added lines, -47 removed lines

@@ -28,41 +28,41 @@ 
      */
     public function testFindLicenceFilesPathsAreRelative()
     {
-        $config = $this->createStub(StraussConfig::class);
+        $config = $this->createStub( StraussConfig::class );
         $workingDir = __DIR__ . DIRECTORY_SEPARATOR;
 
         $dependencies = array();
 
-        $dependency = $this->createStub(ComposerPackage::class);
-        $dependency->method('getPath')->willReturn('developer-name/project-name/');
-        $dependencies[] = $dependency;
+        $dependency = $this->createStub( ComposerPackage::class );
+        $dependency->method( 'getPath' )->willReturn( 'developer-name/project-name/' );
+        $dependencies[ ] = $dependency;
 
-        $sut = new Licenser($config, $workingDir, $dependencies, 'BrianHenryIE');
+        $sut = new Licenser( $config, $workingDir, $dependencies, 'BrianHenryIE' );
 
-        $finder = $this->createStub(Finder::class);
+        $finder = $this->createStub( Finder::class );
 
-        $file = $this->createStub(\SplFileInfo::class);
-        $file->method('getPathname')
-            ->willReturn(__DIR__.'/vendor/developer-name/project-name/license.md');
+        $file = $this->createStub( \SplFileInfo::class );
+        $file->method( 'getPathname' )
+            ->willReturn( __DIR__ . '/vendor/developer-name/project-name/license.md' );
 
-        $finderArrayIterator = new ArrayIterator(array(
+        $finderArrayIterator = new ArrayIterator( array(
             $file
-        ));
+        ) );
 
-        $finder->method('getIterator')->willReturn($finderArrayIterator);
+        $finder->method( 'getIterator' )->willReturn( $finderArrayIterator );
 
         // Make the rest fluent.
-        $callableConstraintNotGetIterator = function ($methodName) {
+        $callableConstraintNotGetIterator = function( $methodName ) {
             return 'getIterator' !== $methodName;
         };
-        $finder->method(new Callback($callableConstraintNotGetIterator))->willReturn($finder);
+        $finder->method( new Callback( $callableConstraintNotGetIterator ) )->willReturn( $finder );
 
-        $sut->findLicenseFiles($finder);
+        $sut->findLicenseFiles( $finder );
 
         $result = $sut->getDiscoveredLicenseFiles();
 
         // Currently contains an array entry: /Users/brianhenry/Sites/mozart/mozart/tests/Unit/developer-name/project-name/license.md
-        $this->assertStringContainsString('developer-name/project-name/license.md', $result[0]);
+        $this->assertStringContainsString( 'developer-name/project-name/license.md', $result[ 0 ] );
     }
 
     /**
@@ -84,8 +84,8 @@ 
 
         $author = 'BrianHenryIE';
 
-        $config = $this->createStub(StraussConfig::class);
-        $sut = new Licenser($config, __DIR__, array(), $author);
+        $config = $this->createStub( StraussConfig::class );
+        $sut = new Licenser( $config, __DIR__, array(), $author );
 
         $given = <<<'EOD'
 <?php
@@ -107,9 +107,9 @@ 
 namespace net\authorize\api\contract\v1;
 EOD;
 
-        $actual = $sut->addChangeDeclarationToPhpString($given, '25-April-2021', 'proprietary');
+        $actual = $sut->addChangeDeclarationToPhpString( $given, '25-April-2021', 'proprietary' );
 
-        $this->assertEquals($expected, $actual);
+        $this->assertEquals( $expected, $actual );
     }
 
     /**
@@ -118,9 +118,9 @@ 
     public function testWithLicenceAlreadyInHeader(): void
     {
 
-        $config = $this->createStub(StraussConfig::class);
+        $config = $this->createStub( StraussConfig::class );
         $author = 'BrianHenryIE';
-        $sut = new Licenser($config, __DIR__, array(), $author);
+        $sut = new Licenser( $config, __DIR__, array(), $author );
 
         $given = <<<'EOD'
 <?php // phpcs:ignore WordPress.Files.FileName
@@ -155,9 +155,9 @@ 
 namespace Yeah;
 EOD;
 
-        $actual = $sut->addChangeDeclarationToPhpString($given, '25-April-2021', 'GPL-2.0-or-later');
+        $actual = $sut->addChangeDeclarationToPhpString( $given, '25-April-2021', 'GPL-2.0-or-later' );
 
-        $this->assertEquals($expected, $actual);
+        $this->assertEquals( $expected, $actual );
     }
 
 
@@ -169,9 +169,9 @@ 
     public function testWithTwoCommentsBeforeFirstCode()
     {
 
-        $config = $this->createStub(StraussConfig::class);
+        $config = $this->createStub( StraussConfig::class );
         $author = 'BrianHenryIE';
-        $sut = new Licenser($config, __DIR__, array(), $author);
+        $sut = new Licenser( $config, __DIR__, array(), $author );
 
         $given = <<<'EOD'
 <?php
@@ -219,9 +219,9 @@ 
 EOD;
 
 
-        $actual = $sut->addChangeDeclarationToPhpString($given, '25-April-2021', 'MIT');
+        $actual = $sut->addChangeDeclarationToPhpString( $given, '25-April-2021', 'MIT' );
 
-        $this->assertEquals($expected, $actual);
+        $this->assertEquals( $expected, $actual );
     }
 
     /**
@@ -230,9 +230,9 @@ 
     public function testUnusualHeaderCommentStyle()
     {
 
-        $config = $this->createStub(StraussConfig::class);
+        $config = $this->createStub( StraussConfig::class );
         $author = 'BrianHenryIE';
-        $sut = new Licenser($config, __DIR__, array(), $author);
+        $sut = new Licenser( $config, __DIR__, array(), $author );
 
         $given = <<<'EOD'
 <?php
@@ -264,9 +264,9 @@ 
 define('FPDF_VERSION','1.82');
 EOD;
 
-        $actual = $sut->addChangeDeclarationToPhpString($given, '25-April-2021', 'proprietary');
+        $actual = $sut->addChangeDeclarationToPhpString( $given, '25-April-2021', 'proprietary' );
 
-        $this->assertEquals($expected, $actual);
+        $this->assertEquals( $expected, $actual );
     }
 
     /**
@@ -275,9 +275,9 @@ 
     public function testCommentWithLicenseWord()
     {
 
-        $config = $this->createStub(StraussConfig::class);
+        $config = $this->createStub( StraussConfig::class );
         $author = 'BrianHenryIE';
-        $sut = new Licenser($config, __DIR__, array(), $author);
+        $sut = new Licenser( $config, __DIR__, array(), $author );
 
         $given = <<<'EOD'
 <?php
@@ -318,9 +318,9 @@ 
 namespace Your_Domain\Assert;
 EOD;
 
-        $actual = $sut->addChangeDeclarationToPhpString($given, '25-April-2021', 'MIT');
+        $actual = $sut->addChangeDeclarationToPhpString( $given, '25-April-2021', 'MIT' );
 
-        $this->assertEquals($expected, $actual);
+        $this->assertEquals( $expected, $actual );
     }
 
     /**
@@ -334,9 +334,9 @@ 
     public function testIncorrectlyMatching()
     {
 
-        $config = $this->createStub(StraussConfig::class);
+        $config = $this->createStub( StraussConfig::class );
         $author = 'BrianHenryIE';
-        $sut = new Licenser($config, __DIR__, array(), $author);
+        $sut = new Licenser( $config, __DIR__, array(), $author );
 
         $fileContents = <<<'EOD'
 <?php
@@ -362,9 +362,9 @@ 
 EOD;
 
         // Attempt to replicate the failing test, since the contents seem the same but the input manner is different.
-        $tmpfname = tempnam(sys_get_temp_dir(), 'Strauss-' . __CLASS__ . '-' . __FUNCTION__);
-        file_put_contents($tmpfname, $fileContents);
-        $given = file_get_contents($tmpfname);
+        $tmpfname = tempnam( sys_get_temp_dir(), 'Strauss-' . __CLASS__ . '-' . __FUNCTION__ );
+        file_put_contents( $tmpfname, $fileContents );
+        $given = file_get_contents( $tmpfname );
 
         $expected = <<<'EOD'
 <?php
@@ -393,9 +393,9 @@ 
 EOD;
 
 
-        $actual = $sut->addChangeDeclarationToPhpString($given, '25-April-2021', 'MIT');
+        $actual = $sut->addChangeDeclarationToPhpString( $given, '25-April-2021', 'MIT' );
 
-        $this->assertEquals($expected, $actual);
+        $this->assertEquals( $expected, $actual );
     }
 
     /**
@@ -404,9 +404,9 @@ 
     public function testLicenseDetailsOnlyInsertedOncePerFile()
     {
 
-        $config = $this->createStub(StraussConfig::class);
+        $config = $this->createStub( StraussConfig::class );
         $author = 'BrianHenryIE';
-        $sut = new Licenser($config, __DIR__, array(), $author);
+        $sut = new Licenser( $config, __DIR__, array(), $author );
 
         $fileContents = <<<'EOD'
 <?php
@@ -435,8 +435,8 @@ 
 EOD;
 
 
-        $actual = $sut->addChangeDeclarationToPhpString($fileContents, '25-April-2021', 'MIT');
+        $actual = $sut->addChangeDeclarationToPhpString( $fileContents, '25-April-2021', 'MIT' );
 
-        $this->assertEquals($expected, $actual);
+        $this->assertEquals( $expected, $actual );
     }
 }

Braces +8 added lines, -16 removed lines

@@ -18,16 +18,14 @@ 
  * @package BrianHenryIE\Strauss\Tests\Unit
  * @coversDefaultClass \BrianHenryIE\Strauss\Licenser
  */
-class LicenserTest extends TestCase
-{
+class LicenserTest extends TestCase {
 
 
     /**
      * @covers ::findLicenseFiles
      *
      */
-    public function testFindLicenceFilesPathsAreRelative()
-    {
+    public function testFindLicenceFilesPathsAreRelative() {
         $config = $this->createStub(StraussConfig::class);
         $workingDir = __DIR__ . DIRECTORY_SEPARATOR;
 
@@ -79,8 +77,7 @@ 
      *
      * @covers ::addChangeDeclarationToPhpString
      */
-    public function testAppendHeaderCommentInformationNoHeader()
-    {
+    public function testAppendHeaderCommentInformationNoHeader() {
 
         $author = 'BrianHenryIE';
 
@@ -166,8 +163,7 @@ 
      *
      * @covers ::addChangeDeclarationToPhpString
      */
-    public function testWithTwoCommentsBeforeFirstCode()
-    {
+    public function testWithTwoCommentsBeforeFirstCode() {
 
         $config = $this->createStub(StraussConfig::class);
         $author = 'BrianHenryIE';
@@ -227,8 +223,7 @@ 
     /**
      * @covers ::addChangeDeclarationToPhpString
      */
-    public function testUnusualHeaderCommentStyle()
-    {
+    public function testUnusualHeaderCommentStyle() {
 
         $config = $this->createStub(StraussConfig::class);
         $author = 'BrianHenryIE';
@@ -272,8 +267,7 @@ 
     /**
      * @covers ::addChangeDeclarationToPhpString
      */
-    public function testCommentWithLicenseWord()
-    {
+    public function testCommentWithLicenseWord() {
 
         $config = $this->createStub(StraussConfig::class);
         $author = 'BrianHenryIE';
@@ -331,8 +325,7 @@ 
      *
      * Seems files loaded are treated different to strings passed.
      */
-    public function testIncorrectlyMatching()
-    {
+    public function testIncorrectlyMatching() {
 
         $config = $this->createStub(StraussConfig::class);
         $author = 'BrianHenryIE';
@@ -401,8 +394,7 @@ 
     /**
      * The licence was being inserted after every `<?php` in the file.
      */
-    public function testLicenseDetailsOnlyInsertedOncePerFile()
-    {
+    public function testLicenseDetailsOnlyInsertedOncePerFile() {
 
         $config = $this->createStub(StraussConfig::class);
         $author = 'BrianHenryIE';

Upper-Lower-Casing +14 added lines, -14 removed lines

@@ -91,7 +91,7 @@ 
 <?php
 
 namespace net\authorize\api\contract\v1;
-EOD;
+eod;
 
         //     "license": "proprietary",
 
@@ -105,7 +105,7 @@ 
  */
 
 namespace net\authorize\api\contract\v1;
-EOD;
+eod;
 
         $actual = $sut->addChangeDeclarationToPhpString($given, '25-April-2021', 'proprietary');
 
@@ -135,7 +135,7 @@ 
  */
 
 namespace Yeah;
-EOD;
+eod;
 
         $expected = <<<'EOD'
 <?php // phpcs:ignore WordPress.Files.FileName
@@ -153,7 +153,7 @@ 
  */
 
 namespace Yeah;
-EOD;
+eod;
 
         $actual = $sut->addChangeDeclarationToPhpString($given, '25-April-2021', 'GPL-2.0-or-later');
 
@@ -192,7 +192,7 @@ 
  * Exit if called directly.
  */
 if ( ! defined( 'WPINC' ) ) {
-EOD;
+eod;
 
         $expected = <<<'EOD'
 <?php
@@ -216,7 +216,7 @@ 
  * Exit if called directly.
  */
 if ( ! defined( 'WPINC' ) ) {
-EOD;
+eod;
 
 
         $actual = $sut->addChangeDeclarationToPhpString($given, '25-April-2021', 'MIT');
@@ -245,7 +245,7 @@ 
 *******************************************************************************/
 
 define('FPDF_VERSION','1.82');
-EOD;
+eod;
 
         $expected = <<<'EOD'
 <?php
@@ -262,7 +262,7 @@ 
 */
 
 define('FPDF_VERSION','1.82');
-EOD;
+eod;
 
         $actual = $sut->addChangeDeclarationToPhpString($given, '25-April-2021', 'proprietary');
 
@@ -295,7 +295,7 @@ 
  */
 
 namespace Your_Domain\Assert;
-EOD;
+eod;
 
         $expected = <<<'EOD'
 <?php
@@ -316,7 +316,7 @@ 
  */
 
 namespace Your_Domain\Assert;
-EOD;
+eod;
 
         $actual = $sut->addChangeDeclarationToPhpString($given, '25-April-2021', 'MIT');
 
@@ -359,7 +359,7 @@ 
 if ( ! defined( 'WPINC' ) ) {
 	die;
 }
-EOD;
+eod;
 
         // Attempt to replicate the failing test, since the contents seem the same but the input manner is different.
         $tmpfname = tempnam(sys_get_temp_dir(), 'Strauss-' . __CLASS__ . '-' . __FUNCTION__);
@@ -390,7 +390,7 @@ 
 if ( ! defined( 'WPINC' ) ) {
 	die;
 }
-EOD;
+eod;
 
 
         $actual = $sut->addChangeDeclarationToPhpString($given, '25-April-2021', 'MIT');
@@ -416,7 +416,7 @@ 
 <?php
 
 ?>
-EOD;
+eod;
 
         $expected = <<<'EOD'
 <?php
@@ -432,7 +432,7 @@ 
 <?php
 
 ?>
-EOD;
+eod;
 
 
         $actual = $sut->addChangeDeclarationToPhpString($fileContents, '25-April-2021', 'MIT');

vendor/brianhenryie/strauss/tests/Unit/Console/Commands/ComposeTest.php

Indentation +190 added lines, -190 removed lines

@@ -11,194 +11,194 @@
 class ComposeTest extends TestCase
 {
 
-    /**
-     * When composer.json is absent, instead of failing with:
-     * "failed to open stream: No such file or directory"
-     * a better message should be written to the OutputInterface.
-     *
-     * @test
-     */
-    public function it_fails_gracefully_when_composer_json_absent(): void
-    {
-        chdir(sys_get_temp_dir());
-
-        $inputInterfaceMock = $this->createMock(InputInterface::class);
-        $outputInterfaceMock = $this->createMock(OutputInterface::class);
-
-        $outputInterfaceMock->expects($this->exactly(1))
-             ->method('write');
-
-        new class( $inputInterfaceMock, $outputInterfaceMock ) extends Compose {
-            public function __construct($inputInterfaceMock, $outputInterfaceMock)
-            {
-                parent::__construct();
-
-                $this->execute($inputInterfaceMock, $outputInterfaceMock);
-            }
-        };
-    }
-
-    /**
-     * When json_decode fails, instead of
-     * "Trying to get property 'extra' of non-object"
-     * a better message should be written to the OutputInterface.
-     *
-     * @test
-     */
-    public function it_handles_malformed_json_with_grace(): void
-    {
-
-        $badComposerJson = '{ "name": "coenjacobs/mozart", }';
-
-        $tmpfname = tempnam(sys_get_temp_dir(), 'Strauss-' . __CLASS__ . '-' . __FUNCTION__);
-        file_put_contents($tmpfname, $badComposerJson);
-        chdir(dirname($tmpfname));
-
-        $inputInterfaceMock = $this->createMock(InputInterface::class);
-        $outputInterfaceMock = $this->createMock(OutputInterface::class);
-
-        $outputInterfaceMock->expects($this->exactly(1))
-                            ->method('write');
-
-        new class( $inputInterfaceMock, $outputInterfaceMock ) extends Compose {
-            public function __construct($inputInterfaceMock, $outputInterfaceMock)
-            {
-                parent::__construct();
-
-                $this->execute($inputInterfaceMock, $outputInterfaceMock);
-            }
-        };
-    }
-
-    /**
-     * When composer.json->extra is absent, instead of
-     * "Undefined property: stdClass::$extra"
-     * a better message should be written to the OutputInterface.
-     *
-     * When package name is not set, `\Composer\Composer::getPackage()->getName()` returns '__root__'.
-     *
-     */
-    public function test_it_handles_absent_extra_config_with_grace(): void
-    {
-
-        $badComposerJson = '{ }';
-
-        $tmpfname = tempnam(sys_get_temp_dir(), 'Strauss-' . __CLASS__ . '-' . __FUNCTION__);
-        file_put_contents($tmpfname, $badComposerJson);
-        chdir(dirname($tmpfname));
-
-        $inputInterfaceMock = $this->createMock(InputInterface::class);
-        $outputInterfaceMock = $this->createMock(OutputInterface::class);
-
-        $outputInterfaceMock->expects($this->exactly(1))
-                            ->method('write');
-
-        new class( $inputInterfaceMock, $outputInterfaceMock ) extends Compose {
-            public function __construct($inputInterfaceMock, $outputInterfaceMock)
-            {
-                parent::__construct();
-
-                $this->execute($inputInterfaceMock, $outputInterfaceMock);
-            }
-        };
-    }
-
-
-    /**
-     * When composer.json->extra is not an object, instead of
-     * "Trying to get property 'mozart' of non-object"
-     * a better message should be written to the OutputInterface.
-     *
-     * @test
-     */
-    public function it_handles_malformed_extra_config_with_grace(): void
-    {
-
-        $badComposerJson = '{ "name": "coenjacobs/mozart", "extra": [] }';
-
-        $tmpfname = tempnam(sys_get_temp_dir(), 'Strauss-' . __CLASS__ . '-' . __FUNCTION__);
-        file_put_contents($tmpfname, $badComposerJson);
-        chdir(dirname($tmpfname));
-
-        $inputInterfaceMock = $this->createMock(InputInterface::class);
-        $outputInterfaceMock = $this->createMock(OutputInterface::class);
-
-        $outputInterfaceMock->expects($this->exactly(1))
-                            ->method('write');
-
-        new class( $inputInterfaceMock, $outputInterfaceMock ) extends Compose {
-            public function __construct($inputInterfaceMock, $outputInterfaceMock)
-            {
-                parent::__construct();
-
-                $this->execute($inputInterfaceMock, $outputInterfaceMock);
-            }
-        };
-    }
-
-    /**
-     * When composer.json->extra->mozart is absent, instead of
-     * "Undefined property: stdClass::$mozart"
-     * a better message should be written to the OutputInterface.
-     *
-     * @test
-     */
-    public function it_handles_absent_mozart_config_with_grace(): void
-    {
-
-        $badComposerJson = '{ "name": "coenjacobs/mozart", "extra": { "moozart": {} } }';
-
-        $tmpfname = tempnam(sys_get_temp_dir(), 'Strauss-' . __CLASS__ . '-' . __FUNCTION__);
-        file_put_contents($tmpfname, $badComposerJson);
-        chdir(dirname($tmpfname));
-
-        $inputInterfaceMock = $this->createMock(InputInterface::class);
-        $outputInterfaceMock = $this->createMock(OutputInterface::class);
-
-        $outputInterfaceMock->expects($this->exactly(1))
-                            ->method('write');
-
-        new class( $inputInterfaceMock, $outputInterfaceMock ) extends Compose {
-            public function __construct($inputInterfaceMock, $outputInterfaceMock)
-            {
-                parent::__construct();
-
-                $this->execute($inputInterfaceMock, $outputInterfaceMock);
-            }
-        };
-    }
-
-    /**
-     * When composer.json->extra->mozart is malformed, instead of
-     * "Undefined property: stdClass::$mozart"
-     * a better message should be written to the OutputInterface.
-     *
-     * is_object() added.
-     *
-     * @test
-     */
-    public function it_handles_malformed_mozart_config__with_grace(): void
-    {
-
-        $badComposerJson = '{ "name": "coenjacobs/mozart", "extra": { "mozart": []  }';
-
-        $tmpfname = tempnam(sys_get_temp_dir(), 'Strauss-' . __CLASS__ . '-' . __FUNCTION__);
-        file_put_contents($tmpfname, $badComposerJson);
-        chdir(dirname($tmpfname));
-
-        $inputInterfaceMock = $this->createMock(InputInterface::class);
-        $outputInterfaceMock = $this->createMock(OutputInterface::class);
-
-        $outputInterfaceMock->expects($this->exactly(1))
-                            ->method('write');
-
-        new class( $inputInterfaceMock, $outputInterfaceMock ) extends Compose {
-            public function __construct($inputInterfaceMock, $outputInterfaceMock)
-            {
-                parent::__construct();
-
-                $this->execute($inputInterfaceMock, $outputInterfaceMock);
-            }
-        };
-    }
+	/**
+	 * When composer.json is absent, instead of failing with:
+	 * "failed to open stream: No such file or directory"
+	 * a better message should be written to the OutputInterface.
+	 *
+	 * @test
+	 */
+	public function it_fails_gracefully_when_composer_json_absent(): void
+	{
+		chdir(sys_get_temp_dir());
+
+		$inputInterfaceMock = $this->createMock(InputInterface::class);
+		$outputInterfaceMock = $this->createMock(OutputInterface::class);
+
+		$outputInterfaceMock->expects($this->exactly(1))
+			 ->method('write');
+
+		new class( $inputInterfaceMock, $outputInterfaceMock ) extends Compose {
+			public function __construct($inputInterfaceMock, $outputInterfaceMock)
+			{
+				parent::__construct();
+
+				$this->execute($inputInterfaceMock, $outputInterfaceMock);
+			}
+		};
+	}
+
+	/**
+	 * When json_decode fails, instead of
+	 * "Trying to get property 'extra' of non-object"
+	 * a better message should be written to the OutputInterface.
+	 *
+	 * @test
+	 */
+	public function it_handles_malformed_json_with_grace(): void
+	{
+
+		$badComposerJson = '{ "name": "coenjacobs/mozart", }';
+
+		$tmpfname = tempnam(sys_get_temp_dir(), 'Strauss-' . __CLASS__ . '-' . __FUNCTION__);
+		file_put_contents($tmpfname, $badComposerJson);
+		chdir(dirname($tmpfname));
+
+		$inputInterfaceMock = $this->createMock(InputInterface::class);
+		$outputInterfaceMock = $this->createMock(OutputInterface::class);
+
+		$outputInterfaceMock->expects($this->exactly(1))
+							->method('write');
+
+		new class( $inputInterfaceMock, $outputInterfaceMock ) extends Compose {
+			public function __construct($inputInterfaceMock, $outputInterfaceMock)
+			{
+				parent::__construct();
+
+				$this->execute($inputInterfaceMock, $outputInterfaceMock);
+			}
+		};
+	}
+
+	/**
+	 * When composer.json->extra is absent, instead of
+	 * "Undefined property: stdClass::$extra"
+	 * a better message should be written to the OutputInterface.
+	 *
+	 * When package name is not set, `\Composer\Composer::getPackage()->getName()` returns '__root__'.
+	 *
+	 */
+	public function test_it_handles_absent_extra_config_with_grace(): void
+	{
+
+		$badComposerJson = '{ }';
+
+		$tmpfname = tempnam(sys_get_temp_dir(), 'Strauss-' . __CLASS__ . '-' . __FUNCTION__);
+		file_put_contents($tmpfname, $badComposerJson);
+		chdir(dirname($tmpfname));
+
+		$inputInterfaceMock = $this->createMock(InputInterface::class);
+		$outputInterfaceMock = $this->createMock(OutputInterface::class);
+
+		$outputInterfaceMock->expects($this->exactly(1))
+							->method('write');
+
+		new class( $inputInterfaceMock, $outputInterfaceMock ) extends Compose {
+			public function __construct($inputInterfaceMock, $outputInterfaceMock)
+			{
+				parent::__construct();
+
+				$this->execute($inputInterfaceMock, $outputInterfaceMock);
+			}
+		};
+	}
+
+
+	/**
+	 * When composer.json->extra is not an object, instead of
+	 * "Trying to get property 'mozart' of non-object"
+	 * a better message should be written to the OutputInterface.
+	 *
+	 * @test
+	 */
+	public function it_handles_malformed_extra_config_with_grace(): void
+	{
+
+		$badComposerJson = '{ "name": "coenjacobs/mozart", "extra": [] }';
+
+		$tmpfname = tempnam(sys_get_temp_dir(), 'Strauss-' . __CLASS__ . '-' . __FUNCTION__);
+		file_put_contents($tmpfname, $badComposerJson);
+		chdir(dirname($tmpfname));
+
+		$inputInterfaceMock = $this->createMock(InputInterface::class);
+		$outputInterfaceMock = $this->createMock(OutputInterface::class);
+
+		$outputInterfaceMock->expects($this->exactly(1))
+							->method('write');
+
+		new class( $inputInterfaceMock, $outputInterfaceMock ) extends Compose {
+			public function __construct($inputInterfaceMock, $outputInterfaceMock)
+			{
+				parent::__construct();
+
+				$this->execute($inputInterfaceMock, $outputInterfaceMock);
+			}
+		};
+	}
+
+	/**
+	 * When composer.json->extra->mozart is absent, instead of
+	 * "Undefined property: stdClass::$mozart"
+	 * a better message should be written to the OutputInterface.
+	 *
+	 * @test
+	 */
+	public function it_handles_absent_mozart_config_with_grace(): void
+	{
+
+		$badComposerJson = '{ "name": "coenjacobs/mozart", "extra": { "moozart": {} } }';
+
+		$tmpfname = tempnam(sys_get_temp_dir(), 'Strauss-' . __CLASS__ . '-' . __FUNCTION__);
+		file_put_contents($tmpfname, $badComposerJson);
+		chdir(dirname($tmpfname));
+
+		$inputInterfaceMock = $this->createMock(InputInterface::class);
+		$outputInterfaceMock = $this->createMock(OutputInterface::class);
+
+		$outputInterfaceMock->expects($this->exactly(1))
+							->method('write');
+
+		new class( $inputInterfaceMock, $outputInterfaceMock ) extends Compose {
+			public function __construct($inputInterfaceMock, $outputInterfaceMock)
+			{
+				parent::__construct();
+
+				$this->execute($inputInterfaceMock, $outputInterfaceMock);
+			}
+		};
+	}
+
+	/**
+	 * When composer.json->extra->mozart is malformed, instead of
+	 * "Undefined property: stdClass::$mozart"
+	 * a better message should be written to the OutputInterface.
+	 *
+	 * is_object() added.
+	 *
+	 * @test
+	 */
+	public function it_handles_malformed_mozart_config__with_grace(): void
+	{
+
+		$badComposerJson = '{ "name": "coenjacobs/mozart", "extra": { "mozart": []  }';
+
+		$tmpfname = tempnam(sys_get_temp_dir(), 'Strauss-' . __CLASS__ . '-' . __FUNCTION__);
+		file_put_contents($tmpfname, $badComposerJson);
+		chdir(dirname($tmpfname));
+
+		$inputInterfaceMock = $this->createMock(InputInterface::class);
+		$outputInterfaceMock = $this->createMock(OutputInterface::class);
+
+		$outputInterfaceMock->expects($this->exactly(1))
+							->method('write');
+
+		new class( $inputInterfaceMock, $outputInterfaceMock ) extends Compose {
+			public function __construct($inputInterfaceMock, $outputInterfaceMock)
+			{
+				parent::__construct();
+
+				$this->execute($inputInterfaceMock, $outputInterfaceMock);
+			}
+		};
+	}
 }

Spacing +52 added lines, -52 removed lines

@@ -20,20 +20,20 @@ 
      */
     public function it_fails_gracefully_when_composer_json_absent(): void
     {
-        chdir(sys_get_temp_dir());
+        chdir( sys_get_temp_dir() );
 
-        $inputInterfaceMock = $this->createMock(InputInterface::class);
-        $outputInterfaceMock = $this->createMock(OutputInterface::class);
+        $inputInterfaceMock = $this->createMock( InputInterface::class );
+        $outputInterfaceMock = $this->createMock( OutputInterface::class );
 
-        $outputInterfaceMock->expects($this->exactly(1))
-             ->method('write');
+        $outputInterfaceMock->expects( $this->exactly( 1 ) )
+             ->method( 'write' );
 
         new class( $inputInterfaceMock, $outputInterfaceMock ) extends Compose {
-            public function __construct($inputInterfaceMock, $outputInterfaceMock)
+            public function __construct( $inputInterfaceMock, $outputInterfaceMock )
             {
                 parent::__construct();
 
-                $this->execute($inputInterfaceMock, $outputInterfaceMock);
+                $this->execute( $inputInterfaceMock, $outputInterfaceMock );
             }
         };
     }
@@ -50,22 +50,22 @@ 
 
         $badComposerJson = '{ "name": "coenjacobs/mozart", }';
 
-        $tmpfname = tempnam(sys_get_temp_dir(), 'Strauss-' . __CLASS__ . '-' . __FUNCTION__);
-        file_put_contents($tmpfname, $badComposerJson);
-        chdir(dirname($tmpfname));
+        $tmpfname = tempnam( sys_get_temp_dir(), 'Strauss-' . __CLASS__ . '-' . __FUNCTION__ );
+        file_put_contents( $tmpfname, $badComposerJson );
+        chdir( dirname( $tmpfname ) );
 
-        $inputInterfaceMock = $this->createMock(InputInterface::class);
-        $outputInterfaceMock = $this->createMock(OutputInterface::class);
+        $inputInterfaceMock = $this->createMock( InputInterface::class );
+        $outputInterfaceMock = $this->createMock( OutputInterface::class );
 
-        $outputInterfaceMock->expects($this->exactly(1))
-                            ->method('write');
+        $outputInterfaceMock->expects( $this->exactly( 1 ) )
+                            ->method( 'write' );
 
         new class( $inputInterfaceMock, $outputInterfaceMock ) extends Compose {
-            public function __construct($inputInterfaceMock, $outputInterfaceMock)
+            public function __construct( $inputInterfaceMock, $outputInterfaceMock )
             {
                 parent::__construct();
 
-                $this->execute($inputInterfaceMock, $outputInterfaceMock);
+                $this->execute( $inputInterfaceMock, $outputInterfaceMock );
             }
         };
     }
@@ -83,22 +83,22 @@ 
 
         $badComposerJson = '{ }';
 
-        $tmpfname = tempnam(sys_get_temp_dir(), 'Strauss-' . __CLASS__ . '-' . __FUNCTION__);
-        file_put_contents($tmpfname, $badComposerJson);
-        chdir(dirname($tmpfname));
+        $tmpfname = tempnam( sys_get_temp_dir(), 'Strauss-' . __CLASS__ . '-' . __FUNCTION__ );
+        file_put_contents( $tmpfname, $badComposerJson );
+        chdir( dirname( $tmpfname ) );
 
-        $inputInterfaceMock = $this->createMock(InputInterface::class);
-        $outputInterfaceMock = $this->createMock(OutputInterface::class);
+        $inputInterfaceMock = $this->createMock( InputInterface::class );
+        $outputInterfaceMock = $this->createMock( OutputInterface::class );
 
-        $outputInterfaceMock->expects($this->exactly(1))
-                            ->method('write');
+        $outputInterfaceMock->expects( $this->exactly( 1 ) )
+                            ->method( 'write' );
 
         new class( $inputInterfaceMock, $outputInterfaceMock ) extends Compose {
-            public function __construct($inputInterfaceMock, $outputInterfaceMock)
+            public function __construct( $inputInterfaceMock, $outputInterfaceMock )
             {
                 parent::__construct();
 
-                $this->execute($inputInterfaceMock, $outputInterfaceMock);
+                $this->execute( $inputInterfaceMock, $outputInterfaceMock );
             }
         };
     }
@@ -116,22 +116,22 @@ 
 
         $badComposerJson = '{ "name": "coenjacobs/mozart", "extra": [] }';
 
-        $tmpfname = tempnam(sys_get_temp_dir(), 'Strauss-' . __CLASS__ . '-' . __FUNCTION__);
-        file_put_contents($tmpfname, $badComposerJson);
-        chdir(dirname($tmpfname));
+        $tmpfname = tempnam( sys_get_temp_dir(), 'Strauss-' . __CLASS__ . '-' . __FUNCTION__ );
+        file_put_contents( $tmpfname, $badComposerJson );
+        chdir( dirname( $tmpfname ) );
 
-        $inputInterfaceMock = $this->createMock(InputInterface::class);
-        $outputInterfaceMock = $this->createMock(OutputInterface::class);
+        $inputInterfaceMock = $this->createMock( InputInterface::class );
+        $outputInterfaceMock = $this->createMock( OutputInterface::class );
 
-        $outputInterfaceMock->expects($this->exactly(1))
-                            ->method('write');
+        $outputInterfaceMock->expects( $this->exactly( 1 ) )
+                            ->method( 'write' );
 
         new class( $inputInterfaceMock, $outputInterfaceMock ) extends Compose {
-            public function __construct($inputInterfaceMock, $outputInterfaceMock)
+            public function __construct( $inputInterfaceMock, $outputInterfaceMock )
             {
                 parent::__construct();
 
-                $this->execute($inputInterfaceMock, $outputInterfaceMock);
+                $this->execute( $inputInterfaceMock, $outputInterfaceMock );
             }
         };
     }
@@ -148,22 +148,22 @@ 
 
         $badComposerJson = '{ "name": "coenjacobs/mozart", "extra": { "moozart": {} } }';
 
-        $tmpfname = tempnam(sys_get_temp_dir(), 'Strauss-' . __CLASS__ . '-' . __FUNCTION__);
-        file_put_contents($tmpfname, $badComposerJson);
-        chdir(dirname($tmpfname));
+        $tmpfname = tempnam( sys_get_temp_dir(), 'Strauss-' . __CLASS__ . '-' . __FUNCTION__ );
+        file_put_contents( $tmpfname, $badComposerJson );
+        chdir( dirname( $tmpfname ) );
 
-        $inputInterfaceMock = $this->createMock(InputInterface::class);
-        $outputInterfaceMock = $this->createMock(OutputInterface::class);
+        $inputInterfaceMock = $this->createMock( InputInterface::class );
+        $outputInterfaceMock = $this->createMock( OutputInterface::class );
 
-        $outputInterfaceMock->expects($this->exactly(1))
-                            ->method('write');
+        $outputInterfaceMock->expects( $this->exactly( 1 ) )
+                            ->method( 'write' );
 
         new class( $inputInterfaceMock, $outputInterfaceMock ) extends Compose {
-            public function __construct($inputInterfaceMock, $outputInterfaceMock)
+            public function __construct( $inputInterfaceMock, $outputInterfaceMock )
             {
                 parent::__construct();
 
-                $this->execute($inputInterfaceMock, $outputInterfaceMock);
+                $this->execute( $inputInterfaceMock, $outputInterfaceMock );
             }
         };
     }
@@ -182,22 +182,22 @@ 
 
         $badComposerJson = '{ "name": "coenjacobs/mozart", "extra": { "mozart": []  }';
 
-        $tmpfname = tempnam(sys_get_temp_dir(), 'Strauss-' . __CLASS__ . '-' . __FUNCTION__);
-        file_put_contents($tmpfname, $badComposerJson);
-        chdir(dirname($tmpfname));
+        $tmpfname = tempnam( sys_get_temp_dir(), 'Strauss-' . __CLASS__ . '-' . __FUNCTION__ );
+        file_put_contents( $tmpfname, $badComposerJson );
+        chdir( dirname( $tmpfname ) );
 
-        $inputInterfaceMock = $this->createMock(InputInterface::class);
-        $outputInterfaceMock = $this->createMock(OutputInterface::class);
+        $inputInterfaceMock = $this->createMock( InputInterface::class );
+        $outputInterfaceMock = $this->createMock( OutputInterface::class );
 
-        $outputInterfaceMock->expects($this->exactly(1))
-                            ->method('write');
+        $outputInterfaceMock->expects( $this->exactly( 1 ) )
+                            ->method( 'write' );
 
         new class( $inputInterfaceMock, $outputInterfaceMock ) extends Compose {
-            public function __construct($inputInterfaceMock, $outputInterfaceMock)
+            public function __construct( $inputInterfaceMock, $outputInterfaceMock )
             {
                 parent::__construct();
 
-                $this->execute($inputInterfaceMock, $outputInterfaceMock);
+                $this->execute( $inputInterfaceMock, $outputInterfaceMock );
             }
         };
     }

Braces +7 added lines, -14 removed lines

@@ -8,8 +8,7 @@ 
 use Symfony\Component\Console\Input\InputInterface;
 use Symfony\Component\Console\Output\OutputInterface;
 
-class ComposeTest extends TestCase
-{
+class ComposeTest extends TestCase {
 
     /**
      * When composer.json is absent, instead of failing with:
@@ -29,8 +28,7 @@ 
              ->method('write');
 
         new class( $inputInterfaceMock, $outputInterfaceMock ) extends Compose {
-            public function __construct($inputInterfaceMock, $outputInterfaceMock)
-            {
+            public function __construct($inputInterfaceMock, $outputInterfaceMock) {
                 parent::__construct();
 
                 $this->execute($inputInterfaceMock, $outputInterfaceMock);
@@ -61,8 +59,7 @@ 
                             ->method('write');
 
         new class( $inputInterfaceMock, $outputInterfaceMock ) extends Compose {
-            public function __construct($inputInterfaceMock, $outputInterfaceMock)
-            {
+            public function __construct($inputInterfaceMock, $outputInterfaceMock) {
                 parent::__construct();
 
                 $this->execute($inputInterfaceMock, $outputInterfaceMock);
@@ -94,8 +91,7 @@ 
                             ->method('write');
 
         new class( $inputInterfaceMock, $outputInterfaceMock ) extends Compose {
-            public function __construct($inputInterfaceMock, $outputInterfaceMock)
-            {
+            public function __construct($inputInterfaceMock, $outputInterfaceMock) {
                 parent::__construct();
 
                 $this->execute($inputInterfaceMock, $outputInterfaceMock);
@@ -127,8 +123,7 @@ 
                             ->method('write');
 
         new class( $inputInterfaceMock, $outputInterfaceMock ) extends Compose {
-            public function __construct($inputInterfaceMock, $outputInterfaceMock)
-            {
+            public function __construct($inputInterfaceMock, $outputInterfaceMock) {
                 parent::__construct();
 
                 $this->execute($inputInterfaceMock, $outputInterfaceMock);
@@ -159,8 +154,7 @@ 
                             ->method('write');
 
         new class( $inputInterfaceMock, $outputInterfaceMock ) extends Compose {
-            public function __construct($inputInterfaceMock, $outputInterfaceMock)
-            {
+            public function __construct($inputInterfaceMock, $outputInterfaceMock) {
                 parent::__construct();
 
                 $this->execute($inputInterfaceMock, $outputInterfaceMock);
@@ -193,8 +187,7 @@ 
                             ->method('write');
 
         new class( $inputInterfaceMock, $outputInterfaceMock ) extends Compose {
-            public function __construct($inputInterfaceMock, $outputInterfaceMock)
-            {
+            public function __construct($inputInterfaceMock, $outputInterfaceMock) {
                 parent::__construct();
 
                 $this->execute($inputInterfaceMock, $outputInterfaceMock);

vendor/brianhenryie/strauss/tests/Unit/Console/ApplicationTest.php

Indentation +13 added lines, -13 removed lines

@@ -9,22 +9,22 @@
 class ApplicationTest extends TestCase
 {
 
-    /**
-     * Test the Symfony\Component\Console\Application instance contains the Compose command.
-     */
-    public function testInstantiation()
-    {
+	/**
+	 * Test the Symfony\Component\Console\Application instance contains the Compose command.
+	 */
+	public function testInstantiation()
+	{
 
-        $version = '1.0.0';
+		$version = '1.0.0';
 
-        $sut = new Application($version);
+		$sut = new Application($version);
 
-        $commands = $sut->all();
+		$commands = $sut->all();
 
-        $containsComposeCommand = array_reduce($commands, function ($carry, $item) {
-            return $carry || $item instanceof Compose;
-        }, false);
+		$containsComposeCommand = array_reduce($commands, function ($carry, $item) {
+			return $carry || $item instanceof Compose;
+		}, false);
 
-        $this->assertTrue($containsComposeCommand);
-    }
+		$this->assertTrue($containsComposeCommand);
+	}
 }

Spacing +4 added lines, -4 removed lines

@@ -17,14 +17,14 @@
 
         $version = '1.0.0';
 
-        $sut = new Application($version);
+        $sut = new Application( $version );
 
         $commands = $sut->all();
 
-        $containsComposeCommand = array_reduce($commands, function ($carry, $item) {
+        $containsComposeCommand = array_reduce( $commands, function( $carry, $item ) {
             return $carry || $item instanceof Compose;
-        }, false);
+        }, false );
 
-        $this->assertTrue($containsComposeCommand);
+        $this->assertTrue( $containsComposeCommand );
     }
 }

Braces +2 added lines, -4 removed lines

@@ -6,14 +6,12 @@
 use BrianHenryIE\Strauss\Console\Commands\Compose;
 use PHPUnit\Framework\TestCase;
 
-class ApplicationTest extends TestCase
-{
+class ApplicationTest extends TestCase {
 
     /**
      * Test the Symfony\Component\Console\Application instance contains the Compose command.
      */
-    public function testInstantiation()
-    {
+    public function testInstantiation() {
 
         $version = '1.0.0';