Skip to content
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

Develop Bug #554

Closed
rigelrozanski opened this issue Mar 3, 2018 · 2 comments
Closed

Develop Bug #554

rigelrozanski opened this issue Mar 3, 2018 · 2 comments
Assignees
Labels

Comments

@rigelrozanski
Copy link
Contributor

CLI develop is broken - I think by the go-wire revision, but glide tendermint version is missing from older commits so I can’t test hypothesis 😞 - we need better CLI auto-testing in here asap to avoid situations like this… basecli breaks after this step of attempting to query the genesis account

make get_vendor_deps && make build
rm -rf ~/.basecoind ~/.basecli

Terminal 1:
./build/basecoind init
# copy seed phrase into clipboard
./build/basecoind start

Terminal 2:
./build/basecli keys add demo --recover
# use password “1234567890”, hardcoded in sendtx right now
# enter the seedphrase above, and 
# store the address returned as ADDR
./build/basecli account $ADDR --node=localhost:46657

I traced back the error (even though --trace doesn't work) to coming from the unmarshal step here: https://github.com/cosmos/cosmos-sdk/blob/develop/examples/basecoin/types/account.go#L29

ebuchman added a commit that referenced this issue Mar 3, 2018
@ebuchman
Copy link
Member

ebuchman commented Mar 3, 2018

we need better CLI auto-testing in here asap to avoid situations like this

While I agree we need better CLI testing, this is actually a failure of the units tests.
This kind of basic functionality should absolutely have a test that would have caught this. Note there are no tests in the examples/basecoin/types package!

@ebuchman
Copy link
Member

ebuchman commented Mar 3, 2018

Let's not change anything in the SDK until we write more tests! #555 and #556

@ebuchman ebuchman closed this as completed Mar 3, 2018
rigelrozanski pushed a commit that referenced this issue Mar 8, 2018
added rest server and status endpoint

added get block endpoint

added latest block endpoint

add 404 if height is out of bounds

add version endpoint

add validators endpoint

export GetBlockHeight

add keys endpoints

add txs endpoints

added verb limiters to ednpoints

only output node info + json structure improvement

fixed wrong body parsing

github PR template

crypto.Address -> sdk.Address

revert to old go-wire

update glide

remove print statement and update glide

fix #554

add .DS_Store to .gitignore

Massive consolidation: queue, data storage struct, store, logic, ...

Small fixes
mossid pushed a commit that referenced this issue Mar 12, 2018
ebuchman added a commit that referenced this issue Mar 17, 2018
added rest server and status endpoint

added get block endpoint

added latest block endpoint

add 404 if height is out of bounds

add version endpoint

add validators endpoint

export GetBlockHeight

add keys endpoints

add txs endpoints

added verb limiters to ednpoints

only output node info + json structure improvement

fixed wrong body parsing

github PR template

crypto.Address -> sdk.Address

revert to old go-wire

update glide

remove print statement and update glide

fix #554

add .DS_Store to .gitignore

Massive consolidation: queue, data storage struct, store, logic, ...

Small fixes
chillyvee pushed a commit to chillyvee/cosmos-sdk that referenced this issue Mar 1, 2024
MSalopek pushed a commit to MSalopek/cosmos-sdk that referenced this issue Apr 12, 2024
* upgrade premetheus version

* more tidy
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants