From 5aed917f72aab5ccb9f948ed40f99c9b8e2a02d6 Mon Sep 17 00:00:00 2001
From: Magnus Persson <magnus.e.persson@gmail.com>
Date: Wed, 11 Mar 2015 23:41:12 +0100
Subject: [PATCH] Resolves chrisboulton/php-resque#145

---
 lib/Resque/Redis.php          |  2 +-
 test/Resque/Tests/JobTest.php | 18 ++++++++++++++++--
 2 files changed, 17 insertions(+), 3 deletions(-)

diff --git a/lib/Resque/Redis.php b/lib/Resque/Redis.php
index 59ca215e..5410499d 100644
--- a/lib/Resque/Redis.php
+++ b/lib/Resque/Redis.php
@@ -98,7 +98,7 @@ class Resque_Redis
 	 */
 	public static function prefix($namespace)
 	{
-	    if (strpos($namespace, ':') === false) {
+	    if (substr($namespace, -1) !== ':') {
 	        $namespace .= ':';
 	    }
 	    self::$defaultNamespace = $namespace;
diff --git a/test/Resque/Tests/JobTest.php b/test/Resque/Tests/JobTest.php
index 0636d8bf..156ec04f 100644
--- a/test/Resque/Tests/JobTest.php
+++ b/test/Resque/Tests/JobTest.php
@@ -166,7 +166,21 @@ public function testJobWithTearDownCallbackFiresTearDown()
 		
 		$this->assertTrue(Test_Job_With_TearDown::$called);
 	}
-	
+
+	public function testNamespaceNaming() {
+		$fixture = array(
+			array('test' => 'more:than:one:with:', 'assertValue' => 'more:than:one:with:'),
+			array('test' => 'more:than:one:without', 'assertValue' => 'more:than:one:without:'),
+			array('test' => 'resque', 'assertValue' => 'resque:'),
+			array('test' => 'resque:', 'assertValue' => 'resque:'),
+		);
+
+		foreach($fixture as $item) {
+			Resque_Redis::prefix($item['test']);
+			$this->assertEquals(Resque_Redis::getPrefix(), $item['assertValue']);
+		}
+	}
+
 	public function testJobWithNamespace()
 	{
 	    Resque_Redis::prefix('php');
@@ -176,7 +190,7 @@ public function testJobWithNamespace()
         
         $this->assertEquals(Resque::queues(), array('jobs'));
         $this->assertEquals(Resque::size($queue), 1);
-        
+
         Resque_Redis::prefix('resque');
         $this->assertEquals(Resque::size($queue), 0);        
 	}