unoplatform/uno

Layout unit tests should be migrated to runtime tests

Open

#15.703 aberto em 29 de fev. de 2024

Ver no GitHub
 (1 comment) (1 reaction) (1 assignee)C# (837 forks)batch import
area/tests 🧪difficulty/starter 🚀good first issuekind/bugproject/layout 🧱

Métricas do repositório

Stars
 (9.789 stars)
Métricas de merge de PR
 (Mesclagem média 14d 8h) (130 fundiu PRs em 30d)

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

Guia do colaborador

Direção de pesquisa
Estude os testes unitários de layout existentes e o framework de teste de tempo de execução no Uno. Identifique como converter testes unitários em testes de tempo de execução usando executores específicos da plataforma. Veja exemplos de testes de tempo de execução no repositório e migre os testes de layout de acordo.
Pilha de tecnologia
csharp
Domain
testing
Tipo Issue
Refatoração
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.
Ativo
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 testingXAML
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
40