Updated help around new feature DI. (#51)

Author: EranBoudjnah

cli/src/main/kotlin/com/mitteloupe/cag/cli/ManPagePrinter.kt

@@ -76,8 +76,9 @@ object ManPagePrinter {
                 "[${bold("--dependency-injection=hilt")}|${bold("koin")}|${bold("none")}]]... " +
                 "[${bold("--new-feature")} [${bold("--name=") + italic("FeatureName")} " +
                 "[${bold("--package=") + italic("PackageName")}] [${bold("--ktlint")}] " +
-                "[${bold("--detekt")}] [${bold("--git")}]]... [${bold("--new-datasource")} " +
-                "[${bold("--name=") + italic("DataSourceName")} " +
+                "[${bold("--detekt")}] [${bold("--git")}] " +
+                "[${bold("--dependency-injection=hilt")}|${bold("koin")}|${bold("none")}]]... " +
+                "[${bold("--new-datasource")} [${bold("--name=") + italic("DataSourceName")} " +
                 "[${bold("--with=ktor")}|${bold("retrofit")}|${bold("ktor,retrofit")}] " +
                 "[${bold("--git")}]]... [${bold("--new-use-case")} " +
                 "[${bold("--name=") + italic("UseCaseName")} [${bold("--path=") + italic("TargetPath")}] " +

cli/src/main/kotlin/com/mitteloupe/cag/cli/help/HelpContent.kt

@@ -16,7 +16,7 @@ object HelpContent {
     private const val NEW_ARCHITECTURE_SYNTAX =
         "[--new-architecture [--no-compose] [--ktlint] [--detekt] [--git] [--dependency-injection=hilt|koin|none]]"
     private const val NEW_FEATURE_SYNTAX =
-        "[--new-feature --name=FeatureName [--package=PackageName] [--ktlint] [--detekt] [--git]]"
+        "[--new-feature --name=FeatureName [--package=PackageName] [--ktlint] [--detekt] [--git] [--dependency-injection=hilt|koin|none]]]"
     private const val NEW_DATASOURCE_SYNTAX =
         "[--new-datasource --name=DataSourceName [--with=ktor|retrofit|ktor,retrofit] [--git]]"
     private const val NEW_USE_CASE_SYNTAX = "[--new-use-case --name=UseCaseName [--path=TargetPath] [--git]]"
@@ -106,7 +106,9 @@ object HelpContent {
     --detekt | -d
         Enable detekt for the preceding feature (adds plugin and detekt.yml if missing)
     --git | -g
-        Automatically stage changes to git repository""",
+        Automatically stage changes to git repository
+    --dependency-injection=hilt[default]|koin|none | -DI hilt[default]|koin|none
+        Specify the dependency injection library""",
                     examples =
                         """
                         Examples:

cli/src/test/kotlin/com/mitteloupe/cag/cli/MainTest.kt

@@ -19,7 +19,8 @@ private const val NEW_PROJECT_SYNTAX =
     "[--new-project --name=ProjectName --package=PackageName [--no-compose] [--ktlint] [--detekt] [--ktor] [--retrofit] [--git] [--dependency-injection=hilt|koin|none]]"
 private const val NEW_ARCHITECTURE_SYNTAX =
     "[--new-architecture [--no-compose] [--ktlint] [--detekt] [--git] [--dependency-injection=hilt|koin|none]]"
-private const val NEW_FEATURE_SYNTAX = "[--new-feature --name=FeatureName [--package=PackageName] [--ktlint] [--detekt] [--git]]"
+private const val NEW_FEATURE_SYNTAX =
+    "[--new-feature --name=FeatureName [--package=PackageName] [--ktlint] [--detekt] [--git] [--dependency-injection=hilt|koin|none]]]"
 private const val NEW_DATASOURCE_SYNTAX = "[--new-datasource --name=DataSourceName [--with=ktor|retrofit|ktor,retrofit] [--git]]"
 private const val NEW_USE_CASE_SYNTAX = "[--new-use-case --name=UseCaseName [--path=TargetPath] [--git]]"
 private const val NEW_VIEW_MODEL_SYNTAX = "[--new-view-model --name=ViewModelName [--path=TargetPath] [--git]]"
@@ -160,6 +161,8 @@ Options:
         Enable detekt for the preceding feature (adds plugin and detekt.yml if missing)
     --git | -g
         Automatically stage changes to git repository
+    --dependency-injection=hilt[default]|koin|none | -DI hilt[default]|koin|none
+        Specify the dependency injection library
   --new-datasource | -nds
       Generate a new data source
     --name=DataSourceName | -n=DataSourceName | -n DataSourceName | -nDataSourceName

cli/src/test/kotlin/com/mitteloupe/cag/cli/ManPagePrinterTest.kt

@@ -16,7 +16,7 @@ private const val HEADER =
 cag - generate Clean Architecture Android code
 .SH SYNOPSIS
 .B cag
-[\fB--new-project\fR \fB--name=\fR\fIProjectName\fR \fB--package=\fR\fIPackageName\fR [\fB--no-compose\fR] [\fB--ktlint\fR] [\fB--detekt\fR] [\fB--ktor\fR] [\fB--retrofit\fR] [\fB--git\fR] [\fB--dependency-injection=hilt\fR|\fBkoin\fR|\fBnone\fR]]... [\fB--new-architecture\fR [\fB--no-compose\fR] [\fB--ktlint\fR] [\fB--detekt\fR] [\fB--git\fR] [\fB--dependency-injection=hilt\fR|\fBkoin\fR|\fBnone\fR]]... [\fB--new-feature\fR [\fB--name=\fR\fIFeatureName\fR [\fB--package=\fR\fIPackageName\fR] [\fB--ktlint\fR] [\fB--detekt\fR] [\fB--git\fR]]... [\fB--new-datasource\fR [\fB--name=\fR\fIDataSourceName\fR [\fB--with=ktor\fR|\fBretrofit\fR|\fBktor,retrofit\fR] [\fB--git\fR]]... [\fB--new-use-case\fR [\fB--name=\fR\fIUseCaseName\fR [\fB--path=\fR\fITargetPath\fR] [\fB--git\fR]]... [\fB--new-view-model\fR [\fB--name=\fR\fIViewModelName\fR [\fB--path=\fR\fITargetPath\fR] [\fB--git\fR]]...
+[\fB--new-project\fR \fB--name=\fR\fIProjectName\fR \fB--package=\fR\fIPackageName\fR [\fB--no-compose\fR] [\fB--ktlint\fR] [\fB--detekt\fR] [\fB--ktor\fR] [\fB--retrofit\fR] [\fB--git\fR] [\fB--dependency-injection=hilt\fR|\fBkoin\fR|\fBnone\fR]]... [\fB--new-architecture\fR [\fB--no-compose\fR] [\fB--ktlint\fR] [\fB--detekt\fR] [\fB--git\fR] [\fB--dependency-injection=hilt\fR|\fBkoin\fR|\fBnone\fR]]... [\fB--new-feature\fR [\fB--name=\fR\fIFeatureName\fR [\fB--package=\fR\fIPackageName\fR] [\fB--ktlint\fR] [\fB--detekt\fR] [\fB--git\fR] [\fB--dependency-injection=hilt\fR|\fBkoin\fR|\fBnone\fR]]... [\fB--new-datasource\fR [\fB--name=\fR\fIDataSourceName\fR [\fB--with=ktor\fR|\fBretrofit\fR|\fBktor,retrofit\fR] [\fB--git\fR]]... [\fB--new-use-case\fR [\fB--name=\fR\fIUseCaseName\fR [\fB--path=\fR\fITargetPath\fR] [\fB--git\fR]]... [\fB--new-view-model\fR [\fB--name=\fR\fIViewModelName\fR [\fB--path=\fR\fITargetPath\fR] [\fB--git\fR]]...
 .SH DESCRIPTION
 .B cag
 generates Android Clean Architecture scaffolding and components.
@@ -85,6 +85,8 @@ private val FULL_HELP =
             Enable detekt for the preceding feature (adds plugin and detekt.yml if missing)
         --git | -g
             Automatically stage changes to git repository
+        --dependency-injection=hilt[default]|koin|none | -DI hilt[default]|koin|none
+            Specify the dependency injection library
     Examples:
       cag --new-feature --name=Profile
       cag --new-feature --name=Profile --package=com.example.feature.profile

cli/src/test/kotlin/com/mitteloupe/cag/cli/argument/AppArgumentProcessorTest.kt

@@ -18,14 +18,13 @@ import org.junit.runners.Suite.SuiteClasses
 
 @RunWith(Enclosed::class)
 @SuiteClasses(
-    AppArgumentProcessorTest.Features::class,
     AppArgumentProcessorTest.Help::class,
-    AppArgumentProcessorTest.NewFeature::class,
     AppArgumentProcessorTest.AppArgumentProcessorArchitectureTest::class,
     AppArgumentProcessorTest.AppArgumentProcessorDataSourcesTest::class,
+    AppArgumentProcessorTest.AppArgumentProcessorFeatures::class,
+    AppArgumentProcessorTest.AppArgumentProcessorProjectTemplateTest::class,
     AppArgumentProcessorTest.AppArgumentProcessorUseCasesTest::class,
     AppArgumentProcessorTest.AppArgumentProcessorViewModelsTest::class,
-    AppArgumentProcessorTest.AppArgumentProcessorProjectTemplateTest::class,
     AppArgumentProcessorTest.UnknownFlagsValidation::class
 )
 class AppArgumentProcessorTest {
@@ -62,7 +61,7 @@ class AppArgumentProcessorTest {
         }
     }
 
-    class NewFeature {
+    class AppArgumentProcessorFeatures {
         private lateinit var classUnderTest: AppArgumentProcessor
 
         @Before
@@ -135,15 +134,6 @@ class AppArgumentProcessorTest {
             // Then
             assertEquals(expectedRequests, result)
         }
-    }
-
-    class Features {
-        private lateinit var classUnderTest: AppArgumentProcessor
-
-        @Before
-        fun setUp() {
-            classUnderTest = AppArgumentProcessor()
-        }
 
         @Test
         fun `Given features with optional packages when getNewFeatures then maps in order`() {