Merge remote-tracking branch 'upstream/develop' into develop

8 files changed, 27 insertions, 20 deletions

diff --git a/tests/codeigniter/core/Loader_test.php b/tests/codeigniter/core/Loader_test.php
index 93ca5b223..9e2092e05 100644
--- a/tests/codeigniter/core/Loader_test.php
+++ b/tests/codeigniter/core/Loader_test.php
@@ -405,12 +405,19 @@ class Loader_test extends CI_TestCase {
 		// Create model in VFS package path
 		$dir = 'third-party';
 		$lib = 'unit_test_package';
-		$class = 'CI_'.ucfirst($lib);
+		$class = ucfirst($lib);
 		$this->ci_vfs_create(ucfirst($lib), '<?php class '.$class.' { }', $this->ci_app_root, array($dir, 'libraries'));
 
 		// Get paths
 		$paths = $this->load->get_package_paths(TRUE);
 
+		// Test failed load without path
+		$this->setExpectedException(
+			'RuntimeException',
+			'CI Error: Unable to load the requested class: '.ucfirst($lib)
+		);
+		$this->load->library($lib);
+
 		// Add path and verify
 		$path = APPPATH.$dir.'/';
 		$this->assertInstanceOf('CI_Loader', $this->load->add_package_path($path));
@@ -432,13 +439,6 @@ class Loader_test extends CI_TestCase {
 		// Remove path and verify restored paths
 		$this->assertInstanceOf('CI_Loader', $this->load->remove_package_path($path));
 		$this->assertEquals($paths, $this->load->get_package_paths(TRUE));
-
-		// Test failed load without path
-		$this->setExpectedException(
-			'RuntimeException',
-			'CI Error: Unable to load the requested class: '.ucfirst($lib)
-		);
-		$this->load->library($lib);
 	}
 
 	// --------------------------------------------------------------------
diff --git a/tests/codeigniter/core/URI_test.php b/tests/codeigniter/core/URI_test.php
index 50663d54a..42dff3639 100644
--- a/tests/codeigniter/core/URI_test.php
+++ b/tests/codeigniter/core/URI_test.php
@@ -252,6 +252,3 @@ class URI_test extends CI_TestCase {
 	}
 
 }
-
-/* End of file URI_test.php */
-/* Location: ./tests/core/URI_test.php */
\ No newline at end of file
diff --git a/tests/codeigniter/core/compat/standard_test.php b/tests/codeigniter/core/compat/standard_test.php
index a3a6d9552..4077a5c7c 100644
--- a/tests/codeigniter/core/compat/standard_test.php
+++ b/tests/codeigniter/core/compat/standard_test.php
@@ -449,7 +449,6 @@ class standard_test extends CI_TestCase {
 			return $this->markTestSkipped('quoted_printable_encode() is already available on PHP 5.3');
 		}
 
-
 		// These are actually imap_8bit() tests:
 		$this->assertEquals("String with CRLF at end=20\r\n", quoted_printable_encode("String with CRLF at end \r\n"));
 		// ext/imap/tests/imap_8bit_basic.phpt says for this line:
@@ -459,6 +458,11 @@ class standard_test extends CI_TestCase {
 		$this->assertEquals("String with tab at end =09", quoted_printable_encode("String with tab at end \t"));
 		$this->assertEquals("=00=01=02=03=04=FE=FF=0A=0D", quoted_printable_encode("\x00\x01\x02\x03\x04\xfe\xff\x0a\x0d"));
 
+		if (function_exists('imap_8bit'))
+		{
+			return $this->markTestIncomplete('imap_8bit() exists and is called as an alias for quoted_printable_encode()');
+		}
+
 		// And these are from ext/standard/tests/strings/quoted_printable_encode_002.phpt:
 		$this->assertEquals(
 			"=00=00=00=00=00=00=00=00=00=00=00=00=00=00=00=00=00=00=00=00=00=00=00=00=00=\r\n"
diff --git a/tests/codeigniter/helpers/date_helper_test.php b/tests/codeigniter/helpers/date_helper_test.php
index 0f16e6c49..c80e92cd0 100644
--- a/tests/codeigniter/helpers/date_helper_test.php
+++ b/tests/codeigniter/helpers/date_helper_test.php
@@ -315,5 +315,3 @@ class Date_helper_test extends CI_TestCase {
 	}
 
 }
-
-/* End of file date_helper_test.php */
\ No newline at end of file
diff --git a/tests/codeigniter/helpers/directory_helper_test.php b/tests/codeigniter/helpers/directory_helper_test.php
index ac71dfaf8..253c1e368 100644
--- a/tests/codeigniter/helpers/directory_helper_test.php
+++ b/tests/codeigniter/helpers/directory_helper_test.php
@@ -57,5 +57,3 @@ class Directory_helper_test extends CI_TestCase {
 	}
 
 }
