flutter/flutter

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

Offen

#134.471 geöffnet am 11.09.2023

Auf GitHub ansehenContributor Guide
 (9 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)Dart (30.366 Forks)batch import
P2c: tech-debtgood first issueteam-enginetriaged-engine

Repository-Metriken

Stars
 (176.322 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 7T 7h) (491 gemergte PRs in 30 T)

Beschreibung

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.

Contributor Guide

Research-Richtung
Untersuche die flutter::Settings Struktur in switches.h und die Funktion SettingsFromCommandLine. Identifiziere, welche Einstellungen für Test Harnesses relevant sind. Erwäge die Aufteilung der Einstellungen in Basis und testspezifische Kategorien. Schau dir die vorhandenen Einstiegspunkte von Test Harnesses an, um die Befehlszeilenanalyse zu integrieren.
Tech Stack
cpp
Domain
testing
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Halber Tag
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Meist klar
Voraussetzungen
C++Flutter testing infrastructure
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
40