Skip to content
This repository has been archived by the owner on Aug 2, 2022. It is now read-only.

add --no-auto-keosd to cli_test #10134

Merged
merged 1 commit into from
Apr 28, 2021
Merged

add --no-auto-keosd to cli_test #10134

merged 1 commit into from
Apr 28, 2021

Conversation

spoonincode
Copy link
Contributor

Change Description

Best I can tell, the cli_test does not need keosd to test what it is trying to test. So add --no-auto-keosd on the cleos calls to preven keosd from starting. Otherwise, a keosd using the default wallet path is left running after the tests are completed.

This leftover keosd is probably quite benign in this case, though imo it's nice to keep the ctest -LE _tests variety of tests ("parallelizable tests") self contained.

Change Type

Select ONE:

  • Documentation
  • Stability bug fix
  • Other
  • Other - special case

Testing Changes

Select ANY that apply:

  • New Tests
  • Existing Tests
  • Test Framework
  • CI System
  • Other

Consensus Changes

  • Consensus Changes

API Changes

  • API Changes

Documentation Additions

  • Documentation Additions

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants