[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[nongnu] elpa/subed e195756 179/389: Rewrite adjusting tests
From: |
ELPA Syncer |
Subject: |
[nongnu] elpa/subed e195756 179/389: Rewrite adjusting tests |
Date: |
Fri, 3 Dec 2021 11:00:21 -0500 (EST) |
branch: elpa/subed
commit e19575679c75bc3b4a4b144428aaecba2c4e42b9
Author: Random User <rndusr@posteo.de>
Commit: Random User <rndusr@posteo.de>
Rewrite adjusting tests
They were still using the global subed-srt-increase/decrease-start/stop-time
functions.
---
tests/test-subed-srt.el | 205 ++++++++++++++++++++----------------------------
1 file changed, 85 insertions(+), 120 deletions(-)
diff --git a/tests/test-subed-srt.el b/tests/test-subed-srt.el
index dac451b..1d13f02 100644
--- a/tests/test-subed-srt.el
+++ b/tests/test-subed-srt.el
@@ -611,167 +611,136 @@ Baz.
(add-hook 'subed-subtitle-time-adjusted-hook 'foo)
(with-temp-buffer
(insert mock-srt-data)
- (expect (subed-increase-start-time) :to-equal 100)
- (expect 'foo :to-have-been-called-with 183550)
- (expect 'foo :to-have-been-called-times 1)
(subed-srt--jump-to-subtitle-id 1)
- (expect (subed-increase-stop-time) :to-equal 100)
- (expect 'foo :to-have-been-called-with 61000)
+ (subed-srt--adjust-subtitle-start 100)
+ (expect 'foo :to-have-been-called-with 61100)
+ (expect 'foo :to-have-been-called-times 1)
+ (subed-srt--adjust-subtitle-stop 123)
+ (expect 'foo :to-have-been-called-with 61100)
(expect 'foo :to-have-been-called-times 2)
- (subed-srt--jump-to-subtitle-end 2)
- (expect (subed-decrease-start-time) :to-equal -100)
- (expect 'foo :to-have-been-called-with 122134)
+ (subed-srt--jump-to-subtitle-id 2)
+ (subed-srt--adjust-subtitle-start 6)
+ (expect 'foo :to-have-been-called-with 122240)
(expect 'foo :to-have-been-called-times 3)
- (subed-srt--jump-to-subtitle-text 3)
- (expect (subed-decrease-stop-time) :to-equal -100)
- (expect 'foo :to-have-been-called-with 183550)
+ (subed-srt--adjust-subtitle-stop 123)
+ (expect 'foo :to-have-been-called-with 122240)
(expect 'foo :to-have-been-called-times 4))
(remove-hook 'subed-subtitle-time-adjusted-hook 'foo)))
(it "adjusts the start/stop time."
(with-temp-buffer
(insert mock-srt-data)
(subed-srt--jump-to-subtitle-id 1)
- (expect (subed-increase-start-time) :to-equal 100)
+ (expect (subed-srt--adjust-subtitle-start 100) :to-equal 100)
(expect (save-excursion
(subed-srt--jump-to-subtitle-time-start)
(thing-at-point 'line)) :to-equal
"00:01:01,100 --> 00:01:05,123\n")
- (expect (subed-decrease-start-time) :to-equal -100)
- (expect (subed-decrease-start-time) :to-equal -100)
+ (expect (subed-srt--adjust-subtitle-start -200) :to-equal -200)
(expect (save-excursion
(subed-srt--jump-to-subtitle-time-start)
(thing-at-point 'line)) :to-equal
"00:01:00,900 --> 00:01:05,123\n")
- (expect (subed-increase-stop-time) :to-equal 100)
- (expect (subed-increase-stop-time) :to-equal 100)
+ (expect (subed-srt--adjust-subtitle-stop 200) :to-equal 200)
(expect (save-excursion
(subed-srt--jump-to-subtitle-time-start)
(thing-at-point 'line)) :to-equal
"00:01:00,900 --> 00:01:05,323\n")
- (expect (subed-decrease-stop-time) :to-equal -100)
+ (expect (subed-srt--adjust-subtitle-stop -100) :to-equal -100)
(expect (save-excursion
(subed-srt--jump-to-subtitle-time-start)
(thing-at-point 'line)) :to-equal
"00:01:00,900 --> 00:01:05,223\n")))
(it "adjusts the start/stop time if milliseconds lack digits."
(with-temp-buffer
(insert mock-srt-data)
(subed-srt--jump-to-subtitle-id 3)
- (expect (subed-increase-start-time 200) :to-equal 200)
- (expect (subed-decrease-start-time 100) :to-equal -100)
+ (expect (subed-srt--adjust-subtitle-start 200) :to-equal 200)
+ (expect (subed-srt--adjust-subtitle-start -100) :to-equal -100)
(expect (save-excursion
(subed-srt--jump-to-subtitle-time-start)
(thing-at-point 'line)) :to-equal
"00:03:03,550 --> 00:03:15,5\n")
- (expect (subed-increase-stop-time 100) :to-equal 100)
- (expect (subed-decrease-stop-time 200) :to-equal -200)
+ (expect (subed-srt--adjust-subtitle-stop 100) :to-equal 100)
+ (expect (subed-srt--adjust-subtitle-stop -200) :to-equal -200)
(expect (save-excursion (subed-srt--jump-to-subtitle-time-stop)
(thing-at-point 'line)) :to-equal
"00:03:03,550 --> 00:03:15,400\n")))
- (it "sets the number of milliseconds if given an argument."
- (with-temp-buffer
- (insert mock-srt-data)
- (subed-srt--jump-to-subtitle-id 1)
- (expect (subed-increase-start-time 200) :to-equal 200)
- (expect (subed-increase-start-time) :to-equal 200)
- (expect (subed-increase-start-time) :to-equal 200)
- (expect (save-excursion
(subed-srt--jump-to-subtitle-time-start)
- (thing-at-point 'line)) :to-equal
"00:01:01,600 --> 00:01:05,123\n")
- (expect (subed-decrease-start-time 50) :to-be -50)
- (expect (subed-decrease-start-time) :to-be -50)
- (expect (save-excursion
(subed-srt--jump-to-subtitle-time-start)
- (thing-at-point 'line)) :to-equal
"00:01:01,500 --> 00:01:05,123\n")
- (expect (subed-decrease-stop-time 1000) :to-be -1000)
- (expect (subed-decrease-stop-time) :to-be -1000)
- (expect (subed-decrease-stop-time) :to-be -1000)
- (expect (save-excursion
(subed-srt--jump-to-subtitle-time-start)
- (thing-at-point 'line)) :to-equal
"00:01:01,500 --> 00:01:02,123\n")
- (expect (subed-increase-stop-time 2000) :to-be 2000)
- (expect (subed-increase-stop-time) :to-be 2000)
- (expect (save-excursion
(subed-srt--jump-to-subtitle-time-start)
- (thing-at-point 'line)) :to-equal
"00:01:01,500 --> 00:01:06,123\n")))
(describe "enforces limits"
(describe "when decreasing start time"
(it "of the first subtitle."
(with-temp-buffer
- (insert mock-srt-data)
- (subed-srt--jump-to-subtitle-id 1)
- (expect (subed-decrease-start-time 60999)
:to-be -60999)
+ (insert (concat "1\n"
+ "00:00:01,000 -->
00:00:02,000\n"
+ "Foo.\n"))
+ (expect (subed-srt--adjust-subtitle-start
-999) :to-be -999)
(expect (subed-srt--subtitle-msecs-start)
:to-be 1)
- (expect (subed-decrease-start-time 1)
:to-be -1)
+ (expect (subed-srt--adjust-subtitle-start
-1) :to-be -1)
(expect (subed-srt--subtitle-msecs-start)
:to-be 0)
- (expect (subed-decrease-start-time 1)
:to-be nil)
+ (expect (subed-srt--adjust-subtitle-start
-1) :to-be nil)
(expect (subed-srt--subtitle-msecs-start)
:to-be 0)))
(it "of a non-first subtitle."
(with-temp-buffer
- (insert mock-srt-data)
+ (insert (concat "1\n"
+ "00:00:01,000 -->
00:00:02,000\n"
+ "Foo.\n\n"
+ "2\n"
+ "00:00:03,000 -->
00:00:04,000\n"
+ "Bar.\n\n"))
(subed-srt--jump-to-subtitle-id 2)
- (message (buffer-string))
- (expect (subed-decrease-start-time (-
(subed-srt--subtitle-msecs-start 2)
-
(subed-srt--subtitle-msecs-stop 1)
-
subed-subtitle-spacing
- 1))
:to-be -57010)
- (expect (subed-srt--subtitle-msecs-start)
:to-be (+ (subed-srt--subtitle-msecs-stop 1)
-
subed-subtitle-spacing
-
1))
- (expect (subed-decrease-start-time 1)
:to-be -1)
- (expect (subed-srt--subtitle-msecs-start)
:to-be (+ (subed-srt--subtitle-msecs-stop 1)
-
subed-subtitle-spacing))
- (expect (subed-decrease-start-time 1)
:to-be nil)
- (expect (subed-srt--subtitle-msecs-start)
:to-be (+ (subed-srt--subtitle-msecs-stop 1)
-
subed-subtitle-spacing))))
+ (expect (subed-srt--adjust-subtitle-start
-899) :to-be -899)
+ (expect (subed-srt--subtitle-msecs-start)
:to-be 2101)
+ (expect (subed-srt--adjust-subtitle-start
-1) :to-be -1)
+ (expect (subed-srt--subtitle-msecs-start)
:to-be 2100)
+ (expect (subed-srt--adjust-subtitle-start
-1) :to-be nil)
+ (expect (subed-srt--subtitle-msecs-start)
:to-be 2100)))
)
(it "when increasing start time."
(with-temp-buffer
+ (insert (concat "1\n"
+ "00:00:01,000 --> 00:00:02,000\n"
+ "Foo.\n\n"
+ "2\n"
+ "00:00:03,000 --> 00:00:04,000\n"
+ "Bar.\n\n"))
(insert mock-srt-data)
(subed-srt--jump-to-subtitle-id 2)
- (expect (subed-increase-start-time (-
(subed-srt--subtitle-msecs-stop 2)
-
(subed-srt--subtitle-msecs-start 2)
- 1)) :to-be
8110)
- (expect (subed-srt--subtitle-msecs-start 2) :to-be
(- (subed-srt--subtitle-msecs-stop 2) 1))
- (expect (subed-increase-start-time 1) :to-be 1)
- (expect (subed-srt--subtitle-msecs-start 2) :to-be
(subed-srt--subtitle-msecs-stop 2))
- (expect (subed-increase-start-time 1) :to-be nil)
- (expect (subed-srt--subtitle-msecs-start 2) :to-be
(subed-srt--subtitle-msecs-stop 2))))
+ (expect (subed-srt--adjust-subtitle-start 999)
:to-be 999)
+ (expect (subed-srt--subtitle-msecs-start) :to-be
3999)
+ (expect (subed-srt--adjust-subtitle-start 1) :to-be
1)
+ (expect (subed-srt--subtitle-msecs-start) :to-be
4000)
+ (expect (subed-srt--adjust-subtitle-start 1) :to-be
nil)
+ (expect (subed-srt--subtitle-msecs-start) :to-be
4000)))
(it "when decreasing stop time."
(with-temp-buffer
- (insert mock-srt-data)
+ (insert (concat "1\n"
+ "00:00:01,000 --> 00:00:02,000\n"
+ "Foo.\n\n"
+ "2\n"
+ "00:00:03,000 --> 00:00:04,000\n"
+ "Bar.\n\n"))
(subed-srt--jump-to-subtitle-id 2)
- (expect (subed-decrease-stop-time (-
(subed-srt--subtitle-msecs-stop 2)
-
(subed-srt--subtitle-msecs-start 2)
- 1)) :to-be
-8110)
- (expect (subed-srt--subtitle-msecs-stop 2) :to-be (+
(subed-srt--subtitle-msecs-start 2) 1))
- (expect (subed-decrease-stop-time 1) :to-be -1)
- (expect (subed-srt--subtitle-msecs-stop 2) :to-be
(subed-srt--subtitle-msecs-start 2))
- (expect (subed-decrease-stop-time 1) :to-be nil)
- (expect (subed-srt--subtitle-msecs-stop 2) :to-be
(subed-srt--subtitle-msecs-start 2))))
+ (expect (subed-srt--adjust-subtitle-stop -999)
:to-be -999)
+ (expect (subed-srt--subtitle-msecs-stop) :to-be 3001)
+ (expect (subed-srt--adjust-subtitle-stop -1) :to-be
-1)
+ (expect (subed-srt--subtitle-msecs-stop) :to-be 3000)
+ (expect (subed-srt--adjust-subtitle-stop -1) :to-be
nil)
+ (expect (subed-srt--subtitle-msecs-stop) :to-be
3000)))
(describe "when increasing stop time"
(it "of the last subtitle."
(with-temp-buffer
- (insert mock-srt-data)
+ (insert (concat "1\n"
+ "00:00:01,000 -->
00:00:02,000\n"
+ "Foo.\n\n"
+ "2\n"
+ "00:00:03,000 -->
00:00:04,000\n"
+ "Bar.\n\n"))
(subed-srt--jump-to-subtitle-id 3)
- (expect (subed-increase-stop-time (- (* 99
3600000)
-
(subed-srt--subtitle-msecs-stop 3)))
- :to-be 356204500)
- (expect (subed-srt--subtitle-msecs-stop 3)
:to-be (* 99 3600000))
- (expect (subed-increase-stop-time (* 59
60000)) :to-be (* 59 60000))
- (expect (subed-srt--subtitle-msecs-stop 3)
:to-be (+ (* 99 3600000)
-
(* 59 60000)))
- (expect (subed-increase-stop-time (* 59
1000)) :to-be (* 59 1000))
- (expect (subed-srt--subtitle-msecs-stop 3)
:to-be (+ (* 99 3600000)
-
(* 59 60000)
-
(* 59 1000)))
- (expect (subed-increase-stop-time 999)
:to-be 999)
- (expect (subed-srt--subtitle-msecs-stop 3)
:to-be (+ (* 99 3600000)
-
(* 59 60000)
-
(* 59 1000)
-
999))))
+ (expect (subed-srt--adjust-subtitle-stop
1000000):to-be 1000000)
+ (expect (subed-srt--subtitle-msecs-stop)
:to-be 1004000)))
(it "of a non-last subtitle."
(with-temp-buffer
- (insert mock-srt-data)
- (subed-srt--jump-to-subtitle-id 2)
- (expect (subed-increase-stop-time (-
(subed-srt--subtitle-msecs-start 3)
-
(subed-srt--subtitle-msecs-stop 2)
-
subed-subtitle-spacing
- 1))
:to-be 53004)
- (expect (subed-srt--subtitle-msecs-stop 2)
:to-be (- (subed-srt--subtitle-msecs-start 3)
-
subed-subtitle-spacing
-
1))
- (expect (subed-increase-stop-time 1)
:to-be 1)
- (expect (subed-srt--subtitle-msecs-stop 2)
:to-be (- (subed-srt--subtitle-msecs-start 3)
-
subed-subtitle-spacing))
- (expect (subed-increase-stop-time 1)
:to-be nil)
- (expect (subed-srt--subtitle-msecs-stop 2)
:to-be (- (subed-srt--subtitle-msecs-start 3)
-
subed-subtitle-spacing))))
+ (insert (concat "1\n"
+ "00:00:01,000 -->
00:00:02,000\n"
+ "Foo.\n\n"
+ "2\n"
+ "00:00:03,000 -->
00:00:04,000\n"
+ "Bar.\n\n"))
+ (subed-srt--jump-to-subtitle-id 1)
+ (expect (subed-srt--adjust-subtitle-stop
899) :to-be 899)
+ (expect (subed-srt--subtitle-msecs-stop)
:to-be 2899)
+ (expect (subed-srt--adjust-subtitle-stop
1) :to-be 1)
+ (expect (subed-srt--subtitle-msecs-stop)
:to-be 2900)
+ (expect (subed-srt--adjust-subtitle-stop
1) :to-be nil)
+ (expect (subed-srt--subtitle-msecs-stop)
:to-be 2900)))
)
(it "without undershooting the target time."
(with-temp-buffer
@@ -782,7 +751,7 @@ Baz.
"00:00:02,000 --> 00:00:03,000\n"
"Bar.\n"))
(subed-jump-to-subtitle-id 1)
- (expect (subed-increase-stop-time 1) :to-be nil)
+ (expect (subed-srt--adjust-subtitle-stop 1) :to-be
nil)
(expect (subed-subtitle-msecs-stop) :to-equal 2000)))
(it "without overshooting the target time."
(with-temp-buffer
@@ -793,20 +762,16 @@ Baz.
"00:00:02,000 --> 00:00:03,000\n"
"Bar.\n"))
(subed-jump-to-subtitle-id 2)
- (expect (subed-decrease-start-time 1) :to-be nil)
+ (expect (subed-srt--adjust-subtitle-start -1) :to-be
nil)
(expect (subed-subtitle-msecs-start) :to-equal
2000)))
)
(it "does nothing if no timestamp can be found."
(with-temp-buffer
(insert "foo")
(goto-char (point-min))
- (expect (subed-increase-start-time) :to-be nil)
- (expect (buffer-string) :to-equal "foo")
- (expect (subed-decrease-start-time) :to-be nil)
- (expect (buffer-string) :to-equal "foo")
- (expect (subed-increase-stop-time) :to-be nil)
+ (expect (subed-srt--adjust-subtitle-start 123) :to-be nil)
(expect (buffer-string) :to-equal "foo")
- (expect (subed-decrease-stop-time) :to-be nil)
+ (expect (subed-srt--adjust-subtitle-start -123) :to-be nil)
(expect (buffer-string) :to-equal "foo")))
)
- [nongnu] elpa/subed 53bad4a 149/389: Explain what moving does in general in comment, (continued)
- [nongnu] elpa/subed 53bad4a 149/389: Explain what moving does in general in comment, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 800d4b3 150/389: Return new start/stop time when adjusting, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 6122de0 151/389: Rename adjust-subtitle-start/stop-relative to adjust-subtitle-start/stop, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 06a4681 153/389: Adjusting functions return milliseconds moved, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed c7d3fcb 156/389: Add autoloading, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 20c70e8 160/389: Make more functions public, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed af2004c 172/389: subed-mode-enable: Add autoload cookie, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed a5d750d 162/389: Inser empty line after section header comment, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed e22f4b7 157/389: Make subed-srt-* functions private, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 3af0bb2 169/389: subed-guess-video-file: Don't croak if buffer-file-name returns nil, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed e195756 179/389: Rewrite adjusting tests,
ELPA Syncer <=
- [nongnu] elpa/subed 3e818aa 173/389: Remove useless (progn ...) in (if ...), ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 4116ea7 171/389: Add Makefile, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 37bd5f7 168/389: Don't provide sub-id to subtitle-time-adjusted-hook functions, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 3289584 170/389: Pacify checkdoc, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 645efb2 175/389: Don't test for undefined return value when moving subtitles, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 908b1b0 176/389: Simplify tests for moving in active region, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed c1c707e 182/389: Rename subed-adjust-to-player-start/stop-timestamp, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed f6b13f6 197/389: Adjust error message when autoloading video fails, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed e9f3e4e 215/389: Move tests for killing SRT subtitles to test-subed-srt.el, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 5b64bcc 203/389: Fix typo in docstring, ELPA Syncer, 2021/12/03