Skip to content

Conversation

clark-ja
Copy link
Collaborator

@clark-ja clark-ja commented Jan 12, 2022

Add tests for heap bins small, large and unsorted cmd

Description/Motivation/Screenshots

This PR adds some missing tests for:

  • heap bins small
  • heap bins large
  • heap bins unsorted

All 3 new tests use the same binary file heap-bins.out.

How Has This Been Tested?

Architecture Yes/No Comments
x86-32 ✖️
x86-64 ✔️
ARM ✖️
AARCH64 ✖️
MIPS ✖️
POWERPC ✖️
SPARC ✖️
RISC-V ✖️
make test ✔️

Checklist

  • My PR was done against the dev branch, not master.
  • My code follows the code style of this project.
  • My change includes a change to the documentation, if required.
  • My change adds tests as appropriate.
  • I have read and agree to the CONTRIBUTING document.

Copy link
Owner

@hugsy hugsy left a comment

Choose a reason for hiding this comment

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

nice & easy

@hugsy hugsy merged commit 2975d5f into hugsy:dev Jan 12, 2022
@clark-ja clark-ja deleted the test_heap_bins branch January 12, 2022 23:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants