Large-scale cleanup (#794)

* Large-scale cleanup * Large-scale cleanup * Fix tests * Cleanup * Fix project description * Rebuilds
duckie-team · Jul 7, 2023 · 373a0a6 · 373a0a6
1 parent aa92056
commit 373a0a6
Show file tree

Hide file tree

Showing 136 changed files with 1,401 additions and 2,123 deletions.
diff --git a/.gitignore b/.gitignore
@@ -5,6 +5,7 @@
 !/.idea/codeStyleSettings.xml
 !/.idea/codeStyles
 !/.idea/dictionaries
+!/.idea/inspectionProfiles
 *.iml
 .gradle
 build

diff --git a/.idea/inspectionProfiles/Project_Default.xml b/.idea/inspectionProfiles/Project_Default.xml
diff --git a/.run/catalog.run.xml b/.run/catalog.run.xml
@@ -0,0 +1,68 @@
+<component name="ProjectRunConfigurationManager">
+  <configuration default="false" name="catalog" type="AndroidRunConfigurationType" factoryName="Android App" activateToolWindowBeforeRun="false">
+    <module name="quack-quack.catalog.main" />
+    <option name="DEPLOY" value="true" />
+    <option name="DEPLOY_APK_FROM_BUNDLE" value="false" />
+    <option name="DEPLOY_AS_INSTANT" value="false" />
+    <option name="ARTIFACT_NAME" value="" />
+    <option name="PM_INSTALL_OPTIONS" value="" />
+    <option name="ALL_USERS" value="false" />
+    <option name="ALWAYS_INSTALL_WITH_PM" value="false" />
+    <option name="CLEAR_APP_STORAGE" value="false" />
+    <option name="ACTIVITY_EXTRA_FLAGS" value="" />
+    <option name="MODE" value="default_activity" />
+    <option name="CLEAR_LOGCAT" value="false" />
+    <option name="SHOW_LOGCAT_AUTOMATICALLY" value="false" />
+    <option name="INSPECTION_WITHOUT_ACTIVITY_RESTART" value="false" />
+    <option name="TARGET_SELECTION_MODE" value="DEVICE_AND_SNAPSHOT_COMBO_BOX" />
+    <option name="SELECTED_CLOUD_MATRIX_CONFIGURATION_ID" value="-1" />
+    <option name="SELECTED_CLOUD_MATRIX_PROJECT_ID" value="" />
+    <option name="DEBUGGER_TYPE" value="Java" />
+    <Auto>
+      <option name="USE_JAVA_AWARE_DEBUGGER" value="false" />
+      <option name="SHOW_STATIC_VARS" value="true" />
+      <option name="WORKING_DIR" value="" />
+      <option name="TARGET_LOGGING_CHANNELS" value="lldb process:gdb-remote packets" />
+      <option name="SHOW_OPTIMIZED_WARNING" value="true" />
+      <option name="ATTACH_ON_WAIT_FOR_DEBUGGER" value="false" />
+      <option name="DEBUG_SANDBOX_SDK" value="false" />
+    </Auto>
+    <Hybrid>
+      <option name="USE_JAVA_AWARE_DEBUGGER" value="false" />
+      <option name="SHOW_STATIC_VARS" value="true" />
+      <option name="WORKING_DIR" value="" />
+      <option name="TARGET_LOGGING_CHANNELS" value="lldb process:gdb-remote packets" />
+      <option name="SHOW_OPTIMIZED_WARNING" value="true" />
+      <option name="ATTACH_ON_WAIT_FOR_DEBUGGER" value="false" />
+      <option name="DEBUG_SANDBOX_SDK" value="false" />
+    </Hybrid>
+    <Java>
+      <option name="ATTACH_ON_WAIT_FOR_DEBUGGER" value="false" />
+      <option name="DEBUG_SANDBOX_SDK" value="false" />
+    </Java>
+    <Native>
+      <option name="USE_JAVA_AWARE_DEBUGGER" value="false" />
+      <option name="SHOW_STATIC_VARS" value="true" />
+      <option name="WORKING_DIR" value="" />
+      <option name="TARGET_LOGGING_CHANNELS" value="lldb process:gdb-remote packets" />
+      <option name="SHOW_OPTIMIZED_WARNING" value="true" />
+      <option name="ATTACH_ON_WAIT_FOR_DEBUGGER" value="false" />
+      <option name="DEBUG_SANDBOX_SDK" value="false" />
+    </Native>
+    <Profilers>
+      <option name="ADVANCED_PROFILING_ENABLED" value="false" />
+      <option name="STARTUP_PROFILING_ENABLED" value="false" />
+      <option name="STARTUP_CPU_PROFILING_ENABLED" value="false" />
+      <option name="STARTUP_CPU_PROFILING_CONFIGURATION_NAME" value="Java/Kotlin Method Sample (legacy)" />
+      <option name="STARTUP_NATIVE_MEMORY_PROFILING_ENABLED" value="false" />
+      <option name="NATIVE_MEMORY_SAMPLE_RATE_BYTES" value="2048" />
+    </Profilers>
+    <option name="DEEP_LINK" value="" />
+    <option name="ACTIVITY_CLASS" value="" />
+    <option name="SEARCH_ACTIVITY_IN_GLOBAL_SCOPE" value="false" />
+    <option name="SKIP_ACTIVITY_VALIDATION" value="false" />
+    <method v="2">
+      <option name="Android.Gradle.BeforeRunTask" enabled="true" />
+    </method>
+  </configuration>
+</component>
diff --git a/README.md b/README.md
@@ -3,18 +3,19 @@
 </p>
 <h1 align="center">QuackQuack</h1>
 
-<img src="assets/QuackTextSnapshot_ModifierSpan.png" align="right"/>
+<img src="assets/awesome-quackquack.png" align="right"/>
 
 ```kotlin
-QuackText(
+
+ import java.awt.ColorQuackText(
   modifier = Modifier
     .background(color = Color.White)
     .span(
       texts = listOf("QuackQuack", "Duckie"),
       style = SpanStyle(color = QuackColor.DuckieOrange),
     )
     .padding(30.dp),
-  text = "QuackQuack is an awesome ui kit created by the Duckie team.",
+  text = "QuackQuack is an awesome design system created by the Duckie team.",
   typography = QuackTypography.Body1,
 )
 ```

diff --git a/animation/build.gradle.kts b/animation/build.gradle.kts
@@ -5,13 +5,13 @@
  * Please see full license: https://github.com/duckie-team/quack-quack-android/blob/main/LICENSE
  */
 
-@file:Suppress("INLINE_FROM_HIGHER_PLATFORM")
-
 plugins {
   quackquack("android-library")
   quackquack("android-compose")
   quackquack("android-compose-metrics")
   quackquack("kotlin-explicit-api")
+  quackquack("test-junit")
+  quackquack("test-roborazzi")
   quackquack("quack-publishing")
 }
 
@@ -25,4 +25,5 @@ dependencies {
     libs.compose.animation,
     projects.util.orArtifact(),
   )
+  testImplementation(libs.compose.foundation)
 }
