Additional cases for JSON formater tests. (empty string/false/true/incorrect json)

But I am not sure if in case of failure we should return false or null like is everywhere i.e. cache, dbresult, etc.

Author: nowackipawel

tests/system/Test/FeatureResponseTest.php

@@ -211,14 +211,47 @@ public function testGetJSON()
 		$this->assertEquals($formatter->format(['foo' => 'bar']), $this->feature->getJSON());
 	}
 
-	public function testInvalidJSON()
+	public function testEmptyJSON()
+	{
+		$this->getFeatureResponse('<h1>Hello World</h1>');
+		$this->response->setJSON('');
+		$config    = new \Config\Format();
+		$formatter = $config->getFormatter('application/json');
+
+		// this should be "" - json_encode('');
+		$this->assertEquals("", $this->feature->getJSON());
+	}
+	
+	public function testFalseJSON()
 	{
 		$this->getFeatureResponse('<h1>Hello World</h1>');
 		$this->response->setJSON('');
 		$config    = new \Config\Format();
 		$formatter = $config->getFormatter('application/json');
 
-		// this should fail because of empty JSON
+		// this should be FALSE - json_encode(false)
+		$this->assertFalse($this->feature->getJSON());
+	}
+	
+	public function testTrueJSON()
+	{
+		$this->getFeatureResponse('<h1>Hello World</h1>');
+		$this->response->setJSON('');
+		$config    = new \Config\Format();
+		$formatter = $config->getFormatter('application/json');
+
+		// this should be TRUE - json_encode(true)
+		$this->assertTrue($this->feature->getJSON());
+	}
+	
+	public function testInvalidJSON()
+	{
+		$this->getFeatureResponse('<h1>Hello World</h1>');
+		$this->response->setBody(' test " case ');
+		$config    = new \Config\Format();
+		$formatter = $config->getFormatter('application/json');
+
+		// this should be FALSE - invalid JSON - will see if this is working that way ;-)
 		$this->assertFalse($this->feature->getJSON());
 	}