Skip to content

Update requirements and enhance validation tests #4387

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Update requirements and enhance validation tests #4387

wants to merge 1 commit into from

Conversation

hamzaMissewi
Copy link

  • Added new test cases in test_validate_format.py for handling None and empty string inputs in HTTPS validation.
  • Introduced additional tests in test_validate_links.py to verify link extraction and link status checking functionality.
  • Improved check_https function to handle invalid inputs more gracefully.
  • Enhanced check_if_list_of_links_are_working to utilize threading for better performance.
  • My submission is formatted according to the guidelines in the contributing guide
  • My addition is ordered alphabetically
  • My submission has a useful description
  • The description does not have more than 100 characters
  • The description does not end with punctuation
  • Each table column is padded with one space on either side
  • I have searched the repository for any relevant issues or pull requests
  • Any category I am creating has the minimum requirement of 3 items
  • All changes have been squashed into a single commit

Signed By: Hamza missaoui

@hamzaMissewi
Update requirements and enhance validation tests
7bf4508 
- Updated `requirements.txt` to ensure consistent package versions.
- Added new test cases in `test_validate_format.py` for handling None and empty string inputs in HTTPS validation.
- Introduced additional tests in `test_validate_links.py` to verify link extraction and link status checking functionality.
- Improved `check_https` function to handle invalid inputs more gracefully.
- Enhanced `check_if_list_of_links_are_working` to utilize threading for better performance.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@hamzaMissewi