Kai/habitica
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix tests

Browse Source
This commit is contained in:
Phillip Thelen
2021-12-10 17:09:01 +01:00
parent bfdfcc9901
commit 0901c717c6
3 changed files with 56 additions and 27 deletions
Download Patch File Download Diff File Expand all files Collapse all files
test/api/v3/integration/user/auth/GET-user_auth_apple.test.js
+5 -3
View File
@@ -1,3 +1,4 @@
import { v4 as generateUUID } from 'uuid';
import {
generateUser,
requester,
@@ -12,14 +13,15 @@ describe('GET /user/auth/apple', () => {
let randomAppleId = '123456';
beforeEach(async () => {
api = requester();
user = await generateUser();
randomAppleId = generateUUID();
const expectedResult = { id: randomAppleId, name: 'an apple user' };
sandbox.stub(appleAuth, 'appleProfile').returns(Promise.resolve(expectedResult));
});
beforeEach(async () => {
api = requester();
user = await generateUser();
afterEach(async () => {
appleAuth.appleProfile.restore();
});
it('registers a new user', async () => {
test/api/v3/integration/user/auth/POST-user_auth_social.test.js
+43 -19
View File
@@ -1,11 +1,11 @@
import passport from 'passport';
import { v4 as generateUUID } from 'uuid';
import {
generateUser,
requester,
translate as t,
getProperty,
} from '../../../../../helpers/api-integration/v3';
import { v4 as generateUUID } from 'uuid';
describe('POST /user/auth/social', () => {
let api;
@@ -36,9 +36,13 @@ describe('POST /user/auth/social', () => {
describe('facebook', () => {
beforeEach(async () => {
randomFacebookId = generateUUID();
const expectedResult = { id: randomFacebookId, displayName: 'a facebook user', emails: [
{ value: `${user.auth.local.username}+facebook@example.com`}
] };
const expectedResult = {
id: randomFacebookId,
displayName: 'a facebook user',
emails: [
{ value: `${user.auth.local.username}+facebook@example.com` },
],
};
sandbox.stub(passport._strategies.facebook, 'userProfile').yields(null, expectedResult);
network = 'facebook';
});
@@ -83,9 +87,13 @@ describe('POST /user/auth/social', () => {
it('logs an existing user in if they have local auth with matching email', async () => {
passport._strategies.facebook.userProfile.restore();
const expectedResult = { id: randomFacebookId, displayName: 'a facebook user', emails: [
{ value: user.auth.local.email},
] };
const expectedResult = {
id: randomFacebookId,
displayName: 'a facebook user',
emails: [
{ value: user.auth.local.email },
],
};
sandbox.stub(passport._strategies.facebook, 'userProfile').yields(null, expectedResult);
const response = await api.post(endpoint, {
@@ -106,9 +114,13 @@ describe('POST /user/auth/social', () => {
expect(registerResponse.newUser).to.be.true;
// This is important for existing accounts before the new social handling
passport._strategies.facebook.userProfile.restore();
const expectedResult = { id: randomFacebookId, displayName: 'a facebook user', emails: [
{ value: user.auth.local.email},
] };
const expectedResult = {
id: randomFacebookId,
displayName: 'a facebook user',
emails: [
{ value: user.auth.local.email },
],
};
sandbox.stub(passport._strategies.facebook, 'userProfile').yields(null, expectedResult);
const response = await api.post(endpoint, {
@@ -164,9 +176,13 @@ describe('POST /user/auth/social', () => {
describe('google', () => {
beforeEach(async () => {
randomGoogleId = generateUUID();
const expectedResult = { id: randomGoogleId, displayName: 'a google user', emails: [
{ value: `${user.auth.local.username}+google@example.com`}
] };
const expectedResult = {
id: randomGoogleId,
displayName: 'a google user',
emails: [
{ value: `${user.auth.local.username}+google@example.com` },
],
};
sandbox.stub(passport._strategies.google, 'userProfile').yields(null, expectedResult);
network = 'google';
});
@@ -208,9 +224,13 @@ describe('POST /user/auth/social', () => {
it('logs an existing user in if they have local auth with matching email', async () => {
passport._strategies.google.userProfile.restore();
const expectedResult = { id: randomGoogleId, displayName: 'a google user', emails: [
{ value: user.auth.local.email},
] };
const expectedResult = {
id: randomGoogleId,
displayName: 'a google user',
emails: [
{ value: user.auth.local.email },
],
};
sandbox.stub(passport._strategies.google, 'userProfile').yields(null, expectedResult);
const response = await api.post(endpoint, {
@@ -231,9 +251,13 @@ describe('POST /user/auth/social', () => {
expect(registerResponse.newUser).to.be.true;
// This is important for existing accounts before the new social handling
passport._strategies.google.userProfile.restore();
const expectedResult = { id: randomGoogleId, displayName: 'a google user', emails: [
{ value: user.auth.local.email},
] };
const expectedResult = {
id: randomGoogleId,
displayName: 'a google user',
emails: [
{ value: user.auth.local.email },
],
};
sandbox.stub(passport._strategies.google, 'userProfile').yields(null, expectedResult);
const response = await api.post(endpoint, {
test/api/v3/integration/user/auth/POST-user_reset_password.test.js
+8 -5
View File
@@ -26,16 +26,19 @@ describe('POST /user/reset-password', async () => {
});
it('resets password for social users', async () => {
const email = `${user.auth.local.username}+google@example.com`
user.auth.google.emails = [email];
const email = `${user.auth.local.username}+google@example.com`;
await user.update({ 'auth.google.emails': [{ value: email }] });
await user.sync();
const previousPassword = user.auth.local.hashed_password;
const previousPassword = user.auth.local.passwordResetCode;
console.log(previousPassword);
const response = await user.post(endpoint, {
email: email,
email,
});
expect(response).to.eql({ data: {}, message: t('passwordReset') });
await user.sync();
expect(user.auth.local.hashed_password).to.not.eql(previousPassword);
console.log(user.auth)
console.log(user.auth.local.hashed_password + " OLD " + previousPassword);
expect(user.auth.local.passwordResetCode).to.not.eql(previousPassword);
});
it('same message on error as on success', async () => {