Drop support for pass-by-reference

Supporting pass-by-reference was such a hack and now side effects of
that hack were reported. Upstream won't support pass-by-reference
anyways, so I'm going to drop that here as well. If you need to mock a
function which uses pass-by-reference (e.g. exec()) feel free to use
another testing framework (e.g. phpunit with php-mock-phpunit).

See also: #1, phpspec/prophecy#225

Author: malkusch

README.md

@@ -51,9 +51,6 @@ This library comes with the same restrictions as the underlying
 
 * Additionally it shares restrictions from Prophecy as well:
   Prophecy [doesn't support pass-by-reference](https://github.com/phpspec/prophecy/issues/225).
-  To support pass-by-reference here, an
-  [implicit feature of composer](https://github.com/composer/composer/issues/4732)
-  is used. It may work accidentially but can change at any moment.
   If you need pass-by-reference in prophecies, consider using another framework
   (e.g. [php-mock-phpunit](https://github.com/php-mock/php-mock-phpunit)).
 

composer.json

@@ -15,8 +15,7 @@
     ],
     "autoload": {
         "psr-4": {
-            "phpmock\\prophecy\\": "classes/",
-            "Prophecy\\": "overwrites/Prophecy/"
+            "phpmock\\prophecy\\": "classes/"
         }
     },
     "require": {

overwrites/Prophecy/Doubler/ClassPatch/ProphecySubjectPatch.php

@@ -56,4 +56,14 @@ public function testBackupStaticAttributes()
     {
         $this->markTestSkipped("Skip until PHPUnit#2016 is resolved");
     }
+
+    /**
+     * Pass-By-Reference is not supported in Prophecy.
+     *
+     * @see https://github.com/phpspec/prophecy/issues/225
+     */
+    public function testPassingByReference()
+    {
+        $this->markTestSkipped("Pass-By-Reference is not supported in Prophecy");
+    }
 }