From ddb60915770de1ed28609918c925ec0222ea35ce Mon Sep 17 00:00:00 2001
From: Jonathan Leibiusky <ionathan@gmail.com>
Date: Thu, 24 Jun 2010 21:27:54 -0400
Subject: [PATCH] Added smembers

---
 src/main/java/redis/clients/jedis/Jedis.java     |  7 +++++++
 .../jedis/tests/commands/SetCommandsTest.java    | 16 +++++++++-------
 2 files changed, 16 insertions(+), 7 deletions(-)

diff --git a/src/main/java/redis/clients/jedis/Jedis.java b/src/main/java/redis/clients/jedis/Jedis.java
index ec39fa4..c53f97f 100644
--- a/src/main/java/redis/clients/jedis/Jedis.java
+++ b/src/main/java/redis/clients/jedis/Jedis.java
@@ -3,8 +3,10 @@ package redis.clients.jedis;
 import java.util.ArrayList;
 import java.util.HashMap;
 import java.util.Iterator;
+import java.util.LinkedHashSet;
 import java.util.List;
 import java.util.Map;
+import java.util.Set;
 
 public class Jedis extends Client {
     public Jedis(String host) {
@@ -269,4 +271,9 @@ public class Jedis extends Client {
     public int sadd(String key, String member) throws JedisException {
 	return sendCommand("SADD", key, member).getIntegerReply();
     }
+
+    public Set<String> smembers(String key) throws JedisException {
+	List<String> members = sendCommand("SMEMBERS", key).getMultiBulkReply();
+	return new LinkedHashSet<String>(members);
+    }
 }
diff --git a/src/test/java/redis/clients/jedis/tests/commands/SetCommandsTest.java b/src/test/java/redis/clients/jedis/tests/commands/SetCommandsTest.java
index 075d3f6..db33bdc 100644
--- a/src/test/java/redis/clients/jedis/tests/commands/SetCommandsTest.java
+++ b/src/test/java/redis/clients/jedis/tests/commands/SetCommandsTest.java
@@ -1,6 +1,5 @@
 package redis.clients.jedis.tests.commands;
 
-import java.util.HashSet;
 import java.util.LinkedHashSet;
 import java.util.Set;
 
@@ -36,16 +35,19 @@ public class SetCommandsTest extends Assert {
 	status = jedis.sadd("foo", "a");
 	assertEquals(0, status);
     }
-    
+
     @Test
     public void smembers() throws JedisException {
 	jedis.sadd("foo", "a");
 	jedis.sadd("foo", "b");
-	
+
 	Set<String> expected = new LinkedHashSet<String>();
-	expected
-	
-	assertEquals(0, status);
+	expected.add("a");
+	expected.add("b");
+
+	Set<String> members = jedis.smembers("foo");
+
+	assertEquals(expected, members);
     }
-    
+
 }