Update User#identifier to conform to Gitolite 2.x's user pattern

Also modifies the specs a bit because I can't help myself. Closes #480

Update User#identifier to conform to Gitolite 2.x's user pattern
Also modifies the specs a bit because I can't help myself. Closes #480
Robert Speicher
1 parent b44e9a08
Showing 3 changed files with 44 additions and 27 deletions Show diff stats
app/roles/account.rb
spec/factories.rb
spec/models/user_spec.rb
-module Account 
+module Account
+  # Returns a string for use as a Gitolite user identifier
+  #
+  # Note that Gitolite 2.x requires the following pattern for users:
+  #
+  #   ^@?[0-9a-zA-Z][0-9a-zA-Z._\@+-]*$
   def identifier
-    email.gsub /[^[:alnum:]]/, "_"
+    # Replace non-word chars with underscores, then make sure it starts with
+    # valid chars
+    email.gsub(/\W/, '_').gsub(/\A([\W\_])+/, '')
   end
   def is_admin?
@@ -28,7 +28,7 @@ FactoryGirl.define do
     email { Faker::Internet.email }
     name
     password "123456"
-    password_confirmation "123456"
+    password_confirmation { password }
     trait :admin do
       admin true
@@ -31,36 +31,46 @@ describe User do
     it { should respond_to(:private_token) }
   end
-  it "should return valid identifier" do
-    user = User.new(email: "test@mail.com")
-    user.identifier.should == "test_mail_com"
-  end
+  describe '#identifier' do
+    it "should return valid identifier" do
+      user = build(:user, email: "test@mail.com")
+      user.identifier.should == "test_mail_com"
+    end
-  it "should return identifier without + sign" do
-    user = User.new(email: "test+foo@mail.com")
-    user.identifier.should == "test_foo_mail_com"
-  end
+    it "should return identifier without + sign" do
+      user = build(:user, email: "test+foo@mail.com")
+      user.identifier.should == "test_foo_mail_com"
+    end
-  it "should execute callback when force_random_password specified" do
-    user = User.new(email: "test@mail.com", force_random_password: true)
-    user.should_receive(:generate_password)
-    user.save
+    it "should conform to Gitolite's required identifier pattern" do
+      user = build(:user, email: "_test@example.com")
+      user.identifier.should == 'test_example_com'
+    end
   end
-  it "should not generate password by default" do
-    user = Factory(:user, password: 'abcdefg', password_confirmation: 'abcdefg')
-    user.password.should == 'abcdefg'
-  end
+  describe '#generate_password' do
+    it "should execute callback when force_random_password specified" do
+      user = build(:user, force_random_password: true)
+      user.should_receive(:generate_password)
+      user.save
+    end
+
+    it "should not generate password by default" do
+      user = create(:user, password: 'abcdefg')
+      user.password.should == 'abcdefg'
+    end
-  it "should generate password when forcing random password" do
-    Devise.stub(:friendly_token).and_return('123456789')
-    user = User.create(email: "test1@mail.com", force_random_password: true)
-    user.password.should == user.password_confirmation
-    user.password.should == '12345678'
+    it "should generate password when forcing random password" do
+      Devise.stub(:friendly_token).and_return('123456789')
+      user = create(:user, password: 'abcdefg', force_random_password: true)
+      user.password.should == '12345678'
+    end
   end
-  it "should have authentication token" do
-    user = Factory(:user)
-    user.authentication_token.should_not == ""
+  describe 'authentication token' do
+    it "should have authentication token" do
+      user = Factory(:user)
+      user.authentication_token.should_not be_blank
+    end
   end
 end
1	-module Account	1	+module Account
		2	+ # Returns a string for use as a Gitolite user identifier
		3	+ #
		4	+ # Note that Gitolite 2.x requires the following pattern for users:
		5	+ #
		6	+ # ^@?[0-9a-zA-Z][0-9a-zA-Z._\@+-]*$
2	def identifier	7	def identifier
3	- email.gsub /[^[:alnum:]]/, "_"	8	+ # Replace non-word chars with underscores, then make sure it starts with
		9	+ # valid chars
		10	+ email.gsub(/\W/, '_').gsub(/\A([\W\_])+/, '')
4	end	11	end
5		12
6	def is_admin?	13	def is_admin?
	@@ -28,7 +28,7 @@ FactoryGirl.define do		@@ -28,7 +28,7 @@ FactoryGirl.define do
28	email { Faker::Internet.email }	28	email { Faker::Internet.email }
29	name	29	name
30	password "123456"	30	password "123456"
31	- password_confirmation "123456"	31	+ password_confirmation { password }
32		32
33	trait :admin do	33	trait :admin do
34	admin true	34	admin true