From 2f456ff371f0d9a3810dad499becff54cb83284f Mon Sep 17 00:00:00 2001
From: simeng-li <simeng@silverhand.io>
Date: Mon, 24 Jun 2024 13:42:50 +0800
Subject: [PATCH] refactor: refactor integration test ci job (#6095)

* fix: remove dev feature diff check

remove dev feature diff check

* refactor: refactor alteration integration test ci job

refactor alteration integration test ci job

* fix: fix the dev feature disbaled integration tests

fix the dev feature disabled integration tests

* fix: fix alteration-compatibility-test

fix alteration-compatibility-test
---
 ...eration-compatibility-integration-test.yml |  8 ++---
 .../dev-feature-disabled-integration-test.yml | 36 -------------------
 2 files changed, 4 insertions(+), 40 deletions(-)

diff --git a/.github/workflows/alteration-compatibility-integration-test.yml b/.github/workflows/alteration-compatibility-integration-test.yml
index b4334641a..2ec977a46 100644
--- a/.github/workflows/alteration-compatibility-integration-test.yml
+++ b/.github/workflows/alteration-compatibility-integration-test.yml
@@ -37,10 +37,10 @@ jobs:
 
           if [ -n "$CHANGE_FILES" ]; then
             echo "$CHANGE_FILES"
-            echo "::set-output name=has-alteration-changes::true"
+            echo "has-alteration-changes=true" >> $GITHUB_OUTPUT
             echo "Alteration changes detected"
           else
-            echo "::set-output name=has-alteration-changes::false"
+            echo "has-alteration-changes=false" >> $GITHUB_OUTPUT
             echo "No alteration changes detected"
           fi
 
@@ -50,7 +50,7 @@ jobs:
     if: ${{needs.check-alteration-changes.outputs.has-alteration-changes == 'true'}}
     env:
       INTEGRATION_TEST: true
-      DEV_FEATURES_ENABLED: true
+      DEV_FEATURES_ENABLED: false
     steps:
       - uses: logto-io/actions-package-logto-artifact@v2
         with:
@@ -67,7 +67,7 @@ jobs:
     runs-on: ubuntu-latest
     env:
       INTEGRATION_TEST: true
-      DEV_FEATURES_ENABLED: true
+      DEV_FEATURES_ENABLED: false
       DB_URL: postgres://postgres:postgres@localhost:5432/postgres
 
     steps:
diff --git a/.github/workflows/dev-feature-disabled-integration-test.yml b/.github/workflows/dev-feature-disabled-integration-test.yml
index 75bf7db83..4413a0acb 100644
--- a/.github/workflows/dev-feature-disabled-integration-test.yml
+++ b/.github/workflows/dev-feature-disabled-integration-test.yml
@@ -10,44 +10,8 @@ concurrency:
   cancel-in-progress: true
 
 jobs:
-  detect-dev-feature-changes:
-    runs-on: ubuntu-latest
-    outputs:
-      has-dev-feature-changes: ${{ steps.changes-detection.outputs.has-dev-feature-changes }}
-
-    steps:
-      - uses: actions/checkout@v4
-        with:
-          fetch-depth: 0
-
-      # compare the current codebase with HEAD and check if there are any new lines with isDevFeaturesEnabled
-      - name: Get the diff and filter added lines
-        run: |
-          if [[ "${{ github.event_name }}" == "pull_request" ]]; then
-            BASE=$(git merge-base origin/${{github.base_ref}} HEAD)
-          else
-            BASE=${{ github.event.before }}
-          fi
-
-          git diff $BASE --unified=0 | grep -E '^\+' > added_lines.txt
-
-          sed -i '/^+++ /d' added_lines.txt
-
-      - name: Check for isDevFeaturesEnabled in added lines
-        id: changes-detection
-        run: |
-          if grep -q 'isDevFeaturesEnabled' added_lines.txt; then
-            echo "Dev features enabled changes detected"
-            echo "::set-output name=has-dev-feature-changes::true"
-          else
-            echo "No dev features enabled changes detected"
-            echo "::set-output name=has-dev-feature-changes::false"
-          fi
-
   package:
-    needs: detect-dev-feature-changes
     runs-on: ubuntu-latest
-    if: ${{needs.detect-dev-feature-changes.outputs.has-dev-feature-changes == 'true'}}
     env:
       INTEGRATION_TEST: true
       DEV_FEATURES_ENABLED: false