discourse/spec/services/username_checker_service_spec.rb
Sam 262f561a87 FEATURE: relax username rules to allow - and . and leading _
This relaxes our very strict username rules to allow for some long asked for requests

- leading _ is now allowed
- . is allowed except for trailing char and confusing extensions like .gif .json
- dash (-) is now permitted
2015-09-02 12:13:44 +10:00

50 lines
1.5 KiB
Ruby

require 'spec_helper'
describe UsernameCheckerService do
describe 'check_username' do
before do
@service = UsernameCheckerService.new
@nil_email = nil
@email = 'vincentvega@example.com'
end
context 'Username invalid' do
it 'rejects too short usernames' do
result = @service.check_username('a', @nil_email)
expect(result).to have_key(:errors)
end
it 'rejects too long usernames' do
result = @service.check_username('a123456789b123456789c123456789', @nil_email)
expect(result).to have_key(:errors)
end
it 'rejects usernames with invalid characters' do
result = @service.check_username('vincent-', @nil_email)
expect(result).to have_key(:errors)
end
it 'rejects usernames that do not start with an alphanumeric character' do
result = @service.check_username('.vincent', @nil_email)
expect(result).to have_key(:errors)
end
end
it 'username not available locally' do
User.stubs(:username_available?).returns(false)
UserNameSuggester.stubs(:suggest).returns('einar-j')
result = @service.check_username('vincent', @nil_email)
expect(result[:available]).to eq(false)
expect(result[:suggestion]).to eq('einar-j')
end
it 'username available locally' do
User.stubs(:username_available?).returns(true)
result = @service.check_username('vincent', @nil_email)
expect(result[:available]).to eq(true)
end
end
end