diff --git a/animation/report/compose-metrics/animation_debug-module.json b/animation/report/compose-metrics/animation_debug-module.json
@@ -1,25 +1,25 @@
 {
- "skippableComposables": 2,
- "restartableComposables": 2,
+ "skippableComposables": 0,
+ "restartableComposables": 0,
  "readonlyComposables": 0,
- "totalComposables": 5,
- "restartGroups": 2,
- "totalGroups": 5,
- "staticArguments": 1,
- "certainArguments": 25,
- "knownStableArguments": 42,
- "knownUnstableArguments": 9,
- "unknownStableArguments": 1,
- "totalArguments": 52,
+ "totalComposables": 3,
+ "restartGroups": 0,
+ "totalGroups": 3,
+ "staticArguments": 0,
+ "certainArguments": 18,
+ "knownStableArguments": 30,
+ "knownUnstableArguments": 11,
+ "unknownStableArguments": 0,
+ "totalArguments": 41,
  "markedStableClasses": 0,
  "inferredStableClasses": 0,
- "inferredUnstableClasses": 1,
+ "inferredUnstableClasses": 0,
  "inferredUncertainClasses": 0,
  "effectivelyStableClasses": 0,
- "totalClasses": 1,
- "memoizedLambdas": 1,
- "singletonLambdas": 1,
+ "totalClasses": 0,
+ "memoizedLambdas": 0,
+ "singletonLambdas": 0,
  "singletonComposableLambdas": 0,
  "composableLambdas": 0,
- "totalLambdas": 1
+ "totalLambdas": 0
 }
