testing via CI all of the possible configurations and refactored run_codegen_test script (#54)

Author: SockworkOrange

@@ -52,7 +52,6 @@ jobs:
       matrix:
         os: [ 'ubuntu-latest' ]
         architecture: [ 'x64', 'arm64' ]
-        dotnet-version: [ '8.0.x' ]
 
     steps:
       - uses: actions/checkout@v4
@@ -61,10 +60,10 @@ jobs:
         with:
           load-mode: strict
 
-      - name: Setup Dotnet ${{ matrix.dotnet-version }}  
+      - name: Setup Dotnet 
         uses: actions/setup-dotnet@v4
         with:
-          dotnet-version: ${{ matrix.dotnet-version }}
+          dotnet-version: '8.0.x'
 
       - name: install Wasi workload
         run: dotnet workload install wasi-experimental
@@ -88,20 +87,21 @@ jobs:
           ./scripts/wasm/copy_plugin_to.sh dist
           
       - name: upload wasm plugin as artifact
-        if: matrix.architecture == 'arm64'
+        if: matrix.architecture == 'arm64' # this needs to happen once across matrix
         uses: actions/upload-artifact@v4
         with:
           name: wasm-file
           path: dist/plugin.wasm
 
-  generate-tests:
-    name: Codegen Tests
+  codegen-tests:
+    name: Codegen Test
     runs-on: ubuntu-latest
     needs: [build]
     strategy:
       matrix:
         file-per-query: [ 'true', 'false' ]
         generate-csproj: [ 'true', 'false' ]
+        target-framework: [ 'net8.0', 'netstandard2.0', 'netstandard2.1' ]
 
     steps:
       - uses: actions/checkout@v4
@@ -119,27 +119,22 @@ jobs:
         with:
           sqlc-version: '1.25.0'
 
-      - name: Updating configuration
-        env:
-          FILE_PER_QUERY: ${{ matrix.file-per-query }}
-          GENERATE_CSPROJ: ${{ matrix.generate-csproj }}
-        run: |
-          ./scripts/wasm/update_sha.sh sqlc.ci.yaml
-          ./scripts/update_config.sh sqlc.ci.yaml $FILE_PER_QUERY $GENERATE_CSPROJ
+      - name: Updating plugin sha
+        run: ./scripts/wasm/update_sha.sh sqlc.ci.yaml
 
-      - name: Sqlc generate using Wasm plugin
-        env:
-          FILE_PER_QUERY: ${{ matrix.file-per-query }}
-          GENERATE_CSPROJ: ${{ matrix.generate-csproj }}
+      - name: Codegen Test
         run: |
-          rm MySqlConnectorExample/*.cs NpgsqlExample/*.cs
-          sqlc -f sqlc.ci.yaml generate
-          ./scripts/run_codegen_tests.sh $FILE_PER_QUERY $GENERATE_CSPROJ
+          ./scripts/run_codegen_test.sh sqlc.ci.yaml \
+              ${{ matrix.file-per-query }} ${{ matrix.generate-csproj }} ${{ matrix.target-framework }}
           
+      - uses: actions/setup-dotnet@v4
+        with:
+          dotnet-version: ${{ matrix.target-framework == 'net8.0' && '8.0.x' || '3.1.x' }}
+        
   end2end-tests:
     name: End-to-End Tests
     runs-on: ubuntu-latest
-    needs: [generate-tests]
+    needs: [codegen-tests]
 
     steps:
       - uses: actions/checkout@v4

.github/workflows/main.yml

@@ -0,0 +1,83 @@
+#!/usr/bin/env bash
+
+set -e
+
+declare -a examples=("MySqlConnectorExample" "NpgsqlExample")
+
+generated_files_cleanup() {
+    generate_csproj=$1
+    for example_dir in "${examples[@]}"
+    do
+        echo "Deleting .cs files" && find "${example_dir}/" -type f -name "*.cs" -exec rm -f {} \;
+        if [ "${generate_csproj}" = "true" ]; then
+            echo "Deleting .csproj file" && rm "${example_dir}/${example_dir}.csproj"
+        fi
+    done
+}
+
+check_cs_file_count() {
+    file_per_query=$1
+    for example_dir in "${examples[@]}"
+    do
+      file_count=$(find "${example_dir}/" -maxdepth 1 -name "*.cs" 2>/dev/null | wc -l)
+      if [[ "${file_per_query}" = "true" && "${file_count}" -le 2 ]]; then
+          echo "Assertion failed: Not more than 2 .cs files in the directory ${example_dir}."
+          return 1
+      elif [[ "${file_per_query}" = "false" && "${file_count}" -ne 2 ]]; then
+          echo "Assertion failed: Not exactly 2 .cs files in the directory ${example_dir}."
+          return 1
+      fi
+    done
+}
+
+check_csproj_file() {
+    for example_dir in "${examples[@]}"
+    do
+      if [ ! -f "${example_dir}/${example_dir}.csproj" ]; then
+          echo "Assertion failed: A .csproj file is not present in the directory ${example_dir}."
+          return 1
+      fi
+    done
+}
+
+check_project_compiles() {
+    for example_dir in "${examples[@]}"
+    do
+      dotnet build "${example_dir}/"
+    done
+}
+
+config_file=$1
+file_per_query=$2
+generate_csproj=$3
+target_framework=$4
+
+yq -i "
+  .sql[0].codegen[0].options.filePerQuery = ${file_per_query} |
+  .sql[1].codegen[0].options.filePerQuery = ${file_per_query} |
+  .sql[0].codegen[0].options.generateCsproj = ${generate_csproj} |
+  .sql[1].codegen[0].options.generateCsproj = ${generate_csproj} |
+  .sql[0].codegen[0].options.targetFramework = \"${target_framework}\" |
+  .sql[1].codegen[0].options.targetFramework = \"${target_framework}\"
+" "${config_file}"
+
+generated_files_cleanup "${generate_csproj}"
+echo "Using the following codegen config:" && \
+  yq '.sql[0].codegen[0]' "${config_file}" && \
+  yq '.sql[1].codegen[0]' "${config_file}"
+sqlc -f "${config_file}" generate
+
+status_code=$(check_cs_file_count "${file_per_query}")
+if [ "${status_code}" -ne 0 ]; then
+    exit "${status_code}"
+fi
+status_code=$(check_csproj_file)
+if [ "${status_code}" -ne 0 ]; then
+    exit "${status_code}"
+fi
+if [ "${generate_csproj}" = "true" ]; then
+    status_code=$(check_project_compiles "${target_framework}")
+    if [ "${status_code}" -ne 0 ]; then
+        exit "${status_code}"
+    fi
+fi