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.

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

Add support for a repl backend to test handlers via a cli interface #62

Closed
adamzaninovich opened this issue Apr 18, 2016 · 4 comments · Fixed by #133
Closed

Add support for a repl backend to test handlers via a cli interface #62

adamzaninovich opened this issue Apr 18, 2016 · 4 comments · Fixed by #133
Assignees

Comments

@adamzaninovich
Copy link
Member

First priority should be a Slack backend and a command line repl backend

@adamzaninovich
Copy link
Member Author

Also a test backend, to help with #47

@adamzaninovich adamzaninovich added this to the Alice v0.4 milestone Jun 16, 2016
@adamzaninovich adamzaninovich changed the title Add support for multiple chat backends (not just Slack) Add support for a repl backend to test handlers via a cli interface Apr 11, 2020
@adamzaninovich adamzaninovich removed this from the Alice 2 milestone Apr 11, 2020
@NateBarnes NateBarnes self-assigned this Apr 12, 2020
@NateBarnes
Copy link
Member

I'll take a look at this one since I was just touching these parts of the code for the handler testing issue. I imagine general pathway will be similar.

@adamzaninovich
Copy link
Member Author

adamzaninovich commented Apr 12, 2020

@NateBarnes check out closed PR #70 where I started working on this before. Good luck and godspeed.

Notably:

All in all, I liked where that PR was going, it was just too big of a change overall. It tried to change a lot more than just adding a repl adapter. But in some ways, it's also a decent preview of what I'd like alice to look like someday.

@NateBarnes
Copy link
Member

Yeah, that PR is awesome, but definitely more ambitious than I was envisioning. My initial thought was to use something like the existing spy adapter to allow the user to do so from iex. You can actually do a hacky version of it from the console now:

Screen Shot 2020-04-11 at 8 22 17 PM

I would need to restructure some things and put some syntactic sugar around it, but that's the general thrust I was thinking.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants