gratipay / grtp.co

Gratipay Widgets + API
https://grtp.co/
MIT License
26 stars 19 forks source link

Rename tests to identify which test file they are in #113

Closed mattbk closed 6 years ago

mattbk commented 8 years ago

Current tests give output that doesn't help find the problem. Suggest adding identifier to the test name that is output.

E.g., below could be 'default-widget: readystatus should be ready': function(test) {

module.exports = {

    'readystatus should be ready': function(test) {
        test.open('http://localhost:9537/test/default-widget.html')
            .waitFor(function() {
                return !!document.querySelector('[data-gratipay-readystatus="ready"]');
            }, [], 2000)
            .assert.attr('[data-gratipay-username]', 'data-gratipay-readystatus').is('ready', 'readystatus should be ready')
            .done();
    },

    'it should fill the standard values': function(test) {
        test.open('http://localhost:9537/test/default-widget.html')
            .waitFor(function() {
                return !!document.querySelector('[data-gratipay-readystatus="ready"]');
            }, [], 2000)
            .assert.exists('iframe')
            .toFrame('iframe')
                .assert.exists('.gratipay-identity')
                .assert.text('.gratipay-identity').is.not('', 'identity should not be empty')
                .assert.exists('.gratipay-receiving')
                .assert.text('.gratipay-receiving').is.not('', 'receiving should not be empty')
            .toParent()
            .done();
    },

};
mattbk commented 8 years ago

Closed in https://github.com/gratipay/grtp.co/commit/1166a521db910d04af0a18f67efe1819e938254d.

mattbk commented 8 years ago

Oops, should be https://github.com/gratipay/grtp.co/pull/132 now.