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

Wlb/adding nodeos param tests 2.0.x #10135

Merged
merged 11 commits into from
Mar 16, 2021

Conversation

williamblevins
Copy link
Collaborator

@williamblevins williamblevins commented Mar 15, 2021

Change Description

Adding tests for nodeos parameters --full-version and --print-build-info.

Test for --print-build-info checks boost_version. I had originally planned to check the compiler output also, but it was a little less intuitive.

Output from the unit tests taken from Mac 10.14 https://buildkite.com/EOSIO/eosio/builds/27460#94aa4efc-0891-4ba4-95e5-1fa62e276da5 which I checked specifically because of needing to and the env parameters.

test 103
--
  | Start 103: version-label-test
  |  
  | 103: Test command: /Users/anka/eos/build/tests/version-label.sh
  | 103: Test timeout computed to be: 1500
  | 103: ##### Nodeos Version Label Test #####
  | 103: Using EOSIO_ROOT="/Users/anka/eos/".
  | 103: Parsing "/Users/anka/eos//build/CMakeCache.txt"...
  | 103: Expecting "v2.0.10"...
  | 103: Passed with "v2.0.10".
  | 89/92 Test #103: version-label-test .................................   Passed    0.03 sec
  | test 104
  | Start 104: full-version-label-test
  |  
  | 104: Test command: /Users/anka/eos/build/tests/full-version-label.sh
  | 104: Test timeout computed to be: 1500
  | 104: ##### Nodeos Full Version Label Test #####
  | 104: Using EOSIO_ROOT="/Users/anka/eos/".
  | 104: Parsing "/Users/anka/eos//build/CMakeCache.txt"...
  | 104: Expecting "v2.0.10-8a46d87e01d28b365f78e8ab00f48ee0d21a5927"...
  | 104: Passed with "v2.0.10-8a46d87e01d28b365f78e8ab00f48ee0d21a5927".
  | 90/92 Test #104: full-version-label-test ............................   Passed    0.03 sec
  | test 105
  | Start 105: print-build-info-test
  |  
  | 105: Test command: /Users/anka/eos/build/tests/print-build-info.sh
  | 105: Test timeout computed to be: 1500
  | 105: ##### Nodeos Print Build Info Test #####
  | 105: Using EOSIO_ROOT="/Users/anka/eos/".
  | 105: $ /Users/anka/eos//build/bin/nodeos --print-build-info 2>&1 \| tee >(cat - >&9) \|\| :
  | 105: info  2021-03-15T22:04:17.056 thread-0  chain_plugin.cpp:628          plugin_initialize    ] initializing chain plugin
  | 105: info  2021-03-15T22:04:17.056 thread-0  chain_plugin.cpp:643          plugin_initialize    ] Build environment JSON:
  | 105: {
  | 105:   "debug": false,
  | 105:   "os": "OS_MACOS",
  | 105:   "arch": "ARCH_X86_64",
  | 105:   "boost_version": 107100,
  | 105:   "compiler": "4.2.1 Compatible Clang 8.0.1 (https://git.llvm.org/git/clang.git a03da8be08a208122e292016cb6cea1f30229677) (https://git.llvm.org/git/llvm.git 18e41dc964f916504ec90dba523826ac74d235c4)"
  | 105: }
  | 105: warn  2021-03-15T22:04:17.057 thread-0  chain_plugin.cpp:1138         plugin_initialize    ] 3100009 node_management_success: Node management operation successfully executed
  | 105: reported build environment information
  | 105:     {}
  | 105:     thread-0  chain_plugin.cpp:660 plugin_initialize
  | 105:
  | 105: Failed to initialize
  | 105: ARCH: "ARCH_X86_64"
  | 105: BOOST_VERSION: 107100
  | 105: COMPILER: "4.2.1 Compatible Clang 8.0.1 (https://git.llvm.org/git/clang.git a03da8be08a208122e292016cb6cea1f30229677) (https://git.llvm.org/git/llvm.git 18e41dc964f916504ec90dba523826ac74d235c4)"
  | 105: DEBUG: false
  | 105: OS: "OS_MACOS"
  | 105: Verifying boost version: "107100" == "107100".
  | 105: Validation of build info complete.
  | 91/92 Test #105: print-build-info-test ..............................   Passed    0.20 sec

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

@williamblevins williamblevins changed the base branch from master to release/2.0.x March 15, 2021 20:00
@williamblevins williamblevins marked this pull request as ready for review March 16, 2021 00:08
Copy link
Contributor

@kj4ezj kj4ezj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🍳

@kj4ezj kj4ezj merged commit ccc288b into release/2.0.x Mar 16, 2021
@kj4ezj kj4ezj deleted the wlb/adding_nodeos_param_tests_2.0.x branch March 16, 2021 13:40
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.

4 participants