src/test/java/com/restfb/types/instagram/IgContainerTest.java (1)

34-45: ⚡ Quick win

Add a failure-path test for missing copyright_check_status.

Per the coding guidelines, unit tests should assert both success and failure paths. The current test only covers the happy path. A second test covering a JSON payload without copyright_check_status should be added to verify that getCopyrightCheckStatus() returns null and the container is still deserialised correctly.

🧪 Suggested failure-path test

Add a second JSON fixture, e.g. src/test/resources/json/instagram/container_no_copyright.json:

{
  "id": "17889615691921648",
  "status": "in_progress",
  "status_code": "IN_PROGRESS"
}

Then extend the test class:

+import static org.junit.jupiter.api.Assertions.assertNull;
+
 class IgContainerTest extends AbstractJsonMapperTests {

   `@Test`
   void checkJson() {
     // ... existing assertions ...
   }
+
+  `@Test`
+  void checkJsonWithoutCopyrightCheckStatus() {
+    IgContainer igContainer = createJsonMapper().toJavaObject(
+        jsonFromClasspath("instagram/container_no_copyright"), IgContainer.class);
+
+    assertNotNull(igContainer);
+    assertEquals("17889615691921648", igContainer.getId());
+    assertNull(igContainer.getCopyrightCheckStatus());
+  }
 }

As per coding guidelines: "Unit tests should assert both success and failure paths."

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@src/test/java/com/restfb/types/instagram/IgContainerTest.java` around lines
34 - 45, Add a failing-path unit test in the IgContainerTest class that
deserializes a JSON fixture without copyright_check_status (e.g.,
container_no_copyright.json) using createJsonMapper().toJavaObject(...) into
IgContainer and asserts the object is non-null, id/status/statusCode match
expected values ("17889615691921648", "in_progress", "IN_PROGRESS"), and that
igContainer.getCopyrightCheckStatus() returns null; implement this as a separate
`@Test` method (e.g., checkJson_missingCopyrightCheckStatus) to complement the
existing checkJson test.