test: update BCCR account_id tests and display tests

Author: gdalcengio

bc_obps/compliance/tests/api/_bccr/_accounts/test_account_id.py

@@ -46,7 +46,7 @@ def test_successful_account_details_retrieval(self, mock_permission, mock_servic
         # Assert
         mock_service.assert_called_once_with(account_id=VALID_ACCOUNT_ID)
         assert response.status_code == 200
-        assert response.json() == {"bccr_trading_name": "Test Account Inc."}
+        assert response.json() == {"bccr_trading_name": "Test Account Inc.", "has_remote_bccr_errors": False}
 
     @patch(VALIDATE_PERMISSION_PATH)
     def test_invalid_account_id_format(self, mock_permission):
@@ -68,7 +68,7 @@ def test_empty_account_details_response(self, mock_permission, mock_service):
         response = self.client.get(self._get_endpoint_url(VALID_ACCOUNT_ID, COMPLIANCE_REPORT_VERSION_ID))
         # Assert
         assert response.status_code == 200
-        assert response.json() == {"bccr_trading_name": None}
+        assert response.json() == {"bccr_trading_name": None, "has_remote_bccr_errors": False}
 
     @patch(BCCR_SERVICE_PATH)
     @patch(VALIDATE_PERMISSION_PATH)
@@ -79,12 +79,12 @@ def test_service_error_handling(self, mock_permission, mock_service):
         # Act
         response = self.client.get(self._get_endpoint_url(VALID_ACCOUNT_ID, COMPLIANCE_REPORT_VERSION_ID))
         # Assert
-        message = "The system cannot connect to the external application. Please try again later. If the problem persists, contact GHGRegulator@gov.bc.ca for help."
-        assert_error_response(
-            response,
-            status_code=400,
-            message=message,
-        )
+        assert response.status_code == 200
+
+        response_json = response.json()
+
+        assert response_json["bccr_trading_name"] is None
+        assert response_json["has_remote_bccr_errors"] is True
 
     @patch(BCCR_SERVICE_PATH)
     @patch(VALIDATE_PERMISSION_PATH)
@@ -104,7 +104,8 @@ def test_account_details_with_null_type_of_account_holder(self, mock_permission,
         mock_service.assert_called_once_with(account_id=VALID_ACCOUNT_ID)
         assert response.status_code == 200
         assert response.json() == {
-            "bccr_trading_name": "Test Account Inc."
+            "bccr_trading_name": "Test Account Inc.",
+            "has_remote_bccr_errors": False,
         }  # Should still work with null type_of_account_holder
 
     @patch(BCCR_SERVICE_PATH)

bciers/apps/compliance/src/tests/components/widgets/BccrHoldingAccountWidget.test.tsx

@@ -149,6 +149,31 @@ describe("BccrHoldingAccountWidget", () => {
     });
   });
 
+  it("shows error message for remote BCCR API errors", async () => {
+    render(<BccrHoldingAccountWidget {...defaultProps} />);
+    mockValidateBccrAccount.mockResolvedValueOnce({
+      bccr_trading_name: null,
+      has_remote_bccr_errors: true,
+    });
+    const input = screen.getByRole("textbox");
+
+    fireEvent.change(input, { target: { value: "123456789012345" } });
+
+    await waitFor(() => {
+      expect(input).toHaveAttribute("aria-invalid", "true");
+      expect(
+        screen.getByText(
+          /Remote BC Carbon Registry system issues, please try again later or contact/i,
+        ),
+      ).toBeVisible();
+      expect(
+        screen.getByRole("link", {
+          name: /ghgregulator@gov\.bc\.ca/i,
+        }),
+      ).toHaveAttribute("href", "mailto:GHGRegulator@gov.bc.ca");
+    });
+  });
+
   it("renders input with correct help text", () => {
     render(<BccrHoldingAccountWidget {...defaultProps} />);
     expect(screen.getByText(/no account\? in bccr\./i)).toBeVisible();