Command designations of repl and buttercup

[jgrey4296]

eask repl is incorrectly set as eask-special-p, which means eask-setup-paths isn't called.

Meanwhile, eask test buttercup is not marked as an eask-execution-p.
This means the following works fine:

(require 'buttercup)

(describe "error sanity"
  (it "handles user error"
    (expect (user-error "blah") :to-throw))

  (it "handles signal"
    (expect (signal 'error "blah") :to-throw))

However, this will crash the test runner:

(require 'buttercup)

(describe "error failure"
  (it "fails on error"
    (expect (error "%s" "blah") :to-throw)))

Because of the advice added to error and warn.

I'd put this as a pull request, but that seems overkill for such minimal changes.

[jcs090218]

eask repl is incorrectly set as eask-special-p, which means eask-setup-paths isn't called.

Meanwhile, eask test buttercup is not marked as an eask-execution-p.

I’m a bit confused—these are two separate commands. How do they affect each other? 🤔

I'd put this as a pull request, but that seems overkill for such minimal changes.

Feel free to open PRs! Eask could definitely use more tests—supporting 80+ commands makes it easy to overlook small issues. I also believe that understanding users’ thoughts helps create a better user experience. 😄

[jgrey4296]

I’m a bit confused—these are two separate commands. How do they affect each other? 🤔

Mainly conceptually, them both being executed differently from expected meant tracking down why my buttercup test was failing when run from the cli but not from the repl, once the repl could load the files correctly.

Feel free to open PRs! Eask could definitely use more tests—supporting 80+ commands makes it easy to overlook small issues. I also believe that understanding users’ thoughts helps create a better user experience. 😄

will do.

[jcs090218]

Should be fixed in #386. :D