[ie/Hotstar] year added in hotstar metadata

[Rajeshwaran2001]

IMPORTANT: PRs without the template will be CLOSED

Description of your pull request and other information

HotStar Year MetaData Added

Fixes #

Template

Before submitting a pull request make sure you have:

• At least skimmed through contributing guidelines including yt-dlp coding conventions
• Searched the bugtracker for similar pull requests
• Checked the code with flake8 and ran relevant tests

In order to be accepted and merged into yt-dlp each piece of code must be in public domain or released under Unlicense. Check all of the following options that apply:

• I am the original author of this code and I am willing to release it under Unlicense
• I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

• Fix or improvement to an extractor (Make sure to add/update tests)
• New extractor (Piracy websites will not be accepted)
• Core bug fix/improvement
• New feature (It is strongly recommended to open an issue first)

Copilot Summary

🤖 Generated by Copilot at a7d72ea

Summary

✨🎥🇮🇳

Add year metadata field to HotStar extractor. This improves the accuracy and completeness of the video information extracted from the HotStar API.

year field added
from video_data dict
autumn of metadata

Walkthrough

• Add year field to video metadata from video_data dictionary (link)

[bashonly]

Please fill out the template checklist or else the PR cannot be accepted:

Before submitting a pull request make sure you have:

• At least skimmed through contributing guidelines including yt-dlp coding conventions
• Searched the bugtracker for similar pull requests
• Checked the code with flake8 and ran relevant tests

In order to be accepted and merged into yt-dlp each piece of code must be in public domain or released under Unlicense. Check all of the following options that apply:

• I am the original author of this code and I am willing to release it under Unlicense
• I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

• Fix or improvement to an extractor (Make sure to add/update tests)
• New extractor (Piracy websites will not be accepted)
• Core bug fix/improvement
• New feature (It is strongly recommended to open an issue first)

[pukkandan]

And update tests

[bashonly]

Suggested change

	'year': video_data.get('year'),
	'release_year': int_or_none(video_data.get('year')),

[Rajeshwaran2001]

suggestion updated and ready to merge

[Rajeshwaran2001]

The suggestion for the year has been updated.

[bashonly]

Update the tests with the release_year

[Rajeshwaran2001]

are you talking about unittest? if it is it is done

[bashonly]

I am talking about these:

    _TESTS = [{
        'url': 'https://www.hotstar.com/can-you-not-spread-rumours/1000076273',
        'info_dict': {
            'id': '1000076273',
            'ext': 'mp4',
            'title': 'Can You Not Spread Rumours?',
            'description': 'md5:c957d8868e9bc793ccb813691cc4c434',
            'timestamp': 1447248600,
            'upload_date': '20151111',
            'duration': 381,
            'episode': 'Can You Not Spread Rumours?',
        },
        'params': {'skip_download': 'm3u8'},
    }, {
        'url': 'https://www.hotstar.com/tv/ek-bhram-sarvagun-sampanna/s-2116/janhvi-targets-suman/1000234847',
        'info_dict': {
            'id': '1000234847',
            'ext': 'mp4',
            'title': 'Janhvi Targets Suman',
            'description': 'md5:78a85509348910bd1ca31be898c5796b',
            'timestamp': 1556670600,
            'upload_date': '20190501',
            'duration': 1219,
            'channel': 'StarPlus',
            'channel_id': 3,
            'series': 'Ek Bhram - Sarvagun Sampanna',
            'season': 'Chapter 1',
            'season_number': 1,
            'season_id': 6771,
            'episode': 'Janhvi Targets Suman',
            'episode_number': 8,
        }
    }, {

and all the rest

[Rajeshwaran2001]

All Done and working fine

[bashonly]

All Done and working fine

No, you haven't updated the tests. You need to actually run the tests, and then update them by adding release_year values.

python3 test/test_download.py TestDownload.test_HotStar
python3 test/test_download.py TestDownload.test_HotStar_1
python3 test/test_download.py TestDownload.test_HotStar_2

etc

[Rajeshwaran2001]

Result of Test1:
Result of Test2:
Result of Test3:
Result of Test4:

Test 3 failed first due to a broken link and I have updated the test with a working link and all tests ok

[bashonly]

Why do none of the tests return a release_year field, though? The point of adding/updating tests is so we can test that release_year is succesfully extracted

[Rajeshwaran2001]

Why do none of the tests return a release_year field, though? The point of adding/updating tests is so we can test that release_year is successfully extracted

release_year is mainly for movies and There is no test for that. You can see release_year extracted for a movie in the info.json file

.

I can write a new test for this if you want.

[bashonly]

Yes please do

[Rajeshwaran2001]

A new test was added and verified