[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[nongnu] elpa/subed dc771c8 067/389: Fix/Add tests
From: |
ELPA Syncer |
Subject: |
[nongnu] elpa/subed dc771c8 067/389: Fix/Add tests |
Date: |
Fri, 3 Dec 2021 10:59:58 -0500 (EST) |
branch: elpa/subed
commit dc771c83e899301f9a0579c3e370c7537fc592f9
Author: Random User <rndusr@posteo.de>
Commit: Random User <rndusr@posteo.de>
Fix/Add tests
---
tests/test-subed-srt.el | 24 ++++++++++++++++++++++--
1 file changed, 22 insertions(+), 2 deletions(-)
diff --git a/tests/test-subed-srt.el b/tests/test-subed-srt.el
index 129b001..8386f05 100644
--- a/tests/test-subed-srt.el
+++ b/tests/test-subed-srt.el
@@ -895,8 +895,7 @@ Baz.
(describe "Validating"
(it "works in empty buffer."
(with-temp-buffer
- (expect (subed-srt-validate) :to-throw
- 'error '("Found invalid subtitle ID: \"\""))))
+ (expect (subed-srt-validate) :to-be nil)))
(it "reports invalid IDs."
(with-temp-buffer
(insert mock-srt-data)
@@ -931,6 +930,14 @@ Baz.
(expect (subed-srt-validate) :to-throw
'error '("Found invalid separator between start and
stop time: \"00:01:01,000 -->00:01:05,123\""))
(expect (point) :to-equal 15)))
+ (it "does not report error when last subtitle text is empty."
+ (with-temp-buffer
+ (insert mock-srt-data)
+ (subed-srt-move-to-subtitle-text 3)
+ (kill-whole-line)
+ (forward-char -2)
+ (subed-srt-validate)
+ (expect (point) :to-equal 107)))
(it "preserves point if there is no error."
(with-temp-buffer
(insert mock-srt-data)
@@ -1057,6 +1064,19 @@ Baz.
"3\n"
"00:03:03,456 -->
00:03:15,567\n"
"\n"))))
+ (it "ensures single space before and after time separators."
+ (with-temp-buffer
+ (insert mock-srt-data)
+ (goto-char (point-min))
+ (re-search-forward " --> ")
+ (replace-match " --> ")
+ (re-search-forward " --> ")
+ (replace-match " --> ")
+ (re-search-forward " --> ")
+ (replace-match "-->")
+ (expect (buffer-string) :not :to-equal mock-srt-data)
+ (subed-srt-sanitize)
+ (expect (buffer-string) :to-equal mock-srt-data)))
)
(describe "Renumbering"
- [nongnu] elpa/subed c865969 058/389: Toggle subed-mode when `subed-mode' is called, (continued)
- [nongnu] elpa/subed c865969 058/389: Toggle subed-mode when `subed-mode' is called, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 65bfba6 057/389: When enabling debugging, expect the debug window to exist, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed dc60c27 060/389: subed-srt--subtitle-id-at-msecs: Try to ignore invalid time stamps, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 0781285 061/389: Make mpv RPC socket and client buffer unique per buffer, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 2f86a8e 030/389: Add function: subed-srt--subtitle-text, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 17f3ae5 034/389: Add/fix tests for srt parsing changes, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed fb9354c 039/389: Adjust tests for subed-srt-move-to-subtitle-end, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 4e3e42f 044/389: subed-srt-sort: Wrap function body in atomic-change-group, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 33f27c5 063/389: Fix docstring, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 7932aee 066/389: subed-srt-sort: Sanitize before validating, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed dc771c8 067/389: Fix/Add tests,
ELPA Syncer <=
- [nongnu] elpa/subed 23156e3 071/389: subed-srt-subtitle-insert: Make argument optional, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 0e05076 072/389: subed-srt-sanitize: Properly trim trailing newlines, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 49e8e72 073/389: subed-srt-move-to-subtitle-end: Adjust docstring, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 473f041 077/389: Add ogv to video file extensions, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed bd4c44d 081/389: Fix typo, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed bd2c0e4 082/389: Add LICENSE, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 07c6b2b 083/389: Show more helpful error message if mpv isn't available, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 150a781 084/389: Rename *-move-to-* functions to *-jump-to-*, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed d2d3532 085/389: Rename get-adjust-time to get-seconds-adjust, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 73d303d 087/389: Adjust some docstrings, ELPA Syncer, 2021/12/03