fix: classify microsoft quotaLimitReached code as dest. full (#1447)

unfortunately this isn't perfectly correct; `quotaLimitReached` can mean storage quota but _appears_ it also can be used to mean rate-limiting. Unclear, for now.

Author: jzacsh

extensions/data-transfer/portability-data-transfer-microsoft/src/main/java/org/datatransferproject/transfer/microsoft/MicrosoftApiResponse.java

@@ -223,7 +223,7 @@ private FatalState toFatalState() {
     }
     // Nit: we _could_ just parse the body into json properly and make sure the JSON body "message"
     // field has this string. This seems fine for now.
-    if (httpStatus() == 507 && bodyContains("Insufficient Space Available")) {
+    if (httpStatus() == 507 && (bodyContains("Insufficient Space Available") || bodyContains("quotaLimitReached"))) {
       return FatalState.FATAL_STATE_FATAL_DESTINATION_FULL;
     }
     return FatalState.FATAL_STATE_FATAL_UNSPECIFIED;

extensions/data-transfer/portability-data-transfer-microsoft/src/test/java/org/datatransferproject/transfer/microsoft/MicrosoftApiResponseTest.java

@@ -95,6 +95,29 @@ public void testDestinationFull() throws IOException {
         });
   }
 
+  // Alternative way Microsoft APIs respond with destination-full errors.
+  @Test
+  public void testDestinationFull_quotaLimitReached() throws IOException {
+    Response networkResponse = fakeResponse(507, "",
+        "{" +
+            "\"error\": {" +
+                "\"code\":\"quotaLimitReached\"," +
+                "\"message\":\"Quota limit reached\"," +
+                "\"innerError\": {\"date\":\"2024-12-12T13:30:20\",\"request-id\":\"fake-request-id\",\"client-request-id\":\"fake-client-request-id\"}" +
+            "}" +
+        "}"
+        ).build();
+
+    MicrosoftApiResponse response = MicrosoftApiResponse.ofResponse(networkResponse);
+
+    assertThat(response.isOkay()).isFalse();
+    assertThrows(
+        DestinationMemoryFullException.class,
+        () -> {
+          response.throwDtpException("unit testing");
+        });
+  }
+
   @Test
   public void testErrorUnknown() throws IOException {
     Response networkResponse =