GET /auth/name/<name> now serves possible free names

server/routes/auth.coffee

@@ -157,13 +157,30 @@ module.exports.setup = (app) ->
         res.send msg + '<p><a href="/account/settings">Account settings</a></p>'
         res.end()
 
+  app.get '/auth/name/*', (req, res) ->
+    parts = req.path.split '/'
+    console.log parts
+    originalName = parts[3]
+
+    return errors.badInput res, 'No name provided.' unless originalName and originalName isnt ''
+    return errors.notFound res if parts.length isnt 4
+
+    User.unconflictName originalName, (err, name) ->
+      return errors.serverError res, err if err
+      response = name: name
+      if originalName is name
+        res.send 200, response
+      else
+        errors.conflict res, response
+
+
 module.exports.loginUser = loginUser = (req, res, user, send=true, next=null) ->
   user.save((err) ->
     return errors.serverError res, err if err?
 
     req.logIn(user, (err) ->
       return errors.serverError res, err if err?
-      return res.send user if send
+      return res.send(user) and res.end() if send
       next() if next
     )
   )

test/server/functional/auth.spec.coffee

@@ -152,3 +152,27 @@ describe '/auth/unsubscribe', ->
           expect(user.get('emails').recruitNotes.enabled).toBe(false)
           expect(user.isEmailSubscriptionEnabled('generalNews')).toBeTruthy()
           done()
+
+describe '/auth/name', ->
+  url = '/auth/name'
+
+  it 'must provide a name to check with', (done) ->
+    request.get {url: getURL(url + '/'), json: {}}, (err, response) ->
+      expect(err).toBeNull()
+      expect(response.statusCode).toBe 422
+      done()
+
+  it 'can GET a non-conflicting name', (done) ->
+    request.get {url: getURL(url + '/Gandalf'), json: {}}, (err, response) ->
+      expect(err).toBeNull()
+      expect(response.statusCode).toBe 200
+      expect(response.body.name).toBe 'Gandalf'
+      done()
+
+  it 'can GET a new name in case of conflict', (done) ->
+    request.get {url: getURL(url + '/joe'), json: {}}, (err, response) ->
+      expect(err).toBeNull()
+      expect(response.statusCode).toBe 409
+      expect(response.body.name).not.toBe 'joe'
+      expect(response.body.name.length).toBe 4 # 'joe' and a random number
+      done()