unitytk 0.3.0__tar.gz → 0.4.0__tar.gz

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

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: unitytk
-Version: 0.3.0
+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.3.0 → 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.3.0 → 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)
@@ -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:
@@ -2701,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;
 
@@ -2710,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"]);
@@ -2776,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.3.0 → 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.3.0 → 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.
@@ -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,18 +801,26 @@ class TestAudioCarrierPrecedence(unittest.TestCase):
 
     @classmethod
     def setUpClass(cls):
-        cls.temp_dir = TempProjects.allocate("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},
             },
         )
 

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

@@ -656,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."""
@@ -679,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))
@@ -694,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.3.0 → 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."""

{unitytk-0.3.0 → 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"

{unitytk-0.3.0 → 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)
@@ -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

{unitytk-0.3.0 → unitytk-0.4.0}/test/test_clip_name_contract_integration.py

@@ -3,7 +3,7 @@
 """Integration test: the Maya→Unity clip-name JOIN contract under stress.
 
 The whole shots→Unity scheme hinges on one identity: the shot name == the FBX
-take name == the imported ``AnimationClip.name`` == the ``shot_metadata`` ``clip``
+take name == the imported ``AnimationClip.name`` == the ``animation`` record's ``clip``
 join key.  The store refuses any name outside ``ShotStore.NAME_PATTERN``
 (``[A-Za-z0-9_]``, unique ignoring case) instead of respelling it on export, so
 what the user typed is what ships.  This test proves both halves end to end:
@@ -97,7 +97,7 @@ view = store.to_export_view()
 store.publish_export_view()
 n_takes = FbxUtils.apply_takes_from_node()
 # The take list the split realizes, read back off the carrier through the ONE
-# reader of it (each shot_metadata clip's own range).
+# reader of it (each animation-record clip's own range).
 takes = ptk.SceneRecords.declared_takes(
     lambda key: ptk.SceneRecords.resolve(key).load(DataNodes)
 )
@@ -116,7 +116,7 @@ results = {
     "fbx_exists": os.path.exists(fbx_path),
     "n_takes": n_takes,
     "clip_keys": [t["name"] for t in takes],
-    "meta_keys": [s["clip"] for s in view["shot_metadata"]["shots"]],
+    "meta_keys": [s["clip"] for s in view[ptk.SceneRecords.SHOTS.key]["shots"]],
     "refused": refused,
 }
 with open(os.path.join(output_dir, "maya_results.json"), "w") as f:
@@ -168,7 +168,7 @@ public class ClipNameContractVerifier
             .Select(c => c.name)
             .ToList();
 
-        // shot_metadata clip keys, via the attached controller (reflection).
+        // The animation record's clip keys, via the attached controller (reflection).
         var clipKeys = new List<string>();
         var smcType = (Type.GetType("ShotMetadataController, M3trik.Unitytk") ?? Type.GetType("ShotMetadataController, Assembly-CSharp"));
         var recType = (Type.GetType("ShotRecord, M3trik.Unitytk") ?? Type.GetType("ShotRecord, Assembly-CSharp"));
@@ -214,16 +214,12 @@ class TestClipNameContract(unittest.TestCase):
 
     @classmethod
     def setUpClass(cls):
-        cls.temp_dir = TempProjects.allocate("clip_name_contract")
+        cls.temp_dir = TempProjects.allocate("clip_name_contract", cls)
         cls.fbx_path = None
         cls.maya_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}")
-
     @MayaEnv.skip_unless_available()
     def test_01_maya_export(self):
         script_path = os.path.join(self.temp_dir, "maya_export.py")

{unitytk-0.3.0 → unitytk-0.4.0}/test/test_controller_runtime_integration.py

@@ -168,14 +168,10 @@ class TestControllerRuntimeIntegration(unittest.TestCase):
 
     @classmethod
     def setUpClass(cls):
-        cls.temp_dir = TempProjects.allocate("ctrl_runtime_integ")
+        cls.temp_dir = TempProjects.allocate("ctrl_runtime_integ", cls)
         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}")
-
     @EnvSkip.unless(_unity_available(), "no Unity editor found")
     def test_01_unity_runtime(self):
         """Drive both controllers in a real Unity; capture the readback."""