-
Notifications
You must be signed in to change notification settings - Fork 108
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chores(engine-primitives): nit #2068
chores(engine-primitives): nit #2068
Conversation
@@ -63,18 +63,16 @@ type PayloadAttributes[ | |||
ParentBeaconBlockRoot common.Root `json:"parentBeaconBlockRoot"` | |||
} | |||
|
|||
// NewPayloadAttributes creates a new PayloadAttributes. | |||
func NewPayloadAttributes[ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is not really needed and used only in tests. The New
method is required by a few interfaces
|
||
func TestPayloadAttributes(t *testing.T) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
consolidated tests into a single, table driven test
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## update-mockery-v2-46-3 #2068 +/- ##
==========================================================
- Coverage 22.24% 22.20% -0.04%
==========================================================
Files 356 352 -4
Lines 16034 15976 -58
Branches 12 0 -12
==========================================================
- Hits 3566 3547 -19
+ Misses 12316 12277 -39
Partials 152 152
|
Dropped constructor used in unit tests only. Marginally improved test coverage. Consolidated unit tests into table driven unit test.