Use asset_manager_id instead of attachment_data_id

When sending attachment information to publishing api,
use asset_manager_id instead of attachment_data_id,
as this is the id that the asset manager expects.

Author: unoduetre

app/models/file_attachment.rb

@@ -55,7 +55,7 @@ def publishing_api_details_for_format
       filename:,
       number_of_pages:,
       preview_url:,
-      attachment_data_id:,
+      asset_manager_id:,
     }
   end
 
@@ -67,10 +67,10 @@ def alternative_format_contact_email
     nil
   end
 
-  def attachment_data_id
+  def asset_manager_id
     return unless csv? && attachable.is_a?(Edition) && attachment_data.all_asset_variants_uploaded?
 
-    attachment_data.id
+    attachment_data.assets.first.asset_manager_id
   end
 
   def preview_url

test/unit/app/models/file_attachment_test.rb

@@ -69,8 +69,9 @@ def assert_delegated(attachment, method)
     assert_equal Plek.asset_root + "/media/#{attachment.attachment_data.id}/sample.csv/preview", attachment.publishing_api_details_for_format[:preview_url]
   end
 
-  test "return media attachment_data_id if all_asset_variants_uploaded?" do
+  test "#asset_manager_id returns asset manager id if all_asset_variants_uploaded?" do
     attachment = create(:csv_attachment, attachable: create(:edition))
-    assert_equal attachment.attachment_data.id, attachment.publishing_api_details_for_format[:attachment_data_id]
+    assert_not_nil attachment.attachment_data.assets.first.asset_manager_id
+    assert_equal attachment.attachment_data.assets.first.asset_manager_id, attachment.publishing_api_details_for_format[:asset_manager_id]
   end
 end