diff --git a/components/remote_settings/src/client.rs b/components/remote_settings/src/client.rs index 07ff37b16b..d8dca71ff6 100644 --- a/components/remote_settings/src/client.rs +++ b/components/remote_settings/src/client.rs @@ -221,7 +221,7 @@ impl RemoteSettingsClient { } // Try to download the attachment because neither the storage nor the local data had it - let attachment = inner.api_client.get_attachment(&metadata.location)?; + let attachment = inner.api_client.fetch_attachment(&metadata.location)?; // Verify downloaded data if attachment.len() as u64 != metadata.size { @@ -287,7 +287,7 @@ pub trait ApiClient { fn fetch_changeset(&mut self, timestamp: Option) -> Result; /// Fetch an attachment from the server - fn get_attachment(&mut self, attachment_location: &str) -> Result>; + fn fetch_attachment(&mut self, attachment_location: &str) -> Result>; /// Check if this client is pointing to the production server fn is_prod_server(&self) -> Result; @@ -397,7 +397,7 @@ impl ApiClient for ViaductApiClient { } } - fn get_attachment(&mut self, attachment_location: &str) -> Result> { + fn fetch_attachment(&mut self, attachment_location: &str) -> Result> { let attachments_base_url = match &self.remote_state.attachments_base_url { Some(attachments_base_url) => attachments_base_url.to_owned(), None => { @@ -2259,7 +2259,7 @@ mod test_packaged_metadata { .returning(move || collection_url.clone()); api_client.expect_is_prod_server().returning(|| Ok(true)); api_client - .expect_get_attachment() + .expect_fetch_attachment() .returning(move |_| Ok(mock_api_data.clone())); let rs_client =