Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Update Sort Function unit tests #87

Open
wants to merge 1 commit into
base: feature/sort-function-weighting
Choose a base branch
from
Open

Update Sort Function unit tests #87

wants to merge 1 commit into from

Commits on Mar 22, 2017

  1. Update Sort Function unit tests 

    Since we are changing the concept of a `SortFunction` the unit tests have to change with it. This means that rather than comparing the timing we would compare the weights returned by the `SortFunction`.

However, this changes the concept of the unit tests entirely so we also need to add in unit tests for the `TimingFunction`'s as well. To make sure that the refactor returns the same values, we would also need a combination of unit tests. This means the sort function would have to return the same values if we passed the weights into a standard timing function.
    @jacksontaylor13
    jacksontaylor13 committed Mar 22, 2017
    Configuration menu
    Copy the full SHA
    8db39e2 View commit details
    Browse the repository at this point in the history