From 1b1445d782e64e5253e0175e601d5ca73af7543a Mon Sep 17 00:00:00 2001
From: AntonioTerceiro <AntonioTerceiro@3f533792-8f58-4932-b0fe-aaf55b0a4547>
Date: Wed, 27 Feb 2008 15:17:16 +0000
Subject: [PATCH] ActionItem36: don't suggest duplicated friend groups

---
 app/models/person.rb     |  2 +-
 test/unit/person_test.rb |  9 +++++++++
 2 files changed, 10 insertions(+), 1 deletion(-)

diff --git a/app/models/person.rb b/app/models/person.rb
index f1a888d..c5f73c6 100644
--- a/app/models/person.rb
+++ b/app/models/person.rb
@@ -6,7 +6,7 @@ class Person < Profile
   has_many :friends, :class_name => 'Person', :through => :friendships
 
   def suggested_friend_groups
-    friend_groups + [ _('friends'), _('work'), _('school'), _('family') ]
+    (friend_groups + [ _('friends'), _('work'), _('school'), _('family') ]).uniq
   end
 
   def friend_groups
diff --git a/test/unit/person_test.rb b/test/unit/person_test.rb
index 61adf28..051d18c 100644
--- a/test/unit/person_test.rb
+++ b/test/unit/person_test.rb
@@ -182,4 +182,13 @@ class PersonTest < Test::Unit::TestCase
     assert_equivalent ['group1', 'group2'], p1.friend_groups
   end
 
+  should 'not suggest duplicated friend groups' do
+    p1 = create_user('testuser1').person
+    p2 = create_user('testuser2').person
+   
+    p1.add_friend(p2, 'friends')
+
+    assert_equal p1.suggested_friend_groups, p1.suggested_friend_groups.uniq
+  end
+
 end
--
libgit2 0.21.2