fix(KC): remove try catch

Author: unknownunknown1

contracts/src/arbitration/KlerosCore.sol

@@ -139,11 +139,6 @@ contract KlerosCore is IArbitratorV2, Initializable, UUPSProxiable {
     /// @param _voteID ID of the vote given to the drawn juror.
     event Draw(address indexed _address, uint256 indexed _disputeID, uint256 _roundID, uint256 _voteID);
 
-    /// @notice Emitted when the dispute is successfully appealed.
-    /// @param _disputeID ID of the related dispute.
-    /// @param _arbitrable The arbitrable contract.
-    event ArbitrableReverted(uint256 indexed _disputeID, IArbitrableV2 indexed _arbitrable);
-
     /// @notice Emitted when a new court is created.
     /// @param _courtID ID of the new court.
     /// @param _parent ID of the parent court.
@@ -1073,9 +1068,7 @@ contract KlerosCore is IArbitratorV2, Initializable, UUPSProxiable {
         (uint256 winningChoice, , ) = currentRuling(_disputeID);
         dispute.ruled = true;
         emit Ruling(dispute.arbitrated, _disputeID, winningChoice);
-        try dispute.arbitrated.rule(_disputeID, winningChoice) {} catch {
-            emit ArbitrableReverted(_disputeID, dispute.arbitrated);
-        }
+        dispute.arbitrated.rule(_disputeID, winningChoice);
     }
 
     // ************************************* //

contracts/test/foundry/KlerosCore_Execution.t.sol

@@ -665,60 +665,6 @@ contract KlerosCore_ExecutionTest is KlerosCore_TestBase {
         assertEq(ruled, true, "Should be ruled");
     }
 
-    function test_executeRuling_arbitrableRevert() public {
-        MaliciousArbitrableMock maliciousArbitrable = new MaliciousArbitrableMock(
-            core,
-            templateData,
-            templateDataMappings,
-            arbitratorExtraData,
-            registry,
-            feeToken
-        );
-        uint256 disputeID = 0;
-
-        vm.prank(staker1);
-        core.setStake(GENERAL_COURT, 20000);
-        vm.prank(disputer);
-        maliciousArbitrable.createDispute{value: feeForJuror * DEFAULT_NB_OF_JURORS}("Action");
-        vm.warp(block.timestamp + minStakingTime);
-        sortitionModule.passPhase(); // Generating
-        vm.warp(block.timestamp + rngLookahead);
-        sortitionModule.passPhase(); // Drawing phase
-
-        core.draw(disputeID, DEFAULT_NB_OF_JURORS);
-        vm.warp(block.timestamp + timesPerPeriod[0]);
-        core.passPeriod(disputeID); // Vote
-
-        uint256[] memory voteIDs = new uint256[](3);
-        voteIDs[0] = 0;
-        voteIDs[1] = 1;
-        voteIDs[2] = 2;
-
-        vm.prank(staker1);
-        disputeKit.castVote(disputeID, voteIDs, 2, 0, "XYZ");
-        core.passPeriod(disputeID); // Appeal
-
-        vm.warp(block.timestamp + timesPerPeriod[3]);
-        core.passPeriod(disputeID); // Execution
-
-        core.executeRuling(disputeID);
-
-        (, , , bool ruled, ) = core.disputes(disputeID);
-        assertEq(ruled, true, "Should be ruled");
-
-        (bool isRuled, , ) = maliciousArbitrable.disputes(disputeID);
-        assertEq(isRuled, false, "Should be false");
-
-        vm.expectRevert(KlerosCore.RulingAlreadyExecuted.selector);
-        core.executeRuling(disputeID);
-
-        maliciousArbitrable.changeBehaviour(false);
-
-        // If the first revert was accidental arbitrable will be locked out.
-        vm.expectRevert(KlerosCore.RulingAlreadyExecuted.selector);
-        core.executeRuling(disputeID);
-    }
-
     function test_executeRuling_appealSwitch() public {
         // Check that the ruling switches if only one side was funded
         uint256 disputeID = 0;