Skip to content
This repository has been archived by the owner on Apr 4, 2024. It is now read-only.

Fix unit test non-determinism #1446

Closed
fedekunze opened this issue Nov 9, 2022 · 1 comment · Fixed by #1447
Closed

Fix unit test non-determinism #1446

fedekunze opened this issue Nov 9, 2022 · 1 comment · Fixed by #1447
Labels
good first issue Good for newcomers help wanted Extra attention is needed pinned Pinned issues that won't be closed by stalebot Type: Tests issues and PR related to tests

Comments

@fedekunze
Copy link
Contributor

--- FAIL: TestKeyring (0.22s)
    algorithm_test.go:78: 
        	Error Trace:	/home/runner/work/ethermint/ethermint/crypto/hd/algorithm_test.go:78
        	Error:      	Not equal: 
        	            	expected: "0x6306EcD6E3d3E269C6E1328951873Eb3AEDd0DD2"
        	            	actual  : "0xaFF332245CD714842BF88374ea62f131ec906c49"
        	            	
        	            	Diff:
        	            	--- Expected
        	            	+++ Actual
        	            	@@ -1 +1 @@
        	            	-0x6306EcD6E3d3E269C6E132895[187](https:/evmos/ethermint/actions/runs/3429370302/jobs/5714895630#step:5:188)3Eb3AEDd0DD2
        	            	+0xaFF332245CD714842BF88374ea62f131ec906c49
        	Test:       	TestKeyring
FAIL
coverage: 86.2% of statements
@fedekunze fedekunze added good first issue Good for newcomers help wanted Extra attention is needed Type: Tests issues and PR related to tests pinned Pinned issues that won't be closed by stalebot labels Nov 9, 2022
@GAtom22 GAtom22 assigned GAtom22 and unassigned GAtom22 Nov 14, 2022
@GAtom22
Copy link
Contributor

GAtom22 commented Nov 14, 2022

Already being addressed in PR #1447

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
good first issue Good for newcomers help wanted Extra attention is needed pinned Pinned issues that won't be closed by stalebot Type: Tests issues and PR related to tests
Projects
No open projects
Status: Canceled
Development

Successfully merging a pull request may close this issue.

2 participants