Merge pull request #1126 from Adyen/improve-github-workflow

Improve GitHub CI workflow

Author: ayodejidev

Diff for: .github/workflows/dotnetcore.yml

@@ -1,32 +1,63 @@
 name: .NET Core
 
-on: [ push ]
+on:
+  push:
+    branches:
+      - main
+      - sdk-automation/models
+      - promote/main
+  pull_request:
+    branches:
+      - main
+      - sdk-automation/models
+      - promote/main
+  workflow_dispatch: {}
 
 jobs:
-  build:
-    name: Build and test
-
+  build-test:
+    name: Build and Test
     runs-on: ${{ matrix.os }}
+    
     strategy:
       matrix:
         os: [ ubuntu-latest, windows-latest ]
+        dotnet-version: [ '6.0.x' ]
+
     steps:
       - uses: actions/checkout@v4
+      
       - name: Setup .NET
         uses: actions/setup-dotnet@v4
         with:
-          dotnet-version: |
-            6.0.x
-      - name: Install tools
-        run: |
-          dotnet restore
-
+          dotnet-version: ${{ matrix.dotnet-version }}
+        
       - name: Build Debug
         run: dotnet build -c Debug
-
-      - name: Run test suite net6.0
+        
+      - name: Run unit tests
         run: dotnet test --no-build -c Debug -f net6.0 Adyen.Test/Adyen.Test.csproj
 
-      - name: Run integration test suite net6.0
-        run: dotnet test --no-build -c Debug -f net6.0 Adyen.IntegrationTest/Adyen.IntegrationTest.csproj
-        if: ${{ contains(github.event.pull_request.labels.*.name, 'release') }}
+  integration-tests:
+    name: Integration Tests
+    runs-on: ubuntu-latest
+    needs: build-test
+    
+    #Check if event is a pull request and has a release tag and is not from a forked repo
+    if: |
+      github.event_name == 'pull_request' &&
+      contains(github.event.pull_request.labels.*.name, 'release') &&
+      github.event.pull_request.head.repo.full_name == github.repository
+
+    steps:
+      - uses: actions/checkout@v4
+      
+      - name: Setup .NET
+        uses: actions/setup-dotnet@v4
+        with:
+          dotnet-version: '6.0.x'
+        
+      - name: Build Debug
+        run: dotnet build -c Debug
+        
+      - name: Run integration tests
+        run: dotnet test --no-build -c Debug -f net6.0 Adyen.IntegrationTest/Adyen.IntegrationTest.csproj

Diff for: .github/workflows/sonarcloud.yml

@@ -0,0 +1,50 @@
+name: SonarQube Analysis
+on:
+  schedule:
+    - cron: '0 1 * * 0'
+  workflow_dispatch: {}
+
+permissions:
+  contents: read
+  pull-requests: read
+
+jobs:
+  build:
+    name: Build and analyze
+    runs-on: windows-latest
+    steps:
+      - name: Set up JDK 17
+        uses: actions/setup-java@v4
+        with:
+          java-version: 17
+          distribution: 'zulu'
+      - uses: actions/checkout@v4
+        with:
+          fetch-depth: 0
+      - name: Cache SonarQube Cloud packages
+        uses: actions/cache@v4
+        with:
+          path: ~\sonar\cache
+          key: ${{ runner.os }}-sonar
+          restore-keys: ${{ runner.os }}-sonar
+      - name: Cache SonarQube Cloud scanner
+        id: cache-sonar-scanner
+        uses: actions/cache@v4
+        with:
+          path: .\.sonar\scanner
+          key: ${{ runner.os }}-sonar-scanner
+          restore-keys: ${{ runner.os }}-sonar-scanner
+      - name: Install SonarQube Cloud scanner
+        if: steps.cache-sonar-scanner.outputs.cache-hit != 'true'
+        shell: powershell
+        run: |
+          New-Item -Path .\.sonar\scanner -ItemType Directory
+          dotnet tool update dotnet-sonarscanner --tool-path .\.sonar\scanner
+      - name: Build and analyze
+        env:
+          SONAR_TOKEN: ${{ secrets.SONAR_TOKEN }}
+        shell: powershell
+        run: |
+          .\.sonar\scanner\dotnet-sonarscanner begin /k:"Adyen_adyen-dotnet-api-library" /o:"adyen" /d:sonar.token="${{ secrets.SONAR_TOKEN }}" /d:sonar.host.url="https://sonarcloud.io"
+          dotnet build
+          .\.sonar\scanner\dotnet-sonarscanner end /d:sonar.token="${{ secrets.SONAR_TOKEN }}"