Add debug tracers needed for modularized workflow #11477
101 new issues (0 max.) of at least minor severity.
Here is an overview of what got changed by this pull request:
Issues
======
+ Solved 15
- Added 101
Complexity increasing per file
==============================
- web3/src/main/java/org/hiero/mirror/web3/evm/contracts/execution/traceability/MirrorOperationTracer.java 2
- web3/src/test/java/org/hiero/mirror/web3/evm/contracts/execution/traceability/MirrorOperationActionTracerTest.java 1
- web3/src/main/java/org/hiero/mirror/web3/evm/contracts/execution/traceability/MirrorOperationActionTracer.java 9
- web3/src/test/java/org/hiero/mirror/web3/evm/contracts/execution/traceability/OpcodeActionTracerTest.java 7
- web3/src/main/java/org/hiero/mirror/web3/evm/contracts/execution/traceability/OpcodeActionTracer.java 7
- web3/src/main/java/org/hiero/mirror/web3/evm/contracts/execution/traceability/TracerUtils.java 5
Clones added
============
- web3/src/test/java/org/hiero/mirror/web3/evm/contracts/execution/traceability/MirrorOperationTracerTest.java 11
- web3/src/main/java/org/hiero/mirror/web3/evm/contracts/execution/traceability/OpcodeTracer.java 4
- web3/src/test/java/org/hiero/mirror/web3/evm/contracts/execution/traceability/OpcodeTracerTest.java 38
- web3/src/test/java/org/hiero/mirror/web3/evm/contracts/execution/traceability/MirrorOperationActionTracerTest.java 9
- web3/src/main/java/org/hiero/mirror/web3/evm/contracts/execution/traceability/MirrorOperationActionTracer.java 1
- web3/src/test/java/org/hiero/mirror/web3/evm/contracts/execution/traceability/OpcodeActionTracerTest.java 30
- web3/src/main/java/org/hiero/mirror/web3/evm/contracts/execution/traceability/OpcodeActionTracer.java 5
See the complete overview on Codacy
Annotations
codacy-production / Codacy Static Code Analysis
web3/src/main/java/com/hedera/node/app/service/contract/impl/exec/processors/CustomMessageCallProcessor.java#L3
A high number of imports can indicate a high degree of coupling within an object.
codacy-production / Codacy Static Code Analysis
web3/src/main/java/com/hedera/node/app/service/contract/impl/exec/processors/CustomMessageCallProcessor.java#L3
Too many static imports may lead to messy code
codacy-production / Codacy Static Code Analysis
web3/src/main/java/com/hedera/node/app/service/contract/impl/exec/processors/CustomMessageCallProcessor.java#L76
This class has too many methods, consider refactoring it.
codacy-production / Codacy Static Code Analysis
web3/src/main/java/com/hedera/node/app/service/contract/impl/exec/processors/CustomMessageCallProcessor.java#L96
Method ForLazyCreation::CustomMessageCallProcessor has 6 parameters (limit is 4)
codacy-production / Codacy Static Code Analysis
web3/src/main/java/com/hedera/node/app/service/contract/impl/exec/processors/CustomMessageCallProcessor.java#L129
Method ForLazyCreation::start has 44 lines of code (limit is 30)
codacy-production / Codacy Static Code Analysis
web3/src/main/java/com/hedera/node/app/service/contract/impl/exec/processors/CustomMessageCallProcessor.java#L129
Method ForLazyCreation::start has a cyclomatic complexity of 14 (limit is 12)
codacy-production / Codacy Static Code Analysis
web3/src/main/java/com/hedera/node/app/service/contract/impl/exec/processors/CustomMessageCallProcessor.java#L129
The method 'start(MessageFrame, OperationTracer)' has a cyclomatic complexity of 14.
codacy-production / Codacy Static Code Analysis
web3/src/main/java/com/hedera/node/app/service/contract/impl/exec/processors/CustomMessageCallProcessor.java#L129
The method 'start(MessageFrame, OperationTracer)' has an NPath complexity of 1080, current threshold is 200
codacy-production / Codacy Static Code Analysis
web3/src/main/java/com/hedera/node/app/service/contract/impl/exec/processors/CustomMessageCallProcessor.java#L141
Deeply nested if..then statements are hard to read
codacy-production / Codacy Static Code Analysis
web3/src/main/java/org/hiero/mirror/web3/evm/config/EvmConfiguration.java#L272
If you run in Java5 or newer and have concurrent access, you should use the ConcurrentHashMap implementation
codacy-production / Codacy Static Code Analysis
web3/src/main/java/org/hiero/mirror/web3/evm/contracts/execution/traceability/MirrorOperationActionTracer.java#L34
Method MirrorOperationActionTracer::tracePostExecution has 34 lines of code (limit is 30)
codacy-production / Codacy Static Code Analysis
web3/src/main/java/org/hiero/mirror/web3/evm/contracts/execution/traceability/MirrorOperationActionTracer.java#L34
Method MirrorOperationActionTracer::tracePostExecution has a cyclomatic complexity of 6 (limit is 5)
codacy-production / Codacy Static Code Analysis
web3/src/test/java/org/hiero/mirror/web3/evm/contracts/execution/MirrorEvmMessageCallProcessorTest.java#L48
Avoid unused private fields such as 'contextMockedStatic'.
codacy-production / Codacy Static Code Analysis
web3/src/test/java/org/hiero/mirror/web3/evm/contracts/execution/traceability/MirrorOperationActionTracerTest.java#L32
The constant name 'contract' doesn't match '[A-Z][A-Z_0-9]*'
codacy-production / Codacy Static Code Analysis
web3/src/test/java/org/hiero/mirror/web3/evm/contracts/execution/traceability/MirrorOperationActionTracerTest.java#L33
The constant name 'recipient' doesn't match '[A-Z][A-Z_0-9]*'
codacy-production / Codacy Static Code Analysis
web3/src/test/java/org/hiero/mirror/web3/evm/contracts/execution/traceability/MirrorOperationActionTracerTest.java#L36
The constant name 'input' doesn't match '[A-Z][A-Z_0-9]*'
codacy-production / Codacy Static Code Analysis
web3/src/test/java/org/hiero/mirror/web3/evm/contracts/execution/traceability/MirrorOperationActionTracerTest.java#L37
The constant name 'operation' doesn't match '[A-Z][A-Z_0-9]*'
codacy-production / Codacy Static Code Analysis
web3/src/test/java/org/hiero/mirror/web3/evm/contracts/execution/traceability/MirrorOperationActionTracerTest.java#L103
The String literal "type=MESSAGE_CALL" appears 4 times in this file; the first occurrence is on line 102
codacy-production / Codacy Static Code Analysis
web3/src/test/java/org/hiero/mirror/web3/evm/contracts/execution/traceability/MirrorOperationActionTracerTest.java#L106
The String literal "input=0x696e70757444617461" appears 4 times in this file; the first occurrence is on line 105
codacy-production / Codacy Static Code Analysis
web3/src/test/java/org/hiero/mirror/web3/evm/contracts/execution/traceability/MirrorOperationActionTracerTest.java#L108
The String literal "output=0x6f757470757444617461" appears 4 times in this file; the first occurrence is on line 107
codacy-production / Codacy Static Code Analysis
web3/src/test/java/org/hiero/mirror/web3/evm/contracts/execution/traceability/MirrorOperationActionTracerTest.java#L109
The String literal "remainingGas=1000" appears 4 times in this file; the first occurrence is on line 108
codacy-production / Codacy Static Code Analysis
web3/src/test/java/org/hiero/mirror/web3/evm/contracts/execution/traceability/MirrorOperationActionTracerTest.java#L110
The String literal "return=0x72657475726e44617461" appears 4 times in this file; the first occurrence is on line 109
codacy-production / Codacy Static Code Analysis
web3/src/test/java/org/hiero/mirror/web3/evm/contracts/execution/traceability/MirrorOperationActionTracerTest.java#L111
The String literal "revertReason=" appears 4 times in this file; the first occurrence is on line 110
codacy-production / Codacy Static Code Analysis
web3/src/test/java/org/hiero/mirror/web3/evm/contracts/execution/traceability/MirrorOperationActionTracerTest.java#L164
Method OutputCaptureExtension::tracePostExecution has 48 lines of code (limit is 30)
codacy-production / Codacy Static Code Analysis
web3/src/test/java/org/hiero/mirror/web3/evm/contracts/execution/traceability/OpcodeActionTracerTest.java#L632
Array initialization can be written shorter