Merge pull request #4 from adityadroid/ConversationUI-Test

Tests for Conversation Page
adityadroid · Aug 17, 2019 · 3f59b1b · 3f59b1b
2 parents 0fe61cf + fb04bed
commit 3f59b1b
Show file tree

Hide file tree

Showing 13 changed files with 235 additions and 99 deletions.
diff --git a/lib/pages/ConversationPage.dart b/lib/pages/ConversationPage.dart
@@ -4,8 +4,10 @@ import 'package:messio/widgets/ChatListWidget.dart';
 import 'package:messio/widgets/InputWidget.dart';
 
 class ConversationPage extends StatefulWidget {
+
   @override
   _ConversationPageState createState() => _ConversationPageState();
+  const ConversationPage();
 }
 
 class _ConversationPageState extends State<ConversationPage> {

diff --git a/lib/pages/ConversationPageList.dart b/lib/pages/ConversationPageList.dart
@@ -6,6 +6,8 @@ class ConversationPageList extends StatefulWidget {
 
   @override
   _ConversationPageListState createState() => _ConversationPageListState();
+
+ const ConversationPageList();
 }
 
 class _ConversationPageListState extends State<ConversationPageList> {
@@ -20,6 +22,5 @@ class _ConversationPageListState extends State<ConversationPageList> {
       ],
     );
 
-
   }
 }
diff --git a/lib/widgets/ChatAppBar.dart b/lib/widgets/ChatAppBar.dart
@@ -4,96 +4,113 @@ import 'package:messio/config/Palette.dart';
 
 class ChatAppBar extends StatelessWidget implements PreferredSizeWidget {
   final double height = 100;
+
+  const ChatAppBar();
+
   @override
   Widget build(BuildContext context) {
-    var textHeading = TextStyle(color: Palette.primaryTextColor, fontSize: 20); // Text style for the name
-    var textStyle = TextStyle(color: Palette.secondaryTextColor); // Text style for everything else
+    var textHeading = TextStyle(
+        color: Palette.primaryTextColor,
+        fontSize: 20); // Text style for the name
+    var textStyle = TextStyle(
+        color: Palette.secondaryTextColor); // Text style for everything else
 
-    double width = MediaQuery.of(context).size.width; // calculate the screen width
     return Material(
         child: Container(
-            decoration: new BoxDecoration(boxShadow: [ //adds a shadow to the appbar
+            decoration: new BoxDecoration(boxShadow: [
+              //adds a shadow to the appbar
               new BoxShadow(
-              color: Colors.black,
-              blurRadius: 5.0,
-            )]),
-            child:  Container(
+                color: Colors.black,
+                blurRadius: 5.0,
+              )
+            ]),
+            child: Container(
+                padding: EdgeInsets.only(top: 10, bottom: 10),
                 color: Palette.primaryBackgroundColor,
                 child: Row(children: <Widget>[
-              Expanded( //we're dividing the appbar into 7 : 3 ratio. 7 is for content and 3 is for the display picture.
-                  flex: 7,
-                  child: Center(
-                      child: Column(
-                    mainAxisSize: MainAxisSize.min,
-                    children: <Widget>[
-                      Container(
-                          height: 70 - (width * .06),
-                          child: Row(
-                            crossAxisAlignment: CrossAxisAlignment.start,
-                            children: <Widget>[
-                              Expanded(
-                                  flex: 2,
-                                  child: Center(
-                                      child: Icon(
-                                    Icons.attach_file,
-                                    color: Palette.secondaryColor,
-                                  ))),
-                              Expanded(
-                                  flex: 6,
-                                  child: Container(
-                                      child: Column(
+                  Expanded(
+                      //we're dividing the appbar into 7 : 3 ratio. 7 is for content and 3 is for the display picture.
+                      flex: 7,
+                      child: Center(
+                          child: Column(
+                        mainAxisSize: MainAxisSize.min,
+                        children: <Widget>[
+                          Expanded(
+                              flex: 7,
+                              child: Container(
+                                  child: Row(
+                                crossAxisAlignment: CrossAxisAlignment.start,
+                                children: <Widget>[
+                                  Expanded(
+                                      flex: 2,
+                                      child: Center(
+                                          child: IconButton(
+                                              icon: Icon(
+                                                Icons.attach_file,
+                                                color: Palette.secondaryColor,
+                                              ),
+                                              onPressed: ()=>{}))),
+                                  Expanded(
+                                      flex: 6,
+                                      child: Container(
+                                          child: Column(
+                                        crossAxisAlignment:
+                                            CrossAxisAlignment.start,
+                                        mainAxisAlignment:
+                                            MainAxisAlignment.center,
+                                        mainAxisSize: MainAxisSize.min,
+                                        children: <Widget>[
+                                          Text('Aditya Gurjar',
+                                              style: textHeading),
+                                          Text('@adityagurjar',
+                                              style: textStyle)
+                                        ],
+                                      ))),
+                                ],
+                              ))),
+                          //second row containing the buttons for media
+                          Expanded(
+                              flex: 3,
+                              child: Container(
+                                  padding: EdgeInsets.fromLTRB(20, 5, 5, 0),
+                                  child: Row(
+                                    mainAxisAlignment: MainAxisAlignment.start,
                                     crossAxisAlignment:
-                                        CrossAxisAlignment.start,
-                                    mainAxisAlignment: MainAxisAlignment.center,
-                                    mainAxisSize: MainAxisSize.min,
+                                        CrossAxisAlignment.center,
                                     children: <Widget>[
-                                      Text('Aditya Gurjar', style: textHeading),
-                                      Text('@adityagurjar', style: textStyle)
+                                      Text(
+                                        'Photos',
+                                        style: textStyle,
+                                      ),
+                                      VerticalDivider(
+                                        width: 30,
+                                        color: Palette.primaryTextColor,
+                                      ),
+                                      Text(
+                                        'Videos',
+                                        style: textStyle,
+                                      ),
+                                      VerticalDivider(
+                                        width: 30,
+                                        color: Palette.primaryTextColor,
+                                      ),
+                                      Text('Files', style: textStyle)
                                     ],
                                   ))),
-                            ],
-                          )),
-                      //second row containing the buttons for media
-                      Container(
-                          height: 23,
-                          padding: EdgeInsets.fromLTRB(20, 5, 5, 0),
-                          child: Row(
-                            mainAxisAlignment: MainAxisAlignment.start,
-                            crossAxisAlignment: CrossAxisAlignment.center,
-                            children: <Widget>[
-                              Text(
-                                'Photos',
-                                style: textStyle,
-                              ),
-                              VerticalDivider(
-                                width: 30,
-                                color: Palette.primaryTextColor,
-                              ),
-                              Text(
-                                'Videos',
-                                style: textStyle,
-                              ),
-                              VerticalDivider(
-                                width: 30,
-                                color: Palette.primaryTextColor,
-                              ),
-                              Text('Files', style: textStyle)
-                            ],
-                          )),
-                    ],
-                  ))),
-                //This is the display picture
-              Expanded(
-                  flex: 3,
-                  child: Container(
-                      child: Center(
-                          child: CircleAvatar(
-                    radius: (80 - (width * .06)) / 2,
-                    backgroundImage: Image.asset(
-                      Assets.user,
-                    ).image,
-                  )))),
-            ]))));
+                        ],
+                      ))),
+                  //This is the display picture
+                  Expanded(
+                      flex: 3,
+                      child: Container(
+                          child: Center(
+                              child: CircleAvatar(
+                        radius: 30,
+                        backgroundImage: Image.asset(
+                          Assets.user,
+                        ).image,
+                      )))),
+                ]))));
   }
 
   @override