-
-/* End of file directory_helper_test.php */
\ No newline at end of file
diff --git a/tests/codeigniter/helpers/form_helper_test.php b/tests/codeigniter/helpers/form_helper_test.php
index e234f9c83..b5fe99b96 100644
--- a/tests/codeigniter/helpers/form_helper_test.php
+++ b/tests/codeigniter/helpers/form_helper_test.php
@@ -288,5 +288,3 @@ EOH;
 	}
 
 }
-
-/* End of file form_helper_test.php */
\ No newline at end of file
diff --git a/tests/codeigniter/helpers/path_helper_test.php b/tests/codeigniter/helpers/path_helper_test.php
index d25c3ed9b..9d325e70d 100644
--- a/tests/codeigniter/helpers/path_helper_test.php
+++ b/tests/codeigniter/helpers/path_helper_test.php
@@ -28,5 +28,3 @@ class Path_helper_test extends CI_TestCase {
 	}
 
 }
-
-/* End of file path_helper_test.php */
\ No newline at end of file
diff --git a/tests/codeigniter/libraries/Session_test.php b/tests/codeigniter/libraries/Session_test.php
index 6f1332384..76a4fcc98 100644
--- a/tests/codeigniter/libraries/Session_test.php
+++ b/tests/codeigniter/libraries/Session_test.php
@@ -19,6 +19,7 @@ class Session_test extends CI_TestCase {
 	 */
 	public function set_up()
 	{
+return;
 		// Override settings
 		foreach ($this->settings as $name => $value) {
 			$this->setting_vals[$name] = ini_get('session.'.$name);
@@ -68,6 +69,7 @@ class Session_test extends CI_TestCase {
 	 */
 	public function tear_down()
 	{
+return;
 		// Restore environment
 		if (session_id()) session_destroy();
 		$_SESSION = array();
@@ -84,6 +86,7 @@ class Session_test extends CI_TestCase {
 	 */
 	public function test_set_userdata()
 	{
+return;
 		// Set userdata values for each driver
 		$key1 = 'test1';
 		$ckey2 = 'test2';
@@ -115,6 +118,7 @@ class Session_test extends CI_TestCase {
 	 */
 	public function test_has_userdata()
 	{
+return;
 		// Set a userdata value for each driver
 		$key = 'hastest';
 		$cmsg = 'My test data';
@@ -137,6 +141,7 @@ class Session_test extends CI_TestCase {
 	 */
 	public function test_all_userdata()
 	{
+return;
 		// Set a specific series of data for each driver
 		$cdata = array(
 			'one' => 'first',
@@ -171,6 +176,7 @@ class Session_test extends CI_TestCase {
 	 */
 	public function test_unset_userdata()
 	{
+return;
 		// Set a userdata message for each driver
 		$key = 'untest';
 		$cmsg = 'Other test data';
@@ -194,6 +200,7 @@ class Session_test extends CI_TestCase {
 	 */
 	public function test_flashdata()
 	{
+return;
 		// Set flashdata message for each driver
 		$key = 'fltest';
 		$cmsg = 'Some flash data';
@@ -223,6 +230,7 @@ class Session_test extends CI_TestCase {
 	 */
 	public function test_keep_flashdata()
 	{
+return;
 		// Set flashdata message for each driver
 		$key = 'kfltest';
 		$cmsg = 'My flash data';
@@ -255,6 +263,7 @@ class Session_test extends CI_TestCase {
 
 	public function test_keep_flashdata_with_array()
 	{
+return;
 		// Set flashdata array for each driver
 		$cdata = array(
 			'one' => 'first',
@@ -308,6 +317,7 @@ class Session_test extends CI_TestCase {
 	 */
 	public function test_all_flashdata()
 	{
+return;
 		// Set a specific series of data for each driver
 		$cdata = array(
 			'one' => 'first',
@@ -338,6 +348,7 @@ class Session_test extends CI_TestCase {
 	 */
 	public function test_set_tempdata()
 	{
+return;
 		// Set tempdata message for each driver - 1 second timeout
 		$key = 'tmptest';
 		$cmsg = 'Some temp data';
@@ -364,6 +375,7 @@ class Session_test extends CI_TestCase {
 	 */
 	public function test_unset_tempdata()
 	{
+return;
 		// Set tempdata message for each driver - 1 second timeout
 		$key = 'utmptest';
 		$cmsg = 'My temp data';
@@ -387,6 +399,7 @@ class Session_test extends CI_TestCase {
 	 */
 	public function test_sess_regenerate()
 	{
+return;
 		// Get current session id, regenerate, and compare
 		// Cookie driver
 		$oldid = $this->session->cookie->userdata('session_id');
@@ -406,6 +419,7 @@ class Session_test extends CI_TestCase {
 	 */
 	public function test_sess_destroy()
 	{
+return;
 		// Set a userdata message, destroy session, and verify absence
 		$key = 'dsttest';
 		$msg = 'More test data';