Skip to content
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.

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

Parser testing #157

Closed
wants to merge 5 commits into from
Closed

Parser testing #157

wants to merge 5 commits into from

Conversation

aarosan
Copy link
Collaborator

@aarosan aarosan commented Sep 25, 2024

Added the following unit tests for the parser in test-unittest.py

test_parser_class_and_method
test_parser_directories_single_file
test_parser_directories_multiple_files
test_parser_list_of_single_html_file
test_parser_list_of_single_html_file_by_casenumber
test_parser_list_of_multiple_html_files
test_parser_get_list_of_html_error_handling
test_get_html_path
test_write_json_data
test_write_error_log

- Added type specification to methods
- Added logging functionality
- Added error handling
- Refactored into smaller, more readble methods
@aarosan aarosan closed this Sep 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant