chore" fixed comments

VGVentures · Nov 22, 2023 · 29ff973 · 29ff973
1 parent bb43f55
commit 29ff973
Show file tree

Hide file tree

Showing 6 changed files with 29 additions and 25 deletions.
diff --git a/lib/home/widgets/logo.dart b/lib/home/widgets/logo.dart
@@ -20,7 +20,7 @@ class Logo extends StatelessWidget {
           ),
         ),
         const SizedBox(width: 4),
-        VertexIcons.asterisk.image(),
+        vertexIcons.asterisk.image(),
         const SizedBox(width: 4),
         Text(
           l10n.flutter,

diff --git a/lib/home/widgets/welcome_view.dart b/lib/home/widgets/welcome_view.dart
@@ -25,7 +25,7 @@ class WelcomeView extends StatelessWidget {
             ),
             const SizedBox(height: 40),
             CTAButton(
-              icon: VertexIcons.arrowForward.image(),
+              icon: vertexIcons.arrowForward.image(),
               label: l10n.startAsking,
             ),
           ],

diff --git a/packages/app_ui/lib/app_ui.dart b/packages/app_ui/lib/app_ui.dart
@@ -3,5 +3,4 @@ import 'package:app_ui/src/generated/assets.gen.dart';
 export 'src/widgets/widgets.dart';
 
 /// Global reference to actual app_ui icons.
-// ignore: constant_identifier_names
-const VertexIcons = Assets.icons;
+const vertexIcons = Assets.icons;
diff --git a/packages/app_ui/test/src/helpers/helpers.dart b/packages/app_ui/test/src/helpers/helpers.dart
@@ -0,0 +1 @@
+export 'pump_app.dart';
diff --git a/packages/app_ui/test/src/helpers/pump_app.dart b/packages/app_ui/test/src/helpers/pump_app.dart
@@ -0,0 +1,12 @@
+import 'package:flutter/material.dart';
+import 'package:flutter_test/flutter_test.dart';
+
+extension PumpApp on WidgetTester {
+  Future<void> pumpApp(Widget widget) {
+    return pumpWidget(
+      MaterialApp(
+        home: widget,
+      ),
+    );
+  }
+}
diff --git a/packages/app_ui/test/src/widgets/cta_button_test.dart b/packages/app_ui/test/src/widgets/cta_button_test.dart
@@ -3,21 +3,15 @@ import 'package:app_ui/src/generated/assets.gen.dart';
 import 'package:flutter/material.dart';
 import 'package:flutter_test/flutter_test.dart';
 
+import '../helpers/helpers.dart';
+
 void main() {
   group('CTAButton', () {
-    Widget bootstrap({
-      required Widget child,
-    }) {
-      return MaterialApp(home: child);
-    }
-
     testWidgets('renders correctly', (tester) async {
-      await tester.pumpWidget(
-        bootstrap(
-          child: CTAButton(
-            icon: Assets.icons.arrowForward.image(),
-            label: 'label',
-          ),
+      await tester.pumpApp(
+        CTAButton(
+          icon: Assets.icons.arrowForward.image(),
+          label: 'label',
         ),
       );
 
@@ -27,15 +21,13 @@ void main() {
     testWidgets('calls onPressed when tap', (tester) async {
       var called = false;
 
-      await tester.pumpWidget(
-        bootstrap(
-          child: CTAButton(
-            icon: Assets.icons.arrowForward.image(),
-            label: 'label',
-            onPressed: () {
-              called = true;
-            },
-          ),
+      await tester.pumpApp(
+        CTAButton(
+          icon: Assets.icons.arrowForward.image(),
+          label: 'label',
+          onPressed: () {
+            called = true;
+          },
         ),
       );