Merge branch 'release-3.x.x' into TASK-5318

.github/workflows/pull-request-approved.yml

@@ -1,15 +1,40 @@
 name: Pull request approve workflow
+run-name: 'Pull request approve workflow ${{ github.event.pull_request.head.ref }} -> ${{ github.event.pull_request.base.ref }} by @${{ github.actor }}'
 
 on:
   pull_request_review:
     types: [ submitted ]
 
 jobs:
-  build:
-    uses: opencb/java-common-libs/.github/workflows/build-java-app-workflow.yml@develop
+  calculate-xetabase-branch:
+    name: Calculate Xetabase branch
+    runs-on: ubuntu-22.04
+    outputs:
+      xetabase_branch: ${{ steps.get_xetabase_branch.outputs.xetabase_branch }}
+    steps:
+      - name: Clone java-common-libs
+        uses: actions/checkout@v4
+        with:
+          fetch-depth: '10'
+          ## This is important to avoid the error in the next step: "fatal: repository 'https://github.com/zetta-genomics/opencga-enterprise.git/' not found"
+          persist-credentials: false
+      - id: get_xetabase_branch
+        name: "Get current branch for Xetabase from target branch"
+        run: |
+          chmod +x ./.github/workflows/scripts/get-xetabase-branch.sh
+          echo "github.event.pull_request.base.ref: ${{ github.event.pull_request.base.ref }}"
+          echo "github.event.pull_request.head.ref: ${{ github.event.pull_request.head.ref }}"
+          xetabase_branch=$(./.github/workflows/scripts/get-xetabase-branch.sh ${{ github.event.pull_request.base.ref }})
+          echo "__Xetabase ref:__ \"${xetabase_branch}\"" | tee -a ${GITHUB_STEP_SUMMARY}
+          echo "xetabase_branch=${xetabase_branch}" >> $GITHUB_OUTPUT
+        env:
+          ZETTA_REPO_ACCESS_TOKEN: ${{ secrets.ZETTA_REPO_ACCESS_TOKEN }}
 
   test:
-    name: "Test analysis"
-    uses: ./.github/workflows/test-analysis.yml
-    needs: build
-    secrets: inherit
+    name: "Run all tests before merging"
+    needs: calculate-xetabase-branch
+    uses: opencb/java-common-libs/.github/workflows/test-xetabase-workflow.yml@develop
+    with:
+      branch: ${{ needs.calculate-xetabase-branch.outputs.xetabase_branch }}
+      task: ${{ github.event.pull_request.head.ref }}
+    secrets: inherit

.github/workflows/scripts/get-xetabase-branch.sh

