add API to ContactsExtension that can be used from kts buildscripts

fixes #12

Author: wakingrufus

.github/workflows/nebula.yml

@@ -2,7 +2,7 @@ name: Nebula Build
 on:
   push:
     branches:
-      - '*'
+      - 'main'
     tags:
       - v*.*.*
       - v*.*.*-rc.*
@@ -38,20 +38,10 @@ jobs:
             17
             ${{ matrix.java }}
           java-package: jdk
-      - uses: actions/cache@v4
-        id: gradle-cache
+      - name: Setup Gradle
+        uses: gradle/actions/setup-gradle@v5
         with:
-          path: ~/.gradle/caches
-          key: ${{ runner.os }}-gradle-${{ hashFiles('**/gradle/dependency-locks/*.lockfile') }}
-          restore-keys: |
-            - ${{ runner.os }}-gradle-
-      - uses: actions/cache@v4
-        id: gradle-wrapper-cache
-        with:
-          path: ~/.gradle/wrapper
-          key: ${{ runner.os }}-gradlewrapper-${{ hashFiles('gradle/wrapper/*') }}
-          restore-keys: |
-            - ${{ runner.os }}-gradlewrapper-
+          cache-overwrite-existing: true
       - name: Gradle build
         run: ./gradlew --info --stacktrace build
         env:
@@ -78,20 +68,10 @@ jobs:
             17
             21
           java-package: jdk
-      - uses: actions/cache@v4
-        id: gradle-cache
-        with:
-          path: ~/.gradle/caches
-          key: ${{ runner.os }}-gradle-${{ hashFiles('**/gradle/dependency-locks/*.lockfile') }}
-          restore-keys: |
-            - ${{ runner.os }}-gradle-
-      - uses: actions/cache@v4
-        id: gradle-wrapper-cache
+      - name: Setup Gradle
+        uses: gradle/actions/setup-gradle@v5
         with:
-          path: ~/.gradle/wrapper
-          key: ${{ runner.os }}-gradlewrapper-${{ hashFiles('gradle/wrapper/*') }}
-          restore-keys: |
-            - ${{ runner.os }}-gradlewrapper-
+          cache-overwrite-existing: true
       - name: Verify plugin publication
         if: |
           startsWith(github.ref, 'refs/tags/v') &&
@@ -123,20 +103,10 @@ jobs:
             17
             21
           java-package: jdk
-      - uses: actions/cache@v4
-        id: gradle-cache
-        with:
-          path: ~/.gradle/caches
-          key: ${{ runner.os }}-gradle-${{ hashFiles('**/gradle/dependency-locks/*.lockfile') }}
-          restore-keys: |
-            - ${{ runner.os }}-gradle-
-      - uses: actions/cache@v4
-        id: gradle-wrapper-cache
+      - name: Setup Gradle
+        uses: gradle/actions/setup-gradle@v5
         with:
-          path: ~/.gradle/wrapper
-          key: ${{ runner.os }}-gradlewrapper-${{ hashFiles('gradle/wrapper/*') }}
-          restore-keys: |
-            - ${{ runner.os }}-gradlewrapper-
+          cache-overwrite-existing: true
       - name: Publish candidate
         if: |
           startsWith(github.ref, 'refs/tags/v') &&

.java-version

@@ -14,7 +14,7 @@
  * limitations under the License.
  */
 plugins {
-    id 'com.netflix.nebula.plugin-plugin' version '22.0.2'
+    id("com.netflix.nebula.plugin-plugin") version ("23.+")
 }
 
 description 'Plugin to specify and publish who the owner of a project is'
@@ -26,6 +26,10 @@ contacts {
     }
 }
 
