unitytk 0.2.2__tar.gz → 0.4.0__tar.gz

{unitytk-0.2.2/unitytk.egg-info → unitytk-0.4.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: unitytk
-Version: 0.2.2
+Version: 0.4.0
 Summary: A modular Python toolkit for Unity interaction.
 Author-email: Ryan Simpson <m3trik@outlook.com>
 License: MIT
@@ -13,7 +13,7 @@ Classifier: Operating System :: POSIX :: Linux
 Requires-Python: >=3.9
 Description-Content-Type: text/markdown
 License-File: LICENSE
-Requires-Dist: pythontk>=0.13.1
+Requires-Dist: pythontk>=0.17.0
 Dynamic: license-file
 
 [![Tests](https://img.shields.io/badge/Tests-146%20passed-brightgreen.svg)](../test/)

{unitytk-0.2.2 → unitytk-0.4.0}/pyproject.toml

@@ -17,7 +17,12 @@ classifiers = [
 ]
 # The floor is HAND-RATCHETED: unitytk is off-chain from m3trik/push.ps1, so no
 # release cascade bumps it.
-# pythontk 0.13.1, the release this one ships and was tested with: an FBX
+# pythontk 0.17.0, the release this one ships and was tested with: the
+# templates read the scene records by their renamed keys alone (`animation`,
+# `audio`, `lightmaps`, `shadows`, `materials`, `handoff` v3 -- was
+# `shot_metadata`, `audio_manifest`, `lightmap_metadata`, `shadow_metadata`),
+# and only pythontk 0.17.0 writes them; an older one's FBX imports with no
+# shots, audio, lightmaps or shadows. Before it, pythontk 0.13.1: an FBX
 # deliverable carries the files its records name through
 # `FbxMedia.embed_dependencies` (new in 0.13.1; on 0.13.0 the copy deliverer
 # ships the FBX without its maps), and the suite holds the templates to its
@@ -27,7 +32,7 @@ classifiers = [
 # needs 0.8.84: it scans through `AppLauncher.scan_install_dirs` (new in
 # 0.8.72) and needs its pattern order to be a priority (0.8.84), so the Hub's
 # chosen install location wins over the default.
-dependencies = ["pythontk>=0.13.1"]
+dependencies = ["pythontk>=0.17.0"]
 
 [project.urls]
 Homepage = "https://github.com/m3trik/unitytk"

{unitytk-0.2.2 → unitytk-0.4.0}/test/test_articulated_rig_runtime.py

@@ -857,7 +857,9 @@ class LiveExport:
 
 
 def _invalid_rigs():
-    """Rigs the reference refuses (ValueError); the port must refuse them too."""
+    """Rigs the reference refuses (ValueError); the port must refuse them too.
+    Each is a rig as the record states it -- every key its shape requires --
+    so each is refused for its own fault, not a key left out."""
     base = {"name": "bad", "joints": [], "grab": []}
 
     def rig(*joints):
@@ -865,6 +867,9 @@ def _invalid_rigs():
         out["joints"] = list(joints)
         return out
 
+    def channel(name, lo=None, hi=None):
+        return {"channel": name, "min": lo, "max": hi, "weight": 1.0}
+
     ok = {
         "name": "a",
         "parent": None,
@@ -876,10 +881,10 @@ def _invalid_rigs():
     return [
         rig(ok, dict(ok, name="b", parent=1)),  # parent not earlier
         rig(dict(ok, rotate_order="xxy")),  # unknown order
-        rig(dict(ok, channels=[{"channel": "sx"}])),  # unknown channel
-        rig(dict(ok, channels=[{"channel": "rx"}, {"channel": "rx"}])),  # twice
-        rig(dict(ok, channels=[{"channel": "rz", "min": 10, "max": -10}])),  # reversed
-        rig(ok, dict(ok, name="b", parent=0, channels=[{"channel": "tx", "min": None, "max": 3}])),  # valid
+        rig(dict(ok, channels=[channel("sx")])),  # unknown channel
+        rig(dict(ok, channels=[channel("rx"), channel("rx")])),  # twice
+        rig(dict(ok, channels=[channel("rz", lo=10, hi=-10)])),  # reversed
+        rig(ok, dict(ok, name="b", parent=0, channels=[channel("tx", hi=3)])),  # valid
     ]  # fmt: skip
 
 
@@ -915,7 +920,7 @@ public class ArticulatedRigVerifier
     static M.Frame[] Frames(object o) => L(o).Select(f => new M.Frame(Vec(L(f)[0]), Qt(L(f)[1]))).ToArray();
 
     // ---- JSON out ---------------------------------------------------------
-    static string N(double v) => ArticulationJson.Number(v);
+    static string N(double v) => UnitytkJson.Number(v);
     static string B(bool b) => b ? "true" : "false";
     static string S(string s) => "\"" + (s ?? "").Replace("\\", "\\\\").Replace("\"", "\\\"").Replace("\n", "\\n").Replace("\r", "") + "\"";
     static string A(double[] a) => a == null ? "null" : "[" + string.Join(",", a.Select(N)) + "]";
@@ -934,7 +939,7 @@ public class ArticulatedRigVerifier
         try
         {
             AssetDatabase.Refresh(ImportAssetOptions.ForceSynchronousImport);
-            inputs = O(ArticulationJson.Parse(File.ReadAllText(INPUTS)));
+            inputs = O(UnitytkJson.Parse(File.ReadAllText(INPUTS)));
         }
         catch (Exception e) { Put("inputs_error", S(e.ToString())); }
         if (inputs != null)
@@ -1671,7 +1676,7 @@ def _run_unity(test, temp_dir, execute_method, prepare, timeout=600):
     verifier; returns the parsed results (or skips on the license wall)."""
     editor = os.environ.get("UNITYTK_TEST_EDITOR")
     launcher = UnityLauncher(editor) if editor else UnityLauncher()
-    project_path = os.path.join(temp_dir, "UnityProject")
+    project_path = TempProjects.project(temp_dir)
     if not launcher.create_project(project_path, batch_mode=True):
         test.skipTest(UnityEnv.project_creation_reason())
     results_path = os.path.join(temp_dir, "unity_results.json")
@@ -1720,7 +1725,7 @@ class TestArticulatedRigRuntime(unittest.TestCase):
 
     @classmethod
     def setUpClass(cls):
-        cls.temp_dir = TempProjects.allocate("articulated_rig_runtime")
+        cls.temp_dir = TempProjects.allocate("articulated_rig_runtime", cls)
         cls.unity = None
         cls.doc = ptk.ArticulationConformance.cases(seed=SEED, per_rig=PER_RIG)
         cls.scenario = ControllerScenario(cls.doc)
@@ -1756,10 +1761,6 @@ class TestArticulatedRigRuntime(unittest.TestCase):
         }
         logger.info(f"Temp dir: {cls.temp_dir}")
 
-    @classmethod
-    def tearDownClass(cls):
-        logger.info(f"Results preserved in: {cls.temp_dir}")
-
     def _results(self):
         unity = self.__class__.unity
         if not unity:
@@ -1865,9 +1866,7 @@ class TestArticulatedRigRuntime(unittest.TestCase):
             for name in _bundle_assets()
         }
         self.assertEqual(got, expected, "Unity named a deployed asset differently")
-        package = TemplateDeployer.package_dir(
-            os.path.join(self.temp_dir, "UnityProject")
-        )
+        package = TemplateDeployer.package_dir(TempProjects.project(self.temp_dir))
         rewritten = []
         for name in _bundle_assets():
             path = os.path.join(package, *name.split("/")) + ".meta"
@@ -2703,7 +2702,7 @@ public class ArticulatedRigXRVerifier
     const string INPUTS = @"__INPUTS_PATH__";
 
     static string B(bool b) => b ? "true" : "false";
-    static string N(double v) => ArticulationJson.Number(v);
+    static string N(double v) => UnitytkJson.Number(v);
     static IList<object> L(object o) => (IList<object>)o;
     static float F(object o) => (float)(double)o;
 
@@ -2712,7 +2711,7 @@ public class ArticulatedRigXRVerifier
         var sb = new StringBuilder("{\n");
         try
         {
-            var inputs = (IDictionary<string, object>)ArticulationJson.Parse(File.ReadAllText(INPUTS));
+            var inputs = (IDictionary<string, object>)UnitytkJson.Parse(File.ReadAllText(INPUTS));
             var names = L(inputs["names"]);
             var parents = L(inputs["parents"]);
             var locals = L(inputs["locals"]);
@@ -2778,7 +2777,7 @@ class TestArticulatedRigXRI(unittest.TestCase):
 
     @classmethod
     def setUpClass(cls):
-        cls.temp_dir = TempProjects.allocate("articulated_rig_xri")
+        cls.temp_dir = TempProjects.allocate("articulated_rig_xri", cls)
         cls.unity = None
 
     @EnvSkip.unless(_unity_available(), "no Unity editor found")

{unitytk-0.2.2 → unitytk-0.4.0}/test/test_asset_delivery.py

@@ -394,16 +394,13 @@ class SelfContainedDeliveryTest(unittest.TestCase):
     decision of 2026-10-05: every dependency embedded, zero sidecar files)."""
 
     RECORDS = {
-        "lightmap_metadata": {
-            "version": 1,
+        "lightmaps": {
+            "version": 2,
+            "unit_scale": 0.01,
             "objects": [{"name": "Crate", "map": "Crate_Lightmap.exr", "uvIndex": 1}],
-            "probe": {
-                "map": "Room_Probe.exr",
-                "position": [0, 0, 0],
-                "unit_scale": 0.01,
-            },
+            "probe": {"map": "Room_Probe.exr", "position": [0, 0, 0], "box": None},
         },
-        "shadow_metadata": {
+        "shadows": {
             "version": 2,
             "planes": [
                 {

{unitytk-0.2.2 → unitytk-0.4.0}/test/test_audio_events_integration.py

@@ -6,13 +6,13 @@ Validates the new single-carrier audio events pipeline end-to-end:
     1. Maya (mayapy): Author audio tracks on the canonical ``data_internal``
        carrier via :class:`mayatk.audio_utils.AudioUtils`, then call
        :meth:`mayatk.audio_utils.audio_clips.AudioClips.prepare_for_export`
-       to bake ``audio_manifest`` as a plain string channel on ``data_export``
+       to bake the ``audio`` record as a plain string channel on ``data_export``
        (the authoring state stays keyed on ``data_internal`` and never exports).
        Animate a cube so the FBX contains at least one ``AnimationClip``
        for the audio events to bind to.  Export FBX.
     2. Unity (batchmode): Import FBX with the real
        ``AudioEventController.cs`` deployed.  Verify:
-         a. the ``audio_manifest`` string arrives on the ``data_export``
+         a. the ``audio`` record arrives on the ``data_export``
             GameObject (raw user-property),
          b. the importer auto-attaches an ``AudioEventController`` to the
             prefab **root** (and explicitly NOT to the ``data_export``
@@ -414,17 +414,13 @@ class TestAudioEventsIntegration(unittest.TestCase):
 
     @classmethod
     def setUpClass(cls):
-        cls.temp_dir = TempProjects.allocate("audio_events_integ")
+        cls.temp_dir = TempProjects.allocate("audio_events_integ", cls)
         cls.fbx_path = None
         cls.maya_results = None
         cls.unity_results = None
         cls.postproc = None
         logger.info(f"Temp dir: {cls.temp_dir}")
 
-    @classmethod
-    def tearDownClass(cls):
-        logger.info(f"Results preserved in: {cls.temp_dir}")
-
     # ------------------------------------------------------------------
     # Step 1: Maya export via AudioClips.prepare_for_export
     # ------------------------------------------------------------------
@@ -469,7 +465,7 @@ class TestAudioEventsIntegration(unittest.TestCase):
         self.__class__.fbx_path = r["fbx_path"]
         self.assertTrue(r["fbx_exists"])
         self.assertEqual(r["carrier"], "data_internal")
-        self.assertEqual(r["manifest_attr"], "audio_manifest")
+        self.assertEqual(r["manifest_attr"], "audio")
         # v2 manifest: versioned JSON. No shots in this scene, so events are
         # unscoped (clip "") with frames RELATIVE to Maya's playback min
         # (= 1 here) — Unity's clip clock maps Maya frame `min` to time 0.
@@ -499,7 +495,7 @@ class TestAudioEventsIntegration(unittest.TestCase):
         # as the sibling integration tests.
         editor = os.environ.get("UNITYTK_TEST_EDITOR")
         launcher = UnityLauncher(editor) if editor else UnityLauncher()
-        project_path = os.path.join(self.temp_dir, "UnityProject")
+        project_path = TempProjects.project(self.temp_dir)
 
         logger.info(f"Creating Unity project: {project_path}")
         if not launcher.create_project(project_path, batch_mode=True):
@@ -623,11 +619,11 @@ class TestAudioEventsIntegration(unittest.TestCase):
             f"in postprocessor results: {list(postproc)}",
         )
         self.assertIn(
-            "audio_manifest",
+            "audio",
             props,
-            f"audio_manifest user-property missing on {name}; saw: {list(props)}",
+            f"audio record missing on {name}; saw: {list(props)}",
         )
-        manifest = props["audio_manifest"]
+        manifest = props["audio"]
         for label in ("footstep", "jump", "land"):
             self.assertIn(label, manifest)
 
@@ -648,7 +644,7 @@ class TestAudioEventsIntegration(unittest.TestCase):
 
         # (c) the real AudioEventImporter auto-attached the controller to
         # the prefab ROOT (where the Animator will live), NOT to the GO
-        # carrying audio_manifest — this is the runtime-dispatch invariant.
+        # carrying the audio record — this is the runtime-dispatch invariant.
         self.assertTrue(
             unity.get("controller_on_root"),
             "AudioEventController should live on the prefab root so "
@@ -739,7 +735,7 @@ class TestAudioEventsIntegration(unittest.TestCase):
 
 
 # ======================================================================
-# Carrier precedence: one audio_manifest per model, the scene's own first
+# Carrier precedence: one audio record per model, the scene's own first
 # ======================================================================
 
 CARRIER_VERIFIER_CS = r"""
@@ -784,7 +780,7 @@ public class AudioCarrierVerifier
 
 
 class TestAudioCarrierPrecedence(unittest.TestCase):
-    """An FBX carrying ``audio_manifest`` on several carriers plays the
+    """An FBX carrying the ``audio`` record on several carriers plays the
     scene's own only.
 
     An assembly ships a stray ``data_export1`` (and a module's
@@ -805,23 +801,31 @@ class TestAudioCarrierPrecedence(unittest.TestCase):
 
     @classmethod
     def setUpClass(cls):
-        cls.temp_dir = TempProjects.allocate("audio_carrier_precedence")
+        cls.temp_dir = TempProjects.allocate("carrier_precedence", cls)
 
     @BlenderEnv.skip_unless_available()
     @EnvSkip.unless(_unity_available(), "no Unity editor found")
     def test_the_scenes_own_carrier_wins(self):
-        own = {"version": 2, "events": [{"clip": "", "frame": 0, "name": "footstep"}]}
-        stray = {"version": 2, "events": [{"clip": "", "frame": 3, "name": "door"}]}
+        own = {
+            "version": 2,
+            "fps": 24.0,
+            "events": [{"clip": "", "frame": 0, "name": "footstep"}],
+        }
+        stray = {
+            "version": 2,
+            "fps": 24.0,
+            "events": [{"clip": "", "frame": 3, "name": "door"}],
+        }
         fixture = CarrierFbx.build(
             os.path.join(self.temp_dir, "assembly.fbx"),
             {
-                "data_export": {"audio_manifest": own},
-                "data_export1": {"audio_manifest": stray},
+                "data_export": {"audio": own},
+                "data_export1": {"audio": stray},
             },
         )
 
         launcher = UnityLauncher()
-        project = os.path.join(self.temp_dir, "UnityProject")
+        project = TempProjects.project(self.temp_dir)
         if not launcher.create_project(project, batch_mode=True):
             self.skipTest(UnityEnv.project_creation_reason())
         delivered = os.path.join(project, "Assets", "Delivered")

{unitytk-0.2.2 → unitytk-0.4.0}/test/test_base_test.py

@@ -42,6 +42,7 @@ from base_test import (
     MayaEnv,
     SuiteOptions,
     TemplateSources,
+    TempProjects,
     TestAssets,
     UnityEnv,
     UserPropsProbe,
@@ -486,6 +487,83 @@ class UnityEnvTest(unittest.TestCase):
         self.assertEqual(result.errors, [])
         self.assertEqual(result.testsRun, 3)
 
+    def test_make_urp_fails_fast_on_a_project_too_deep_for_the_editor(self):
+        """The editor is not long-path aware: past MAX_PATH its package imports
+        fail, URP's Autodesk Interactive graph resolves to nothing, and the
+        material suite read the fallback pick as a product bug (2026-10-19)."""
+        deep = "C:\\" + "d" * (260 - UnityEnv.URP_IMPORT_DEPTH - 3)  # one past the limit
+
+        class Case(unittest.TestCase):
+            def test_deep(self):
+                UnityEnv.make_urp(self, None, deep, "unused.log")
+
+        result = _run_case(Case)
+        self.assertEqual(result.errors, [])
+        self.assertEqual(len(result.failures), 1)
+        self.assertIn("UNITYTK_TEMP_PROJECTS", result.failures[0][1])
+
+    def test_the_depth_gate_covers_every_shader_graph_urp_ships(self):
+        """The gate measured the Autodesk Interactive graph; URP's Arnold
+        transparent graph sits 25 characters deeper, so a 107-character project
+        passed it, that graph failed to import, URP's Arnold mapper left
+        aiStandardSurface on plain Lit and the material suite failed on every
+        run from the in-repo base (2026-10-19). Measured against each installed
+        editor's bundled URP, not a list of the graphs the suites use today."""
+        from unitytk import UnityFinder
+
+        packages = [
+            p
+            for p in map(UnityEnv.bundled_urp, UnityFinder.find_editors().values())
+            if os.path.isdir(p)
+        ]
+        if not packages:
+            self.skipTest(EnvSkip.reason(UnityEnv.NO_BUNDLED_URP))
+        deepest = UnityEnv.URP_IMPORT_DEPTH - len(UnityEnv.URP_PACKAGE_CACHE)
+        for package in packages:
+            for dirpath, dirnames, filenames in os.walk(package):
+                # A folder ending "~" (samples, docs) or hidden is never imported.
+                dirnames[:] = [d for d in dirnames if not d.endswith("~") and not d.startswith(".")]
+                for name in filenames:
+                    if name.endswith((".shadergraph", ".shadersubgraph")):
+                        below = os.path.join(dirpath, name)[len(package):]
+                        self.assertLessEqual(len(below), deepest, below)
+
+    @unittest.skipUnless(
+        os.name == "nt", "MAX_PATH is the Windows editor's limit; no Unity runs here"
+    )
+    def test_every_suite_fits_a_urp_project_under_the_default_base(self):
+        """A plain ``run_tests.py --all`` builds its projects under the in-repo
+        base, where ``<run>\\urp\\UnityProject`` put the material suite's URP
+        project at 107 characters: past the gate, so every URP suite failed at
+        once (2026-10-19). Every suite's prefix, not only today's URP suites:
+        any of them may call ``make_urp`` on its project, or add a ``"urp"`` one."""
+        prefixes = set()
+        # The allocator's own tests allocate into a sandboxed base, retired prefixes included.
+        for path in _source_files(".py", exclude=("test_temp_projects.py", "test_run_tests.py")):
+            with open(path, encoding="utf-8", errors="replace") as fh:
+                tree = ast.parse(fh.read(), filename=path)
+            for node in ast.walk(tree):
+                if (
+                    isinstance(node, ast.Call)
+                    and isinstance(node.func, ast.Attribute)
+                    and node.func.attr == "allocate"
+                    and getattr(node.func.value, "id", None) == "TempProjects"
+                    and node.args
+                    and isinstance(node.args[0], ast.Constant)
+                    and isinstance(node.args[0].value, str)
+                ):
+                    prefixes.add(node.args[0].value)
+        self.assertGreater(len(prefixes), 10, "the suites' allocations were not found")
+        # TempArtifacts' tag: the clock (16 hex digits), the pid in hex (6: Linux's
+        # pid_max ceiling; Windows ids run below it) and 4 random digits.
+        tag = "f" * (16 + 6 + 4)
+        budget = 259 - UnityEnv.URP_IMPORT_DEPTH
+        for prefix in sorted(prefixes):
+            run = os.path.join(TempProjects.DEFAULT_BASE, f"{prefix}_{tag}")
+            for project in (TempProjects.project(run), TempProjects.project(run, "urp")):
+                with self.subTest(project=project):
+                    self.assertLessEqual(len(project), budget, project)
+
 
 class SuiteOptionsTest(unittest.TestCase):
     """The opt-in switches are named once, neutrally."""
@@ -578,6 +656,64 @@ class TemplateSourcesTest(unittest.TestCase):
         with self.assertRaises(FileNotFoundError):  # a bare, pre-move name
             TemplateSources.copy_flat(["RenderEffectsController.cs"], dest)
 
+    def test_a_staged_set_must_carry_the_generated_records_it_reads(self):
+        """AudioEventController.cs staged without AudioRecord.g.cs compiled to
+        CS0103 in Unity and only the batch editor said so (2026-10-23)."""
+        controller = "AudioEvent/AudioEventController.cs"
+        record = "AudioEvent/AudioRecord.g.cs"
+        self.assertEqual(TemplateSources.missing_records([controller]), [record])
+        self.assertEqual(TemplateSources.missing_records([controller, record]), [])
+        # A record named only in a comment is no dependency.
+        self.assertEqual(TemplateSources.missing_records(["UnitytkSettings.cs"]), [])
+
+        import pythontk as ptk
+
+        store = ptk.TempArtifacts("unitytk_template_sources", policy="scoped")
+        self.addCleanup(store.cleanup)
+        dest = os.path.join(store.dir_path(), "Scripts")
+        with self.assertRaises(ValueError) as ctx:
+            TemplateSources.copy_flat(["UnitytkSettings.cs", controller], dest)
+        self.assertIn(record, str(ctx.exception))
+        self.assertFalse(os.path.exists(dest), "a refused set copies nothing")
+
+    def test_every_suite_stages_a_compile_closed_set(self):
+        """Each literal set a suite hands TemplateSources carries the records
+        its files read -- checked here, where no Unity is needed, because the
+        suites that stage them skip on every machine without one."""
+        calls = {"copy_flat", "available"}
+        offenders = []
+        for path in _source_files(".py", exclude=("base_test.py",)):
+            with open(path, encoding="utf-8", errors="replace") as fh:
+                tree = ast.parse(fh.read(), path)
+            consts = {
+                t.id: node.value
+                for node in tree.body
+                if isinstance(node, ast.Assign)
+                for t in node.targets
+                if isinstance(t, ast.Name)
+            }
+            for node in ast.walk(tree):
+                if not (
+                    isinstance(node, ast.Call)
+                    and isinstance(node.func, ast.Attribute)
+                    and node.func.attr in calls
+                    and isinstance(node.func.value, ast.Name)
+                    and node.func.value.id == "TemplateSources"
+                    and node.args
+                ):
+                    continue
+                arg = node.args[0]
+                arg = consts.get(arg.id, arg) if isinstance(arg, ast.Name) else arg
+                try:
+                    names = ast.literal_eval(arg)
+                except ValueError:
+                    continue  # built at run time: copy_flat checks it then
+                missing = TemplateSources.missing_records(names)
+                if missing:
+                    rel = os.path.relpath(path, REPO_ROOT)
+                    offenders.append(f"{rel}:{node.lineno} lacks {missing}")
+        self.assertEqual(offenders, [], "\n".join(offenders))
+
     def test_no_suite_joins_the_templates_dir_itself(self):
         """Five suites joined their own templates dir with a bare file name;
         one of them silently skipped a copy that no longer resolved."""
@@ -601,10 +737,10 @@ class UserPropsProbeTest(unittest.TestCase):
         self.assertFalse(UserPropsProbe.is_carrier(None))
 
     def test_find_carrier_returns_the_matching_entry(self):
-        postproc = {"root": {}, "data_export 1": {"shot_metadata": "{}"}}
+        postproc = {"root": {}, "data_export 1": {"animation": "{}"}}
         name, props = UserPropsProbe.find_carrier(postproc)
         self.assertEqual(name, "data_export 1")
-        self.assertEqual(props, {"shot_metadata": "{}"})
+        self.assertEqual(props, {"animation": "{}"})
 
     def test_find_carrier_is_none_when_absent(self):
         self.assertEqual(UserPropsProbe.find_carrier({"root": {}}), (None, None))
@@ -616,10 +752,8 @@ class UserPropsProbeTest(unittest.TestCase):
         self.assertFalse(UserPropsProbe.is_carrier("root/data_export/child"))
 
     def test_carrier_props_returns_just_the_props(self):
-        postproc = {"root": {}, "root/data_export": {"shot_metadata": "{}"}}
-        self.assertEqual(
-            UserPropsProbe.carrier_props(postproc), {"shot_metadata": "{}"}
-        )
+        postproc = {"root": {}, "root/data_export": {"animation": "{}"}}
+        self.assertEqual(UserPropsProbe.carrier_props(postproc), {"animation": "{}"})
 
     def test_carrier_props_is_none_when_absent(self):
         self.assertIsNone(UserPropsProbe.carrier_props({"root": {}}))

{unitytk-0.2.2 → unitytk-0.4.0}/test/test_blender_render_effects_e2e.py

@@ -306,15 +306,11 @@ class TestBlenderRenderEffectsE2E(unittest.TestCase):
 
     @classmethod
     def setUpClass(cls):
-        cls.temp_dir = TempProjects.allocate("blender_effects_e2e")
+        cls.temp_dir = TempProjects.allocate("blender_effects_e2e", cls)
         cls.blender_results = None
         cls.unity_results = None
         logger.info(f"Temp dir: {cls.temp_dir}")
 
-    @classmethod
-    def tearDownClass(cls):
-        logger.info(f"Results preserved in: {cls.temp_dir}")
-
     @BlenderEnv.skip_unless_available()
     def test_01_blender_export(self):
         """Author both channels and the keyed visibility; write both ways."""
@@ -351,7 +347,7 @@ class TestBlenderRenderEffectsE2E(unittest.TestCase):
             self.skipTest("Blender results not available -- test_01 must run first")
         editor = os.environ.get("UNITYTK_TEST_EDITOR")
         launcher = UnityLauncher(editor) if editor else UnityLauncher()
-        project_path = os.path.join(self.temp_dir, "UnityProject")
+        project_path = TempProjects.project(self.temp_dir)
         if not launcher.create_project(project_path, batch_mode=True):
             self.skipTest(UnityEnv.project_creation_reason())
         assets_path = os.path.join(project_path, "Assets")

{unitytk-0.2.2 → unitytk-0.4.0}/test/test_blender_shadow_plane_integration.py

@@ -198,7 +198,7 @@ ok = exporter.perform_export(
     tasks={"export_data_node": True},
 )
 fbx_path = os.path.join(output_dir, __FBX_NAME__ + ".fbx")
-payload = btk.DataNodes.read(ptk.Scope.DELIVERABLE, ShadowRig.SHADOW_METADATA)
+payload = btk.DataNodes.read(ptk.Scope.DELIVERABLE, ptk.SceneRecords.SHADOWS.key)
 
 # The maps ship loose beside the FBX: the records name them by file name.
 textures = []
@@ -542,15 +542,11 @@ class TestBlenderShadowPlaneIntegration(unittest.TestCase):
 
     @classmethod
     def setUpClass(cls):
-        cls.temp_dir = TempProjects.allocate("blender_shadow_integ")
+        cls.temp_dir = TempProjects.allocate("blender_shadow_integ", cls)
         cls.blender_results = None
         cls.unity_results = None
         logger.info(f"Temp dir: {cls.temp_dir}")
 
-    @classmethod
-    def tearDownClass(cls):
-        logger.info(f"Results preserved in: {cls.temp_dir}")
-
     def _blender(self):
         blender = self.__class__.blender_results
         if not blender:
@@ -590,7 +586,7 @@ class TestBlenderShadowPlaneIntegration(unittest.TestCase):
         self.assertAlmostEqual(payload.get("unit_scale"), 1.0, places=6)
         records = {p["name"]: p for p in payload.get("planes", [])}
         for key, rig in r["rigs"].items():
-            self.assertIn(rig["plane"], records, f"{key}: no shadow_metadata record")
+            self.assertIn(rig["plane"], records, f"{key}: no shadows record")
         self.assertEqual(records[r["rigs"]["horizon"]["plane"]]["type"], "horizon")
         self.assertEqual(
             records[r["rigs"]["sun"]["plane"]]["source_type"], "directional"
@@ -606,7 +602,7 @@ class TestBlenderShadowPlaneIntegration(unittest.TestCase):
         blender = self._blender()
         editor = os.environ.get("UNITYTK_TEST_EDITOR")
         launcher = UnityLauncher(editor) if editor else UnityLauncher()
-        project_path = os.path.join(self.temp_dir, "UnityProject")
+        project_path = TempProjects.project(self.temp_dir)
         if not launcher.create_project(project_path, batch_mode=True):
             self.skipTest(UnityEnv.project_creation_reason())
 

{unitytk-0.2.2 → unitytk-0.4.0}/test/test_blender_shot_metadata_integration.py

@@ -151,7 +151,7 @@ for name, mode in VARIANTS.items():
     )
     path = os.path.join(output_dir, name + ".fbx")
     exists = bool(ok) and os.path.exists(path)
-    meta = btk.DataNodes.read(ptk.Scope.DELIVERABLE, btk.DataNodes.SHOT_METADATA)
+    meta = btk.DataNodes.read(ptk.Scope.DELIVERABLE, ptk.SceneRecords.SHOTS.key)
     variants[name] = {
         "mode": mode,
         "fbx_exists": exists,
@@ -181,16 +181,12 @@ class TestBlenderShotMetadataIntegration(unittest.TestCase):
 
     @classmethod
     def setUpClass(cls):
-        cls.temp_dir = TempProjects.allocate("blender_shot_meta_integ")
+        cls.temp_dir = TempProjects.allocate("blender_shot_meta_integ", cls)
         cls.blender_results = None
         cls.unity_results = None
         cls.postproc = None
         logger.info(f"Temp dir: {cls.temp_dir}")
 
-    @classmethod
-    def tearDownClass(cls):
-        logger.info(f"Results preserved in: {cls.temp_dir}")
-
     # ---- Step 1: Blender export ----------------------------------------
 
     @BlenderEnv.skip_unless_available()
@@ -217,7 +213,6 @@ class TestBlenderShotMetadataIntegration(unittest.TestCase):
                 {c: (s["start"], s["end"]) for c, s in shots.items()}, SHOTS, name
             )
             self.assertEqual(shots["A01"]["description"], "Open the hangar doors")
-            self.assertEqual(shots["A01"]["section"], "A")
         # Shots Only ships the shots; Full Sequence Only ships one take.
         self.assertEqual(r["variants"]["shot_clips_shots"]["takes"], sorted(SHOTS))
         self.assertEqual(len(r["variants"]["shot_clips_full"]["takes"]), 1)
@@ -232,7 +227,7 @@ class TestBlenderShotMetadataIntegration(unittest.TestCase):
             self.skipTest("Blender results not available -- test_01 must run first")
         editor = os.environ.get("UNITYTK_TEST_EDITOR")
         launcher = UnityLauncher(editor) if editor else UnityLauncher()
-        project_path = os.path.join(self.temp_dir, "UnityProject")
+        project_path = TempProjects.project(self.temp_dir)
         if not launcher.create_project(project_path, batch_mode=True):
             self.skipTest(UnityEnv.project_creation_reason())
 
@@ -313,8 +308,8 @@ class TestBlenderShotMetadataIntegration(unittest.TestCase):
         self.assertTrue(unity.get("postprocessor_fired"))
         carrier = UserPropsProbe.carrier_props(self.__class__.postproc or {})
         self.assertIsNotNone(carrier, "no data_export carrier arrived")
-        self.assertIn("shot_metadata", carrier)
-        self.assertIn("Open the hangar doors", carrier["shot_metadata"])
+        self.assertIn("animation", carrier)
+        self.assertIn("Open the hangar doors", carrier["animation"])
 
         self.assertTrue(unity.get("prefab_loaded"))
         self.assertTrue(unity.get("data_export_found"))
@@ -326,7 +321,6 @@ class TestBlenderShotMetadataIntegration(unittest.TestCase):
         self.assertEqual(unity.get("record_count"), 2)
         recs = {r.split("|")[0]: r.split("|") for r in unity.get("records", [])}
         self.assertEqual(recs["A01"][1], "Open the hangar doors")
-        self.assertEqual(recs["A01"][2], "A")
         self.assertEqual(recs["A02"][1], "Raise the platform")
         clip_names = set(unity.get("clip_names", []))
         for clip in recs:  # every record's clip resolves to an imported clip