diff options
author | Winnie Hellmann <winnie@gitlab.com> | 2017-05-18 18:27:18 +0000 |
---|---|---|
committer | Winnie Hellmann <winnie@gitlab.com> | 2017-05-18 18:27:18 +0000 |
commit | fe3458e36b63686db5d7989e122a4a260d3fd2de (patch) | |
tree | 81b735229dfd3ba4ccd2d40dca816c13104f2877 /doc | |
parent | e4eec191565a053f687911d80865ad43241453f8 (diff) | |
download | gitlab-ce-fe3458e36b63686db5d7989e122a4a260d3fd2de.tar.gz |
Minor changes to Testing Promises sectionwinh-testing-promises-docspatch-20
Diffstat (limited to 'doc')
-rw-r--r-- | doc/development/fe_guide/testing.md | 13 |
1 files changed, 7 insertions, 6 deletions
diff --git a/doc/development/fe_guide/testing.md b/doc/development/fe_guide/testing.md index 5c6bdc97ef6..0ef9fc61a61 100644 --- a/doc/development/fe_guide/testing.md +++ b/doc/development/fe_guide/testing.md @@ -74,7 +74,7 @@ When testing Promises you should always make sure that the test is asynchronous Your Promise chain should therefore end with a call of the `done` callback and `done.fail` in case an error occurred. ```javascript -/// Good +// Good it('tests a promise', (done) => { promise .then((data) => { @@ -84,9 +84,10 @@ it('tests a promise', (done) => { .catch(done.fail); }); -/// Good +// Good it('tests a promise rejection', (done) => { promise + .then(done.fail) .catch((error) => { expect(error).toBe(expectedError); }) @@ -94,7 +95,7 @@ it('tests a promise rejection', (done) => { .catch(done.fail); }); -/// Bad (missing done callback) +// Bad (missing done callback) it('tests a promise', () => { promise .then((data) => { @@ -102,7 +103,7 @@ it('tests a promise', () => { }) }); -/// Bad (missing catch) +// Bad (missing catch) it('tests a promise', (done) => { promise .then((data) => { @@ -111,7 +112,7 @@ it('tests a promise', (done) => { .then(done) }); -/// Bad (use done.fail in asynchronous tests) +// Bad (use done.fail in asynchronous tests) it('tests a promise', (done) => { promise .then((data) => { @@ -121,7 +122,7 @@ it('tests a promise', (done) => { .catch(fail) }); -/// Bad (missing catch) +// Bad (missing catch) it('tests a promise rejection', (done) => { promise .catch((error) => { |