Remove skipping logic and ignore a test entirely if not a snapshot te…

…st (#253)

* Skip marking test as ignored for test discovery phase to prevent skipped tests from being handled during execution

* Test

* Tweak

* Lint

snapshots/sample/app/src/androidTest/kotlin/com/emergetools/snapshots/sample/ExampleSkippedTest.kt

@@ -0,0 +1,24 @@
+package com.emergetools.snapshots.sample
+
+import android.util.Log
+import androidx.test.ext.junit.rules.ActivityScenarioRule
+import androidx.test.ext.junit.runners.AndroidJUnit4
+import com.emergetools.snapshots.EmergeSnapshots
+import org.junit.Rule
+import org.junit.Test
+import org.junit.runner.RunWith
+
+@RunWith(AndroidJUnit4::class)
+class ExampleSkippedTest {
+
+  @get:Rule
+  val activityScenarioRule = ActivityScenarioRule(MainActivity::class.java)
+
+  @Test
+  fun basicActivityView() {
+    val scenario = activityScenarioRule.scenario
+    scenario.onActivity {
+      Log.d("ExampleSkippedTest", "basicActivityView")
+    }
+  }
+}

snapshots/snapshots/src/main/kotlin/com/emergetools/snapshots/runner/SnapshotsRunner.kt

@@ -26,8 +26,9 @@ internal class SnapshotsRunner(
       Log.d(TAG, "Running test class: ${testClass.simpleName}")
       super.run(notifier)
     } else {
+      // Note: We intentionally do not mark the test as ignored as we
+      // don't care to acknowledge the test to ensure it doesn't take test run time/get assigned to an emulator
       Log.d(TAG, "Ignoring test class: ${testClass.simpleName}")
-      notifier.fireTestIgnored(description)
     }
   }