unoplatform/uno

Layout unit tests should be migrated to runtime tests

Open

#15 703 ouverte le 29 févr. 2024

Voir sur GitHub
 (1 commentaire) (1 réaction) (1 assigné)C# (837 forks)batch import
area/tests 🧪difficulty/starter 🚀good first issuekind/bugproject/layout 🧱

Métriques du dépôt

Stars
 (9 789 stars)
Métriques de merge PR
 (Merge moyen 14j 8h) (130 PRs mergées en 30 j)

Description

Current behavior

Using unit tests for layout-related stuff isn't a good idea for the following reasons:

  1. They are run against a "mock" layouting implementation, which can be very different from the actual implementation. So, results can be very deceiving.
  2. For some scenarios, we could be having different behavior on different platforms. We definitely want to catch those so that we are aware of them. Having the test to verify the actual behavior on all supported platforms is a lot better.

Expected behavior

Migrate to runtime tests

How to reproduce it (as minimally and precisely as possible)

No response

Workaround

No response

Works on UWP/WinUI

None

Environment

No response

NuGet package version(s)

No response

Affected platforms

No response

IDE

No response

IDE version

No response

Relevant plugins

No response

Anything else we need to know?

No response

Guide contributeur

Direction de recherche
Étudiez les tests unitaires de mise en page existants et le framework de test d'exécution dans Uno. Identifiez comment convertir les tests unitaires en tests d'exécution à l'aide d'exécuteurs spécifiques à la plateforme. Regardez des exemples de tests d'exécution dans le dépôt, puis migrez les tests de mise en page en conséquence.
Stack technique
csharp
Domaine
testing
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-2 jours
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
C#Unit testingXAML
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
40