Merge pull request #257 from wordpress-mobile/issue/extend_wpanalytics_to_descentralize_tracks

Extend WPAnalytics to decentralize tracks

Author: leandroalonso

WordPressShared.podspec

@@ -1,6 +1,6 @@
 Pod::Spec.new do |s|
   s.name          = "WordPressShared"
-  s.version       = "1.8.15"
+  s.version       = "1.8.16-beta.1"
   s.summary       = "Shared components used in building the WordPress iOS apps and other library components."
 
   s.description   = <<-DESC

WordPressShared/Core/Analytics/WPAnalytics.h

@@ -201,8 +201,6 @@ typedef NS_ENUM(NSUInteger, WPAnalyticsStat) {
     WPAnalyticsStatLoginSocialErrorUnknownUser,
     WPAnalyticsStatLogout,
     WPAnalyticsStatLowMemoryWarning,
-    WPAnalyticsStatMediaEditorShown,
-    WPAnalyticsStatMediaEditorUsed,
     WPAnalyticsStatMediaLibraryDeletedItems,
     WPAnalyticsStatMediaLibraryEditedItemMetadata,
     WPAnalyticsStatMediaLibraryPreviewedItem,
@@ -603,6 +601,8 @@ extern NSString *const WPAnalyticsStatEditorPublishedPostPropertyVideo;
 + (void)endTimerForStat:(WPAnalyticsStat)stat withProperties:(NSDictionary *)properties;
 + (void)track:(WPAnalyticsStat)stat;
 + (void)track:(WPAnalyticsStat)stat withProperties:(NSDictionary *)properties;
++ (void)trackString:(NSString *)event;
++ (void)trackString:(NSString *)event withProperties:(NSDictionary *)properties;
 + (void)endSession;
 + (void)clearQueuedEvents;
 
@@ -612,6 +612,8 @@ extern NSString *const WPAnalyticsStatEditorPublishedPostPropertyVideo;
 
 - (void)track:(WPAnalyticsStat)stat;
 - (void)track:(WPAnalyticsStat)stat withProperties:(NSDictionary *)properties;
+- (void)trackString:(NSString *)event;
+- (void)trackString:(NSString *)event withProperties:(NSDictionary *)properties;
 
 @optional
 - (void)beginSession;

WordPressShared/Core/Analytics/WPAnalytics.m

@@ -63,6 +63,21 @@ + (void)track:(WPAnalyticsStat)stat withProperties:(NSDictionary *)properties
     }
 }
 
++ (void)trackString:(NSString *)event
+{
+    for (id<WPAnalyticsTracker> tracker in [self trackers]) {
+        [tracker trackString:event];
+    }
+}
+
++ (void)trackString:(NSString *)event withProperties:(NSDictionary *)properties
+{
+    NSParameterAssert(properties != nil);
+    for (id<WPAnalyticsTracker> tracker in [self trackers]) {
+        [tracker trackString:event withProperties:properties];
+    }
+}
+
 + (void)beginSession
 {
     for (id<WPAnalyticsTracker> tracker in [self trackers]) {

WordPressSharedTests/TestAnalyticsTracker.m

@@ -27,4 +27,14 @@ - (void)refreshMetadata
     // No-op
 }
 
+- (void)trackString:(NSString *)event {
+    // No-op
+}
+
+
+- (void)trackString:(NSString *)event withProperties:(NSDictionary *)properties {
+    // No-op
+}
+
+
 @end

WordPressSharedTests/WPAnalyticsTests.m

@@ -117,4 +117,27 @@
     itShouldBehaveLike(@"a WPAnalyticsTracker method", @{@"invocation": invocation});
 });
 
-SpecEnd
+describe(@"trackString:", ^{
+    NSMethodSignature *signature = [TestAnalyticsTracker instanceMethodSignatureForSelector:@selector(trackString:)];
+    NSInvocation *invocation  = [NSInvocation invocationWithMethodSignature:signature];
+    [invocation setSelector:@selector(trackString:)];
+    NSString *event = @"my_event";
+    [invocation setArgument:&event atIndex:2];
+
+    itShouldBehaveLike(@"a WPAnalyticsTracker method", @{@"invocation": invocation});
+});
+
+describe(@"trackString:withProperties:", ^{
+    NSMethodSignature *signature = [TestAnalyticsTracker instanceMethodSignatureForSelector:@selector(trackString:withProperties:)];
+    NSInvocation *invocation  = [NSInvocation invocationWithMethodSignature:signature];
+    [invocation setSelector:@selector(trackString:withProperties:)];
+
+    NSString *event = @"my_event";
+    NSDictionary *dict = @{};
+    [invocation setArgument:&event atIndex:2];
+    [invocation setArgument:&dict atIndex:3];
+
+    itShouldBehaveLike(@"a WPAnalyticsTracker method", @{@"invocation": invocation});
+});
+
+SpecEnd