Sepal tests - Githubissues

CactusDev / CactusBot

An open source, community-written service-agnostic chat bot

MIT License

30 stars 6 forks source link

Sepal tests #282

Closed Innectic closed 7 years ago

Innectic commented 7 years ago

What does this change?

Adds sepal tests.

Requirements

[x] Only PG-rated language used (code, commits, etc.)
[x] Descriptive commit messages
[x] Changes have been tested
[x] Changes do not break any existing functionalities

2Cubed commented 7 years ago

@Innectic Looks like you're going to want to pass [CONFIG_PACKET], as it's looking for a list. https://github.com/CactusDev/CactusBot/blob/develop/cactusbot/sepal.py#L81

Innectic commented 7 years ago

@2Cubed I don't think so. That just makes it error more, because the actual parser method isn't looking for that

2Cubed commented 7 years ago

@Innectic Oh, derp, right, sorry. Missed what it was iterating over.

2Cubed commented 7 years ago

Note: This has not been integration-tested! We currently have no confirmation that the packets work in a production environment. (Need Sepal config packet sending to be fixed first. cc: @Innectic)