karma-runner/karma

logger.js ignores layout.pattern

Open

#2 414 ouverte le 19 oct. 2016

Voir sur GitHub
 (1 commentaire) (0 réactions) (0 assignés)JavaScript (11 918 stars) (1 703 forks)batch import
help wantedneeds: investigation

Description

Expected behaviour

In karma/lib/logger.js, setup should use the layout.pattern string if it exists.

Actual behaviour

When layout.type === 'pattern', setup only uses one of the built-in constants:

    var pattern = colors ? constant.COLOR_PATTERN : constant.NO_COLOR_PATTERN

Environment Details

  • Karma version (output of karma --version): 1.3.0
  • Relevant part of your karma.config.js file
    loggers: [
        {
            type: 'console',
            layout: {
                type: 'pattern',
                pattern: '%[%d{ISO8601}:%p [%c]: %]%m'
            }
        }
    ]

Steps to reproduce the behaviour

  1. Supply a custom logger in karma.config.js as specified above.

Comment

If this is indeed broken, I can submit a PR.

Further, I realize that fixing this to use the provided pattern will then ignore the colors boolean. This might be addressed by having layout.colorPattern and layout.noColorPattern which get assigned appropriately to layout.pattern. I can submit a PR for this as well.

Thanks.

Guide contributeur

Stack technique
javascriptnodejs
Domaine
testing
Type d'issue
bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 hours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
stale
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
Node.jsKarma configuration
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70
Direction de recherche
Examine `karma/lib/logger.js` line 23 and the `setup` function. Understand how `layout.type === 'pattern'` is handled and why the user provided `pattern` string is ignored. Consider implementing a fix that uses `layout.pattern` when `layout.type === 'pattern`, and optionally handle the `colors` boolean via separate properties as suggested in the issue comments. See the issue comment by the reporter for additional details.