chore: fixed UI test

SchweizerischeBundesbahnen · Nov 28, 2024 · b5e9c8e · b5e9c8e
1 parent e45884d
commit b5e9c8e
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 12 deletions.
diff --git a/das_client/lib/app/pages/journey/train_journey/widgets/table/base_row_builder.dart b/das_client/lib/app/pages/journey/train_journey/widgets/table/base_row_builder.dart
@@ -44,11 +44,11 @@ class BaseRowBuilder<T extends BaseData> extends DASTableRowBuilder {
 
   DASTableCell kilometreCell(BuildContext context) {
     if (data.kilometre.isEmpty) {
-      return DASTableCell.empty();
+      return DASTableCell.empty(color: specialCellColor);
     }
 
     return DASTableCell(
-        color: getSpecialCellColor(),
+        color: specialCellColor,
         child: Column(
           mainAxisAlignment: MainAxisAlignment.end,
           crossAxisAlignment: CrossAxisAlignment.start,
@@ -62,7 +62,7 @@ class BaseRowBuilder<T extends BaseData> extends DASTableRowBuilder {
 
   DASTableCell routeCell(BuildContext context) {
     return DASTableCell(
-      color: getSpecialCellColor(),
+      color: specialCellColor,
       padding: EdgeInsets.all(0.0),
       alignment: null,
       child: RouteCellBody(
@@ -74,7 +74,7 @@ class BaseRowBuilder<T extends BaseData> extends DASTableRowBuilder {
   }
 
   DASTableCell timeCell(BuildContext context) {
-    return DASTableCell.empty();
+    return DASTableCell.empty(color: specialCellColor);
   }
 
   DASTableCell informationCell(BuildContext context) {
@@ -112,11 +112,8 @@ class BaseRowBuilder<T extends BaseData> extends DASTableRowBuilder {
     return DASTableCell.empty();
   }
 
-  Color? getSpecialCellColor() {
-    return getAdditionalSpeedRestriction() != null
-        ? AdditionalSpeedRestrictionRow.additionalSpeedRestrictionColor
-        : null;
-  }
+  Color? get specialCellColor =>
+      getAdditionalSpeedRestriction() != null ? AdditionalSpeedRestrictionRow.additionalSpeedRestrictionColor : null;
 
   AdditionalSpeedRestriction? getAdditionalSpeedRestriction() {
     return metadata.additionalSpeedRestrictions

diff --git a/das_client/lib/app/pages/journey/train_journey/widgets/table/service_point_row.dart b/das_client/lib/app/pages/journey/train_journey/widgets/table/service_point_row.dart
@@ -38,7 +38,7 @@ class ServicePointRow extends BaseRowBuilder<ServicePoint> {
 
   @override
   DASTableCell timeCell(BuildContext context) {
-    return DASTableCell(child: Text('06:05:52'), alignment: defaultAlignment, color: getSpecialCellColor());
+    return DASTableCell(child: Text('06:05:52'), alignment: defaultAlignment, color: specialCellColor);
   }
 
   @override
@@ -68,7 +68,7 @@ class ServicePointRow extends BaseRowBuilder<ServicePoint> {
   @override
   DASTableCell routeCell(BuildContext context) {
     return DASTableCell(
-      color: getSpecialCellColor(),
+      color: specialCellColor,
       padding: EdgeInsets.all(0.0),
       alignment: null,
       child: RouteCellBody(

diff --git a/das_client/lib/app/widgets/table/das_table_cell.dart b/das_client/lib/app/widgets/table/das_table_cell.dart
@@ -18,7 +18,7 @@ class DASTableCell {
     this.clipBehaviour = Clip.hardEdge,
   });
 
-  const DASTableCell.empty() : this(child: const SizedBox.shrink());
+  const DASTableCell.empty({Color? color}) : this(child: const SizedBox.shrink(), color: color);
 
   final BoxBorder? border;
   final Widget child;