Add test, fix bug test found 😅

Author: vade

Sources/objc/opentimelineio.mm

@@ -638,7 +638,7 @@ bool media_reference_available_image_bounds(CxxRetainer* self, CGRect* rect) {
         rect->origin.x = iBox2D->min.x;
         rect->origin.y = iBox2D->min.y;
         rect->size.width = iBox2D->max.x - iBox2D->min.x;
-        rect->size.height = iBox2D->max.y - iBox2D->max.y;
+        rect->size.height = iBox2D->max.y - iBox2D->min.y;
 
         return true;
     }

Tests/OpenTimelineIOTests/testTimeline.swift

@@ -50,4 +50,33 @@ final class testTimeline: XCTestCase {
             XCTFail("Cannot read OTIO file `\(timelineInputPath)`: \(error)")
         }
     }
+    
+    func testTimelineClipAvailableBounds() {
+        let inputName = "data/clip_example.otio"
+        
+        guard let timelineInputPath = Bundle.module.path(forResource: inputName, ofType: "") else {
+            XCTFail("Missing test data `\(inputName)`")
+            return
+        }
+
+        do {
+            let otio = try SerializableObject.fromJSON(filename: timelineInputPath)
+            
+            guard let timeline = otio as? Timeline else {
+                XCTFail("Could not create Timeline object from \(timelineInputPath)")
+                return
+            }
+
+            if let firstClip = timeline.videoTracks.first!.children[1] as? Clip,
+               let mediaReference = firstClip.mediaReference,
+               let availableBounds = mediaReference.availableImageBounds
+            {
+                XCTAssertEqual(availableBounds, CGRect(origin: .zero, size: CGSize(width: 16, height: 9)))
+            }
+
+        } catch let error {
+            XCTFail("Cannot read OTIO file `\(timelineInputPath)`: \(error)")
+        }
+    }
+    
 }

Tests/data/clip_example.otio

@@ -0,0 +1,139 @@
+{
+  "OTIO_SCHEMA": "Timeline.1",
+  "metadata": {},
+  "name": "transition_test",
+  "tracks": {
+    "OTIO_SCHEMA": "Stack.1",
+    "children": [
+      {
+        "OTIO_SCHEMA": "Track.1",
+        "children": [
+          {
+            "OTIO_SCHEMA": "Gap.1",
+            "effects": [],
+            "markers": [],
+            "enabled": true,
+            "metadata": {},
+            "name": "Gap A",
+            "source_range": {
+              "OTIO_SCHEMA": "TimeRange.1",
+              "duration": {
+                "OTIO_SCHEMA": "RationalTime.1",
+                "rate": 24,
+                "value": 8
+              },
+              "start_time": {
+                "OTIO_SCHEMA": "RationalTime.1",
+                "rate": 24,
+                "value": 0
+              }
+            }
+          },
+          {
+            "OTIO_SCHEMA": "Clip.1",
+            "effects": [],
+            "markers": [],
+            "enabled": true,
+            "media_reference": {
+              "OTIO_SCHEMA": "MissingReference.1",
+              "available_range": {
+                "OTIO_SCHEMA": "TimeRange.1",
+                "duration": {
+                  "OTIO_SCHEMA": "RationalTime.1",
+                  "rate": 24,
+                  "value": 8
+                },
+                "start_time": {
+                  "OTIO_SCHEMA": "RationalTime.1",
+                  "rate": 24,
+                  "value": 0
+                }
+              },
+              "metadata": {},
+              "available_image_bounds": {
+                "OTIO_SCHEMA": "Box2d.1",
+                "min": {
+                  "OTIO_SCHEMA":"V2d.1",
+                  "x": 0.0,
+                  "y": 0.0
+                },
+                "max": {
+                  "OTIO_SCHEMA":"V2d.1",
+                  "x": 16.0,
+                  "y": 9.0
+                }
+              },
+              "name": null
+            },
+            "metadata": {},
+            "name": "Clip-001",
+            "source_range": {
+              "OTIO_SCHEMA": "TimeRange.1",
+              "duration": {
+                "OTIO_SCHEMA": "RationalTime.1",
+                "rate": 24,
+                "value": 3
+              },
+              "start_time": {
+                "OTIO_SCHEMA": "RationalTime.1",
+                "rate": 24,
+                "value": 3
+              }
+            }
+          },
+          {
+            "OTIO_SCHEMA": "Transition.1",
+            "metadata": {},
+            "name": "Dissolve",
+            "transition_type": "SMPTE_Dissolve",
+            "parameters": {},
+            "in_offset": {
+              "OTIO_SCHEMA": "RationalTime.1",
+              "rate": 24,
+              "value": 2
+            },
+            "out_offset": {
+              "OTIO_SCHEMA": "RationalTime.1",
+              "rate": 24,
+              "value": 1
+            }
+          },
+          {
+            "OTIO_SCHEMA": "Gap.1",
+            "effects": [],
+            "markers": [],
+            "enabled": true,
+            "metadata": {},
+            "name": "Gap B",
+            "source_range": {
+              "OTIO_SCHEMA": "TimeRange.1",
+              "duration": {
+                "OTIO_SCHEMA": "RationalTime.1",
+                "rate": 24,
+                "value": 8
+              },
+              "start_time": {
+                "OTIO_SCHEMA": "RationalTime.1",
+                "rate": 24,
+                "value": 0
+              }
+            }
+          }
+        ],
+        "effects": [],
+        "kind": "Video",
+        "markers": [],
+        "enabled": true,
+        "metadata": {},
+        "name": "Track-001",
+        "source_range": null
+      }
+    ],
+    "effects": [],
+    "markers": [],
+    "enabled": true,
+    "metadata": {},
+    "name": "tracks",
+    "source_range": null
+  }
+}