diff --git a/animation/report/compose-metrics/animation_debugUnitTest-module.json b/animation/report/compose-metrics/animation_debugUnitTest-module.json
@@ -0,0 +1,25 @@
+{
+ "skippableComposables": 3,
+ "restartableComposables": 3,
+ "readonlyComposables": 0,
+ "totalComposables": 3,
+ "restartGroups": 3,
+ "totalGroups": 3,
+ "staticArguments": 7,
+ "certainArguments": 0,
+ "knownStableArguments": 40,
+ "knownUnstableArguments": 3,
+ "unknownStableArguments": 0,
+ "totalArguments": 43,
+ "markedStableClasses": 0,
+ "inferredStableClasses": 0,
+ "inferredUnstableClasses": 0,
+ "inferredUncertainClasses": 3,
+ "effectivelyStableClasses": 0,
+ "totalClasses": 3,
+ "memoizedLambdas": 6,
+ "singletonLambdas": 0,
+ "singletonComposableLambdas": 3,
+ "composableLambdas": 3,
+ "totalLambdas": 6
+}
diff --git a/animation/report/compose-metrics/animation_release-module.json b/animation/report/compose-metrics/animation_release-module.json
@@ -1,25 +1,25 @@
 {
- "skippableComposables": 2,
- "restartableComposables": 2,
+ "skippableComposables": 0,
+ "restartableComposables": 0,
  "readonlyComposables": 0,
- "totalComposables": 5,
- "restartGroups": 2,
- "totalGroups": 5,
- "staticArguments": 1,
- "certainArguments": 25,
- "knownStableArguments": 42,
- "knownUnstableArguments": 9,
- "unknownStableArguments": 1,
- "totalArguments": 52,
+ "totalComposables": 3,
+ "restartGroups": 0,
+ "totalGroups": 3,
+ "staticArguments": 0,
+ "certainArguments": 18,
+ "knownStableArguments": 30,
+ "knownUnstableArguments": 11,
+ "unknownStableArguments": 0,
+ "totalArguments": 41,
  "markedStableClasses": 0,
  "inferredStableClasses": 0,
- "inferredUnstableClasses": 1,
+ "inferredUnstableClasses": 0,
  "inferredUncertainClasses": 0,
  "effectivelyStableClasses": 0,
- "totalClasses": 1,
- "memoizedLambdas": 1,
- "singletonLambdas": 1,
+ "totalClasses": 0,
+ "memoizedLambdas": 0,
+ "singletonLambdas": 0,
  "singletonComposableLambdas": 0,
  "composableLambdas": 0,
- "totalLambdas": 1
+ "totalLambdas": 0
 }
