Miner: Export RepayDebt

Author: arajasek

actors/miner/src/lib.rs

@@ -130,6 +130,7 @@ pub enum Method {
     WithdrawBalanceExported = frc42_dispatch::method_hash!("WithdrawBalance"),
     ChangeMultiaddrsExported = frc42_dispatch::method_hash!("ChangeMultiaddrs"),
     ConfirmUpdateWorkerKeyExported = frc42_dispatch::method_hash!("ConfirmUpdateWorkerKey"),
+    RepayDebtExported = frc42_dispatch::method_hash!("RepayDebt"),
     ChangeBenificiaryExported = frc42_dispatch::method_hash!("ChangeBeneficiary"),
     GetBeneficiaryExported = frc42_dispatch::method_hash!("GetBeneficiary"),
     GetOwnerExported = frc42_dispatch::method_hash!("GetOwner"),
@@ -5018,7 +5019,7 @@ impl ActorCode for Actor {
                 Self::confirm_update_worker_key(rt)?;
                 Ok(RawBytes::default())
             }
-            Some(Method::RepayDebt) => {
+            Some(Method::RepayDebt) | Some(Method::RepayDebtExported) => {
                 Self::repay_debt(rt)?;
                 Ok(RawBytes::default())
             }

actors/miner/tests/repay_debts.rs

@@ -1,7 +1,12 @@
-use fil_actor_miner::locked_reward_from_reward;
+use fil_actor_miner::{locked_reward_from_reward, Actor, Method};
+use fil_actors_runtime::test_utils::{expect_abort_contains_message, make_identity_cid};
+use fil_actors_runtime::BURNT_FUNDS_ACTOR_ADDR;
+use fvm_ipld_encoding::RawBytes;
 use fvm_shared::bigint::Zero;
 use fvm_shared::clock::ChainEpoch;
 use fvm_shared::econ::TokenAmount;
+use fvm_shared::error::ExitCode;
+use fvm_shared::METHOD_SEND;
 
 mod util;
 use util::*;
@@ -48,6 +53,52 @@ fn pay_debt_entirely_from_balance() {
     h.check_state(&rt);
 }
 
+#[test]
+fn repay_debt_restricted_correctly() {
+    let h = ActorHarness::new(PERIOD_OFFSET);
+    let mut rt = h.new_runtime();
+    h.construct_and_verify(&mut rt);
+
+    // introduce fee debt
+    let mut st = h.get_state(&rt);
+    let fee_debt: TokenAmount = 4 * &*BIG_BALANCE;
+    st.fee_debt = fee_debt.clone();
+    rt.replace_state(&st);
+
+    rt.set_caller(make_identity_cid(b"1234"), h.owner);
+
+    // fail to call the unexported method
+    expect_abort_contains_message(
+        ExitCode::USR_FORBIDDEN,
+        "must be built-in",
+        rt.call::<Actor>(Method::RepayDebt as u64, &RawBytes::default()),
+    );
+
+    // can call the exported method
+
+    rt.expect_validate_caller_addr(h.caller_addrs());
+
+    rt.add_balance(fee_debt.clone());
+    rt.set_received(fee_debt.clone());
+
+    rt.expect_send(
+        BURNT_FUNDS_ACTOR_ADDR,
+        METHOD_SEND,
+        RawBytes::default(),
+        fee_debt.clone(),
+        RawBytes::default(),
+        ExitCode::OK,
+    );
+
+    rt.call::<Actor>(Method::RepayDebtExported as u64, &RawBytes::default()).unwrap();
+
+    rt.verify();
+
+    let st = h.get_state(&rt);
+    assert!(st.fee_debt.is_zero());
+    h.check_state(&rt);
+}
+
 #[test]
 fn partially_repay_debt() {
     let h = ActorHarness::new(PERIOD_OFFSET);