[Test] Remove code duplication, use getCredentials method

tests/data/api-data.js

@@ -36,7 +36,7 @@ export function log(res) {
 	});
 }
 
-export function getCredentials() {
+export function getCredentials(done = function() {}) {
 	request.post(api('login'))
 		.send(login)
 		.expect('Content-Type', 'application/json')
@@ -44,6 +44,7 @@ export function getCredentials() {
 		.expect((res) => {
 			credentials['X-Auth-Token'] = res.body.data.authToken;
 			credentials['X-User-Id'] = res.body.data.userId;
-		});
+		})
+		.end(done);
 }
 

tests/end-to-end/api/00-miscellaneous.js

@@ -9,17 +9,7 @@ import supertest from 'supertest';
 describe('miscellaneous', function() {
 	this.retries(0);
 
-	before((done) => {
-		request.post(api('login'))
-		.send(login)
-		.expect('Content-Type', 'application/json')
-		.expect(200)
-		.expect((res) => {
-			credentials['X-Auth-Token'] = res.body.data.authToken;
-			credentials['X-User-Id'] = res.body.data.userId;
-		})
-		.end(done);
-	});
+	before(done => getCredentials(done));
 
 	describe('API default', () => {
 	// Required by mobile apps

tests/end-to-end/api/01-users.js

@@ -10,17 +10,7 @@ import supertest from 'supertest';
 describe('Users', function() {
 	this.retries(0);
 
-	before((done) => {
-		request.post(api('login'))
-		.send(login)
-		.expect('Content-Type', 'application/json')
-		.expect(200)
-		.expect((res) => {
-			credentials['X-Auth-Token'] = res.body.data.authToken;
-			credentials['X-User-Id'] = res.body.data.userId;
-		})
-		.end(done);
-	});
+	before(done => getCredentials(done));
 
 	it('/users.create', (done) => {
 		request.post(api('users.create'))

tests/end-to-end/api/02-channels.js

@@ -22,17 +22,7 @@ function getRoomInfo(roomId) {
 describe('channels', function() {
 	this.retries(0);
 
-	before((done) => {
-		request.post(api('login'))
-		.send(login)
-		.expect('Content-Type', 'application/json')
-		.expect(200)
-		.expect((res) => {
-			credentials['X-Auth-Token'] = res.body.data.authToken;
-			credentials['X-User-Id'] = res.body.data.userId;
-		})
-		.end(done);
-	});
+	before(done => getCredentials(done));
 
 	it('/channels.create', (done) => {
 		request.post(api('channels.create'))

tests/end-to-end/api/03-groups.js

@@ -22,17 +22,7 @@ function getRoomInfo(roomId) {
 describe('groups', function() {
 	this.retries(0);
 
-	before((done) => {
-		request.post(api('login'))
-		.send(login)
-		.expect('Content-Type', 'application/json')
-		.expect(200)
-		.expect((res) => {
-			credentials['X-Auth-Token'] = res.body.data.authToken;
-			credentials['X-User-Id'] = res.body.data.userId;
-		})
-		.end(done);
-	});
+	before(done => getCredentials(done));
 
 	it('/groups.create', (done) => {
 		request.post(api('groups.create'))

tests/end-to-end/api/04-direct-message.js

@@ -9,17 +9,8 @@ import supertest from 'supertest';
 describe('direct messages', function() {
 	this.retries(0);
 
-	before((done) => {
-		request.post(api('login'))
-		.send(login)
-		.expect('Content-Type', 'application/json')
-		.expect(200)
-		.expect((res) => {
-			credentials['X-Auth-Token'] = res.body.data.authToken;
-			credentials['X-User-Id'] = res.body.data.userId;
-		})
-		.end(done);
-	});
+	before(done => getCredentials(done));
+
 	it('/chat.postMessage', (done) => {
 		request.post(api('chat.postMessage'))
 			.set(credentials)

tests/end-to-end/api/05-chat.js

@@ -9,17 +9,7 @@ import supertest from 'supertest';
 describe('chat', function() {
 	this.retries(0);
 
-	before((done) => {
-		request.post(api('login'))
-		.send(login)
-		.expect('Content-Type', 'application/json')
-		.expect(200)
-		.expect((res) => {
-			credentials['X-Auth-Token'] = res.body.data.authToken;
-			credentials['X-User-Id'] = res.body.data.userId;
-		})
-		.end(done);
-	});
+	before(done => getCredentials(done));
 
 	it('/chat.postMessage', (done) => {
 		request.post(api('chat.postMessage'))