Refactor/remove demo provider

CHANGELOG.md

@@ -7,6 +7,12 @@ and this project adheres to Semantic Versioning.
 
 ## [Unreleased]
 
+### Changed
+- **BREAKING refactor(enums)**: remove demo enums from production code
+- **refactor**(remote_configs): remove unused ad platform identifiers
+- **fix(tests)**: remove demo enums from tests
+
+
 ## [1.6.0] - 2025-12-31
 
 ### Added

lib/src/enums/ad_platform_type.dart

@@ -4,13 +4,4 @@
 enum AdPlatformType {
   /// Google AdMob platform.
   admob,
-
-  /// Represents a placeholder ad platform for demonstration purposes.
-  ///
-  /// When this type is selected, it indicates that ads should be handled
-  /// by a demo-specific ad provider, which displays placeholder ads ,
-  /// without initializing external ad SDKs. This is
-  /// primarily used in the `demo` environment to avoid actual ad network
-  /// calls and `MissingPluginException` on unsupported platforms (e.g., web).
-  demo,
 }

lib/src/enums/analytics_provider.dart

@@ -15,8 +15,4 @@ enum AnalyticsProvider {
   /// Mixpanel Analytics.
   @JsonValue('mixpanel')
   mixpanel,
-
-  /// A demo provider that logs events to the console for development.
-  @JsonValue('demo')
-  demo,
 }

lib/src/fixtures/remote_configs.dart

@@ -116,11 +116,6 @@ final remoteConfigsFixturesData = <RemoteConfig>[
             bannerAdId: 'ca-app-pub-3940256099942544/6300978111',
             interstitialAdId: 'ca-app-pub-3940256099942544/1033173712',
           ),
-          AdPlatformType.demo: AdPlatformIdentifiers(
-            nativeAdId: '_',
-            bannerAdId: '_',
-            interstitialAdId: '_',
-          ),
         },
         feedAdConfiguration: FeedAdConfiguration(
           enabled: true,

test/src/enums/ad_platform_type_test.dart

@@ -5,17 +5,14 @@ void main() {
   group('AdPlatformType', () {
     test('has correct string values', () {
       expect(AdPlatformType.admob.name, 'admob');
-      expect(AdPlatformType.demo.name, 'demo');
     });
 
     test('can be created from string values', () {
       expect(AdPlatformType.values.byName('admob'), AdPlatformType.admob);
-      expect(AdPlatformType.values.byName('demo'), AdPlatformType.demo);
     });
 
     test('has correct toString representation', () {
       expect(AdPlatformType.admob.toString(), 'AdPlatformType.admob');
-      expect(AdPlatformType.demo.toString(), 'AdPlatformType.demo');
     });
   });
 }

test/src/enums/analytics_provider_test.dart

@@ -6,18 +6,13 @@ void main() {
     test('has correct values', () {
       expect(
         AnalyticsProvider.values,
-        containsAll([
-          AnalyticsProvider.firebase,
-          AnalyticsProvider.mixpanel,
-          AnalyticsProvider.demo,
-        ]),
+        containsAll([AnalyticsProvider.firebase, AnalyticsProvider.mixpanel]),
       );
     });
 
     test('has correct string names', () {
       expect(AnalyticsProvider.firebase.name, 'firebase');
       expect(AnalyticsProvider.mixpanel.name, 'mixpanel');
-      expect(AnalyticsProvider.demo.name, 'demo');
     });
 
     test('can be created from string names', () {

test/src/models/analytics/payloads/ad_clicked_payload_test.dart

@@ -23,12 +23,12 @@ void main() {
     group('Equatable', () {
       test('should equate two identical instances', () {
         const instance1 = AdClickedPayload(
-          adProvider: AdPlatformType.demo,
+          adProvider: AdPlatformType.admob,
           adType: AdType.banner,
           adPlacement: 'bottom',
         );
         const instance2 = AdClickedPayload(
-          adProvider: AdPlatformType.demo,
+          adProvider: AdPlatformType.admob,
           adType: AdType.banner,
           adPlacement: 'bottom',
         );
@@ -38,8 +38,8 @@ void main() {
       test('should not equate instances with different properties', () {
         const instance1 = payload;
         const instance2 = AdClickedPayload(
-          adProvider: AdPlatformType.demo,
-          adType: AdType.native,
+          adProvider: AdPlatformType.admob,
+          adType: AdType.banner,
           adPlacement: 'feed',
         );
         expect(instance1, isNot(equals(instance2)));

test/src/models/analytics/payloads/ad_impression_payload_test.dart

@@ -23,12 +23,12 @@ void main() {
     group('Equatable', () {
       test('should equate two identical instances', () {
         const instance1 = AdImpressionPayload(
-          adProvider: AdPlatformType.demo,
+          adProvider: AdPlatformType.admob,
           adType: AdType.banner,
           adPlacement: 'bottom',
         );
         const instance2 = AdImpressionPayload(
-          adProvider: AdPlatformType.demo,
+          adProvider: AdPlatformType.admob,
           adType: AdType.banner,
           adPlacement: 'bottom',
         );
@@ -38,8 +38,8 @@ void main() {
       test('should not equate instances with different properties', () {
         const instance1 = payload;
         const instance2 = AdImpressionPayload(
-          adProvider: AdPlatformType.demo,
-          adType: AdType.native,
+          adProvider: AdPlatformType.admob,
+          adType: AdType.banner,
           adPlacement: 'feed',
         );
         expect(instance1, isNot(equals(instance2)));

test/src/models/analytics/payloads/ad_load_failed_payload_test.dart

@@ -23,12 +23,12 @@ void main() {
     group('Equatable', () {
       test('should equate two identical instances', () {
         const instance1 = AdLoadFailedPayload(
-          adProvider: AdPlatformType.demo,
+          adProvider: AdPlatformType.admob,
           adType: AdType.banner,
           errorCode: 404,
         );
         const instance2 = AdLoadFailedPayload(
-          adProvider: AdPlatformType.demo,
+          adProvider: AdPlatformType.admob,
           adType: AdType.banner,
           errorCode: 404,
         );
@@ -38,7 +38,7 @@ void main() {
       test('should not equate instances with different properties', () {
         const instance1 = payload;
         const instance2 = AdLoadFailedPayload(
-          adProvider: AdPlatformType.demo,
+          adProvider: AdPlatformType.admob,
           adType: AdType.interstitial,
           errorCode: 2,
         );

test/src/models/config/analytics_config_test.dart

@@ -55,11 +55,7 @@ void main() {
       test('should not equate instances with different properties', () {
         final config1 = analyticsConfigFixture.copyWith();
         final config2 = analyticsConfigFixture.copyWith(enabled: false);
-        final config3 = analyticsConfigFixture.copyWith(
-          activeProvider: AnalyticsProvider.demo,
-        );
         expect(config1, isNot(equals(config2)));
-        expect(config1, isNot(equals(config3)));
       });
 
       test('props list should contain all relevant fields', () {

test/src/models/config/remote_config_test.dart

@@ -54,7 +54,6 @@ void main() {
       final newFeatures = remoteConfigFixture.features.copyWith(
         analytics: remoteConfigFixture.features.analytics.copyWith(
           enabled: false,
-          activeProvider: AnalyticsProvider.demo,
         ),
       );