diff --git a/animation/report/compose-metrics/animation_releaseUnitTest-module.json b/animation/report/compose-metrics/animation_releaseUnitTest-module.json
@@ -0,0 +1,25 @@
+{
+ "skippableComposables": 3,
+ "restartableComposables": 3,
+ "readonlyComposables": 0,
+ "totalComposables": 3,
+ "restartGroups": 3,
+ "totalGroups": 3,
+ "staticArguments": 7,
+ "certainArguments": 0,
+ "knownStableArguments": 40,
+ "knownUnstableArguments": 3,
+ "unknownStableArguments": 0,
+ "totalArguments": 43,
+ "markedStableClasses": 0,
+ "inferredStableClasses": 0,
+ "inferredUnstableClasses": 0,
+ "inferredUncertainClasses": 3,
+ "effectivelyStableClasses": 0,
+ "totalClasses": 3,
+ "memoizedLambdas": 6,
+ "singletonLambdas": 0,
+ "singletonComposableLambdas": 3,
+ "composableLambdas": 3,
+ "totalLambdas": 6
+}
diff --git a/animation/report/compose-reports/animation_debug-classes.txt b/animation/report/compose-reports/animation_debug-classes.txt
@@ -1,4 +0,0 @@
-unstable class QuackAnimationSpec {
-  stable var snapMode: Boolean
-  <runtime stability> = Unstable
-}

diff --git a/animation/report/compose-reports/animation_debug-composables.csv b/animation/report/compose-reports/animation_debug-composables.csv
@@ -1,6 +1,4 @@
 package,name,composable,skippable,restartable,readonly,inline,isLambda,hasDefaults,defaultsGroup,groups,calls,
-team.duckie.quackquack.animation.QuackAnimatedContent,QuackAnimatedContent,1,1,1,0,0,0,0,0,1,1,
-team.duckie.quackquack.animation.QuackAnimatedVisibility,QuackAnimatedVisibility,1,1,1,0,0,0,0,0,1,1,
 team.duckie.quackquack.animation.animatedQuackBorderAsState,animatedQuackBorderAsState,1,0,0,0,0,0,0,0,1,2,
 team.duckie.quackquack.animation.animateQuackColorAsState,animateQuackColorAsState,1,0,0,0,0,0,0,0,1,2,
-team.duckie.quackquack.animation.animatedQuackTextStyleAsState,animatedQuackTextStyleAsState,1,0,0,0,0,0,0,0,1,5,
+team.duckie.quackquack.animation.animatedQuackTypographyAsState,animatedQuackTypographyAsState,1,0,0,0,0,0,0,0,1,5,
diff --git a/animation/report/compose-reports/animation_debug-composables.txt b/animation/report/compose-reports/animation_debug-composables.txt
@@ -1,31 +1,20 @@
-restartable skippable fun QuackAnimatedContent(
-  stable modifier: Modifier? = @static Companion
-  targetState: T
-  stable label: String? = @static "AnimatedContent"
-  stable content: @[ExtensionFunctionType] Function4<AnimatedVisibilityScope, @[ParameterName(name = 'animatedTargetState')] T, Composer, Int, Unit>
-)
-restartable skippable scheme("[0, [0]]") fun QuackAnimatedVisibility(
-  stable modifier: Modifier? = @static Companion
-  stable visible: Boolean
-  stable label: String? = @static "AnimatedVisibility"
-  stable otherEnterAnimation: EnterTransition? = @static null
-  stable otherExitAnimation: ExitTransition? = @static null
-  stable content: @[ExtensionFunctionType] Function3<AnimatedVisibilityScope, Composer, Int, Unit>
-)
 fun animatedQuackBorderAsState(
   stable targetValue: QuackBorder
-  stable label: String? = @static "QuackBorderAnimation"
+  unstable animationSpec: AnimationSpec<Any>? = @static quackTween()
+  stable label: String? = @static "QuackBorder"
   stable widthAnimationfinishedListener: Function1<@[ParameterName(name = 'dp')] Dp, Unit>? = @static null
   stable colorAnimationFinishedListener: Function1<@[ParameterName(name = 'color')] QuackColor, Unit>? = @static null
 ): QuackBorder
 fun animateQuackColorAsState(
   stable targetValue: QuackColor
-  stable label: String? = @static "animateQuackColorAsState"
+  unstable animationSpec: AnimationSpec<QuackColor>? = @static quackTween()
+  stable label: String? = @static "QuackColor"
   stable finishedListener: Function1<@[ParameterName(name = 'color')] QuackColor, Unit>? = @static null
 ): State<QuackColor>
