Merge branch 'master' into linked-user-email

This commit is contained in:
Moriz Wahl 2022-03-23 20:16:49 +01:00 committed by GitHub
commit 0ed539b098
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
18 changed files with 219 additions and 131 deletions

View File

@ -38,8 +38,8 @@ EMAIL_SENDER=info@gradido.net
EMAIL_PASSWORD=xxx EMAIL_PASSWORD=xxx
EMAIL_SMTP_URL=gmail.com EMAIL_SMTP_URL=gmail.com
EMAIL_SMTP_PORT=587 EMAIL_SMTP_PORT=587
EMAIL_LINK_VERIFICATION=http://localhost/checkEmail/{code} EMAIL_LINK_VERIFICATION=http://localhost/checkEmail/{optin}{code}
EMAIL_LINK_SETPASSWORD=http://localhost/reset/{code} EMAIL_LINK_SETPASSWORD=http://localhost/reset/{optin}
EMAIL_CODE_VALID_TIME=10 EMAIL_CODE_VALID_TIME=10
# Webhook # Webhook

View File

@ -62,9 +62,9 @@ const email = {
EMAIL_SMTP_URL: process.env.EMAIL_SMTP_URL || 'gmail.com', EMAIL_SMTP_URL: process.env.EMAIL_SMTP_URL || 'gmail.com',
EMAIL_SMTP_PORT: process.env.EMAIL_SMTP_PORT || '587', EMAIL_SMTP_PORT: process.env.EMAIL_SMTP_PORT || '587',
EMAIL_LINK_VERIFICATION: EMAIL_LINK_VERIFICATION:
process.env.EMAIL_LINK_VERIFICATION || 'http://localhost/checkEmail/{code}', process.env.EMAIL_LINK_VERIFICATION || 'http://localhost/checkEmail/{optin}{code}',
EMAIL_LINK_SETPASSWORD: EMAIL_LINK_SETPASSWORD:
process.env.EMAIL_LINK_SETPASSWORD || 'http://localhost/reset-password/{code}', process.env.EMAIL_LINK_SETPASSWORD || 'http://localhost/reset-password/{optin}',
EMAIL_CODE_VALID_TIME: process.env.EMAIL_CODE_VALID_TIME EMAIL_CODE_VALID_TIME: process.env.EMAIL_CODE_VALID_TIME
? parseInt(process.env.EMAIL_CODE_VALID_TIME) || 10 ? parseInt(process.env.EMAIL_CODE_VALID_TIME) || 10
: 10, : 10,

View File

@ -16,4 +16,7 @@ export default class CreateUserArgs {
@Field(() => Int, { nullable: true }) @Field(() => Int, { nullable: true })
publisherId: number publisherId: number
@Field(() => String, { nullable: true })
redeemCode?: string | null
} }

View File

