Add a requireCommittee option so that you can specify a specific comm…

…ittee

index.bs

@@ -16,7 +16,14 @@ export (options) ->
             else
                 user = data.user
                 if user.auth
-                    if options.requireAbakom
+                    if options.requireCommittee
+                        if user.committees.indexOf(options.requireCommittee) >= 0
+                            user.username = username
+                            done(null, user)
+                        else
+                            done(null, false, { message: 'Not in #{options.requireCommittee}' })
+
+                    elif options.requireAbakom
                         if user.is_abakom
                             user.username = username
                             done(null, user)
@@ -25,6 +32,7 @@ export (options) ->
                     else
                         user.username = username
                         done(null, user)
+
                 else
                     done(null, false, { message: 'Bad credentials' })
         )

test/index.bs

@@ -71,6 +71,53 @@ describe('passport-abakus', () ->
                   }
                 ).authenticate()
         )
+
+        describe('requireCommittee option', () ->
+            it('fail when an user is not in a provided committee', (done) ->
+                createApiMock({ user: {
+                    username: 'test',
+                    committees: ['Webkom'],
+                    is_abakom: true,
+                    auth: true
+                }})
+
+                chai.passport.use(abakusStrategy({ requireCommittee: 'LaBamba' }))
+                    .fail((info) ->
+                        expect(info.message).to.equal('Not in LaBamba')
+                        done()
+                    ).req((req) ->
+                        req.query = {
+                            username: 'test',
+                            password: 'test'
+                        }
+                    ).authenticate()
+            )
+
+            it('success when user is a member of a provided committee', (done) ->
+                createApiMock({ user: {
+                    username: 'test',
+                    committees: ['Webkom'],
+                    is_abakom: true,
+                    auth: true
+                }})
+
+                chai.passport.use(abakusStrategy({ requireCommittee: 'Webkom' }))
+                    .success((user, info) ->
+                        expect(user.username).to.equal('test')
+                        expect(user.committees.indexOf('Webkom')).to.be.above(-1)
+                        expect(user.is_abakom).to.be.true
+                        expect(user.auth).to.be.true
+                        done()
+                    ).req((req) ->
+                        req.query = {
+                            username: 'test',
+                            password: 'test'
+                        }
+                    ).authenticate()
+
+            )
+        )
+
         describe('requireAbakom option', () ->
             it('fail when user is not a member of abakom', (done) ->
                 createApiMock({ user: {