[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[nongnu] elpa/subed 382e83c 126/389: Add more tests for moving subtitles
From: |
ELPA Syncer |
Subject: |
[nongnu] elpa/subed 382e83c 126/389: Add more tests for moving subtitles |
Date: |
Fri, 3 Dec 2021 11:00:11 -0500 (EST) |
branch: elpa/subed
commit 382e83cf99f071442396bbe8cc394fa8d61c9a23
Author: Random User <rndusr@posteo.de>
Commit: Random User <rndusr@posteo.de>
Add more tests for moving subtitles
---
tests/test-subed.el | 59 +++++++++++++++++++++++++++++++++++++++++++++++++++++
1 file changed, 59 insertions(+)
diff --git a/tests/test-subed.el b/tests/test-subed.el
index b867e42..99bae2d 100644
--- a/tests/test-subed.el
+++ b/tests/test-subed.el
@@ -153,6 +153,65 @@
(expect (subed--subtitle-msecs-stop 3) :to-equal
orig-stop-3)
)))
)
+ (it "disables subtitle replay while moving subtitles."
+ (with-temp-buffer
+ (insert mock-srt-data)
+ (subed-enable-replay-adjusted-subtitle :quiet)
+ (spy-on 'subed-enable-replay-adjusted-subtitle
:and-call-through)
+ (spy-on 'subed-disable-replay-adjusted-subtitle
:and-call-through)
+ (spy-on 'subed--adjust-subtitle-start-relative :and-call-fake
+ (lambda (msecs) (expect
(subed-replay-adjusted-subtitle-p) :to-be nil)))
+ (spy-on 'subed--adjust-subtitle-stop-relative :and-call-fake
+ (lambda (msecs) (expect
(subed-replay-adjusted-subtitle-p) :to-be nil)))
+ (subed-move-subtitle-forward 100)
+ (expect 'subed-disable-replay-adjusted-subtitle
:to-have-been-called-times 1)
+ (expect 'subed-enable-replay-adjusted-subtitle
:to-have-been-called-times 1)
+ (subed-move-subtitle-backward 100)
+ (expect 'subed-disable-replay-adjusted-subtitle
:to-have-been-called-times 2)
+ (expect 'subed-enable-replay-adjusted-subtitle
:to-have-been-called-times 2)))
+ (it "does not enable subtitle replay afterwards if it is disabled."
+ (with-temp-buffer
+ (insert mock-srt-data)
+ (subed-disable-replay-adjusted-subtitle :quiet)
+ (spy-on 'subed-enable-replay-adjusted-subtitle
:and-call-through)
+ (spy-on 'subed-disable-replay-adjusted-subtitle
:and-call-through)
+ (spy-on 'subed--adjust-subtitle-start-relative :and-call-fake
+ (lambda (msecs) (expect
(subed-replay-adjusted-subtitle-p) :to-be nil)))
+ (spy-on 'subed--adjust-subtitle-stop-relative :and-call-fake
+ (lambda (msecs) (expect
(subed-replay-adjusted-subtitle-p) :to-be nil)))
+ (subed-move-subtitle-forward 100)
+ (expect 'subed-disable-replay-adjusted-subtitle
:to-have-been-called-times 1)
+ (expect 'subed-enable-replay-adjusted-subtitle
:to-have-been-called-times 0)
+ (subed-move-subtitle-backward 100)
+ (expect 'subed-disable-replay-adjusted-subtitle
:to-have-been-called-times 2)
+ (expect 'subed-enable-replay-adjusted-subtitle
:to-have-been-called-times 0)))
+ (it "seeks player to current subtitle if region is not active."
+ (with-temp-buffer
+ (insert mock-srt-data)
+ (spy-on 'subed-replay-adjusted-subtitle-p :and-return-value t)
+ (spy-on 'subed-mpv-jump)
+ (subed-move-subtitle-forward 100)
+ (expect 'subed-mpv-jump :to-have-been-called-times 1)
+ (expect 'subed-mpv-jump :to-have-been-called-with 183550)
+ (subed-move-subtitle-backward 200)
+ (expect 'subed-mpv-jump :to-have-been-called-times 2)
+ (expect 'subed-mpv-jump :to-have-been-called-with 183350)))
+ (it "seeks player to first subtitle in active region."
+ (with-temp-buffer
+ (insert mock-srt-data)
+ (let ((beg 15)
+ (end (point-max)))
+ (spy-on 'use-region-p :and-return-value t)
+ (spy-on 'region-beginning :and-return-value beg)
+ (spy-on 'region-end :and-return-value end)
+ (spy-on 'subed-replay-adjusted-subtitle-p :and-return-value
t)
+ (spy-on 'subed-mpv-jump)
+ (subed-move-subtitle-forward 100)
+ (expect 'subed-mpv-jump :to-have-been-called-times 1)
+ (expect 'subed-mpv-jump :to-have-been-called-with '61100)
+ (subed-move-subtitle-backward 300)
+ (expect 'subed-mpv-jump :to-have-been-called-times 2)
+ (expect 'subed-mpv-jump :to-have-been-called-with '60800))))
)
(describe "Syncing player to point"
- [nongnu] elpa/subed bdcdb0a 100/389: Remove deprecated variables, (continued)
- [nongnu] elpa/subed bdcdb0a 100/389: Remove deprecated variables, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 181b9f0 099/389: Test for lacking digits in millisecond, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 03dd3c6 106/389: Make subed-move-subtitle-forward/backward work with region (sort of), ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 0f385d6 110/389: Keep logging debug messages when debugging is disabled, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 612f12a 111/389: Add "moving marked subtitles forward/backward in time" to features, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 4522e1a 113/389: Change keybindings to better conform with major mode conventions, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed ecbfd7f 119/389: Remove useless (progn ...), ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 1950bb0 121/389: subed--replay-adjusted-subtitle: Show debug message before doing the deed, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed f06d5f8 124/389: Add macro: subed--with-subtitle-replay-disabled, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 3fafab8 125/389: Disable subtitle replay during subtitle movement, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 382e83c 126/389: Add more tests for moving subtitles,
ELPA Syncer <=
- [nongnu] elpa/subed d1a0b2f 127/389: Remove unnecessary spy-on, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 61cf9ce 129/389: Remove garbage tests for point-to-player syncing, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 3aa9a71 131/389: Pause player by default when editing subtitle text, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed d85f988 136/389: Add keybindings for subed-shift-subtitle-forward/backward, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 5b90306 140/389: Round mpv's playback position to prevent float weirdness (n.99999...), ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 0f3add7 141/389: Temporarily disable point-to-player syncing when adjusting time, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 3081983 144/389: Add commentary section to header, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed c069e23 145/389: subed is named "subed", ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 522017e 146/389: Add packaging information to header, ELPA Syncer, 2021/12/03
- [nongnu] elpa/subed 3bc609d 147/389: Make run-subtitle-time-adjusted-hook not srt specific, ELPA Syncer, 2021/12/03