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

Fixes to native tester macros - develop #642

Merged
merged 5 commits into from
Aug 28, 2019

Conversation

arhag
Copy link
Contributor

@arhag arhag commented Aug 28, 2019

Change Description

  • Fixes to properly support multiple usages of EOSIO_TEST in the same executable.
  • The original value of ___disable_output is now properly restored in cases where something is forced to be printed.
  • Using !(X == Y) instead of (X != Y) in CHECK_EQUAL to be consistent with REQUIRE_EQUAL. (In case the user defined != operator is not consistently the negation of the == operator for particular types.)
  • The unit test failure message now distinguishes whether the failure was due to an early abort of the unit test (e.g. due to REQUIRE_ASSERT, REQUIRE_PRINT, REQUIRE_EQUAL, or a direct call to eosio_assert) or if it the unit test completed but still failed (e.g. due to CHECK_ASSERT, CHECK_PRINT, CHECK_EQUAL). In the former case, there will be an additional text (aborted) added to the end of the failure message.
  • Removed the duplicate libnative header files that were deprecated since the base branch will be used for eosio.cdt v1.7.x which is allowed to remove these deprecated files.
  • Verbose output in unit tests if passed the -v flag.

API Changes

  • API Changes

Documentation Additions

  • Documentation Additions

Copy link
Contributor

@johndebord johndebord left a comment

Choose a reason for hiding this comment

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

Looks good.

@arhag arhag merged commit e38e925 into develop Aug 28, 2019
@arhag arhag deleted the native-tester-macro-fixes-develop branch August 28, 2019 22:17
rise-worlds pushed a commit to potato-coin/eosio.cdt that referenced this pull request Jul 14, 2020
…elop

Fixes to native tester macros - develop
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