-fun animatedQuackTextStyleAsState(
+fun animatedQuackTypographyAsState(
   stable targetValue: QuackTypography
-  stable label: String? = @static "animatedQuackTextStyleAsState"
+  unstable animationSpec: AnimationSpec<Any>? = @static quackTween()
+  stable label: String? = @static "QuackTypography"
   stable colorAnimationFinishedListener: Function1<@[ParameterName(name = 'color')] QuackColor, Unit>? = @static null
   stable sizeAnimationFinishedListener: Function1<@[ParameterName(name = 'size')] Float, Unit>? = @static null
   stable letterSpacingAnimationFinishedListener: Function1<@[ParameterName(name = 'letterSpacing')] Float, Unit>? = @static null

diff --git a/animation/report/compose-reports/animation_debugUnitTest-classes.txt b/animation/report/compose-reports/animation_debugUnitTest-classes.txt
@@ -0,0 +1,12 @@
+runtime class QuackBorderSnapshot {
+  runtime val snapshotPath: SnapshotPathGeneratorRule
+  <runtime stability> = Runtime(SnapshotPathGeneratorRule)
+}
+runtime class QuackColorSnapshot {
+  runtime val snapshotPath: SnapshotPathGeneratorRule
+  <runtime stability> = Runtime(SnapshotPathGeneratorRule)
+}
+runtime class QuackTypographySnapshot {
+  runtime val snapshotPath: SnapshotPathGeneratorRule
+  <runtime stability> = Runtime(SnapshotPathGeneratorRule)
+}
diff --git a/animation/report/compose-reports/animation_debugUnitTest-composables.csv b/animation/report/compose-reports/animation_debugUnitTest-composables.csv
@@ -0,0 +1 @@
+package,name,composable,skippable,restartable,readonly,inline,isLambda,hasDefaults,defaultsGroup,groups,calls,
diff --git a/animation/report/compose-reports/animation_debugUnitTest-composables.txt b/animation/report/compose-reports/animation_debugUnitTest-composables.txt
diff --git a/animation/report/compose-reports/animation_release-classes.txt b/animation/report/compose-reports/animation_release-classes.txt
@@ -1,4 +0,0 @@
-unstable class QuackAnimationSpec {
-  stable var snapMode: Boolean
-  <runtime stability> = Unstable
-}

diff --git a/animation/report/compose-reports/animation_release-composables.csv b/animation/report/compose-reports/animation_release-composables.csv
@@ -1,6 +1,4 @@
 package,name,composable,skippable,restartable,readonly,inline,isLambda,hasDefaults,defaultsGroup,groups,calls,
-team.duckie.quackquack.animation.QuackAnimatedContent,QuackAnimatedContent,1,1,1,0,0,0,0,0,1,1,
-team.duckie.quackquack.animation.QuackAnimatedVisibility,QuackAnimatedVisibility,1,1,1,0,0,0,0,0,1,1,
 team.duckie.quackquack.animation.animatedQuackBorderAsState,animatedQuackBorderAsState,1,0,0,0,0,0,0,0,1,2,
 team.duckie.quackquack.animation.animateQuackColorAsState,animateQuackColorAsState,1,0,0,0,0,0,0,0,1,2,
-team.duckie.quackquack.animation.animatedQuackTextStyleAsState,animatedQuackTextStyleAsState,1,0,0,0,0,0,0,0,1,5,
+team.duckie.quackquack.animation.animatedQuackTypographyAsState,animatedQuackTypographyAsState,1,0,0,0,0,0,0,0,1,5,
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1 @@
		package,name,composable,skippable,restartable,readonly,inline,isLambda,hasDefaults,defaultsGroup,groups,calls,