Raise an exception on undefined clauses.

When the named clause doesn't exist throw an exception. This helps
prevent null checks in userland code and more clearly signals an error.

Refs #10734

src/Database/Query.php

@@ -1695,10 +1695,15 @@ public function getIterator()
      *
      * @param string $name name of the clause to be returned
      * @return mixed
+     * @throws InvalidArgumentException When the named clause does not exist.
      */
     public function clause($name)
     {
-        return isset($this->_parts[$name]) ? $this->_parts[$name] : null;
+        if (!array_key_exists($name, $this->_parts)) {
+            $clauses = implode(', ', array_keys($this->_parts));
+            throw new InvalidArgumentException("The '$name' clause is not defined. Valid clauses are: $clauses");
+        }
+        return $this->_parts[$name];
     }
 
     /**

tests/TestCase/Database/QueryTest.php

@@ -4332,13 +4332,15 @@ public function testSelectWithObjFetchType()
     /**
      * Test that reading an undefined clause does not emit an error.
      *
+     * @expectedException \InvalidArgumentException
+     * @expectedExceptionMessage The 'nope' clause is not defined. Valid clauses are: delete, update
      * @return void
      */
     public function testClauseUndefined()
     {
         $query = new Query($this->connection);
         $this->assertEmpty($query->clause('where'));
-        $this->assertNull($query->clause('nope'));
+        $query->clause('nope');
     }
 
     /**