diff options
| author | Peter Bacon Darwin | 2014-02-20 10:57:40 +0000 | 
|---|---|---|
| committer | Peter Bacon Darwin | 2014-02-20 14:41:32 +0000 | 
| commit | d7a73e41eddf3b8f06b87535c5ba6cc3196cbffd (patch) | |
| tree | 9c73026320b1a17b000eed70500ac02e7a98faf9 /test | |
| parent | e6c35e1b6b22acbf45abb883bf254b7191a42a38 (diff) | |
| download | angular.js-d7a73e41eddf3b8f06b87535c5ba6cc3196cbffd.tar.bz2 | |
chore(doc-gen): add formatted error messages to error pages
This got missed in the doc migration: When there is an error in an
Angular app, extra information is placed in the URL, which can be used
by the docs application to display a more useful message.
This fix adds that back in.  The error message templates are extracted
by the minerr tool during build and put into the errors.json file. The
errors-doc processor will load this up and attach these message templates
to the error docs.
The display of these templates was already in place, via the errorDisplay
directive in docs/app/js/errors.js.
(Also, moved the error.template.html file into the angular.js repository
from the dgeni-packages repository as this is specific to the angular.js
project and all the other error related stuff is in here.
Finally, also, added an e2e test that checks that minerr formatted
messages are displayed correctly.
Closes #6363
Diffstat (limited to 'test')
| -rw-r--r-- | test/e2e/docsAppE2E.js | 9 | 
1 files changed, 9 insertions, 0 deletions
| diff --git a/test/e2e/docsAppE2E.js b/test/e2e/docsAppE2E.js index da0726b8..8fe30c38 100644 --- a/test/e2e/docsAppE2E.js +++ b/test/e2e/docsAppE2E.js @@ -39,22 +39,31 @@ describe('docs.angularjs.org', function () {        expect(code.getText()).toContain('guest!!!');      }); +      it('should be resilient to trailing slashes', function() {        browser.get('index-debug.html#!/api/ng/function/angular.noop/');        var pageBody = element(by.css('h1'));        expect(pageBody.getText()).toEqual('angular.noop');      }); +      it('should be resilient to trailing "index"', function() {        browser.get('index-debug.html#!/api/ng/function/angular.noop/index');        var pageBody = element(by.css('h1'));        expect(pageBody.getText()).toEqual('angular.noop');      }); +      it('should be resilient to trailing "index/"', function() {        browser.get('index-debug.html#!/api/ng/function/angular.noop/index/');        var pageBody = element(by.css('h1'));        expect(pageBody.getText()).toEqual('angular.noop');      }); + + +    it('should display formatted error messages on error doc pages', function() { +      browser.get('index-debug.html#!error/ng/areq?p0=Missing&p1=not%20a%20function,%20got%20undefined'); +      expect(element(by.css('.minerr-errmsg')).getText()).toEqual("Argument 'Missing' is not a function, got undefined"); +    });    });  });
\ No newline at end of file | 
