Merge pull request #40 from JavaSaBr/improve-moules-7

Improve modules, part 7

Author: JavaSaBr

.github/workflows/develop.yml

@@ -8,13 +8,13 @@ on:
 
 jobs:
   build:
-
     runs-on: ubuntu-latest
     permissions:
-      contents: read
-
+      contents: write
+      pull-requests: write
     steps:
       - uses: actions/checkout@v4
+
       - name: Set up JDK 21
         uses: actions/setup-java@v4
         with:
@@ -30,14 +30,28 @@ jobs:
       - name: Run tests for all modules
         run: ./gradlew test
 
-  dependency-submission:
+      - name: Generate jacoco reports
+        run: ./gradlew jacocoTestReport
+
+      - name: Generate aggregated jacoco report
+        run: ./gradlew test-coverage:testCodeCoverageReport
 
+      - name: Add coverage to PR
+        id: jacoco
+        uses: madrapps/jacoco-report@v1.7.2
+        with:
+          paths: ${{ github.workspace }}/test-coverage/build/reports/jacoco/testCodeCoverageReport/testCodeCoverageReport.xml
+          token: ${{ secrets.GITHUB_TOKEN }}
+          min-coverage-overall: 40
+          min-coverage-changed-files: 60
+
+  dependency-submission:
     runs-on: ubuntu-latest
     permissions:
       contents: write
-
     steps:
       - uses: actions/checkout@v4
+
       - name: Set up JDK 21
         uses: actions/setup-java@v4
         with:

build.gradle

@@ -1,4 +1,4 @@
-rootProject.version = "9.10.0"
+rootProject.version = "10.0.alpha"
 group = 'javasabr.rlib'
 
 subprojects {
@@ -8,6 +8,8 @@ subprojects {
   }
 
   apply plugin: "java-library"
+  apply plugin: "jacoco"
+  apply plugin: "jacoco-report-aggregation"
   apply plugin: "java-test-fixtures"
   apply plugin: 'maven-publish'
 
@@ -119,6 +121,15 @@ subprojects {
   tasks.withType(Test).configureEach {
     maxParallelForks = Runtime.runtime.availableProcessors()
   }
+
+  jacocoTestReport {
+    dependsOn test
+    reports {
+      xml.required = false
+      csv.required = false
+      html.outputLocation = layout.buildDirectory.dir('jacocoHtml')
+    }
+  }
 }
 
 wrapper {

rlib-classpath/src/main/java/javasabr/rlib/classpath/impl/ClassPathScannerImpl.java

@@ -18,9 +18,9 @@
 import javasabr.rlib.collections.array.Array;
 import javasabr.rlib.collections.array.ArrayFactory;
 import javasabr.rlib.collections.array.MutableArray;
+import javasabr.rlib.common.util.ArrayUtils;
 import javasabr.rlib.io.impl.ReuseBytesInputStream;
 import javasabr.rlib.io.impl.ReuseBytesOutputStream;
-import javasabr.rlib.common.util.ArrayUtils;
 import javasabr.rlib.io.util.IoUtils;
 import javasabr.rlib.logger.api.Logger;
 import javasabr.rlib.logger.api.LoggerManager;

rlib-collections/src/main/java/javasabr/rlib/collections/array/Array.java

@@ -3,6 +3,7 @@
 import java.io.Serializable;
 import java.util.Collection;
 import java.util.Iterator;
+import java.util.List;
 import java.util.Optional;
 import java.util.RandomAccess;
 import java.util.function.Function;
@@ -203,4 +204,6 @@ default Iterator<E> iterator() {
   ArrayIterationFunctions<E> iterations();
 
   UnsafeArray<E> asUnsafe();
+
+  List<E> toList();
 }

rlib-collections/src/main/java/javasabr/rlib/collections/array/IntArray.java

@@ -34,6 +34,10 @@ static IntArray of(int... elements) {
     return new ImmutableIntArray(elements);
   }
 
+  static IntArray copyOf(IntArray intArray) {
+    return new ImmutableIntArray(intArray.toArray());
+  }
+
   int size();
 
   boolean contains(int value);

rlib-collections/src/main/java/javasabr/rlib/collections/array/LockableArray.java

@@ -1,7 +1,7 @@
 package javasabr.rlib.collections.array;
 
 import javasabr.rlib.collections.operation.LockableSource;
-import javasabr.rlib.common.util.ThreadSafe;
+import javasabr.rlib.common.ThreadSafe;
 
 public interface LockableArray<E> extends MutableArray<E>, LockableSource, ThreadSafe {
 }

rlib-collections/src/main/java/javasabr/rlib/collections/array/LongArray.java

@@ -34,6 +34,10 @@ static LongArray of(long... elements) {
     return new ImmutableLongArray(elements);
   }
 
+  static LongArray copyOf(LongArray intArray) {
+    return new ImmutableLongArray(intArray.toArray());
+  }
+
   int size();
 
   boolean contains(long value);

rlib-collections/src/main/java/javasabr/rlib/collections/array/impl/AbstractArray.java

@@ -4,6 +4,7 @@
 
 import java.util.Arrays;
 import java.util.Collection;
+import java.util.List;
 import java.util.function.Consumer;
 import java.util.function.Function;
 import javasabr.rlib.collections.array.Array;
@@ -246,4 +247,12 @@ public int hashCode() {
   public UnsafeArray<E> asUnsafe() {
     return this;
   }
+
+  @Override
+  public List<E> toList() {
+    if (isEmpty()) {
+      return List.of();
+    }
+    return List.of(Arrays.copyOf(wrapped(), size()));
+  }
 }

rlib-collections/src/main/java/javasabr/rlib/collections/array/impl/CopyOnWriteMutableArray.java

@@ -6,8 +6,8 @@
 import java.util.concurrent.atomic.AtomicReference;
 import javasabr.rlib.collections.array.Array;
 import javasabr.rlib.collections.array.UnsafeMutableArray;
+import javasabr.rlib.common.ThreadSafe;
 import javasabr.rlib.common.util.ArrayUtils;
-import javasabr.rlib.common.util.ThreadSafe;
 import lombok.AccessLevel;
 import lombok.experimental.FieldDefaults;
 import org.jspecify.annotations.Nullable;

rlib-collections/src/main/java/javasabr/rlib/collections/dictionary/Dictionary.java

@@ -1,5 +1,6 @@
 package javasabr.rlib.collections.dictionary;
 
+import java.util.Collection;
 import java.util.Optional;
 import javasabr.rlib.collections.array.Array;
 import javasabr.rlib.collections.array.MutableArray;
@@ -26,10 +27,14 @@ public interface Dictionary<K, V> extends Iterable<V> {
   @Nullable
   V getOrDefault(K key, V def);
 
+  <C extends Collection<K>> C keys(C container);
+
   MutableArray<K> keys(MutableArray<K> container);
 
   Array<K> keys(Class<K> type);
 
+  <C extends Collection<V>> C values(C container);
+
   MutableArray<V> values(MutableArray<V> container);
 
   Array<V> values(Class<V> type);