Commit 93632618 authored by Rene Saarsoo's avatar Rene Saarsoo
Browse files

Remove the separation of error/failure status. 

As the Assert class was removed, this code is currently not needed.
parent 2eaa14ba

template/app/view/doctests/Index.js

+3 −9
Original line number Diff line number Diff line
@@ -243,7 +243,7 @@ Ext.define('Docs.view.doctests.Index', { 
     */

    onPreviewSuccess: function(preview, options, record, config) {

        this.clearTestRunner();

        record.set('status', '<span class="doc-test-success">pass</span>');

        record.set('status', '<span class="doc-test-success">success</span>');

        record.commit();

        config.pass++;

        this.showResult(config);
@@ -265,14 +265,8 @@ Ext.define('Docs.view.doctests.Index', { 
     */

    onPreviewFailure: function(preview, e, obj, record, config) {

        this.clearTestRunner();



        if (e.docAssertFailed) {

            record.set('status', '<span class="doc-test-failure">fail</span>');

        } else {

            record.set('status', '<span class="doc-test-error">error</span>');

        }



        record.set('message', '(exception logged to console): ' + e.toString());

        record.set('status', '<span class="doc-test-failure">failure</span>');

        record.set('message', e.toString());

        record.commit();

        config.fail++;

        this.showResult(config);

template/resources/sass/_doc_tests.scss

+0 −4
Original line number Diff line number Diff line
@@ -3,10 +3,6 @@ 
  font-weight: bold;

  color: #333; }



.doc-test-error {

  font-weight: bold;

  color: orange; }



.doc-test-failure {

  font-weight: bold;

  color: red; }