feedback: rename simple datastore implementation

Author: jdaugherty

build.gradle

@@ -60,14 +60,15 @@ def publishedProjects = [
         'grails-datastore-gorm',
         'grails-datastore-gorm-async',
         //'grails-datastore-gorm-rx',
-        'grails-datastore-gorm-simple',
         'grails-datastore-gorm-support',
         'grails-datastore-gorm-tck',
         'grails-datastore-gorm-tck-base',
         'grails-datastore-gorm-tck-domains',
         'grails-datastore-gorm-validation',
         'grails-datastore-web',
         'grails-gorm-testing-support',
+        // simple
+        'simple-gorm',
         // hibernate5
         'hibernate5-boot-plugin',
         'hibernate5-database-migration',

grails-datastore-gorm-simple/build.gradle renamed to grails-data-simple/build.gradle

@@ -6,6 +6,10 @@ plugins {
 version = projectVersion
 group = 'org.grails'
 
+ext {
+    pomArtifactId = 'grails-datastore-gorm-simple'
+}
+
 dependencies {
 
     implementation platform("org.grails:grails-bom:$grailsVersion")

grails-datastore-gorm-simple/src/main/groovy/org/grails/datastore/mapping/simple/SimpleMapDatastore.java renamed to grails-data-simple/src/main/groovy/org/grails/datastore/mapping/simple/SimpleMapDatastore.java

@@ -8,6 +8,7 @@ group = 'org.grails'
 
 ext {
     includeBaseTckClass = true
+    //TODO: We could publish the test classes once https://github.com/grails/grails-gradle-plugin/pull/425 is merged
 }
 
 dependencies {
@@ -92,7 +93,7 @@ dependencies {
     //testImplementation project(":grails-datastore-gorm-rx")
 
     // Use the simple datastore to ensure the base implementation works correctly
-    api project(':grails-datastore-gorm-simple')
+    api project(':simple-gorm')
 
     testRuntimeOnly 'org.slf4j:slf4j-nop' // Get rid of warning about missing slf4j implementation during tests
 

grails-datastore-gorm-simple/src/main/groovy/org/grails/datastore/mapping/simple/SimpleMapSession.java renamed to grails-data-simple/src/main/groovy/org/grails/datastore/mapping/simple/SimpleMapSession.java

@@ -36,7 +36,6 @@ include(
     'grails-datastore-async',
     'grails-datastore-gorm-async',
     'grails-datastore-gorm-validation',
-    'grails-datastore-gorm-simple',
     'grails-datastore-gorm-support',
     // the tck is split into domains -> base -> tck so that the base class can be swapped out or reused
     'grails-datastore-gorm-tck',
@@ -66,6 +65,9 @@ project(':docs-guide-developer').projectDir = file('docs/guide-developer')
 include 'docs-guide-whats-new'
 project(':docs-guide-whats-new').projectDir = file('docs/guide-whats-new')
 
+// Simple implementation (for simple gorm usage, unit tests, and base implementation testing)
+include 'simple-gorm'
+findProject(':simple-gorm').projectDir = file('grails-data-simple')
 
 // Hibernate 5
 // Core