fix tests

Author: basharh

lib/main.dart

@@ -28,7 +28,7 @@ class WeatherApp extends ConsumerWidget {
     return initInfo.when(
       loading: () => const CircularProgressIndicator(),
       error: (err, stack) {
-        print('Error during initialization: $err');
+        //print('Error during initialization: $err');
         return Text('Error: $err, $stack');
       },
       data: (config) {

lib/services/open_mateo/open_mateo.dart

@@ -8,6 +8,7 @@ class OpenMateoService {
 
   OpenMateoService(this.dio);
 
+  /// Fetch hourly weather data for the given latitude and longitude for the current 24 hours
   Future<Map<String, HourlyData>> fetchHourly({
     required double latitude,
     required double longitude,
@@ -36,6 +37,7 @@ class OpenMateoService {
     return result;
   }
 
+  /// Fetch daily weather data for the given latitude and longitude for the next 7 days
   Future<Map<String, DailyData>> fetchDaily({
     required double latitude,
     required double longitude,

lib/services/open_mateo/open_mateo_url.dart

@@ -24,6 +24,7 @@ class OpenMateoUrlBuilder {
     path: '/v1/forecast',
   );
 
+  /// Get the URL for fetching weather data for 24 hours of today
   String getHourlyDataUrl({
     required double latitude,
     required double longitude,
@@ -42,6 +43,7 @@ class OpenMateoUrlBuilder {
     ).toString();
   }
 
+  /// Get the URL for fetching daily weather data for the next 7 days
   String getDailyDataUrl({
     required double latitude,
     required double longitude,
@@ -60,6 +62,7 @@ class OpenMateoUrlBuilder {
     ).toString();
   }
 
+  /// Get the URL for fetching current weather data at a specific location
   String getWeatherAtLocationUrl({
     required double latitude,
     required double longitude,

test/services/open_mateo_test.dart

@@ -20,15 +20,21 @@ void main() {
       }
     };
 
-    when(client.get(OpenMateoService.hourlyUri.toString())).thenAnswer(
+    when(client.get(any)).thenAnswer(
       (_) async => dio.Response(
         data: data,
         statusCode: 200,
         requestOptions: dio.RequestOptions(),
       ),
     );
 
-    final response = await service.fetchHourly();
+    final response = await service.fetchHourly(
+      latitude: 52.52,
+      longitude: 13.405,
+      timezone: 'America/New_York',
+    );
+
+    // TODO: verify that the get method was called with the correct URL
 
     expect(response, isA<Map<String, HourlyData>>());
     expect(response.length, 3);
@@ -63,15 +69,20 @@ void main() {
       }
     };
 
-    when(client.get(OpenMateoService.dailyUri.toString())).thenAnswer(
+    when(client.get(any)).thenAnswer(
       (_) async => dio.Response(
         data: data,
         statusCode: 200,
         requestOptions: dio.RequestOptions(),
       ),
     );
 
-    final response = await service.fetchDaily();
+    final response = await service.fetchDaily(
+      latitude: 52.52,
+      longitude: 13.405,
+      timezone: 'America/New_York',
+    );
+
     expect(response, isA<Map<String, DailyData>>());
     expect(response.length, 7);
 

test/utils/data.dart

@@ -0,0 +1,20 @@
+import 'package:weather/services/open_mateo/data.dart';
+
+Map<String, HourlyData> getDummyHourlyData() {
+  // Create hourly data for 24 hours
+  Map<String, HourlyData> hourlyData = {};
+
+  var start = DateTime.parse('2024-10-10T00:00');
+
+  for (int i = 0; i < 24; i++) {
+    var time = start.add(Duration(hours: i));
+    var timeString = time.toIso8601String().substring(0, 16);
+    hourlyData[timeString] = HourlyData(
+      weather_code: (i % 12) * 10, // Just a sample pattern for weather codes
+      time: timeString,
+      temperature_2m: 15.0 + i, // Sample temperature increasing over the day
+    );
+  }
+
+  return hourlyData;
+}

test/utils/test_material_app.dart

@@ -24,7 +24,7 @@ class TestMaterialApp extends StatelessWidget {
   Widget build(BuildContext context) {
     return MaterialApp(
       debugShowCheckedModeBanner: false,
-      theme: themeData,
+      theme: getThemeData(Brightness.light),
       home: Scaffold(
         body: Center(
           child: child,

test/widgets/common/hours_row_test.dart

@@ -1,30 +1,15 @@
 import 'package:flutter_test/flutter_test.dart';
-import 'package:weather/services/open_mateo/data.dart';
 import 'package:weather/widgets/common/hours_row.dart';
 
+import '../../utils/data.dart';
 import '../../utils/test_material_app.dart';
 
 void main() {
   group('HoursRow', () {
     testWidgets('golden test', (tester) async {
       await tester.setScreenSize(width: 400, height: 150);
 
-      var start = DateTime.parse('2024-10-10T00:00');
-
-      // Create hourly data for 24 hours
-      Map<String, HourlyData> hourlyData = {};
-
-      for (int i = 0; i < 24; i++) {
-        var time = start.add(Duration(hours: i));
-        var timeString = time.toIso8601String().substring(0, 16);
-        hourlyData[timeString] = HourlyData(
-          weather_code:
-              (i % 12) * 10, // Just a sample pattern for weather codes
-          time: timeString,
-          temperature_2m:
-              15.0 + i, // Sample temperature increasing over the day
-        );
-      }
+      final hourlyData = getDummyHourlyData();
 
       await tester.pumpWidget(
         TestMaterialApp(

test/widgets/day/day_details_card/day_details_card_test.dart

@@ -1,6 +1,7 @@
 import 'package:flutter_test/flutter_test.dart';
 import 'package:weather/widgets/day/day_details_card/day_details_card.dart';
 
+import '../../../utils/data.dart';
 import '../../../utils/test_material_app.dart';
 
 void main() {
@@ -9,8 +10,11 @@ void main() {
       await tester.setScreenSize(width: 250, height: 350);
 
       await tester.pumpWidget(
-        const TestMaterialApp(
-          child: DayDetailsCard(),
+        TestMaterialApp(
+          child: DayDetailsCard(
+            hourlyData: getDummyHourlyData(),
+            localityName: 'New York',
+          ),
         ),
       );