test(project): Add more deep linking tests

Hussein Aladeen · Hussein Aladeen · commit c2c63431e453 · 2017-06-05T14:49:13.000+01:00
diff --git a/extra/gradle/libraries.gradle b/extra/gradle/libraries.gradle
@@ -28,6 +28,8 @@ ext {
     hamcrestVersion = '1.4-atlassian-1'
     espressoVersion = '2.2.2'
     mockitoVersion = '1.10.19'
+    compileTestingVersion = '0.10'
+    googleTruthVersion = '0.30'
 
     javaPoetVersion = '1.8.0'
     autoCommon = '0.8'
@@ -54,7 +56,9 @@ ext {
             testRule          : "com.android.support.test:rules:${tesetRunnerVersion}",
             hamcrest          : "org.hamcrest:hamcrest-library:${hamcrestVersion}",
             espresso          : "com.android.support.test.espresso:espresso-core:${espressoVersion}",
-            mockito           : "org.mockito:mockito-core:${mockitoVersion}"
+            mockito           : "org.mockito:mockito-core:${mockitoVersion}",
+            compileTesting    : "com.google.testing.compile:compile-testing:${compileTestingVersion}",
+            googleTruth       : "com.google.truth:truth:${googleTruthVersion}"
     ]
 
 }
diff --git a/flowr-annotations/build.gradle b/flowr-annotations/build.gradle
@@ -1,5 +1,7 @@
 apply plugin: 'java'
 apply plugin: 'com.github.dcendents.android-maven'
+apply plugin: 'checkstyle'
+apply from: "../extra/checkstyle/checkstyle_java_library.gradle"
 
 group = libraryGroup
 version = libraryVersion
diff --git a/flowr-compiler/build.gradle b/flowr-compiler/build.gradle
@@ -1,5 +1,9 @@
+import org.gradle.internal.jvm.Jvm
+
 apply plugin: 'java'
 apply plugin: 'com.github.dcendents.android-maven'
+apply plugin: 'checkstyle'
+apply from: "../extra/checkstyle/checkstyle_java_library.gradle"
 
 group = libraryGroup
 version = libraryVersion
@@ -12,9 +16,14 @@ dependencies {
     compile libraries.autoCommon
     compile libraries.autoService
 
+    testCompile project(':flowr-mock-classes')
+    testCompile testLibraries.junit
     testCompile testLibraries.hamcrest
     testCompile testLibraries.junit
     testCompile testLibraries.mockito
+    testCompile testLibraries.compileTesting
+    testCompile testLibraries.googleTruth
+    testCompile files(Jvm.current().getToolsJar())
 }
 
 
diff --git a/flowr-compiler/src/main/java/com/fueled/flowr/compilers/DeepLinkAnnotationCompiler.java b/flowr-compiler/src/main/java/com/fueled/flowr/compilers/DeepLinkAnnotationCompiler.java
@@ -15,6 +15,8 @@
 import javax.annotation.processing.Processor;
 import javax.annotation.processing.RoundEnvironment;
 import javax.annotation.processing.SupportedAnnotationTypes;
+import javax.annotation.processing.SupportedSourceVersion;
+import javax.lang.model.SourceVersion;
 import javax.lang.model.element.Element;
 import javax.lang.model.element.Modifier;
 import javax.lang.model.element.TypeElement;
@@ -26,6 +28,7 @@
  */
 @SupportedAnnotationTypes({"com.fueled.flowr.annotations.DeepLink",
         "com.fueled.flowr.annotations.DeepLinkHandler"})
