lukelee-sl

Address the following future/todo messages | app/service/contract/impl/test/handlers/ContractCreateHandlerTest.java | 134 | // FUTURE: uncomment this after JKey removal | unknown what this is about | |---------------------------------------------------------------------------------------------|-----|-------------------------------------------------------------------------------------------------------|------------------------------------------------------------------------------------------------------------------------------------------------------------------| | app/service/contract/impl/exec/gas/CustomGasCalculator.java | 98 |...

Triage all //Future and TODO comments in modularized smart contracts service

3

### Problem Many comments with //FUTURE or TODO exists in the modularized code base Also part of this: consider all comments about specific code that's there for mono-service compatibility to...

Address the following future/todo comments | app/service/contract/impl/handlers/ContractCreateHandler.java | 87 | // FUTURE:Consider reading the init code from a file but this may be too much to do for pure checks...

Address the following todo/future comments | app/service/contract/impl/exec/systemcontracts/hts/transfer/ClassicTransfersCall.java | 235 | // (FUTURE) Remove this divisor special case, done only for mono-service fidelity | |---------------------------------------------------------------------------------------|-----|------------------------------------------------------------------------------------------| | app/service/contract/impl/exec/systemcontracts/hts/create/ClassicCreatesCall.java | 126 | //...

Examine the role of xTests in testing the smart contracts module

1

### Problem xTests are used to test certain functionality. Their use may be discontinued in the future. ### Solution Examine the current test and find an appropriate test frame to...

Remove redundant check for static frame in `CustomSelfDestructOperation`.

2

### Problem The `CustomSelfDestructOperation` does not descend from or use the Besu implementation of `SelfDestructOperation`. Perhaps we should for reference even if we do not call the super class. There...

Address the following future/todo comments | app/service/contract/impl/exec/systemcontracts/hts/iskyc/IsKycCall.java | 80 | // @Future remove to revert #9064 after modularization is completed | |-----------------------------------------------------------------------------------------------------|----|---------------------------------------------------------------------| | app/service/contract/impl/exec/systemcontracts/hts/defaultkycstatus/DefaultKycStatusCall.java | 67 | // @Future remove...

Token view functions revert on incorrect input

4

### Description View functions in mono service that are not from the ERC[20, 721] specifications should not revert if the given input contains an incorrect tokenId. ### Steps to reproduce...

### Problem There is a new function available in the HederaAcountService system contract called `isAuthorizedRaw` ### Solution Add test to demonstrate usage of `isAuthorizedRaw` ### Alternatives _No response_

### Problem From time to time, we need to make additions to or more importantly make modifications to the function signatures for system contracts. For example, a new parameter might...