| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
|
|
| |
Fixes #781
|
|
|
|
|
| |
Prevents showing "http://some.url (http://some.url)" if the url has a
fallback which is equal to the url.
|
| |
|
| |
|
|
|
|
|
|
|
|
|
|
| |
Unfortunately, the date tests of the last commit only work in one time
zone. This changes them to work in UTC. If your machine doesn't use UTC,
you now have to run the tests like this:
TZ=UTC pytest
I'm not sure what the best way to avoid requiring this is.
|
| |
|
| |
|
|
|
|
|
|
| |
Fallbacks in messages are not changed when channels/users/groups are
renamed, so they shouldn't be used unless the ref can't be found. Links
are not refs we look up, so they are still treated the same way.
|
|
|
|
|
| |
This is only used in tests and in render_topic, and I don't see why
render_topic shouldn't follow the global config that is set.
|
|
|
|
|
| |
This is only used in tests, and there we can override the config
instead.
|
|
|
|
|
|
|
|
|
| |
This is for the notifications @channel, @everyone, @group and @here.
@group isn't mentioned in the documentation[0], but it is still rendered
as a highlight in the web client, so I included it.
[0]: https://get.slack.help/hc/en-us/articles/202009646-Notify-a-channel-or-workspace
|
|
|
|
|
|
| |
This mainly adds team, channel_general and user_alice as fixtures, so we
can use those directly instead of picking arbitrary ones from the lists.
It also adds assertions to some tests which where missing it.
|
| |
|
|
|
| |
Add @user-groups with tab-completion. @user-groups will be unfurl into format <!subteam^{ID}|handle> message before sending message via linktext method.
|
| |
|
|
|
|
|
|
| |
This tests the bug that was fixed in commit c667bf6, where a message
that first has an url and then a > character would try to unfurl
everything up to the last >.
|
| |
|
| |
|
|
|
|
|
| |
This provides nicer granular test cases rather than one whole test that
fails.
|
| |
|
| |
|
| |
|
| |
|
|
|
|
|
| |
Instead of blindly splitting on spaces, use something approximating a
parser for parsing refs. Also add another test case.
|
|
The logic for showing/ignoring alt refs was wrong. Fix that, and to
prevent this from showing up again, add a unit test.
|