Issue #373: ANIm should not be symmetric

[baileythegreen]

With this PR, ANIm is no longer treated as symmetric. Comparisons between each pair of genomes are run in both directions; enforced sorting of output file names has been disabled; tests relying on this sorting are skipped; and documentation has been updated.

Closes #373.

Type of change

• Bug fix (non-breaking change that fixes an issue)
• New feature (non-breaking change that adds functionality)
• Breaking change (fix or feature that would cause existing functionality not to work as expected)
• This change requires a documentation update
• This is a documentation update

Action Checklist

• Work on a single issue/concept (if there are multiple separate issues to address, please use a separate pull request for each)
• Fork the pyani repository under your own account (please allow write access for repository maintainers)
• Set up an appropriate development environment (please see CONTRIBUTING.md)
• Create a new branch with a short, descriptive name
• Work on this branch
  • style guidelines have been followed
  • new code is commented, especially in hard-to-understand areas
  • corresponding changes to documentation have been made
  • tests for the change have been added that demonstrate the fix or feature works
• Test locally with pytest -v non-passing code will not be merged
• Rebase against origin/master
• Check changes with flake8 and black before submission
• Commit branch
• Submit pull request, describing the content and intent of the pull request
• Request a code review
• Continue the discussion at Pull requests section in the pyani repository

[widdowquinn]

tests relying on this sorting are skipped

What is the reason for retaining and skipping them? I would have thought deleting/removing the tests would be appropriate.

[baileythegreen]

tests relying on this sorting are skipped

What is the reason for retaining and skipping them? I would have thought deleting/removing the tests would be appropriate.

I wanted to think a bit about whether there might be a different use for it (there's really only one such test), or whether it might be usefully repurposed before completely removing it.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 75.77%. Comparing base (62949c4) to head (8d9cc42).
Report is 117 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #376      +/-   ##
==========================================
- Coverage   75.98%   75.77%   -0.21%     
==========================================
  Files          55       55              
  Lines        3747     3740       -7     
==========================================
- Hits         2847     2834      -13     
- Misses        900      906       +6     

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.