@ -124,7 +124,10 @@ describe('UserResolver', () => {
describe('account activation email', () => { describe('account activation email', () => {
it('sends an account activation email', () => { it('sends an account activation email', () => {
const activationLink = CONFIG.EMAIL_LINK_VERIFICATION.replace(/{code}/g, emailOptIn) const activationLink = CONFIG.EMAIL_LINK_VERIFICATION.replace(
/{optin}/g,
emailOptIn,
).replace(/{code}/g, '')
expect(sendAccountActivationEmail).toBeCalledWith({ expect(sendAccountActivationEmail).toBeCalledWith({
link: activationLink, link: activationLink,
firstName: 'Peter', firstName: 'Peter',

View File

@ -7,6 +7,7 @@ import { getConnection, getCustomRepository, QueryRunner } from '@dbTools/typeor
import CONFIG from '@/config' import CONFIG from '@/config'
import { User } from '@model/User' import { User } from '@model/User'
import { User as DbUser } from '@entity/User' import { User as DbUser } from '@entity/User'
import { TransactionLink as dbTransactionLink } from '@entity/TransactionLink'
import { encode } from '@/auth/JWT' import { encode } from '@/auth/JWT'
import CreateUserArgs from '@arg/CreateUserArgs' import CreateUserArgs from '@arg/CreateUserArgs'
import UnsecureLoginArgs from '@arg/UnsecureLoginArgs' import UnsecureLoginArgs from '@arg/UnsecureLoginArgs'
@ -305,7 +306,8 @@ export class UserResolver {
@Authorized([RIGHTS.CREATE_USER]) @Authorized([RIGHTS.CREATE_USER])
@Mutation(() => User) @Mutation(() => User)
async createUser( async createUser(
@Args() { email, firstName, lastName, language, publisherId }: CreateUserArgs, @Args()
{ email, firstName, lastName, language, publisherId, redeemCode = null }: CreateUserArgs,
): Promise<User> { ): Promise<User> {
// TODO: wrong default value (should be null), how does graphql work here? Is it an required field? // TODO: wrong default value (should be null), how does graphql work here? Is it an required field?
// default int publisher_id = 0; // default int publisher_id = 0;
@ -338,6 +340,12 @@ export class UserResolver {
dbUser.language = language dbUser.language = language
dbUser.publisherId = publisherId dbUser.publisherId = publisherId
dbUser.passphrase = passphrase.join(' ') dbUser.passphrase = passphrase.join(' ')
if (redeemCode) {
const transactionLink = await dbTransactionLink.findOne({ code: redeemCode })
if (transactionLink) {
dbUser.referrerId = transactionLink.userId
}
}
// TODO this field has no null allowed unlike the loginServer table // TODO this field has no null allowed unlike the loginServer table
// dbUser.pubKey = Buffer.from(randomBytes(32)) // Buffer.alloc(32, 0) default to 0000... // dbUser.pubKey = Buffer.from(randomBytes(32)) // Buffer.alloc(32, 0) default to 0000...
// dbUser.pubkey = keyPair[0] // dbUser.pubkey = keyPair[0]
@ -360,9 +368,9 @@ export class UserResolver {
const emailOptIn = await createEmailOptIn(dbUser.id, queryRunner) const emailOptIn = await createEmailOptIn(dbUser.id, queryRunner)
const activationLink = CONFIG.EMAIL_LINK_VERIFICATION.replace( const activationLink = CONFIG.EMAIL_LINK_VERIFICATION.replace(
/{code}/g, /{optin}/g,
emailOptIn.verificationCode.toString(), emailOptIn.verificationCode.toString(),
) ).replace(/{code}/g, redeemCode ? '/' + redeemCode : '')
// eslint-disable-next-line @typescript-eslint/no-unused-vars // eslint-disable-next-line @typescript-eslint/no-unused-vars
const emailSent = await sendAccountActivationEmail({ const emailSent = await sendAccountActivationEmail({
@ -379,6 +387,7 @@ export class UserResolver {
console.log(`Account confirmation link: ${activationLink}`) console.log(`Account confirmation link: ${activationLink}`)
} }
*/ */
await queryRunner.commitTransaction() await queryRunner.commitTransaction()
} catch (e) { } catch (e) {
await queryRunner.rollbackTransaction() await queryRunner.rollbackTransaction()
@ -404,7 +413,7 @@ export class UserResolver {
const emailOptIn = await createEmailOptIn(user.id, queryRunner) const emailOptIn = await createEmailOptIn(user.id, queryRunner)
const activationLink = CONFIG.EMAIL_LINK_VERIFICATION.replace( const activationLink = CONFIG.EMAIL_LINK_VERIFICATION.replace(
/{code}/g, /{optin}/g,
emailOptIn.verificationCode.toString(), emailOptIn.verificationCode.toString(),
) )
@ -443,7 +452,7 @@ export class UserResolver {
const optInCode = await getOptInCode(user.id) const optInCode = await getOptInCode(user.id)
const link = CONFIG.EMAIL_LINK_SETPASSWORD.replace( const link = CONFIG.EMAIL_LINK_SETPASSWORD.replace(
/{code}/g, /{optin}/g,
optInCode.verificationCode.toString(), optInCode.verificationCode.toString(),
) )

View File

@ -45,6 +45,7 @@ export const createUser = gql`
$email: String! $email: String!
$language: String! $language: String!
$publisherId: Int $publisherId: Int
$redeemCode: String
) { ) {
createUser( createUser(
email: $email email: $email
@ -52,6 +53,7 @@ export const createUser = gql`
lastName: $lastName lastName: $lastName
language: $language language: $language
publisherId: $publisherId publisherId: $publisherId
redeemCode: $redeemCode
) { ) {
id id
} }

View File

@ -160,6 +160,9 @@ export default {
}, },
props: { props: {
balance: { type: Number, default: 0 }, balance: { type: Number, default: 0 },
email: { type: String, default: '' },
amount: { type: Number, default: 0 },
memo: { type: String, default: '' },
}, },
inject: ['getTunneledEmail'], inject: ['getTunneledEmail'],
data() { data() {
@ -167,9 +170,9 @@ export default {
amountFocused: false, amountFocused: false,
emailFocused: false, emailFocused: false,
form: { form: {
email: '', email: this.email,
amount: '', amount: this.amount ? String(this.amount) : '',
memo: '', memo: this.memo,
amountValue: 0.0, amountValue: 0.0,
}, },
selected: SEND_TYPES.send, selected: SEND_TYPES.send,

View File

@ -31,7 +31,7 @@
</template> </template>
<script> <script>
export default { export default {
name: 'TransactionResultSend', name: 'TransactionResultSendError',
props: { props: {
error: { type: Boolean, default: false }, error: { type: Boolean, default: false },
errorResult: { type: String, default: '' }, errorResult: { type: String, default: '' },

View File

@ -45,6 +45,7 @@ export const createUser = gql`
$email: String! $email: String!
$language: String! $language: String!
$publisherId: Int $publisherId: Int
$redeemCode: String
) { ) {
createUser( createUser(
email: $email email: $email
@ -52,6 +53,7 @@ export const createUser = gql`
lastName: $lastName lastName: $lastName
language: $language language: $language
publisherId: $publisherId publisherId: $publisherId
redeemCode: $redeemCode
) { ) {
id id
} }

View File

@ -140,14 +140,6 @@ describe('DashboardLayoutGdd', () => {
}) })
}) })
describe('update balance', () => {
it('updates the amount correctelly', async () => {
await wrapper.findComponent({ ref: 'router-view' }).vm.$emit('update-balance', 5)
await flushPromises()
expect(wrapper.vm.balance).toBe(-5)
})
})
describe('update transactions', () => { describe('update transactions', () => {
beforeEach(async () => { beforeEach(async () => {
apolloMock.mockResolvedValue({ apolloMock.mockResolvedValue({

View File

@ -27,7 +27,6 @@
:transactionLinkCount="transactionLinkCount" :transactionLinkCount="transactionLinkCount"
:pending="pending" :pending="pending"
:decayStartBlock="decayStartBlock" :decayStartBlock="decayStartBlock"
@update-balance="updateBalance"
@update-transactions="updateTransactions" @update-transactions="updateTransactions"
@set-tunneled-email="setTunneledEmail" @set-tunneled-email="setTunneledEmail"
></router-view> ></router-view>
@ -119,9 +118,6 @@ export default {
// what to do when loading balance fails? // what to do when loading balance fails?
}) })
}, },
updateBalance(ammount) {
this.balance -= ammount
},
admin() { admin() {
window.location.assign(CONFIG.ADMIN_AUTH_URL.replace('{token}', this.$store.state.token)) window.location.assign(CONFIG.ADMIN_AUTH_URL.replace('{token}', this.$store.state.token))
this.$store.dispatch('logout') // logout without redirect this.$store.dispatch('logout') // logout without redirect

View File

@ -54,25 +54,27 @@ describe('ResetPassword', () => {
describe('mount', () => { describe('mount', () => {
beforeEach(() => { beforeEach(() => {
jest.clearAllMocks()
wrapper = Wrapper() wrapper = Wrapper()
}) })
describe('No valid optin', () => { describe('no valid optin', () => {
it.skip('does not render the Reset Password form when not authenticated', () => { beforeEach(() => {
expect(wrapper.find('form').exists()).toBeFalsy() jest.clearAllMocks()
apolloQueryMock.mockRejectedValue({ message: 'Your time is up!' })
wrapper = Wrapper()
}) })
it.skip('toasts an error when no valid optin is given', () => { it('toasts an error when no valid optin is given', () => {
expect(toastErrorSpy).toHaveBeenCalledWith('error') expect(toastErrorSpy).toHaveBeenCalledWith('Your time is up!')
}) })
it.skip('has a message suggesting to contact the support', () => { it('redirects to /forgot-password/resetPassword', () => {
expect(wrapper.find('div.header').text()).toContain('settings.password.reset') expect(routerPushMock).toBeCalledWith('/forgot-password/resetPassword')
expect(wrapper.find('div.header').text()).toContain('settings.password.not-authenticated')
}) })
}) })
describe('is authenticated', () => { describe('valid optin', () => {
it('renders the Reset Password form when authenticated', () => { it('renders the Reset Password form when authenticated', () => {
expect(wrapper.find('div.resetpwd-form').exists()).toBeTruthy() expect(wrapper.find('div.resetpwd-form').exists()).toBeTruthy()
}) })
@ -148,7 +150,6 @@ describe('ResetPassword', () => {
describe('server response with error code > 10min', () => { describe('server response with error code > 10min', () => {
beforeEach(async () => { beforeEach(async () => {
jest.clearAllMocks()
apolloMutationMock.mockRejectedValue({ message: '...Code is older than 10 minutes' }) apolloMutationMock.mockRejectedValue({ message: '...Code is older than 10 minutes' })
await wrapper.find('form').trigger('submit') await wrapper.find('form').trigger('submit')
await flushPromises() await flushPromises()
@ -163,7 +164,7 @@ describe('ResetPassword', () => {
}) })
}) })
describe('server response with error code > 10min', () => { describe('server response with error', () => {
beforeEach(async () => { beforeEach(async () => {
jest.clearAllMocks() jest.clearAllMocks()
apolloMutationMock.mockRejectedValueOnce({ message: 'Error' }) apolloMutationMock.mockRejectedValueOnce({ message: 'Error' })
@ -178,6 +179,7 @@ describe('ResetPassword', () => {
describe('server response with success on /checkEmail', () => { describe('server response with success on /checkEmail', () => {
beforeEach(async () => { beforeEach(async () => {
jest.clearAllMocks()
mocks.$route.path.mock = 'checkEmail' mocks.$route.path.mock = 'checkEmail'
apolloMutationMock.mockResolvedValue({ apolloMutationMock.mockResolvedValue({
data: { data: {
@ -204,6 +206,28 @@ describe('ResetPassword', () => {
it('redirects to "/thx/checkEmail"', () => { it('redirects to "/thx/checkEmail"', () => {
expect(routerPushMock).toHaveBeenCalledWith('/thx/checkEmail') expect(routerPushMock).toHaveBeenCalledWith('/thx/checkEmail')
}) })
describe('with param code', () => {
beforeEach(async () => {
mocks.$route.params.code = 'the-most-secret-code-ever'
apolloMutationMock.mockResolvedValue({
data: {
resetPassword: 'success',
},
})
wrapper = Wrapper()
await wrapper.findAll('input').at(0).setValue('Aa123456_')
await wrapper.findAll('input').at(1).setValue('Aa123456_')
await wrapper.find('form').trigger('submit')
await flushPromises()
})
it('redirects to "/thx/checkEmail/the-most-secret-code-ever"', () => {
expect(routerPushMock).toHaveBeenCalledWith(
'/thx/checkEmail/the-most-secret-code-ever',
)
})
})
}) })
describe('server response with success on /reset-password', () => { describe('server response with success on /reset-password', () => {

View File

@ -6,11 +6,11 @@
<b-row class="justify-content-center"> <b-row class="justify-content-center">
<b-col xl="5" lg="6" md="8" class="px-2"> <b-col xl="5" lg="6" md="8" class="px-2">
<!-- eslint-disable-next-line @intlify/vue-i18n/no-dynamic-keys--> <!-- eslint-disable-next-line @intlify/vue-i18n/no-dynamic-keys-->
<h1>{{ $t(displaySetup.authenticated) }}</h1> <h1>{{ $t(displaySetup.title) }}</h1>
<div class="pb-4"> <div class="pb-4">
<span> <span>
<!-- eslint-disable-next-line @intlify/vue-i18n/no-dynamic-keys--> <!-- eslint-disable-next-line @intlify/vue-i18n/no-dynamic-keys-->
{{ $t(displaySetup.notAuthenticated) }} {{ $t(displaySetup.text) }}
</span> </span>
</div> </div>
</b-col> </b-col>
@ -53,14 +53,14 @@ import { queryOptIn } from '@/graphql/queries'
const textFields = { const textFields = {
reset: { reset: {
authenticated: 'settings.password.change-password', title: 'settings.password.change-password',
notAuthenticated: 'settings.password.reset-password.text', text: 'settings.password.reset-password.text',
button: 'settings.password.change-password', button: 'settings.password.change-password',
linkTo: '/login', linkTo: '/login',
}, },
checkEmail: { checkEmail: {
authenticated: 'settings.password.set', title: 'settings.password.set',
notAuthenticated: 'settings.password.set-password.text', text: 'settings.password.set-password.text',
button: 'settings.password.set', button: 'settings.password.set',
linkTo: '/login', linkTo: '/login',
}, },
@ -97,7 +97,11 @@ export default {
.then(() => { .then(() => {
this.form.password = '' this.form.password = ''
if (this.$route.path.includes('checkEmail')) { if (this.$route.path.includes('checkEmail')) {
if (this.$route.params.code) {
this.$router.push('/thx/checkEmail/' + this.$route.params.code)
} else {
this.$router.push('/thx/checkEmail') this.$router.push('/thx/checkEmail')
}
} else { } else {
this.$router.push('/thx/resetPassword') this.$router.push('/thx/resetPassword')
} }

View File

@ -18,9 +18,7 @@ describe('Send', () => {
const propsData = { const propsData = {
balance: 123.45, balance: 123.45,
GdtBalance: 1234.56, GdtBalance: 1234.56,
transactions: [{ balance: 0.1 }],
pending: true, pending: true,
currentTransactionStep: TRANSACTION_STEPS.transactionConfirmationSend,
} }
const mocks = { const mocks = {
@ -54,11 +52,10 @@ describe('Send', () => {
}) })
it('has a send field', () => { it('has a send field', () => {
expect(wrapper.find('div.gdd-send').exists()).toBeTruthy() expect(wrapper.find('div.gdd-send').exists()).toBe(true)
}) })
/* SEND */ describe('fill transaction form for send coins', () => {
describe('transaction form send', () => {
beforeEach(async () => { beforeEach(async () => {
wrapper.findComponent({ name: 'TransactionForm' }).vm.$emit('set-transaction', { wrapper.findComponent({ name: 'TransactionForm' }).vm.$emit('set-transaction', {
email: 'user@example.org', email: 'user@example.org',
@ -67,36 +64,35 @@ describe('Send', () => {
selected: SEND_TYPES.send, selected: SEND_TYPES.send,
}) })
}) })
it('steps forward in the dialog', () => { it('steps forward in the dialog', () => {
expect(wrapper.findComponent({ name: 'TransactionConfirmationSend' }).exists()).toBe(true) expect(wrapper.findComponent({ name: 'TransactionConfirmationSend' }).exists()).toBe(true)
}) })
describe('confirm transaction view', () => {
describe('cancel confirmation', () => {
beforeEach(async () => {
await wrapper
.findComponent({ name: 'TransactionConfirmationSend' })
.vm.$emit('on-reset')
}) })
describe('confirm transaction if selected: SEND_TYPES.send', () => { it('shows the transaction formular again', () => {
beforeEach(() => { expect(wrapper.findComponent({ name: 'TransactionForm' }).exists()).toBe(true)
wrapper.setData({ })
currentTransactionStep: TRANSACTION_STEPS.transactionConfirmationSend,
transactionData: { it('restores the previous data in the formular', () => {
email: 'user@example.org', expect(wrapper.find('#input-group-1').find('input').vm.$el.value).toBe(
amount: 23.45, 'user@example.org',
memo: 'Make the best of it!', )
selected: SEND_TYPES.send, expect(wrapper.find('#input-group-2').find('input').vm.$el.value).toBe('23.45')
}, expect(wrapper.find('#input-group-3').find('textarea').vm.$el.value).toBe(
'Make the best of it!',
)
}) })
}) })
it('resets the transaction process when on-reset is emitted', async () => { describe('confirm transaction with server succees', () => {
await wrapper.findComponent({ name: 'TransactionConfirmationSend' }).vm.$emit('on-reset')
expect(wrapper.findComponent({ name: 'TransactionForm' }).exists()).toBeTruthy()
expect(wrapper.vm.transactionData).toEqual({
email: 'user@example.org',
amount: 23.45,
memo: 'Make the best of it!',
selected: SEND_TYPES.send,
})
})
describe('transaction is confirmed and server response is success', () => {
beforeEach(async () => { beforeEach(async () => {
jest.clearAllMocks() jest.clearAllMocks()
await wrapper await wrapper
@ -118,9 +114,9 @@ describe('Send', () => {
) )
}) })
it('emits update-balance', () => { it('emits update transactions', () => {
expect(wrapper.emitted('update-balance')).toBeTruthy() expect(wrapper.emitted('update-transactions')).toBeTruthy()
expect(wrapper.emitted('update-balance')).toEqual([[23.45]]) expect(wrapper.emitted('update-transactions')).toEqual(expect.arrayContaining([[{}]]))
}) })
it('shows the success page', () => { it('shows the success page', () => {
@ -128,7 +124,7 @@ describe('Send', () => {
}) })
}) })
describe('transaction is confirmed and server response is error', () => { describe('confirm transaction with server error', () => {
beforeEach(async () => { beforeEach(async () => {
jest.clearAllMocks() jest.clearAllMocks()
apolloMutationMock.mockRejectedValue({ message: 'recipient not known' }) apolloMutationMock.mockRejectedValue({ message: 'recipient not known' })
@ -137,7 +133,13 @@ describe('Send', () => {
.vm.$emit('send-transaction') .vm.$emit('send-transaction')
}) })
it('shows the error page', () => { it('has a component TransactionResultSendError', () => {
expect(wrapper.findComponent({ name: 'TransactionResultSendError' }).exists()).toBe(
true,
)
})
it('has an standard error text', () => {
expect(wrapper.find('.test-send_transaction_error').text()).toContain( expect(wrapper.find('.test-send_transaction_error').text()).toContain(
'form.send_transaction_error', 'form.send_transaction_error',
) )
@ -150,6 +152,7 @@ describe('Send', () => {
}) })
}) })
}) })
})
/* LINK */ /* LINK */
@ -160,10 +163,11 @@ describe('Send', () => {
}) })
await wrapper.findComponent({ name: 'TransactionForm' }).vm.$emit('set-transaction', { await wrapper.findComponent({ name: 'TransactionForm' }).vm.$emit('set-transaction', {
amount: 56.78, amount: 56.78,
memo: 'Make the best of it link!', memo: 'Make the best of the link!',
selected: SEND_TYPES.link, selected: SEND_TYPES.link,
}) })
}) })
it('steps forward in the dialog', () => { it('steps forward in the dialog', () => {
expect(wrapper.findComponent({ name: 'TransactionConfirmationLink' }).exists()).toBe(true) expect(wrapper.findComponent({ name: 'TransactionConfirmationLink' }).exists()).toBe(true)
}) })
@ -182,18 +186,18 @@ describe('Send', () => {
mutation: createTransactionLink, mutation: createTransactionLink,
variables: { variables: {
amount: 56.78, amount: 56.78,
memo: 'Make the best of it link!', memo: 'Make the best of the link!',
}, },
}), }),
) )
}) })
it.skip('emits update-balance', () => { it('emits update-transactions', () => {
expect(wrapper.emitted('update-balance')).toBeTruthy() expect(wrapper.emitted('update-transactions')).toBeTruthy()
expect(wrapper.emitted('update-balance')).toEqual([[56.78]]) expect(wrapper.emitted('update-transactions')).toEqual(expect.arrayContaining([[{}]]))
}) })
it('find components ClipBoard', () => { it('finds the clip board component', () => {
expect(wrapper.findComponent({ name: 'ClipboardCopy' }).exists()).toBe(true) expect(wrapper.findComponent({ name: 'ClipboardCopy' }).exists()).toBe(true)
}) })
@ -205,7 +209,7 @@ describe('Send', () => {
expect(wrapper.find('div.card-body').text()).toContain('form.close') expect(wrapper.find('div.card-body').text()).toContain('form.close')
}) })
describe('Copy link to Clipboard', () => { describe('copy link to clipboard', () => {
const navigatorClipboard = navigator.clipboard const navigatorClipboard = navigator.clipboard
beforeAll(() => { beforeAll(() => {
delete navigator.clipboard delete navigator.clipboard
@ -260,5 +264,39 @@ describe('Send', () => {
}) })
}) })
}) })
describe('no field selected on send transaction', () => {
const errorHandler = localVue.config.errorHandler
beforeAll(() => {
localVue.config.errorHandler = jest.fn()
})
afterAll(() => {
localVue.config.errorHandler = errorHandler
})
beforeEach(async () => {
await wrapper.setData({
currentTransactionStep: TRANSACTION_STEPS.transactionConfirmationSend,
transactionData: {
email: 'user@example.org',
amount: 23.45,
memo: 'Make the best of it!',
selected: 'not-valid',
},
})
})
it('throws an error', async () => {
try {
await wrapper
.findComponent({ name: 'TransactionConfirmationSend' })
.vm.$emit('send-transaction')
} catch (error) {
expect(error).toBe('undefined transactionData.selected : not-valid')
}
})
})
}) })
}) })

View File

@ -3,7 +3,11 @@
<b-container> <b-container>
<gdd-send :currentTransactionStep="currentTransactionStep" class="pt-3 ml-2 mr-2"> <gdd-send :currentTransactionStep="currentTransactionStep" class="pt-3 ml-2 mr-2">
<template #transactionForm> <template #transactionForm>
<transaction-form :balance="balance" @set-transaction="setTransaction"></transaction-form> <transaction-form
v-bind="transactionData"
:balance="balance"
@set-transaction="setTransaction"
></transaction-form>
</template> </template>
<template #transactionConfirmationSend> <template #transactionConfirmationSend>
<transaction-confirmation-send <transaction-confirmation-send
@ -124,8 +128,9 @@ export default {
}) })
.then(() => { .then(() => {
this.error = false this.error = false
this.$emit('update-balance', this.transactionData.amount)
this.$emit('set-tunneled-email', null) this.$emit('set-tunneled-email', null)
this.updateTransactions({})
this.transactionData = { ...EMPTY_TRANSACTION_DATA }
this.currentTransactionStep = TRANSACTION_STEPS.transactionResultSendSuccess this.currentTransactionStep = TRANSACTION_STEPS.transactionResultSendSuccess
}) })
.catch((err) => { .catch((err) => {
@ -144,6 +149,7 @@ export default {
this.$emit('set-tunneled-email', null) this.$emit('set-tunneled-email', null)
this.code = result.data.createTransactionLink.code this.code = result.data.createTransactionLink.code
this.currentTransactionStep = TRANSACTION_STEPS.transactionResultLink this.currentTransactionStep = TRANSACTION_STEPS.transactionResultLink
this.updateTransactions({})
}) })
.catch((error) => { .catch((error) => {
this.toastError(error) this.toastError(error)
@ -162,7 +168,7 @@ export default {
}, },
}, },
created() { created() {
this.updateTransactions(0) this.updateTransactions({})
}, },
} }
</script> </script>

View File

@ -9,7 +9,11 @@
<!-- eslint-disable-next-line @intlify/vue-i18n/no-dynamic-keys--> <!-- eslint-disable-next-line @intlify/vue-i18n/no-dynamic-keys-->
<p class="h4">{{ $t(displaySetup.subtitle) }}</p> <p class="h4">{{ $t(displaySetup.subtitle) }}</p>
<hr /> <hr />
<b-button v-if="displaySetup.linkTo" :to="displaySetup.linkTo"> <b-button v-if="$route.params.code" :to="`/redeem/${$route.params.code}`">
<!-- eslint-disable-next-line @intlify/vue-i18n/no-dynamic-keys-->
{{ $t(displaySetup.button) }}
</b-button>
<b-button v-else :to="displaySetup.linkTo">
<!-- eslint-disable-next-line @intlify/vue-i18n/no-dynamic-keys--> <!-- eslint-disable-next-line @intlify/vue-i18n/no-dynamic-keys-->
{{ $t(displaySetup.button) }} {{ $t(displaySetup.button) }}
</b-button> </b-button>

View File

@ -112,7 +112,7 @@ describe('router', () => {
}) })
describe('thx', () => { describe('thx', () => {
const thx = routes.find((r) => r.path === '/thx/:comingFrom') const thx = routes.find((r) => r.path === '/thx/:comingFrom/:code?')
it('loads the "Thx" page', async () => { it('loads the "Thx" page', async () => {
const component = await thx.component() const component = await thx.component()
@ -177,7 +177,9 @@ describe('router', () => {
describe('checkEmail', () => { describe('checkEmail', () => {
it('loads the "CheckEmail" page', async () => { it('loads the "CheckEmail" page', async () => {
const component = await routes.find((r) => r.path === '/checkEmail/:optin').component() const component = await routes
.find((r) => r.path === '/checkEmail/:optin/:code?')
.component()
expect(component.default.name).toBe('ResetPassword') expect(component.default.name).toBe('ResetPassword')
}) })
}) })

View File

@ -47,7 +47,7 @@ const routes = [
component: () => import('@/pages/Register.vue'), component: () => import('@/pages/Register.vue'),
}, },
{ {
path: '/thx/:comingFrom', path: '/thx/:comingFrom/:code?',
component: () => import('@/pages/thx.vue'), component: () => import('@/pages/thx.vue'),
beforeEnter: (to, from, next) => { beforeEnter: (to, from, next) => {
const validFrom = ['forgot-password', 'reset-password', 'register', 'login', 'checkEmail'] const validFrom = ['forgot-password', 'reset-password', 'register', 'login', 'checkEmail']
@ -79,7 +79,7 @@ const routes = [
component: () => import('@/pages/ResetPassword.vue'), component: () => import('@/pages/ResetPassword.vue'),
}, },
{ {
path: '/checkEmail/:optin', path: '/checkEmail/:optin/:code?',
component: () => import('@/pages/ResetPassword.vue'), component: () => import('@/pages/ResetPassword.vue'),
}, },
{ {