diff --git a/lib/widgets/ChatItemWidget.dart b/lib/widgets/ChatItemWidget.dart
@@ -3,9 +3,9 @@ import 'package:messio/config/Palette.dart';
 import 'package:intl/intl.dart';
 
 class ChatItemWidget extends StatelessWidget{
-  var index;
+  final int index;
 
-  ChatItemWidget(this.index);
+ const ChatItemWidget(this.index);
 
   @override
   Widget build(BuildContext context) {

diff --git a/lib/widgets/InputWidget.dart b/lib/widgets/InputWidget.dart
@@ -3,7 +3,10 @@ import 'package:messio/config/Palette.dart';
 
 class InputWidget extends StatelessWidget {
 
-  final TextEditingController textEditingController = new TextEditingController();
+  final TextEditingController textEditingController = TextEditingController();
+
+
+   InputWidget();
 
   @override
   Widget build(BuildContext context) {
@@ -16,6 +19,7 @@ class InputWidget extends StatelessWidget {
               child: new IconButton(
                 icon: new Icon(Icons.face),
                 color: Palette.primaryColor,
+                onPressed: ()=>{},
               ),
             ),
             color: Colors.white,

diff --git a/test/main_test.dart b/test/main_test.dart
@@ -0,0 +1,14 @@
+
+import 'package:flutter_test/flutter_test.dart';
+import 'package:messio/main.dart';
+import 'package:messio/pages/ConversationPageList.dart';
+
+void main() {
+  testWidgets('Main UI Test', (WidgetTester tester) async {
+    // Build our app and trigger a frame.
+    await tester.pumpWidget(Messio());
+
+    expect(find.byType(ConversationPageList),findsOneWidget);
+
+  });
+}
diff --git a/test/pages/ConversationPageList_test.dart b/test/pages/ConversationPageList_test.dart
@@ -0,0 +1,19 @@
+import 'package:flutter/material.dart';
+import 'package:flutter_test/flutter_test.dart';
+import 'package:messio/pages/ConversationPage.dart';
+import 'package:messio/pages/ConversationPageList.dart';
+
+void main(){
+  const MaterialApp app = MaterialApp(
+    home: Scaffold(
+        body:  const ConversationPageList()
+    ),
+  );
+  testWidgets('ConversationPageList UI Test', (WidgetTester tester) async {
+    // Build our app and trigger a frame.
+    await tester.pumpWidget(app);
+    expect(find.byType(ConversationPage),findsOneWidget);
+    expect(find.byType(PageView),findsOneWidget);
+
+  });
+}
diff --git a/test/pages/ConversationPage_test.dart b/test/pages/ConversationPage_test.dart
@@ -0,0 +1,25 @@
+import 'package:flutter/material.dart';
+import 'package:flutter_test/flutter_test.dart';
+import 'package:messio/pages/ConversationPage.dart';
+import 'package:messio/widgets/ChatAppBar.dart';
+import 'package:messio/widgets/ChatListWidget.dart';
+import 'package:messio/widgets/InputWidget.dart';
+
+void main(){
+  const MaterialApp app = MaterialApp(
+    home: Scaffold(
+      body:  const ConversationPage()
+    ),
+  );
+
+  testWidgets('ConversationPage UI Test', (WidgetTester tester) async {
+    // Build our app and trigger a frame.
+    await tester.pumpWidget(app);
+
+    expect(find.byType(ChatAppBar),findsOneWidget);
+    expect(find.byType(InputWidget),findsOneWidget);
+    expect(find.byType(ChatListWidget),findsOneWidget);
+
+
+  });
+}
diff --git a/test/widget_test.dart b/test/widget_test.dart
diff --git a/test/widgets/ChatAppBar_test.dart b/test/widgets/ChatAppBar_test.dart
@@ -0,0 +1,20 @@
+import 'package:flutter/material.dart';
+import 'package:flutter_test/flutter_test.dart';
+import 'package:messio/widgets/ChatAppBar.dart';
+
+void main(){
+  const MaterialApp app = MaterialApp(
+    home: Scaffold(
+        body:  const ChatAppBar()
+    ),
+  );
+  testWidgets('ChatAppBar UI Test', (WidgetTester tester) async {
+    // Build our app and trigger a frame.
+    await tester.pumpWidget(app);
+
+    expect(find.text('Aditya Gurjar'), findsOneWidget);
+    expect(find.text('@adityagurjar'), findsOneWidget);
+    expect(find.byType(IconButton),findsNWidgets(1));
+    expect(find.byType(CircleAvatar),findsOneWidget);
+  });
+}
diff --git a/test/widgets/ChatItemWidget_test.dart b/test/widgets/ChatItemWidget_test.dart
@@ -0,0 +1,20 @@
+import 'package:flutter/material.dart';
+import 'package:flutter_test/flutter_test.dart';
+import 'package:messio/widgets/ChatItemWidget.dart';
+
+void main(){
+  const MaterialApp app = MaterialApp(
+    home: Scaffold(
+        body:  const ChatItemWidget(0)
+    ),
+  );
+  testWidgets('ChatItemWidget UI Test', (WidgetTester tester) async {
+    // Build our app and trigger a frame.
+    await tester.pumpWidget(app);
+
+    expect(find.byType(Container),findsNWidgets(3));
+    expect(find.byType(Column),findsNWidgets(1));
+    expect(find.byType(Row),findsNWidgets(2));
+    expect(find.byType(Text),findsNWidgets(2));
+  });
+}
diff --git a/test/widgets/ChatListWidget_test.dart b/test/widgets/ChatListWidget_test.dart
@@ -0,0 +1,12 @@
+import 'package:flutter/material.dart';
+import 'package:flutter_test/flutter_test.dart';
+import 'package:messio/main.dart';
+
+void main(){
+  testWidgets('ChatListWidget UI Test', (WidgetTester tester) async {
+    // Build our app and trigger a frame.
+    await tester.pumpWidget(Messio());
+
+    expect(find.byType(ListView),findsOneWidget);
+  });
+}