diff options
| author | Zeyla Hellyer <[email protected]> | 2018-08-30 18:33:56 -0700 |
|---|---|---|
| committer | Zeyla Hellyer <[email protected]> | 2018-08-30 18:35:40 -0700 |
| commit | e66812aa3b8458634901ca7226e5547f0e4be9eb (patch) | |
| tree | 698bf2341670d78a539f4ff2015f5db8a7949e19 /src/utils | |
| parent | Add more unit tests for MessageBuilder (diff) | |
| download | serenity-e66812aa3b8458634901ca7226e5547f0e4be9eb.tar.xz serenity-e66812aa3b8458634901ca7226e5547f0e4be9eb.zip | |
Add newline in MessageBuilder::push_codeblock_safe
MessageBuilder::push_codeblock_safe did not add a newline after the language or
code fence, resulting in an output like:
```code here```
or:
```rustcode here```
This commit fixes that, and makes the equivalent outputs:
```
code here```
and:
```rust
code here```
Closes #374.
Diffstat (limited to 'src/utils')
| -rw-r--r-- | src/utils/message_builder.rs | 12 |
1 files changed, 7 insertions, 5 deletions
diff --git a/src/utils/message_builder.rs b/src/utils/message_builder.rs index 79c02bb..72f8e9b 100644 --- a/src/utils/message_builder.rs +++ b/src/utils/message_builder.rs @@ -501,6 +501,8 @@ impl MessageBuilder { self.0.push_str(language); } + self.0.push('\n'); + { let mut c = content.into(); c.inner = normalize(&c.inner).replace("```", " "); @@ -1103,23 +1105,23 @@ mod test { fn push_codeblock_safe() { assert_eq!( MessageBuilder::new().push_codeblock_safe("foo", Some("rs")).0, - "```rsfoo```", + "```rs\nfoo```", ); assert_eq!( MessageBuilder::new().push_codeblock_safe("", None).0, - "``````", + "```\n```", ); assert_eq!( MessageBuilder::new().push_codeblock_safe("1 * 2", None).0, - "```1 * 2```", + "```\n1 * 2```", ); assert_eq!( MessageBuilder::new().push_codeblock_safe("`1 * 3`", None).0, - "````1 * 3````", + "```\n`1 * 3````", ); assert_eq!( MessageBuilder::new().push_codeblock_safe("```.```", None).0, - "``` . ```", + "```\n . ```", ); } |