chore!: remove unused response method (#1263)

The `response` method from `ScriptCallHandler`, `ContractCallHandler`
and `MultiContractCallHandler` was unreachable because of:
```rust
let tx_id = self.cached_tx_id.expect("Cached tx_id is missing");
```
Which means that if there is no cached `tx_id` this method will panic.
The only way to get a `tx_id` is by calling `call`, `simulate` or
`submit` and these methods return other types that handle the response.

Even though this method was never used, I will mark the PR as breaking
as the method was `pub`.

BREAKING CHANGE: remove `response` method from `ScriptCallHandler`,
`ContractCallHandler` and `MultiContractCallHandler`.

packages/fuels-programs/src/contract.rs

@@ -603,21 +603,8 @@ where
         Ok(SubmitResponse::new(tx_id, self))
     }
 
-    pub async fn response(self) -> Result<FuelCallResponse<D>> {
-        let provider = self.account.try_provider()?;
-        let tx_id = self.cached_tx_id.expect("Cached tx_id is missing");
-
-        let receipts = provider
-            .tx_status(&tx_id)
-            .await?
-            .take_receipts_checked(Some(&self.log_decoder))?;
-
-        self.get_response(receipts)
-    }
-
     /// Call a contract's method on the node, in a simulated manner, meaning the state of the
     /// blockchain is *not* modified but simulated.
-    ///
     pub async fn simulate(&mut self) -> Result<FuelCallResponse<D>> {
         self.call_or_simulate(true).await
     }
@@ -626,8 +613,7 @@ where
         let tx = self.build_tx().await?;
         let provider = self.account.try_provider()?;
 
-        let chain_id = provider.chain_id();
-        self.cached_tx_id = Some(tx.id(chain_id));
+        self.cached_tx_id = Some(tx.id(provider.chain_id()));
 
         let tx_status = if simulate {
             provider.checked_dry_run(tx).await?
@@ -903,18 +889,6 @@ impl<T: Account> MultiContractCallHandler<T> {
         Ok(SubmitResponseMultiple::new(tx_id, self))
     }
 
-    pub async fn response<D: Tokenizable + Debug>(self) -> Result<FuelCallResponse<D>> {
-        let provider = self.account.try_provider()?;
-        let tx_id = self.cached_tx_id.expect("Cached tx_id is missing");
-
-        let receipts = provider
-            .tx_status(&tx_id)
-            .await?
-            .take_receipts_checked(Some(&self.log_decoder))?;
-
-        self.get_response(receipts)
-    }
-
     /// Call contract methods on the node, in a simulated manner, meaning the state of the
     /// blockchain is *not* modified but simulated.
     /// It is the same as the [call] method because the API is more user-friendly this way.
@@ -930,9 +904,8 @@ impl<T: Account> MultiContractCallHandler<T> {
     ) -> Result<FuelCallResponse<D>> {
         let tx = self.build_tx().await?;
         let provider = self.account.try_provider()?;
-        let chain_id = provider.chain_id();
 
-        self.cached_tx_id = Some(tx.id(chain_id));
+        self.cached_tx_id = Some(tx.id(provider.chain_id()));
 
         let tx_status = if simulate {
             provider.checked_dry_run(tx).await?

packages/fuels-programs/src/script_calls.rs

@@ -253,18 +253,6 @@ where
         Ok(SubmitResponse::new(tx_id, self))
     }
 
-    pub async fn response(self) -> Result<FuelCallResponse<D>> {
-        let tx_id = self.cached_tx_id.expect("Cached tx_id is missing");
-
-        let receipts = self
-            .provider
-            .tx_status(&tx_id)
-            .await?
-            .take_receipts_checked(Some(&self.log_decoder))?;
-
-        self.get_response(receipts)
-    }
-
     /// Call a script on the node, in a simulated manner, meaning the state of the
     /// blockchain is *not* modified but simulated.
     /// It is the same as the [`call`] method because the API is more user-friendly this way.