Added better support for preconditions in use-cases.
This commit is contained in:
parent
bb2b3da9dd
commit
6c02db3331
2 changed files with 40 additions and 5 deletions
|
@ -51,6 +51,15 @@ function UseCaseReporter(runner) {
|
|||
--indents;
|
||||
});
|
||||
|
||||
runner.on('steps', useCase => {
|
||||
++indents;
|
||||
console.log(color('pass', '%s%s'), indent(), useCase.title);
|
||||
});
|
||||
|
||||
runner.on('steps end', () => {
|
||||
--indents;
|
||||
});
|
||||
|
||||
runner.on('step pass', step => {
|
||||
console.log(indent() + color('checkmark', ' ' + Mocha.reporters.Base.symbols.ok) + color('pass', ' %s'), step.title);
|
||||
});
|
||||
|
@ -183,9 +192,26 @@ async function step(name, asyncFn) {
|
|||
}
|
||||
}
|
||||
|
||||
async function steps(name, asyncFn) {
|
||||
try {
|
||||
runner.emit('steps', {title: name});
|
||||
await asyncFn();
|
||||
runner.emit('steps end');
|
||||
} catch (err) {
|
||||
runner.emit('step end');
|
||||
throw err;
|
||||
}
|
||||
}
|
||||
|
||||
async function precondition(preConditionName, useCaseName, asyncFn) {
|
||||
await steps(`Including use case "${useCaseName}" to satisfy precondition "${preConditionName}"`, asyncFn);
|
||||
}
|
||||
|
||||
module.exports = {
|
||||
mocha,
|
||||
useCase,
|
||||
step,
|
||||
steps,
|
||||
precondition,
|
||||
driver
|
||||
};
|
|
@ -1,7 +1,7 @@
|
|||
'use strict';
|
||||
|
||||
const config = require('../lib/config');
|
||||
const { useCase, step, driver } = require('../lib/mocha-e2e');
|
||||
const { useCase, step, precondition, driver } = require('../lib/mocha-e2e');
|
||||
const shortid = require('shortid');
|
||||
const expect = require('chai').expect;
|
||||
|
||||
|
@ -60,6 +60,13 @@ async function subscribe(subscription) {
|
|||
return subscription;
|
||||
}
|
||||
|
||||
async function subscriptionExistsPrecondition(subscription) {
|
||||
await precondition('Subscription exists', 'Subscription to a public list (main scenario)', async () => {
|
||||
await subscribe(subscription);
|
||||
});
|
||||
return subscription;
|
||||
}
|
||||
|
||||
suite('Subscription use-cases', function() {
|
||||
before(() => driver.manage().deleteAllCookies());
|
||||
|
||||
|
@ -86,7 +93,7 @@ suite('Subscription use-cases', function() {
|
|||
});
|
||||
|
||||
useCase('Subscription to a public list (email already registered)', async () => {
|
||||
const subscription = await subscribe({
|
||||
const subscription = await subscriptionExistsPrecondition({
|
||||
email: generateEmail()
|
||||
});
|
||||
|
||||
|
@ -112,7 +119,7 @@ suite('Subscription use-cases', function() {
|
|||
useCase('Subscription to a non-public list');
|
||||
|
||||
useCase('Change profile info', async () => {
|
||||
const subscription = await subscribe({
|
||||
const subscription = await subscriptionExistsPrecondition({
|
||||
email: generateEmail(),
|
||||
firstName: 'John',
|
||||
lastName: 'Doe'
|
||||
|
@ -154,7 +161,7 @@ suite('Subscription use-cases', function() {
|
|||
});
|
||||
|
||||
useCase('Change email', async () => {
|
||||
const subscription = await subscribe({
|
||||
const subscription = await subscriptionExistsPrecondition({
|
||||
email: generateEmail(),
|
||||
firstName: 'John',
|
||||
lastName: 'Doe'
|
||||
|
@ -212,7 +219,7 @@ suite('Subscription use-cases', function() {
|
|||
});
|
||||
|
||||
useCase('Unsubscription from list #1 (one-step, no form).', async () => {
|
||||
const subscription = await subscribe({
|
||||
const subscription = await subscriptionExistsPrecondition({
|
||||
email: generateEmail()
|
||||
});
|
||||
|
||||
|
@ -236,4 +243,6 @@ suite('Subscription use-cases', function() {
|
|||
useCase('Unsubscription from list #4 (two-step, with form).');
|
||||
|
||||
useCase('Unsubscription from list #5 (manual unsubscribe).');
|
||||
|
||||
useCase('Resubscription.'); // This one is supposed to check that values pre-filled in resubscription (i.e. the re-subscribe link in unsubscription confirmation) are the same as the ones used before.
|
||||
});
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue