dotnet/winforms

Add unit test for Link label

Open

#10 679 ouverte le 17 janv. 2024

Voir sur GitHub
 (6 commentaires) (1 réaction) (1 assigné)C# (922 forks)batch import
help wantedtest-enhancement

Métriques du dépôt

Stars
 (4 100 stars)
Métriques de merge PR
 (Merge moyen 14j 22h) (56 PRs mergées en 30 j)

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

Guide contributeur

Direction de recherche
Passez en revue les modifications apportées dans la PR #10642 pour comprendre la correction. Créez une nouvelle classe ou méthode de test unitaire pour LinkLabel qui teste les scénarios avec une propriété Text nulle ou vide. Utilisez l'infrastructure de test WinForms (par exemple, xUnit, modèles de test unitaire WinForms) pour simuler les conditions sans Application.DoEvents(). Assurez vous que le test réussisse sans boucle infinie.
Stack technique
csharp
Domaine
desktop dev
Type d'issue
Test
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 heures
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Git.NET SDK
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75