refactor icaro new here (#31)

Author: LGala

.github/workflows/ci.yml

@@ -0,0 +1,11 @@
+name: CI
+on:
+  [ push ]
+jobs:
+  container-test-job:
+    runs-on: ubuntu-latest
+    container: maven:3.8.6-amazoncorretto-11
+    steps:
+      - uses: actions/checkout@v3
+      - name: tests
+        run: mvn test

pom.xml

@@ -2,8 +2,8 @@
          xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd">
   <modelVersion>4.0.0</modelVersion>
   <groupId>org.icaro</groupId>
-  <artifactId>cli</artifactId>
-  <version>0.1</version>
+  <artifactId>cli-core</artifactId>
+  <version>0.1-alpha.1</version>
   <packaging>jar</packaging>
   <name>Icaro programming language CLI</name>
   <url>http://maven.apache.org</url>
@@ -14,6 +14,11 @@
     <maven.compiler.target>11</maven.compiler.target>
   </properties>
   <dependencies>
+    <dependency>
+      <groupId>commons-io</groupId>
+      <artifactId>commons-io</artifactId>
+      <version>2.11.0</version>
+    </dependency>    
     <dependency>
       <groupId>com.google.code.gson</groupId>
       <artifactId>gson</artifactId>
@@ -71,7 +76,7 @@
             </goals>
             <configuration>
               <sourceDirs>
-                <source>src/main/kotlin</source>
+                <source>src</source>
               </sourceDirs>
             </configuration>
           </execution>
@@ -83,7 +88,7 @@
             </goals>
             <configuration>
               <sourceDirs>
-                <source>src/test/kotlin</source>
+                <source>test</source>
               </sourceDirs>
             </configuration>
           </execution>

src/commands/new-here.kt

@@ -0,0 +1,55 @@
+package commands
+
+import BINARY_DIR_NAME
+import CLASSES_DIR_NAME
+import CLI_VERSION_DEPS_ATTRIBUTE_NAME
+import DEPS_FILE
+import LANG_VERSION_DEPS_ATTRIBUTE_NAME
+import MAIN_SOURCE_FILE
+import SOURCE_DIR_NAME
+import com.google.gson.GsonBuilder
+import picocli.CommandLine
+import java.io.File
+
+@CommandLine.Command(name = "new-here", description = ["the current directory will become an Icaro project"])
+class NewHere : Runnable {
+    override fun run() {
+        try {
+            print("insert cli version: ")
+            val cliVersion = readLine()!!
+
+            print("insert lang version: ")
+            val langVersion = readLine()!!
+
+            createDependenciesFile(cliVersion, langVersion)
+            createSrcAndBinDirs(SOURCE_DIR_NAME, BINARY_DIR_NAME)
+        } catch (e: Throwable) {
+            e.printStackTrace()
+        }
+    }
+}
+
+/*
+doesn't make sense to pass constants but
+1. constants are useful
+2. in this case, we make this function parametrized, or we couldn't test it 
+   because creating a src directory in the root path of this folder conflict with this 
+   project src file (DX with docker is bad because everytime we need to wait for maven to build)
+ */
+fun createSrcAndBinDirs(sourceDirName: String, binaryDirName: String) {
+    File(sourceDirName).mkdir()
+    File("$sourceDirName/$MAIN_SOURCE_FILE").writeText("")
+
+    File("$binaryDirName/$CLASSES_DIR_NAME").mkdirs()
+}
+
+fun createDependenciesFile(cliVersion: String, langVersion: String) {
+    val depsInJson = GsonBuilder().setPrettyPrinting().create().toJson(
+        mapOf(
+            CLI_VERSION_DEPS_ATTRIBUTE_NAME to cliVersion,
+            LANG_VERSION_DEPS_ATTRIBUTE_NAME to langVersion
+        )
+    )
+
+    File(DEPS_FILE).writeText(depsInJson)
+}

src/main/kotlin/main.kt src/main.ktsrc/main/kotlin/main.kt renamed to src/main.kt

@@ -0,0 +1,7 @@
+const val DEPS_FILE = "deps.json"
+const val CLI_VERSION_DEPS_ATTRIBUTE_NAME = "cliVersion"
+const val LANG_VERSION_DEPS_ATTRIBUTE_NAME = "langVersion"
+const val SOURCE_DIR_NAME = "src"
+const val MAIN_SOURCE_FILE = "main.ic"
+const val BINARY_DIR_NAME = ".bin"
+const val CLASSES_DIR_NAME = "classes"

src/main/kotlin/commands/new-here.kt

@@ -0,0 +1,43 @@
+package commands
+
+import CLASSES_DIR_NAME
+import CLI_VERSION_DEPS_ATTRIBUTE_NAME
+import DEPS_FILE
+import LANG_VERSION_DEPS_ATTRIBUTE_NAME
+import MAIN_SOURCE_FILE
+import com.google.gson.Gson
+import org.junit.jupiter.api.Test
+import java.io.File
+import kotlin.test.assertEquals
+import kotlin.test.assertTrue
+
+class NewHereTest {
+    @Test
+    fun shouldCreateDependenciesFile() {
+        val cliVersion = "1.2"
+        val langVersion = "3.1-rc.3"
+
+        createDependenciesFile(cliVersion, langVersion)
+
+        val dependencies = Gson().fromJson(File(DEPS_FILE).readText(), Map::class.java)
+
+        assertEquals(cliVersion, dependencies[CLI_VERSION_DEPS_ATTRIBUTE_NAME])
+        assertEquals(langVersion, dependencies[LANG_VERSION_DEPS_ATTRIBUTE_NAME])
+
+        File(DEPS_FILE).delete()
+    }
+
+    @Test
+    fun shouldCreateSourceAndBinaryDirectories() {
+        val sourceDir = "src43234"
+        val binDir = ".bin43334"
+        
+        createSrcAndBinDirs(sourceDir, binDir)
+
+        assertTrue(File("$sourceDir/$MAIN_SOURCE_FILE").isFile)
+        assertTrue(File("$binDir/$CLASSES_DIR_NAME").isDirectory)
+
+        File(sourceDir).deleteRecursively()
+        File(binDir).deleteRecursively()
+    }
+}