karma-runner/karma

script-launcher : bad script browser recognition

Open

#754 aberto em 23 de set. de 2013

Ver no GitHub
 (2 comments) (0 reactions) (0 assignees)JavaScript (1.703 forks)batch import
help wantedtype: backlogtype: feature

Métricas do repositório

Stars
 (11.918 stars)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

Description

Hi all, just wanted to note that after updating to latest karma (0.10.2) , if someone's got problem with "script browsers" ( we use a .bat to run IE tester with specific IE versions) , it could be because of the "browser type recognition". It gets recognized as "script browser" only if it's path it's absolute

lib\launcher.js line 19

 // TODO(vojta): determine script from name
      if (name.indexOf('/') !== -1) {
        name = 'Script';
      }

we just put a temporary work-around like this

 // TODO(vojta): determine script from name
      if (name.indexOf('/') !== -1  || name.indexOf('.bat') !== -1) {
        name = 'Script';
      }

Guia do colaborador

Direção de pesquisa
Modifique a lógica de detecção do navegador de script em lib/launcher.js para também reconhecer caminhos que terminam com .bat (ou outras extensões de script) se não forem absolutos.
Pilha de tecnologia
javascript
Domain
testing
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Menos de 1 hora
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Antigo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
Git
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
80