Skip to content

Commit

Permalink
#61 - task(firestore service): remove commented out code
Browse files Browse the repository at this point in the history
  • Loading branch information
sdresselmann committed Apr 28, 2024
1 parent 6cd4538 commit be628bc
Show file tree
Hide file tree
Showing 5 changed files with 4 additions and 18 deletions.
2 changes: 1 addition & 1 deletion lib/core/interfaces/json_serializable.dart
@@ -1,6 +1,6 @@
import 'package:lifting_progress_tracker/firebase/firestore_json.dart';

abstract class JsonSerializable<T> {
abstract class JsonSerializable {
// Usage of named constructors can't be enforced, however the usage of
// both methods is heavily encouraged.
JsonSerializable.fromJson();
Expand Down
2 changes: 1 addition & 1 deletion lib/training_plan/models/training_plan.dart
Expand Up @@ -2,7 +2,7 @@ import 'package:lifting_progress_tracker/core/interfaces/json_serializable.dart'
import 'package:lifting_progress_tracker/firebase/firestore_json.dart';
import 'package:lifting_progress_tracker/training_plan/models/training_plan_entry.dart';

class TrainingPlan implements JsonSerializable<TrainingPlan> {
class TrainingPlan implements JsonSerializable {
Map<String, TrainingPlanEntry> planEntries;

TrainingPlan.fromJson(FirestoreJson json)
Expand Down
2 changes: 1 addition & 1 deletion lib/training_plan/models/training_plan_entry.dart
@@ -1,7 +1,7 @@
import 'package:lifting_progress_tracker/core/interfaces/json_serializable.dart';
import 'package:lifting_progress_tracker/firebase/firestore_json.dart';

class TrainingPlanEntry implements JsonSerializable<TrainingPlanEntry> {
class TrainingPlanEntry implements JsonSerializable {
String repeats;
String exerciseName;
String weight;
Expand Down
2 changes: 1 addition & 1 deletion lib/training_plan/models/training_plan_list.dart
Expand Up @@ -2,7 +2,7 @@ import 'package:lifting_progress_tracker/core/interfaces/json_serializable.dart'
import 'package:lifting_progress_tracker/firebase/firestore_json.dart';
import 'package:lifting_progress_tracker/training_plan/models/training_plan.dart';

class TrainingPlanList implements JsonSerializable<TrainingPlanList> {
class TrainingPlanList implements JsonSerializable {
Map<String, TrainingPlan> trainingPlanList;

TrainingPlanList(this.trainingPlanList);
Expand Down
14 changes: 0 additions & 14 deletions test_utils/mocks/firestore_mock_service.dart
Expand Up @@ -5,18 +5,4 @@ import 'firestore_mock_data.dart';

class MockFirestoreService extends Mock implements FirestoreService {
final MockFirestoreData mockFirestoreData = MockFirestoreData();

// @override
// Future<Map<String, dynamic>> getRawData(
// String documentReference,
// String documentId,
// ) async {
// return mockFirestoreData.getAsync();
// }
//
// @override
// Future<void> uploadRawData(
// String collectionName,
// Map<String, dynamic> data,
// ) async {}
}

0 comments on commit be628bc

Please sign in to comment.