Merge pull request #95 from Natifishman/feature/tests

Test: Add Test Infrastructure

Author: Natifishman

app/src/test/java/com/example/partymaker/repository/GroupRepositoryTest.java

@@ -58,7 +58,7 @@ public void setUp() {
   public void testGetGroup_CacheHit_ReturnsFromCache() {
     // Given
     String groupKey = "test-group-key";
-    Group mockGroup = createMockGroup(groupKey, "Test Group");
+    Group mockGroup = createMockGroup(groupKey);
 
     doAnswer(
             invocation -> {
@@ -99,7 +99,7 @@ public void onError(String error) {
   public void testGetGroup_CacheMiss_FallsBackToRemote() {
     // Given
     String groupKey = "test-group-key";
-    Group mockGroup = createMockGroup(groupKey, "Test Group");
+    Group mockGroup = createMockGroup(groupKey);
 
     // Mock cache miss
     doAnswer(
@@ -159,7 +159,7 @@ public void onError(String error) {
   public void testGetGroup_ForceRefresh_SkipsCache() {
     // Given
     String groupKey = "test-group-key";
-    Group mockGroup = createMockGroup(groupKey, "Test Group");
+    Group mockGroup = createMockGroup(groupKey);
 
     // Mock successful remote fetch
     doAnswer(
@@ -200,7 +200,7 @@ public void onError(String error) {
   public void testSaveGroup_Success_SavesRemoteAndLocal() {
     // Given
     String groupKey = "test-group-key";
-    Group mockGroup = createMockGroup(groupKey, "Test Group");
+    Group mockGroup = createMockGroup(groupKey);
 
     // Mock successful remote save
     doAnswer(
@@ -247,7 +247,7 @@ public void onError(String error) {
   public void testObserveGroup_ReturnsLiveData() {
     // Given
     String groupKey = "test-group-key";
-    Group mockGroup = createMockGroup(groupKey, "Test Group");
+    Group mockGroup = createMockGroup(groupKey);
     MutableLiveData<Group> mockLiveData = new MutableLiveData<>();
     mockLiveData.setValue(mockGroup);
 
@@ -263,10 +263,10 @@ public void testObserveGroup_ReturnsLiveData() {
   }
 
   /** Helper method to create a mock Group for testing. */
-  private Group createMockGroup(String key, String name) {
+  private Group createMockGroup(String key) {
     Group group = new Group();
     group.setGroupKey(key);
-    group.setGroupName(name);
+    group.setGroupName("Test Group");
     group.setGroupLocation("Test Location");
     group.setAdminKey("test-admin");
     return group;

build.gradle.kts

@@ -10,31 +10,54 @@ plugins {
 
 buildscript {
     repositories {
+        // Use Google's Maven repository first for Android dependencies
         google()
+        // Use Maven Central for other dependencies  
         mavenCentral()
+        // Use Gradle Plugin Portal for Gradle plugins
+        gradlePluginPortal()
     }
     dependencies {
         // Plugin for managing secrets (API keys, etc.)
         classpath(libs.secrets.gradle.plugin)
+        
+        // Build optimization plugins
+        classpath("com.getkeepsafe.dexcount:dexcount-gradle-plugin:4.0.0")
+        classpath("com.github.ben-manes:gradle-versions-plugin:0.52.0")
     }
 }
 
 // Configure all projects in the build
 allprojects {
+
     // Apply common configurations to all modules
     tasks.withType<org.jetbrains.kotlin.gradle.tasks.KotlinCompile>().configureEach {
         compilerOptions {
-            // Enable Java 8 compatibility
+            // Enable Java 11 compatibility 
             jvmTarget.set(org.jetbrains.kotlin.gradle.dsl.JvmTarget.JVM_11)
 
-            // Use the experimental Kotlin compiler
-            freeCompilerArgs.add("-Xopt-in=kotlin.RequiresOptIn")
+            // Kotlin compiler optimizations
+            freeCompilerArgs.addAll(listOf(
+                "-Xopt-in=kotlin.RequiresOptIn",
+                "-Xjvm-default=all",
+                "-Xuse-k2" // Use K2 compiler for better performance
+            ))
         }
     }
 
     tasks.withType<JavaCompile>().configureEach {
         sourceCompatibility = JavaVersion.VERSION_11.toString()
         targetCompatibility = JavaVersion.VERSION_11.toString()
+        
+        // Java compiler optimizations
+        options.apply {
+            isIncremental = true
+            isFork = true
+            compilerArgs.addAll(listOf(
+                "-Xlint:none", // Disable lint warnings for faster compilation
+                "-nowarn" // Suppress warnings
+            ))
+        }
     }
 }
 
@@ -51,4 +74,15 @@ tasks.withType<Delete> {
     }
 }
 
+// Build performance monitoring (configuration cache compatible)
+tasks.register("buildTimeReport") {
+    group = "build performance"
+    description = "Reports build performance metrics"
+    
+    doLast {
+        println("Build performance monitoring available")
+        println("Use --scan flag for detailed build insights")
+    }
+}
+
 // No dependencies or configuration should be added here unless it applies to ALL modules.