Skip to content

testing dep assess breaks tests on emacs 25 #177

Closed
@dpsutton

Description

@dpsutton

thanks to the really helpful @Silex I've found the reason the tests have been failing.

Assess is a testing library included which requires seq. emacs 25 tops out at seq 2.3 and assess -> m-buffer -> seq 2.14. And there we have our blowup.

Wanted thoughts about how to clean this up:

  1. don't test on emacs 25. Its pretty old and pretty reasonable to expect people to be on a later version. We still support 25 of course just no reason to complicate our tests. Downside is its easy to slip something in that breaks 25 support and we won't know for a while.
  2. Inline what we need from assess. Either the actual functions or don't use the library but roll our own.
  3. Delete these tests. These tests are all about completion bounds. Not sure how much benefit they confer.

Wanted to get your thoughts @bbatsov and then we can get the test runner green again.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions