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

More fixes to native tester macros - develop #660

Merged
merged 3 commits into from
Sep 12, 2019
Merged

More fixes to native tester macros - develop #660

merged 3 commits into from
Sep 12, 2019

Conversation

arhag
Copy link
Contributor

@arhag arhag commented Sep 6, 2019

Change Description

  • Fixed bug in CHECK_ASSERT and CHECK_PRINT macros which could undo a prior recording of a failure.
  • Fixed bug in symbol::print which used the wrong function to print the symbol code. (This bug was exposed from a test failure in symbol_tests after the CHECK_PRINT function was fixed.)
  • Corrected the symbol_tests unit test to use the proper expected print messages.
  • Fixed bugs in eosio::string::pop_back and eosio::string::erase.
  • Corrected wrong expected assert message in string_tests.

API Changes

  • API Changes

Documentation Additions

  • Documentation Additions

@arhag arhag changed the title More native tester macro fixes develop More fixes to native tester macros - develop Sep 6, 2019
@arhag arhag mentioned this pull request Sep 7, 2019
Merged
2 tasks
@arhag arhag merged commit 945eefd into develop Sep 12, 2019
@arhag arhag deleted the more-native-tester-macro-fixes-develop branch September 12, 2019 14:20
rise-worlds pushed a commit to potato-coin/eosio.cdt that referenced this pull request Jul 14, 2020
@arhag
Merge pull request EOSIO#660 from EOSIO/more-native-tester-macro-fixe…
3892c74 
…s-develop

More fixes to native tester macros - develop
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@larryk85 larryk85 larryk85 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@arhag @larryk85