+dependencies {
+    testImplementation("org.assertj:assertj-core:3.27.6")
+}
+
 gradlePlugin {
     plugins {
         contacts {
@@ -45,3 +49,10 @@ gradlePlugin {
     }
 }
 
+testing {
+    suites {
+        test {
+            useJUnitJupiter()
+        }
+    }
+}

build.gradle

@@ -2,17 +2,18 @@
 # Manual edits can break the build and are not advised.
 # This file is expected to be part of source control.
 cglib:cglib-nodep:3.2.2=integTestRuntimeClasspath,testRuntimeClasspath
-com.netflix.nebula:nebula-test:11.0.0=integTestCompileClasspath,integTestRuntimeClasspath,testCompileClasspath,testRuntimeClasspath
-junit:junit:4.13.2=integTestCompileClasspath,integTestRuntimeClasspath,testCompileClasspath,testRuntimeClasspath
-org.apache.groovy:groovy:4.0.4=integTestCompileClasspath,integTestRuntimeClasspath,testCompileClasspath,testRuntimeClasspath
+com.netflix.nebula:nebula-test:11.6.2=integTestCompileClasspath,integTestRuntimeClasspath,testCompileClasspath,testRuntimeClasspath
+net.bytebuddy:byte-buddy:1.17.7=integTestCompileClasspath,integTestRuntimeClasspath,testCompileClasspath,testRuntimeClasspath
 org.apiguardian:apiguardian-api:1.1.2=integTestCompileClasspath,integTestRuntimeClasspath,testCompileClasspath,testRuntimeClasspath
-org.hamcrest:hamcrest-core:1.3=integTestCompileClasspath,integTestRuntimeClasspath,testCompileClasspath,testRuntimeClasspath
-org.hamcrest:hamcrest:2.2=integTestCompileClasspath,integTestRuntimeClasspath,testCompileClasspath,testRuntimeClasspath
-org.junit.platform:junit-platform-commons:1.13.1=integTestCompileClasspath,integTestRuntimeClasspath,testCompileClasspath,testRuntimeClasspath
-org.junit.platform:junit-platform-engine:1.13.1=integTestCompileClasspath,integTestRuntimeClasspath,testCompileClasspath,testRuntimeClasspath
-org.junit.platform:junit-platform-launcher:1.13.1=integTestCompileClasspath,integTestRuntimeClasspath,testCompileClasspath,testRuntimeClasspath
+org.assertj:assertj-core:3.27.6=integTestCompileClasspath,integTestRuntimeClasspath,testCompileClasspath,testRuntimeClasspath
+org.jspecify:jspecify:1.0.0=integTestCompileClasspath,integTestRuntimeClasspath,testCompileClasspath,testRuntimeClasspath
+org.junit.jupiter:junit-jupiter-api:5.12.2=integTestCompileClasspath,integTestRuntimeClasspath,testCompileClasspath,testRuntimeClasspath
+org.junit.jupiter:junit-jupiter-engine:5.12.2=integTestRuntimeClasspath,testRuntimeClasspath
+org.junit.jupiter:junit-jupiter-params:5.12.2=integTestCompileClasspath,integTestRuntimeClasspath,testCompileClasspath,testRuntimeClasspath
+org.junit.jupiter:junit-jupiter:5.12.2=integTestCompileClasspath,integTestRuntimeClasspath,testCompileClasspath,testRuntimeClasspath
+org.junit.platform:junit-platform-commons:1.14.0=integTestCompileClasspath,integTestRuntimeClasspath,testCompileClasspath,testRuntimeClasspath
+org.junit.platform:junit-platform-engine:1.14.0=integTestCompileClasspath,integTestRuntimeClasspath,testCompileClasspath,testRuntimeClasspath
+org.junit.platform:junit-platform-launcher:1.14.0=integTestCompileClasspath,integTestRuntimeClasspath,testCompileClasspath,testRuntimeClasspath
 org.objenesis:objenesis:2.4=integTestRuntimeClasspath,testRuntimeClasspath
 org.opentest4j:opentest4j:1.3.0=integTestCompileClasspath,integTestRuntimeClasspath,testCompileClasspath,testRuntimeClasspath
-org.spockframework:spock-core:2.3-groovy-4.0=integTestCompileClasspath,integTestRuntimeClasspath,testCompileClasspath,testRuntimeClasspath
-org.spockframework:spock-junit4:2.3-groovy-4.0=integTestCompileClasspath,integTestRuntimeClasspath,testCompileClasspath,testRuntimeClasspath
 empty=annotationProcessor,compileClasspath,integTestAnnotationProcessor,runtimeClasspath,testAnnotationProcessor

gradle.lockfile

@@ -8,7 +8,7 @@ import org.gradle.api.Named
  */
 @Canonical
 class Contact implements Named {
-    final String email
+     String email
 
     /**
      * Aka name, but Named interface won't let us call it that
@@ -31,7 +31,8 @@ class Contact implements Named {
     String slack = ""
 
     Set<String> roles = [] as Set
-
+    Contact() {
+    }
     Contact(String email) {
         this.email = email
     }

src/main/groovy/nebula/plugin/contacts/Contact.groovy

@@ -1,5 +1,8 @@
 package nebula.plugin.contacts
 
+import org.gradle.api.Action
+import org.gradle.api.NamedDomainObjectContainer
+
 /**
  * Holds Person for a project.
  * TODO repeat a name and guarantee uniqueness
@@ -8,6 +11,7 @@ class ContactsExtension {
 
     private final String emailPattern = /[_A-Za-z0-9-]+(.[_A-Za-z0-9-]+)*@[A-Za-z0-9]+(.[A-Za-z0-9]+)*(.[A-Za-z]{2,})/
 
+    NamedDomainObjectContainer<Contact> peopleContainer
     final LinkedHashMap<String, Contact> people
     boolean validateEmails
 
@@ -65,6 +69,15 @@ class ContactsExtension {
         return person
     }
 
+//    Contact addPerson(String email, Action<Contact> closure) {
+//        if(validateEmails)
+//            validateEmail(email)
+//        def person = people.containsKey(email) ? BaseContactsPlugin.cloneContact(people.get(email)) : new Contact(email)
+//        closure.execute(person)
+//        people.put(email, person)
+//        return person
+//    }
+
     private validateEmail(String email) {
         if(!(email ==~  emailPattern)) {
             throw new ContactsPluginException("$email is not a valid email")

src/main/groovy/nebula/plugin/contacts/ContactsExtension.groovy

@@ -0,0 +1,41 @@
+package nebula.plugin.contacts;
+
+import nebula.test.dsl.BuildscriptLanguage;
+import nebula.test.dsl.TestProjectBuilder;
+import org.gradle.testkit.runner.BuildResult;
+import org.junit.jupiter.api.Test;
+import org.junit.jupiter.api.io.TempDir;
+
+import java.io.File;
+
+import static nebula.test.dsl.TestKitAssertions.assertThat;
+
+class ContactsPluginTest {
+    @TempDir
+    File projectDir;
+
+    @Test
+    void test_kotlin() {
+        final TestProjectBuilder builder = TestProjectBuilder.testProject(projectDir);
+        builder.rootProject().plugins().id("com.netflix.nebula.contacts");
+        builder.rootProject()
+                .rawBuildScript(
+                        //language=kotlin
+                        """
+contacts {
+    addPerson("example@example.com") {
+        moniker = "Nebula-Plugins maintainers"
+        github = "nebula-plugins"
+    }
+}
+""");
+        final var runner = builder.build(BuildscriptLanguage.KOTLIN);
+
+
+        BuildResult result = runner.run("build");
+        assertThat(result)
+                .hasNoDeprecationWarnings()
+                .hasNoMutableStateWarnings();
+    }
+
+}