updated workflows to use latest versions of action steps

added defineConstraints to AssetConfiguration in export_unity_package.py
added unit tests for defineConstraints in export_unity_package_test.py

Author: StephenHodgson

.github/workflows/build.yaml

@@ -64,7 +64,7 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - name: Fetch All builds
-        uses: actions/download-artifact@v3
+        uses: actions/download-artifact@v4
         with:
           path: built_artifact
 

.github/workflows/build_macos.yaml

@@ -33,12 +33,12 @@ env:
 jobs:
   build_desktop:
     name: build-macOS-unity${{ inputs.unity_version}}
-    runs-on: macos-13
+    runs-on: macos-15
     strategy:
       fail-fast: false
 
     steps:
-      - uses: actions/checkout@v3
+      - uses: actions/checkout@v4
 
       - id: build_setup
         uses: ./gha/build_setup
@@ -89,15 +89,15 @@ jobs:
           fi
 
       - name: Upload build results artifact
-        uses: actions/upload-artifact@v3
+        uses: actions/upload-artifact@v4
         if: ${{ !cancelled() }}
         with:
           name: ${{ env.assetPackageArtifactName }}
           path: build/external-dependency-manager.unitypackage
           retention-days: ${{ env.artifactRetentionDays }}
 
       - name: Upload build results artifact
-        uses: actions/upload-artifact@v3
+        uses: actions/upload-artifact@v4
         if: ${{ !cancelled() }}
         with:
           name: ${{ env.tarballPackageArtifactName }}

.github/workflows/test.yaml

@@ -107,12 +107,12 @@ jobs:
 
   test_on_macos:
     name: test-macOS-unity${{ needs.check_and_prepare.outputs.unity_version }}
-    runs-on: macos-13
+    runs-on: macos-15
     needs: [check_and_prepare]
     strategy:
       fail-fast: false
     steps:
-      - uses: actions/checkout@v3
+      - uses: actions/checkout@v4
       - id: build_setup
         uses: ./gha/build_setup
         timeout-minutes: 30
@@ -168,7 +168,7 @@ jobs:
           release_license: "true"
 
       - name: Upload build logs
-        uses: actions/upload-artifact@v3
+        uses: actions/upload-artifact@v4
         if: ${{ !cancelled() }}
         with:
           name: logs

gha/build_setup/action.yml

@@ -42,7 +42,7 @@ runs:
   using: 'composite'
   steps:
     # Download GHA tools and requirements from Firebase Unity SDK repo
-    - uses: actions/checkout@v3
+    - uses: actions/checkout@v4
       with:
         repository: firebase/firebase-unity-sdk
         path: external/firebase-unity-sdk
@@ -51,7 +51,7 @@ runs:
         sparse-checkout-cone-mode: false
 
     - name: Setup python
-      uses: actions/setup-python@v4
+      uses: actions/setup-python@v5
       with:
         python-version: ${{ inputs.python_version }}
 

source/ExportUnityPackage/export_unity_package.py

@@ -1851,6 +1851,12 @@ def importer_metadata(self):
       if "Android" in platforms and cpu_string != "AnyCPU":
         importer_metadata = Asset.set_cpu_for_android(
             importer_metadata, cpu_string)
+
+      # set define constraints
+      define_constraints = safe_dict_get_value(self._json, "defineConstraints", default_value=None)
+      if define_constraints:
+        importer_metadata["PluginImporter"]["defineConstraints"] = define_constraints
+
     else:
       raise ProjectConfigurationError(
           "Unknown importer type %s for package %s, paths %s" % (

source/ExportUnityPackage/export_unity_package_test.py

@@ -2181,6 +2181,33 @@ def test_importer_metadata_standalone_only_x86_64(self):
                            "platforms": ["Standalone"],
                            "cpu": "x86_64"}).importer_metadata)
 
+  def test_importer_define_constraints(self):
+    """Test defineConstraints property."""
+    define_constraints = ["UNITY_EDITOR", "UNITY_IOS"]
+    self.plugin_metadata["PluginImporter"]["defineConstraints"] = define_constraints
+    self.assertEqual(
+        self.plugin_metadata,
+        export_unity_package.AssetConfiguration(
+            self.package, {
+                "importer": "PluginImporter",
+                "defineConstraints": define_constraints
+            }).importer_metadata)
+
+  def test_importer_define_constraints_upm(self):
+    """Test defineConstraints property for UPM."""
+    define_constraints = ["UNITY_EDITOR", "UNITY_IOS"]
+    self.plugin_metadata["PluginImporter"]["defineConstraints"] = define_constraints
+    self.assertEqual(
+        self.plugin_metadata,
+        export_unity_package.AssetConfiguration(
+            self.package, {
+                "importer": "PluginImporter",
+                "override_metadata_upm": {
+                    "PluginImporter": {
+                        "defineConstraints": define_constraints
+                    }
+                }
+            }).override_metadata_upm)
 
 class AssetPackageAndProjectFileOperationsTest(absltest.TestCase):
   """Tests for file operation methods."""