2022-06-01 12:50:16 +02:00
6 changed files with 202 additions and 243 deletions
--- a/lib/ListViewPage.dart
+++ b/lib/ListViewPage.dart
@ -11,10 +11,11 @@ class ListViewPage extends StatefulWidget {
  const ListViewPage({Key? key}) : super(key: key);

  @override
-  State<ListViewPage> createState() => _ListViewPageState();
+  State<ListViewPage> createState() => ListViewPageState();
 }

-class _ListViewPageState extends State<ListViewPage> {
+@visibleForTesting
+class ListViewPageState extends State<ListViewPage> {
  final List<Venue> allVenues = globals.VENUES;

  @override
--- a/lib/Map.dart
+++ b/lib/Map.dart
@ -1,22 +1,12 @@
-import 'dart:convert';
 import 'package:flutter/material.dart';
 import 'package:flutter_applicationdemo/BottomNavPage.dart';
-import 'package:flutter_applicationdemo/ListViewPage.dart';
-import 'package:flutter_applicationdemo/WeatherData.dart';
 import 'package:flutter_applicationdemo/WebScraper.dart';
-import 'package:font_awesome_flutter/font_awesome_flutter.dart';
 import 'package:google_fonts/google_fonts.dart';
-import 'package:flutter_applicationdemo/HomePage.dart';
 import 'dart:async';
 import 'login/User.dart';
 import 'package:google_maps_flutter/google_maps_flutter.dart';
 import 'package:flutter_google_places/flutter_google_places.dart';
-import 'package:http/http.dart' as http;
-import 'package:http/retry.dart';
-import 'package:intl/number_symbols.dart';
 import 'package:location/location.dart';
-import 'package:geolocator/geolocator.dart';
-import 'package:intl/intl.dart';
 import 'package:flutter_applicationdemo/login/User.dart';
 import 'SettingsPage.dart';
 import 'venuePage.dart';
@ -25,7 +15,6 @@ import 'globals.dart' as globals;

 import 'package:syncfusion_flutter_sliders/sliders.dart';

-import 'HomePage.dart';
 import 'FeedbackPage.dart';
 import 'login/CreateAccountPage.dart';
 import 'login/signInPage.dart';
@ -37,7 +26,7 @@ class Map extends StatefulWidget {

 const kGoogleApiKey = "AIzaSyAUmhd6Xxud8SwgDxJ4LlYlcntm01FGoSk";

-final homeSacffoldKey = GlobalKey<ScaffoldState>();
+final homeScaffoldKey = GlobalKey<ScaffoldState>();

 late CameraPosition _currentCameraPosition;

@ -77,8 +66,7 @@ class MapState extends State<Map> {
  void createBottomSheet(String venueName) async {
    var webScraper = WebScraper();
    await webScraper.getWebsiteData(venueName);
-    Scaffold.of(context).showBottomSheet<void>(
-        ((context) {
+    Scaffold.of(context).showBottomSheet<void>(((context) {
      return Container(
        height: 420,
        color: Colors.white,
@ -87,13 +75,7 @@ class MapState extends State<Map> {
          mainAxisAlignment: MainAxisAlignment.center,
          mainAxisSize: MainAxisSize.min,
          children: <Widget>[
-            /*const Text('BottomSheet'),
-                        ElevatedButton(
-                          child: const Text('Close BottomSheet'),
-                          onPressed: () {Navigator.pop(context);})*/
-            Container(
-              child: Text(webScraper.openingHoursThisWeek.length.toString()),
-            ),
+            Text(webScraper.openingHoursThisWeek.length.toString()),
          ],
        )),
      );
@ -139,6 +121,7 @@ class MapState extends State<Map> {
  final Mode _mode = Mode.fullscreen;

  int currentIndex = 0;
+
  @override
  Widget build(BuildContext context) {
    _currentCameraPosition = _stockholmCity;
@ -146,27 +129,29 @@ class MapState extends State<Map> {
      appBar: AppBar(
        centerTitle: true,
        title: const Text("Sun chasers"),
-        key: homeSacffoldKey,
+        key: homeScaffoldKey,
        actions: <Widget>[createFilterMenuButton()],
        backgroundColor: const Color.fromARGB(255, 190, 146, 160),
      ),
-      drawer : Drawer(
+      drawer: Drawer(
        child: Container(
-          child: globals.LOGGED_IN_USER.userID == 0 ? buildDrawerSignedOut(context) : buildDrawerSignedIn(context),
+          child: globals.LOGGED_IN_USER.userID == 0
+              ? buildDrawerSignedOut(context)
+              : buildDrawerSignedIn(context),
        ),
      ),
-
-      body: Stack (
+      body: Stack(
        children: [
          GoogleMap(
            cameraTargetBounds: CameraTargetBounds(LatLngBounds(
                northeast: const LatLng(59.3474696569038, 18.1001602476002147),
-                southwest: const LatLng(59.297332547922636, 17.999522500277884))),
+                southwest:
+                    const LatLng(59.297332547922636, 17.999522500277884))),
            minMaxZoomPreference: MinMaxZoomPreference(12.5, 18.5),
-            onCameraMove: (CameraPosition camera){
+            onCameraMove: (CameraPosition camera) {
              _currentCameraPosition = camera;
            },
-            onCameraIdle: (){
+            onCameraIdle: () {
              (context as Element).reassemble();
              removeMarkersOutOfRange();
              addMarkersInRange();
@ -339,43 +324,6 @@ class MapState extends State<Map> {
        CameraPosition(target: LatLng(lat, lng), zoom: 14.5)));
  }

-  Widget _boxes(double lat, double lng, String restaurantName) {
-    return GestureDetector(
-      onTap: () {
-        _gotoLocation(lat, lng);
-      },
-      child: Container(
-          child: FittedBox(
-        child: Material(
-          color: Colors.white,
-          elevation: 14.0,
-          borderRadius: BorderRadius.circular(24.0),
-          shadowColor: Color(0x802196F3),
-          child: Row(
-            mainAxisAlignment: MainAxisAlignment.spaceBetween,
-            children: <Widget>[
-              Container(
-                width: 250,
-                height: 200,
-                child: ClipRRect(
-                  borderRadius: new BorderRadius.circular(24.0),
-                  child:
-                      const Image(image: AssetImage('assets/images/bild.png')),
-                ),
-              ),
-              Container(
-                child: Padding(
-                  padding: const EdgeInsets.all(8.0),
-                  child: Text(restaurantName),
-                ),
-              )
-            ],
-          ),
-        ),
-      )),
-    );
-  }
-
  Future<void> _goToCurrentPosition(LatLng latlng) async {
    final GoogleMapController controller = await _controller.future;
    controller.animateCamera(CameraUpdate.newCameraPosition(CameraPosition(
@ -386,26 +334,36 @@ class MapState extends State<Map> {
  }

  void removeMarkersOutOfRange() {
-    for(int i = 0; i<closeByMarkersList.length; i++){
+    for (int i = 0; i < closeByMarkersList.length; i++) {
      Marker marker = closeByMarkersList[i];
-      globals.venueAlreadyAdded(globals.getVenueByID(int.parse(marker.markerId.value))!.venueName);
-      if(marker.position.longitude - _currentCameraPosition.target.longitude > 0.02 || marker.position.latitude - _currentCameraPosition.target.latitude > 0.02){
+      globals.venueAlreadyAdded(
+          globals.getVenueByID(int.parse(marker.markerId.value))!.venueName);
+      if (marker.position.longitude - _currentCameraPosition.target.longitude >
+              0.02 ||
+          marker.position.latitude - _currentCameraPosition.target.latitude >
+              0.02) {
        closeByMarkersList.remove(marker);
-        globals.getVenueByID(int.parse(marker.markerId.value))?.isShownOnMap = false;
+        globals.getVenueByID(int.parse(marker.markerId.value))?.isShownOnMap =
+            false;
        i--;
      }
    }
  }

  void addMarkersInRange() {
-    for(int i = 0; i< globals.VENUES.length; i++){
-      if(!globals.VENUES[i].isShownOnMap && (globals.VENUES[i].position.longitude - _currentCameraPosition.target.longitude < 0.02 && globals.VENUES[i].position.latitude - _currentCameraPosition.target.latitude < 0.02)){
+    for (int i = 0; i < globals.VENUES.length; i++) {
+      if (!globals.VENUES[i].isShownOnMap &&
+          (globals.VENUES[i].position.longitude -
+                      _currentCameraPosition.target.longitude <
+                  0.02 &&
+              globals.VENUES[i].position.latitude -
+                      _currentCameraPosition.target.latitude <
+                  0.02)) {
        Marker marker = Marker(
            markerId: MarkerId(globals.VENUES[i].venueID.toString()),
            position: globals.VENUES[i].position,
            onTap: () => createBottomDrawer(globals.VENUES[i]),
-            icon: globals.VENUES[i].drawIconColor()
-        );
+            icon: globals.VENUES[i].drawIconColor());
        globals.VENUES[i].isShownOnMap = true;
        closeByMarkersList.add(marker);
      }
@ -415,7 +373,9 @@ class MapState extends State<Map> {
  createBottomDrawer(Venue venue) async {
    _bottomSheetIsOpen = true;
    // Scaffold.of(context).showBottomSheet<void>(((context) {
-    showModalBottomSheet(context: context, builder: (BuildContext context) {
+    showModalBottomSheet(
+        context: context,
+        builder: (BuildContext context) {
          return InkWell(
            onTap: () {
              Navigator.push(
@ -454,7 +414,6 @@ class MapState extends State<Map> {
                ],
              ),
              // columnCoveringRating(),
-
            ],
          ),
          const SizedBox(
@ -465,8 +424,8 @@ class MapState extends State<Map> {
            children: [
              Column(
                children: const [
-                  weatherIconRow(),
-                  weatherStatusRow(),
+                  WeatherIconRow(),
+                  WeatherStatusRow(),
                ],
              ),
              columnHandlingReadMoreButton(context, venue),
@ -544,45 +503,14 @@ class MapState extends State<Map> {
  }

  void setAllMarkersAsInvisible() {
-    for(Venue venue in hiddenVenues){
+    for (Venue venue in hiddenVenues) {
      venue.isShownOnMap = false;
    }
  }
-/*
- Future<void> _handelPressButton() async {
-    Prediction? p = await PlacesAutocomplete.show(
-                          context: context,
-                          apiKey: kGoogleApiKey,
-                          mode: _mode, // Mode.fullscreen
-                          language: "en",
-                          strictbounds: false,
-                          decoration: InputDecoration(
-                            hintText:'serach',
-                            focusedBorder: OutlineInputBorder(borderRadius: BorderRadius.circular(20), borderSide: BorderSide(color: Colors.white))
-                          ),
-                          types: [""],
-                          components: [Component(Component.country, "se")]);
-    if (p != null) {
-      displayPrediction(p,homeSacffoldKey.currentState);
-    }
-  }
-  Future<void> displayPrediction(Prediction p, ScaffoldState? currentState) async {
-    GoogleMapsPlaces places = GoogleMapsPlaces(
-      apiKey: kGoogleApiKey,
-      apiHeaders: await const GoogleApiHeaders().getHeaders()
-    );
-    PlacesDetailsResponse detail = await places.getDetailsByPlaceId(p.placeId!);
-    final lat = detail.result.geometry!.location.lat;
-    final lng = detail.result.geometry!.location.lng;
-    markersList.clear();
-    markersList.add(Marker(markerId: const MarkerId("0"), position: LatLng(lat, lng), infoWindow: InfoWindow(title: detail.result.name)));
-    setState(() {});
-    googleMapController.animateCamera(CameraUpdate.newLatLngZoom(LatLng(lat,lng), 14.0));
-  }*/
 }

-class weatherIconRow extends StatelessWidget {
-  const weatherIconRow({
+class WeatherIconRow extends StatelessWidget {
+  const WeatherIconRow({
    Key? key,
  }) : super(key: key);

@ -600,8 +528,8 @@ class weatherIconRow extends StatelessWidget {
  }
 }

-class weatherStatusRow extends StatelessWidget {
-  const weatherStatusRow({
+class WeatherStatusRow extends StatelessWidget {
+  const WeatherStatusRow({
    Key? key,
  }) : super(key: key);

@ -645,36 +573,16 @@ Widget buildDrawerSignedIn(BuildContext context) {
          title: Text('Sign out'),
          onTap: () {
            globals.LOGGED_IN_USER = User(0, "", "");
-            Navigator.push(
-              context,
-              MaterialPageRoute(builder: (context) => BottomNavPage()), //Replace Container() with call to Map-page.
-            );
-          },
-        ),
-        ListTile(
-          leading: Icon(Icons.thumb_up_alt),
-          title: Text('Give feedback'),
-          onTap: () {
            Navigator.push(
              context,
              MaterialPageRoute(
-                builder: (context) => FormForFeedback(),
-              ),
-            );
-          },
-        ),
-        ListTile(
-          leading: Icon(Icons.settings),
-          title: Text('Settings'),
-          onTap: () {
-            Navigator.push(
-              context,
-              MaterialPageRoute(
-                builder: (context) => SettingsPage(),
-              ),
+                  builder: (context) =>
+                      BottomNavPage()), //Replace Container() with call to Map-page.
            );
          },
        ),
+        giveFeedbackTile(context),
+        settingsTile(context),
      ],
    ),
  );
@ -685,9 +593,19 @@ Widget buildDrawerSignedOut(BuildContext context) {
    child: ListView(
      padding: EdgeInsets.zero,
      children: [
-        DrawerHeader(
-          decoration:
-              const BoxDecoration(color: Color.fromARGB(255, 190, 146, 160)),
+        drawerHeader(),
+        createAccountTile(context),
+        logInTile(context),
+        giveFeedbackTile(context),
+        settingsTile(context),
+      ],
+    ),
+  );
+}
+
+DrawerHeader drawerHeader() {
+  return DrawerHeader(
+    decoration: const BoxDecoration(color: Color.fromARGB(255, 190, 146, 160)),
    child: Column(
      children: const <Widget>[
        Text(
@ -697,44 +615,11 @@ Widget buildDrawerSignedOut(BuildContext context) {
        SizedBox(height: 30),
      ],
    ),
-        ),
-        ListTile(
-          leading: Icon(Icons.account_box_rounded),
-          title: Text('Create account'),
-          onTap: () {
-            Navigator.push(
-              context,
-              MaterialPageRoute(
-                builder: (context) => CreateAccountPage(),
-              ),
  );
-          },
-        ),
-        ListTile(
-          leading: Icon(Icons.login),
-          title: Text('Sign in'),
-          onTap: () {
-            Navigator.push(
-              context,
-              MaterialPageRoute(
-                builder: (context) => SignInPage(),
-              ),
-            );
-          },
-        ),
-        ListTile(
-          leading: Icon(Icons.thumb_up_alt),
-          title: Text('Give feedback'),
-          onTap: () {
-            Navigator.push(
-              context,
-              MaterialPageRoute(
-                builder: (context) => FormForFeedback(),
-              ),
-            );
-          },
-        ),
-        ListTile(
+}
+
+ListTile settingsTile(BuildContext context) {
+  return ListTile(
    leading: Icon(Icons.settings),
    title: Text('Settings'),
    onTap: () {
@ -745,16 +630,50 @@ Widget buildDrawerSignedOut(BuildContext context) {
        ),
      );
    },
-        ),
-      ],
-    ),
  );
 }

-class _Marker {
-  var Plats_1;
-  var Gatunr_1;
-  var coordinates;
-
-  _Marker(this.Plats_1, this.Gatunr_1, this.coordinates);
+ListTile giveFeedbackTile(BuildContext context) {
+  return ListTile(
+    leading: Icon(Icons.thumb_up_alt),
+    title: Text('Give feedback'),
+    onTap: () {
+      Navigator.push(
+        context,
+        MaterialPageRoute(
+          builder: (context) => FormForFeedback(),
+        ),
+      );
+    },
+  );
+}
+
+ListTile logInTile(BuildContext context) {
+  return ListTile(
+    leading: Icon(Icons.login),
+    title: Text('Sign in'),
+    onTap: () {
+      Navigator.push(
+        context,
+        MaterialPageRoute(
+          builder: (context) => SignInPage(),
+        ),
+      );
+    },
+  );
+}
+
+ListTile createAccountTile(BuildContext context) {
+  return ListTile(
+    leading: Icon(Icons.account_box_rounded),
+    title: Text('Create account'),
+    onTap: () {
+      Navigator.push(
+        context,
+        MaterialPageRoute(
+          builder: (context) => CreateAccountPage(),
+        ),
+      );
+    },
+  );
 }
--- a/lib/main.dart
+++ b/lib/main.dart
@ -24,13 +24,20 @@ import 'globals.dart' as globals;
 void main() async {
  WidgetsFlutterBinding.ensureInitialized();
  await Firebase.initializeApp();
-  await loadAllVenues();
-  await fetchWeather();
+  // await loadAllVenues();
+  // await fetchWeather();
+  await weatherInstance();
  await loadAllVenuesSQL();

  runApp(MyApp());
 }

+Future weatherInstance() async{
+  WeatherData weather = WeatherData(3, 12);
+  globals.forecast = weather;
+
+}
+
 Future fetchWeather() async {
  WeatherData tempWeather = WeatherData(0, 0);
  Uri weatherDataURI = Uri.parse(
--- a/lib/venuePage.dart
+++ b/lib/venuePage.dart
@ -19,24 +19,18 @@ class VenuePage extends StatefulWidget {
  final Venue venue;

  @override
-  State<VenuePage> createState() => _VenuePageState(venue);
+  State<VenuePage> createState() => VenuePageState(venue);
 }

-class _VenuePageState extends State<VenuePage> {
+@visibleForTesting
+class VenuePageState extends State<VenuePage> {
  late WeatherData currentWeather;
  final String imageLink = '';
  late final Venue venue;
  late VenueInfo venueInfo;

-  _VenuePageState(this.venue);
+  VenuePageState(this.venue);

-  validateAndGetImageLink() {
-    if (imageLink == '') {
-      return 'https://live.staticflickr.com/6205/6081773215_19444220b6_b.jpg';
-    } else {
-      return imageLink;
-    }
-  }

  @override
  void initState() {
@ -64,7 +58,7 @@ class _VenuePageState extends State<VenuePage> {

      setState(() {
        globals.forecast = tempWeather;
-        currentWeather = tempWeather; //Could be a widget instead??
+        currentWeather = tempWeather;
      });
    } else {
      throw const HttpException("Problem fetching the weather data");
--- a/test/ListViewPageTest.dart
+++ b/test/ListViewPageTest.dart
@ -0,0 +1,19 @@
+
+import 'dart:math';
+
+import 'package:flutter_applicationdemo/ListViewPage.dart';
+import 'package:flutter_applicationdemo/Venue.dart';
+import 'package:flutter_applicationdemo/globals.dart' as globals;
+import 'package:google_maps_flutter/google_maps_flutter.dart';
+
+import 'package:test/test.dart';
+
+void main() {
+  test('Test ListViewPage can access venues', () {
+    // ListViewPage listViewPage = const ListViewPage();
+    ListViewPageState listViewPage = ListViewPageState();
+    globals.VENUES.add(Venue(00, 'Aira', 'Biskopsudden', '9', const LatLng(59.354823, 19.29485)));
+
+    expect(globals.VENUES.first, listViewPage.allVenues.first);
+  });
+}
--- a/test/VenuePageTest.dart
+++ b/test/VenuePageTest.dart
@ -0,0 +1,19 @@
+import 'package:flutter_applicationdemo/Venue.dart';
+import 'package:flutter_applicationdemo/WeatherData.dart';
+import 'package:flutter_applicationdemo/venuePage.dart';
+import 'package:google_maps_flutter/google_maps_flutter.dart';
+import 'package:test/test.dart';
+
+
+
+void main() {
+  test('Venue Page Has Access to Venue Instance', () {
+    Venue venue = Venue(00, 'Aira', 'Biskopsudden', '9', const LatLng(59.354823, 19.29485));
+    VenuePage venuePage = VenuePage(venue);
+
+    expect('Aira', venuePage.venue.venueName);
+    expect('Biskopsudden', venuePage.venue.venueAddress);
+    expect('9', venuePage.venue.venueStreetNo);
+
+  });
+}