+@SupportedSourceVersion(SourceVersion.RELEASE_7)
 @AutoService(Processor.class)
 public class DeepLinkAnnotationCompiler extends AbstractProcessor {
 
diff --git a/flowr-compiler/src/test/java/com/fueled/flowr/compilers/DeepLinkAnnotationCompilerTest.java b/flowr-compiler/src/test/java/com/fueled/flowr/compilers/DeepLinkAnnotationCompilerTest.java
@@ -1,12 +1,15 @@
 package com.fueled.flowr.compilers;
 
-import org.junit.Before;
+import com.google.testing.compile.JavaFileObjects;
+
 import org.junit.Test;
 
-import java.util.ArrayList;
-import java.util.List;
+import java.util.Arrays;
+
+import javax.tools.JavaFileObject;
 
-import static org.junit.Assert.assertEquals;
+import static com.google.common.truth.Truth.assertAbout;
+import static com.google.testing.compile.JavaSourcesSubjectFactory.javaSources;
 
 /**
  * Created by julienFueled on 5/25/17.
@@ -15,16 +18,39 @@ public class DeepLinkAnnotationCompilerTest {
 
     private static final String TEST_PACKAGE = "com.fueled.flowr.sample";
 
-    private DeepLinkAnnotationCompiler compiler;
+    private static final JavaFileObject TEST_DEEP_LINK_FRAGMENT = JavaFileObjects
+            .forSourceString("HomeFragment", "package " + TEST_PACKAGE + ";\n" +
+                    "import com.fueled.flowr.AbstractFlowrFragment;\n" +
+                    "import com.fueled.flowr.annotations.DeepLink;\n" +
+                    "@DeepLink(value = {\"/test\"})\n" +
+                    "public class HomeFragment extends AbstractFlowrFragment {\n" +
+                    "}");
+
+    private static final JavaFileObject TEST_DEEP_LINK_HANDLER = JavaFileObjects
+            .forSourceString("TestDeepLinkHandler", "package " + TEST_PACKAGE + ";\n" +
+                    "import com.fueled.flowr.annotations.DeepLinkHandler;\n" +
+                    "@DeepLinkHandler\n" +
+                    "public interface TestDeepLinkHandler {\n" +
+                    "}");
+
+    private static final JavaFileObject TEST_DEEP_GENERATED_HANDLER = JavaFileObjects
+            .forSourceString("TestDeepLinkHandlerImpl", "package com.fueled.flowr.sample;\n" +
+                    "import com.fueled.flowr.internal.AbstractFlowrDeepLinkHandler;\n" +
+                    "public final class TestDeepLinkHandlerImpl extends AbstractFlowrDeepLinkHandler {\n" +
+                    "  public TestDeepLinkHandlerImpl() {\n" +
+                    "    addFragment(\"/test\", HomeFragment.class);\n" +
+                    "  }\n" +
+                    "}");
 
-    @Before
-    public void setup() {
-        compiler = new DeepLinkAnnotationCompiler();
-    }
 
     @Test
     public void process() throws Exception {
-
+        assertAbout(javaSources())
+                .that(Arrays.asList(TEST_DEEP_LINK_FRAGMENT, TEST_DEEP_LINK_HANDLER))
+                .processedWith(new DeepLinkAnnotationCompiler())
+        .compilesWithoutError()
+        .and()
+        .generatesSources(TEST_DEEP_GENERATED_HANDLER);
     }
 
 }
diff --git a/flowr-mock-classes/build.gradle b/flowr-mock-classes/build.gradle
@@ -0,0 +1,8 @@
+apply plugin: 'java'
+
+dependencies {
+    compile fileTree(dir: 'libs', include: ['*.jar'])
+}
+
+sourceCompatibility = "1.7"
+targetCompatibility = "1.7"
diff --git a/flowr-mock-classes/src/main/java/com/fueled/flowr/AbstractFlowrFragment.java b/flowr-mock-classes/src/main/java/com/fueled/flowr/AbstractFlowrFragment.java
@@ -0,0 +1,7 @@
+package com.fueled.flowr;
+
+/**
+ * Mock AbstractFlowrFragment class used for compiler tests.
+ */
+public class AbstractFlowrFragment {
+}
diff --git a/flowr-mock-classes/src/main/java/com/fueled/flowr/internal/AbstractFlowrDeepLinkHandler.java b/flowr-mock-classes/src/main/java/com/fueled/flowr/internal/AbstractFlowrDeepLinkHandler.java
@@ -0,0 +1,12 @@
+package com.fueled.flowr.internal;
+
+/**
+ * Mock AbstractFlowrDeepLinkHandler class used for compiler tests.
+ */
+public class AbstractFlowrDeepLinkHandler {
+
+    protected void addFragment(String url, Class clazz) {
+
+    }
+
+}
diff --git a/flowr/build.gradle b/flowr/build.gradle
@@ -32,4 +32,5 @@ dependencies {
     // Test
     testCompile testLibraries.hamcrest
     testCompile testLibraries.junit
+    testCompile testLibraries.mockito
 }
diff --git a/flowr/src/main/java/com/fueled/flowr/internal/AbstractFlowrDeepLinkHandler.java b/flowr/src/main/java/com/fueled/flowr/internal/AbstractFlowrDeepLinkHandler.java
@@ -5,6 +5,7 @@
 import android.os.Bundle;
 import android.support.annotation.NonNull;
 import android.support.annotation.Nullable;
+import android.support.annotation.VisibleForTesting;
 import android.support.v4.app.Fragment;
 
 import com.fueled.flowr.Flowr;
@@ -52,13 +53,13 @@ protected void addFragment(String url, Class<? extends T> fragment) {
      * @return a bundle containing all path variables and parameters as strings.
      */
     @Nullable
-    private static Bundle bundleUriInfo(Uri uri, String pattern) {
+    private Bundle bundleUriInfo(Uri uri, String pattern) {
         String regex = getRegexPattern(pattern);
         String path = uri.getPath();
         Matcher m = Pattern.compile(regex).matcher(path);
 
         if (m.matches()) {
-            Bundle data = new Bundle();
+            Bundle data = getNewBundle();
             data.putString(Flowr.DEEP_LINK_URL, uri.toString());
             Iterator<String> params = getNamedGroupCandidates(regex).iterator();
 
@@ -76,11 +77,11 @@ private static Bundle bundleUriInfo(Uri uri, String pattern) {
         return null;
     }
 
-    private static String getRegexPattern(String pattern) {
+    private String getRegexPattern(String pattern) {
         return pattern.replaceAll("\\{(.+?)\\}", "(?<$1>\\.+?)");
     }
 
-    private static List<String> getNamedGroupCandidates(String regex) {
+    private List<String> getNamedGroupCandidates(String regex) {
         List<String> namedGroups = new ArrayList<>();
 
         Matcher m = NAMED_PARAM_PATTERN.matcher(regex);
@@ -93,11 +94,16 @@ private static List<String> getNamedGroupCandidates(String regex) {
         return namedGroups;
     }
 
+    @VisibleForTesting
+    protected Bundle getNewBundle() {
+        return new Bundle();
+    }
+
     /**
      * @inheritDoc
      */
     @Nullable
-    public FlowrDeepLinkInfo getDeepLinkInfoForIntent(@NonNull Intent intent) {
+    public FlowrDeepLinkInfo<T> getDeepLinkInfoForIntent(@NonNull Intent intent) {
         Uri uri = intent.getData();
 
         if (uri != null) {
diff --git a/flowr/src/test/java/com/fueled/flowr/internal/AbstractFlowrDeepLinkHandlerTest.java b/flowr/src/test/java/com/fueled/flowr/internal/AbstractFlowrDeepLinkHandlerTest.java
@@ -0,0 +1,106 @@
+package com.fueled.flowr.internal;
+
+import android.content.Intent;
+import android.net.Uri;
+import android.os.Bundle;
+
+import com.fueled.flowr.AbstractFlowrFragment;
+import com.fueled.flowr.Flowr;
+
+import org.junit.Before;
+import org.junit.Test;
+import org.junit.runner.RunWith;
+import org.mockito.Mock;
+import org.mockito.runners.MockitoJUnitRunner;
+
+import static org.junit.Assert.assertEquals;
+import static org.junit.Assert.assertNotNull;
+import static org.junit.Assert.assertNull;
+import static org.mockito.Mockito.spy;
+import static org.mockito.Mockito.verify;
+import static org.mockito.Mockito.verifyNoMoreInteractions;
+import static org.mockito.Mockito.verifyZeroInteractions;
+import static org.mockito.Mockito.when;
+
+/**
+ * Created by hussein@fueled.com on 05/06/2017.
+ * Copyright (c) 2017 Fueled. All rights reserved.
+ */
+@RunWith(MockitoJUnitRunner.class)
+public class AbstractFlowrDeepLinkHandlerTest {
+
+    @Mock Intent intent;
+    @Mock Uri uri;
+    @Mock Bundle bundle;
+
+    private AbstractFlowrDeepLinkHandler<AbstractFlowrFragment> deepLinkHandler;
+
+    @Before
+    public void setup() {
+        deepLinkHandler = spy(new AbstractFlowrDeepLinkHandler<AbstractFlowrFragment>());
+
+        when(intent.getData()).thenReturn(uri);
+        when(deepLinkHandler.getNewBundle()).thenReturn(bundle);
+    }
+
+    @Test
+    public void testMapLinkWithNoArguments() {
+        when(uri.getPath()).thenReturn("/hello");
+        when(uri.toString()).thenReturn("http://fueled.com/hello");
+
+        deepLinkHandler.addFragment("/hi", DemoFragment.class);
+        deepLinkHandler.addFragment("/hello", SampleFragment.class);
+
+        FlowrDeepLinkInfo<AbstractFlowrFragment> info = deepLinkHandler.getDeepLinkInfoForIntent(intent);
+
+        verify(bundle).putString(Flowr.DEEP_LINK_URL, "http://fueled.com/hello");
+        verifyNoMoreInteractions(bundle);
+
+        assertNotNull(info);
+        assertEquals(SampleFragment.class, info.fragment);
+        assertNotNull(info.data);
+    }
+
+    @Test
+    public void testMapLinkWithPathVariable() {
+        when(uri.getPath()).thenReturn("/m/123");
+        when(uri.toString()).thenReturn("http://fueled.com/m/123");
+
+        deepLinkHandler.addFragment("/m/{id}", DemoFragment.class);
+        deepLinkHandler.addFragment("/hello", SampleFragment.class);
+
+        FlowrDeepLinkInfo<AbstractFlowrFragment> info = deepLinkHandler.getDeepLinkInfoForIntent(intent);
+
+        verify(bundle).putString(Flowr.DEEP_LINK_URL, "http://fueled.com/m/123");
+        verify(bundle).putString("id", "123");
+        verifyNoMoreInteractions(bundle);
+
+        assertNotNull(info);
+        assertEquals(DemoFragment.class, info.fragment);
+        assertNotNull(info.data);
+    }
+
+    @Test
+    public void testMapLinkNoMappingFound() {
+        when(uri.getPath()).thenReturn("/m/123");
+        when(uri.toString()).thenReturn("http://fueled.com/m/123");
+
+        deepLinkHandler.addFragment("/hi", DemoFragment.class);
+        deepLinkHandler.addFragment("/hello", SampleFragment.class);
+
+        FlowrDeepLinkInfo<AbstractFlowrFragment> info = deepLinkHandler.getDeepLinkInfoForIntent(intent);
+
+        verifyZeroInteractions(bundle);
+
+        assertNull(info);
+    }
+
+    public static class SampleFragment extends AbstractFlowrFragment {
+
+    }
+
+    public static class DemoFragment extends AbstractFlowrFragment {
+
+    }
+
+}
diff --git a/settings.gradle b/settings.gradle
@@ -1 +1 @@
-include ':flowr', ':sample', ':flowr-annotations', ':flowr-compiler'
+include ':flowr', ':sample', ':flowr-annotations', ':flowr-compiler', ':flowr-mock-classes'

Original file line number	Diff line number	Diff line change
`@@ -32,4 +32,5 @@ dependencies {`
`32`	`32`	`// Test`
`33`	`33`	`testCompile testLibraries.hamcrest`
`34`	`34`	`testCompile testLibraries.junit`
	`35`	`+ testCompile testLibraries.mockito`
`35`	`36`	`}`