flutter/flutter

Respect flutter::Settings from command line arguments specified to test harnesses.

Ouverte

#134 471 ouverte le 11 sept. 2023

Voir sur GitHubGuide contributeur
 (9 commentaires) (0 réaction) (0 personne assignée)Dart (30 366 forks)batch import
P2c: tech-debtgood first issueteam-enginetriaged-engine

Métriques du dépôt

Stars
 (176 322 étoiles)
Métriques de merge PR
 (Merge moyen 7j 7h) (491 PRs mergées en 30 j)

Description

In //flutter/common, there are flutter::Settings in switches.h that can be read from the command line (via flutter::SettingsFromCommandLine). However, these settings are not respected in any of the test harnesses in //flutter/testing. Now, not all the settings are valid in unit-tests. So these likely need to be split apart into base settings for all exectuables. But stuff like log control, log tags, and such should be configurable from the command line in unit-tests.

Guide contributeur

Direction de recherche
Examinez la structure flutter::Settings dans switches.h et la fonction SettingsFromCommandLine. Identifiez les paramètres pertinents pour les harnais de test. Envisagez de diviser les paramètres en catégories de base et spécifiques aux tests. Regardez les points d'entrée des harnais de test existants pour intégrer l'analyse de la ligne de commande.
Stack technique
cpp
Domaine
testing
Type d'issue
Fonctionnalité
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.
Une demi journée
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.
Plutôt claire
Prérequis
C++Flutter testing infrastructure
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
40