dotnet/winforms

Add unit test for Link label

Open

#10.679 aberto em 17 de jan. de 2024

Ver no GitHub
 (6 comments) (1 reaction) (1 assignee)C# (922 forks)batch import
help wantedtest-enhancement

Métricas do repositório

Stars
 (4.100 stars)
Métricas de merge de PR
 (Mesclagem média 14d 22h) (56 fundiu PRs em 30d)

Description

Add unit tests for PR#10642

Using the following code in a .net core project reproduces issue #10515,

  • When the linkLabel.Text is null or empty, the code Application.DoEvents(); will enter an infinite loop before PR#10642
 LinkLabel linkLabel = new();
 // try to comment out  this line 'linkLabel.Text = "2";'
 // and observe the result after click the button
 linkLabel.Text = "";

this.Controls.Add(linkLabel);
this.Invalidate();
Application.DoEvents(); 

MessageBox.Show("button1_Click");

However, Application.DoEvents(); is invalid when used in Unit test, so additional tests need to be added for PR#https://github.com/dotnet/winforms/pull/10642

Guia do colaborador

Direção de pesquisa
Revise as alterações no PR #10642 para entender a correção. Crie uma nova classe ou método de teste unitário para LinkLabel que teste cenários com a propriedade Text nula ou vazia. Use a infraestrutura de teste do WinForms (ex.: xUnit, padrões de teste unitário do WinForms) para simular as condições sem Application.DoEvents(). Garanta que o teste passe sem loop infinito.
Pilha de tecnologia
csharp
Domain
desktop dev
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.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
Git.NET SDK
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
75