fix(dashboard): Charts standardmässig auf gesamten Zeitraum (MAX)#745
Merged
Conversation
Preis- und Portfolio-Chart initialisierten ihren State mit TimePeriod.threeMonths. Default auf TimePeriod.all umgestellt, sodass beide Charts beim Oeffnen den gesamten verfuegbaren Zeitraum zeigen. Veraltete Default-Referenzen in den Cubit-Tests-Kommentaren korrigiert. Das Dashboard-Golden wird auf dem Goldens-Runner regeneriert (geaenderter Perioden-Button-Highlight).
Regression-Guard je Cubit: ein frisch konstruierter Price-/PortfolioChartCubit muss selectedPeriod == TimePeriod.all haben. Faengt ein versehentliches Zurueckdrehen des Defaults, das die bestehenden selectPeriod-Tests nicht abdecken.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Änderung
Preis- und Portfolio-Chart auf dem Dashboard initialisierten ihren State mit
TimePeriod.threeMonths. Der Default ist jetztTimePeriod.all— beide Charts zeigen beim Öffnen den gesamten verfügbaren Zeitraum.lib/screens/dashboard/bloc/price_chart/price_chart_cubit.dartlib/screens/dashboard/bloc/portfolio_chart/portfolio_chart_cubit.dartTests
flutter analyzesauber.price_chart_cubit_test+portfolio_chart_cubit_test: 24/24 grün (alle Tests setzen die Periode explizit viaselectPeriod; der einzige Default-abhängige Test bleibt grün, da seine Datenpunkte auch unterallsichtbar sind).Goldens
Das Dashboard-Golden ändert sich (hervorgehobener Perioden-Button wechselt 3M → MAX). Baselines werden auf dem self-hosted Goldens-Runner via
golden-regenerate.yamlregeneriert und vom Bot auf diesen Branch committet.