@@ -0,0 +1,50 @@
+#!/bin/bash
+
+# Function to calculate the corresponding branch of Xetabase project
+get_xetabase_branch() {
+  # Input parameter (branch name)
+  input_branch="$1"
+
+  # If the branch begins with 'TASK' and exists in the opencga-enterprise repository, I return it
+  if [[ $input_branch == TASK* ]]; then
+    if [ "$(git ls-remote "https://$ZETTA_REPO_ACCESS_TOKEN@github.com/zetta-genomics/opencga-enterprise.git" "$input_branch" )" ] ; then
+      echo $input_branch;
+      return 0;
+    fi
+  fi
+
+  # Check if the branch name is "develop" in that case return the same branch name
+  if [[ "$input_branch" == "develop" ]]; then
+    echo "develop"
+    return 0
+  fi
+
+  # Check if the branch name starts with "release-" and follows the patterns "release-a.x.x" or "release-a.b.x"
+  if [[ "$input_branch" =~ ^release-([0-9]+)\.x\.x$ ]] || [[ "$input_branch" =~ ^release-([0-9]+)\.([0-9]+)\.x$ ]]; then
+    # Extract the MAJOR part of the branch name
+    MAJOR=${BASH_REMATCH[1]}
+    # Calculate the XETABASE_MAJOR by subtracting 1 from MAJOR
+    XETABASE_MAJOR=$((MAJOR - 1))
+    # Check if the XETABASE_MAJOR is negative
+    if (( XETABASE_MAJOR < 0 )); then
+      echo "Error: 'MAJOR' digit after subtraction results in a negative number."
+      return 1
+    fi
+    # Construct and echo the new branch name
+    echo "release-$XETABASE_MAJOR.${input_branch#release-$MAJOR.}"
+    return 0
+  fi
+
+  # If the branch name does not match any of the expected patterns
+  echo "Error: The branch name is not correct."
+  return 1
+}
+
+# Check if the script receives exactly one argument
+if [ "$#" -ne 1 ]; then
+  echo "Usage: $0 <branch-name>"
+  exit 1
+fi
+
+# Call the function with the input branch name
+get_xetabase_branch "$1"

biodata-external/pom.xml

@@ -6,7 +6,7 @@
     <parent>
         <artifactId>biodata</artifactId>
         <groupId>org.opencb.biodata</groupId>
-        <version>3.2.1-SNAPSHOT</version>
+        <version>3.3.0-SNAPSHOT</version>
         <relativePath>../pom.xml</relativePath>
     </parent>
 

biodata-formats/pom.xml

@@ -22,7 +22,7 @@
     <parent>
         <groupId>org.opencb.biodata</groupId>
         <artifactId>biodata</artifactId>
-        <version>3.2.1-SNAPSHOT</version>
+        <version>3.3.0-SNAPSHOT</version>
         <relativePath>../pom.xml</relativePath>
     </parent>
 

biodata-models/pom.xml

@@ -22,7 +22,7 @@
     <parent>
         <groupId>org.opencb.biodata</groupId>
         <artifactId>biodata</artifactId>
-        <version>3.2.1-SNAPSHOT</version>
+        <version>3.3.0-SNAPSHOT</version>
         <relativePath>../pom.xml</relativePath>
     </parent>
 

biodata-models/src/main/java/org/opencb/biodata/models/clinical/interpretation/Interpretation.java

@@ -28,29 +28,29 @@
 
 public class Interpretation {
 
-    private String id;
-    private String uuid;
-    private String description;
-    private String clinicalAnalysisId;
+    protected String id;
+    protected String uuid;
+    protected String name;
+    protected String description;
+    protected String clinicalAnalysisId;
 
     /**
      * Interpretation algorithm tool used to generate this interpretation.
      */
-    private ClinicalAnalyst analyst;
-    private InterpretationMethod method;
+    protected ClinicalAnalyst analyst;
+    protected InterpretationMethod method;
 
-    private List<ClinicalVariant> primaryFindings;
-    private List<ClinicalVariant> secondaryFindings;
+    protected List<ClinicalVariant> primaryFindings;
+    protected List<ClinicalVariant> secondaryFindings;
 
-    private List<ClinicalComment> comments;
+    protected List<ClinicalComment> comments;
 
-    private InterpretationStats stats;
+    protected InterpretationStats stats;
 
-    private boolean locked;
-    private Status status;
-    private String creationDate;
-    private String modificationDate;
-    private int version;
+    protected boolean locked;
+    protected String creationDate;
+    protected String modificationDate;
+    protected int version;
 
     /**
      * Users can add custom information in this field.
@@ -61,12 +61,23 @@ public class Interpretation {
     public Interpretation() {
     }
 
+    @Deprecated
     public Interpretation(String id, String uuid, String description, String clinicalAnalysisId, ClinicalAnalyst analyst,
                           InterpretationMethod method, List<ClinicalVariant> primaryFindings, List<ClinicalVariant> secondaryFindings,
                           List<ClinicalComment> comments, InterpretationStats stats, Status status, String creationDate,
                           String modificationDate, boolean locked, int version, Map<String, Object> attributes) {
+        this(id, uuid, id, description, clinicalAnalysisId, analyst, method, primaryFindings, secondaryFindings,
+                comments, stats, locked, creationDate, modificationDate, version, attributes);
+    }
+
+    public Interpretation(String id, String uuid, String name, String description, String clinicalAnalysisId,
+                          ClinicalAnalyst analyst, InterpretationMethod method, List<ClinicalVariant> primaryFindings,
+                          List<ClinicalVariant> secondaryFindings, List<ClinicalComment> comments,
+                          InterpretationStats stats, boolean locked, String creationDate, String modificationDate,
+                          int version, Map<String, Object> attributes) {
         this.id = id;
         this.uuid = uuid;
+        this.name = name;
         this.description = description;
         this.clinicalAnalysisId = clinicalAnalysisId;
         this.analyst = analyst;
@@ -75,10 +86,9 @@ public Interpretation(String id, String uuid, String description, String clinica
         this.secondaryFindings = secondaryFindings;
         this.comments = comments;
         this.stats = stats;
-        this.status = status;
+        this.locked = locked;
         this.creationDate = creationDate;
         this.modificationDate = modificationDate;
-        this.locked = locked;
         this.version = version;
         this.attributes = attributes;
     }
@@ -88,6 +98,7 @@ public String toString() {
         final StringBuilder sb = new StringBuilder("Interpretation{");
         sb.append("id='").append(id).append('\'');
         sb.append(", uuid='").append(uuid).append('\'');
+        sb.append(", name='").append(name).append('\'');
         sb.append(", description='").append(description).append('\'');
         sb.append(", clinicalAnalysisId='").append(clinicalAnalysisId).append('\'');
         sb.append(", analyst=").append(analyst);
@@ -96,7 +107,6 @@ public String toString() {
         sb.append(", secondaryFindings=").append(secondaryFindings);
         sb.append(", comments=").append(comments);
         sb.append(", stats=").append(stats);
-        sb.append(", status=").append(status);
         sb.append(", creationDate='").append(creationDate).append('\'');
         sb.append(", modificationDate='").append(modificationDate).append('\'');
         sb.append(", locked='").append(locked).append('\'');
@@ -124,6 +134,15 @@ public Interpretation setUuid(String uuid) {
         return this;
     }
 
+    public String getName() {
+        return name;
+    }
+
+    public Interpretation setName(String name) {
+        this.name = name;
+        return this;
+    }
+
     public String getDescription() {
         return description;
     }
@@ -196,15 +215,6 @@ public Interpretation setStats(InterpretationStats stats) {
         return this;
     }
 
-    public Status getStatus() {
-        return status;
-    }
-
-    public Interpretation setStatus(Status status) {
-        this.status = status;
-        return this;
-    }
-
     public String getCreationDate() {
         return creationDate;
     }

biodata-models/src/main/java/org/opencb/biodata/models/common/Status.java

@@ -19,17 +19,22 @@
 public class Status {
 
     protected String id;
-    protected String name;
     protected String description;
     protected String date;
 
     public Status() {
-        this("", "", "", "");
+        this("", "", "");
     }
 
+    public Status(String id, String description, String date) {
+        this.id = id;
+        this.description = description;
+        this.date = date;
+    }
+
+    @Deprecated
     public Status(String id, String name, String description, String date) {
         this.id = id;
-        this.name = name;
         this.description = description;
         this.date = date;
     }
@@ -38,7 +43,6 @@ public Status(String id, String name, String description, String date) {
     public String toString() {
         final StringBuilder sb = new StringBuilder("Status{");
         sb.append("id='").append(id).append('\'');
-        sb.append(", name='").append(name).append('\'');
         sb.append(", description='").append(description).append('\'');
         sb.append(", date='").append(date).append('\'');
         sb.append('}');
@@ -53,15 +57,13 @@ public boolean equals(Object o) {
         Status status = (Status) o;
 
         if (!id.equals(status.id)) return false;
-        if (name != null ? !name.equals(status.name) : status.name != null) return false;
         if (description != null ? !description.equals(status.description) : status.description != null) return false;
         return date != null ? date.equals(status.date) : status.date == null;
     }
 
     @Override
     public int hashCode() {
         int result = id.hashCode();
-        result = 31 * result + (name != null ? name.hashCode() : 0);
         result = 31 * result + (description != null ? description.hashCode() : 0);
         result = 31 * result + (date != null ? date.hashCode() : 0);
         return result;
@@ -76,12 +78,13 @@ public Status setId(String id) {
         return this;
     }
 
+    @Deprecated
     public String getName() {
-        return name;
+        return id;
     }
 
+    @Deprecated
     public Status setName(String name) {
-        this.name = name;
         return this;
     }