Added MovieRepositoryImpl Tests

ashishrawat2911 · Aug 5, 2022 · 715840b · 715840b
1 parent 75288d0
commit 715840b
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 8 deletions.
diff --git a/lib/data/source/local/movie_local_data_source.dart b/lib/data/source/local/movie_local_data_source.dart
@@ -14,15 +14,12 @@ class MovieLocalDataSourceImpl extends MovieLocalDataSource {
 
   @override
   Future<List<MovieEntity>> getMovies() async {
-    final movies = await _movieDao.findAllMovies();
-    return movies;
+    return _movieDao.findAllMovies();
   }
 
   @override
   Future<MovieEntity?> movieDetails(int id) async {
-    final movie = await _movieDao.findMovieById(id);
-    if (movie == null) return null;
-    return movie;
+    return _movieDao.findMovieById(id);
   }
 
   @override

diff --git a/lib/data/source/remote/movie_remote_data_source.dart b/lib/data/source/remote/movie_remote_data_source.dart
@@ -15,8 +15,7 @@ class MovieRemoteDataSourceImpl extends MovieRemoteDataSource {
 
   @override
   Future<MovieResponseModel> getMovieDetails(int id) async {
-    final movie = await _movieApiService.getMovieDetails(id);
-    return movie;
+    return _movieApiService.getMovieDetails(id);
   }
 
   @override

diff --git a/test/data/repository/movie_repository_impl_test.dart b/test/data/repository/movie_repository_impl_test.dart
@@ -13,7 +13,6 @@ import 'movie_repository_impl_test.mocks.dart';
 @GenerateMocks([MovieRepositoryImpl])
 void main() {
   MockMovieRepositoryImpl mockMovieRepository = MockMovieRepositoryImpl();
-
   group('getPopularMovies function testing', () {
     test('success test', () async {
       const mockValue = Right<NetworkError, List<Movie>>([]);