microsoft/Terminal

Validate that VT adapter tests cover all mock possibilities

Open

#987 aberto em 24 de mai. de 2019

Ver no GitHub
 (0 comments) (0 reactions) (0 assignees)C++ (3.212 forks)batch import
Area-CodeHealthArea-VTHelp WantedIssue-TaskProduct-Conhost

Métricas do repositório

Stars
 (35.764 stars)
Métricas de merge de PR
 (Mesclagem média 27d 19h) (24 fundiu PRs em 30d)

Description

All of the various mock definitions don't seem to be in use. When I stood up the original parser years ago and wrote the mock, every mock was checked somehow. Over time, it probably decayed and the only thing left is that the mock has to be implemented to satisfy compilation of the interface.

Originally posted by @miniksa in https://github.com/microsoft/terminal/pull/891/review_comment/create

The goal here would be to run through all the tests and ensure that the mocks are involved somehow in each operation to validate that we're getting what we expect back through the adapter interface for all potential results.

Guia do colaborador

Direção de pesquisa
Examine o conjunto de testes do adaptador VT (provavelmente em src/terminal/adapter/) para identificar todas as classes mock. Para cada mock, verifique se todos os seus métodos são exercitados nos testes existentes. Escreva testes adicionais para cobrir as invocações de métodos mock ausentes. Revise quaisquer alterações recentes no adaptador para garantir que novos mocks também sejam testados.
Pilha de tecnologia
None
Domain
testing
Tipo Issue
Teste
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-2 dias
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Recente
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
C++Unit testing
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
50