[MediaCCC] Fix regressions and improve MediaCCCLiveStreamKioskExtractor #1097
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Regression introduced in #1082
TobiGr
added
bug
TobiGr
force-pushed
the
ccc
branch
3 times, most recently
from
8d83786 to
e9c38bd
Compare
Add and improve tests for MediaCCCLiveStreamKioskExtractor: - test stream items if a live stream is running - use mock tests to check live talk extraction and testing conferences
AudricV
changed the title
AudricV
requested changes
Sep 23, 2023
| } | ||
|
|
||
| @Override | ||
| public String getName() throws ParsingException { | ||
| return roomInfo.getObject("talks").getObject("current").getString("title"); | ||
| if (isBreak()) { | ||
| return roomInfo.getString("display") + " - Pause"; |
There was a problem hiding this comment.
How MediaCCC's website displays paused rooms? I don't think hardcoding - Pause at the end of a room name is a good idea for localization purposes.
Comment on lines
+86
to
+88
| * <p>Reset cached live stream data.</p> | ||
| * This is a temporary method which can be used to reset the cached live stream data until a | ||
| * caching policy for {@link #getLiveStreams(Downloader, Localization)} is implemented. |
There was a problem hiding this comment.
Suggested change
| * <p>Reset cached live stream data.</p> | |
| * This is a temporary method which can be used to reset the cached live stream data until a | |
| * caching policy for {@link #getLiveStreams(Downloader, Localization)} is implemented. | |
| * Reset cached live stream data. | |
| * | |
| * <p> | |
| * This is a temporary method which can be used to reset the cached live stream data until a | |
| * caching policy for {@link #getLiveStreams(Downloader, Localization)} is implemented. | |
| * </p> |
Comment on lines
+8
to
+9
| * Clears static media.ccc.de states. | ||
| * <p>This method needs to be called in every class before running and recording mock tests.</p> |
There was a problem hiding this comment.
Suggested change
| * Clears static media.ccc.de states. | |
| * <p>This method needs to be called in every class before running and recording mock tests.</p> | |
| * Clears static MediaCCC states. | |
| * | |
| * <p> | |
| * This method needs to be called in every class before running and recording mock tests. | |
| * </p> |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Changes
MediaCCCLiveStreamKioskExtractor:To Do
MediaCCCLiveStreamListExtractorTest.[PreparationTest|LiveConferenceTest]always use mocks.