Merge pull request #5 from sirbrillig/add/invalidexpectationexception

Throw exception when accessing invalid property on an Expectation

src/Spies/Expectation.php

@@ -49,6 +49,7 @@ public function __get( $key ) {
 			$this->negation = true;
 			return $this;
 		}
+		throw new InvalidExpectationException( 'Invalid property: "' . $key . '" does not exist on this Expectation' );
 	}
 
 	/**

src/Spies/InvalidExpectationException.php

@@ -0,0 +1,5 @@
+<?php
+namespace Spies;
+
+class InvalidExpectationException extends \Exception {
+}

tests/ExpectationTest.php

@@ -86,6 +86,12 @@ public function test_times_is_not_met_if_spy_is_called_more_than_that_many_times
 		$this->assertInternalType( 'string', $expectation->verify() );
 	}
 
+	public function test_once_as_property_throws_an_error() {
+		$this->expectException( \Spies\InvalidExpectationException::class );
+		$spy = \Spies\make_spy();
+		\Spies\expect_spy( $spy )->to_have_been_called->once;
+	}
+
 	public function test_once_is_met_if_spy_is_called_once() {
 		$spy = \Spies\make_spy();
 		$expectation = \Spies\expect_spy( $spy )->to_have_been_called->once();
@@ -100,6 +106,12 @@ public function test_once_is_not_met_if_spy_is_not_called() {
 		$this->assertInternalType( 'string', $expectation->verify() );
 	}
 
+	public function test_twice_as_property_throws_an_error() {
+		$this->expectException( \Spies\InvalidExpectationException::class );
+		$spy = \Spies\make_spy();
+		\Spies\expect_spy( $spy )->to_have_been_called->twice;
+	}
+
 	public function test_once_is_not_met_if_spy_is_called_twice() {
 		$spy = \Spies\make_spy();
 		$expectation = \Spies\expect_spy( $spy )->to_have_been_called->once();