Skip to content

v0.0.3 #4

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
Merged
merged 10 commits into from
Oct 15, 2024
Merged

v0.0.3 #4

merged 10 commits into from
Oct 15, 2024

Conversation

@sebastienrousseau
Copy link
Owner

@sebastienrousseau sebastienrousseau commented Oct 14, 2024 •
edited
Loading

  • v0.0.3
  • test(rssgen): ✅ add new unit tests
  • test(rssgen): ✅ add new unit tests for data.rs
  • test(rssgen): ✅ add new unit tests for lib.rs
  • test(rssgen): ✅ add new unit tests for data.rs
  • test(rssgen): ✅ add new unit tests for data.rs
  • test(rssgen): ✅ enhance and expand unit tests for validator.rs
  • fix(rssgen): 🎨 refactoring code and various optimisations
  • test(rssgen): ✅ enhance and expand unit tests for parser.rs
  • test(rssgen): ✅ enhance and complete unit tests for parser.rs

@codecov
Copy link

codecov bot commented Oct 14, 2024 •
edited
Loading

Codecov Report

Attention: Patch coverage is 92.84890% with 62 lines in your changes missing coverage. Please review.

Project coverage is 93.23%. Comparing base (4a7e57d) to head (712a086).
Report is 11 commits behind head on main.

Files with missing lines Patch % Lines
src/data.rs 90.21% 27 Missing ⚠️
src/parser.rs 93.31% 26 Missing ⚠️
src/validator.rs 93.26% 7 Missing ⚠️
src/error.rs 80.00% 1 Missing ⚠️
src/lib.rs 98.82% 1 Missing ⚠️
Additional details and impacted files 
@@            Coverage Diff             @@
##             main       #4      +/-   ##
==========================================
+ Coverage   86.44%   93.23%   +6.79%     
==========================================
  Files           7        7              
  Lines        1785     2056     +271     
==========================================
+ Hits         1543     1917     +374     
+ Misses        242      139     -103

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@sebastienrousseau
test(rssgen): ✅ add new unit tests
e0801a8
@sebastienrousseau
test(rssgen): ✅ add new unit tests for data.rs
f22f2a8
@sebastienrousseau
test(rssgen): ✅ add new unit tests for lib.rs
8997ddc 
- Add test for maximum input length boundary
- Add specific tests for HTTP and HTTPS URL schemes
- Improve test coverage for `quick_rss` function
@sebastienrousseau
test(rssgen): ✅ add new unit tests for data.rs
e855603 
- Add tests for RssVersion parsing and display
- Implement comprehensive tests for RssData and RssItem setters
- Add validation tests for URL and date parsing
- Improve test coverage for edge cases and empty inputs
- Add tests for hash map conversion and image setting
@sebastienrousseau
test(rssgen): ✅ add new unit tests for data.rs
ab6ee91 
- Add tests for RssData and RssItem field setters
- Implement tests for URL and date parsing functions
- Add validation tests for RssData and RssItem structs
- Include tests for edge cases and empty inputs
- Improve test coverage for RssVersion enum
@sebastienrousseau
test(rssgen): ✅ enhance and expand unit tests for validator.rs
cf2382d 
- Add comprehensive tests for RssFeedValidator methods
- Implement tests for URL validation and error handling
- Add tests for date parsing and validation
- Include tests for version-specific validations (RSS 1.0, 2.0, and older versions)
- Improve test coverage for GUID uniqueness and atom:link requirements
- Add edge case tests for invalid inputs and empty fields
- Enhance test suite to cover all public and private methods
@sebastienrousseau
fix(rssgen): 🎨 refactoring code and various optimisations
50cb36d
@sebastienrousseau
test(rssgen): ✅ enhance and expand unit tests for parser.rs
6e8af3b 
- Add comprehensive tests for `parse_channel_element` covering all fields (language, copyright, managingEditor, webMaster, pubDate, etc.)
- Implement tests for RSS 1.0-specific elements like `items` and `rdf:Seq`
- Add tests for `parse_item_element` covering item fields such as author, guid, pubDate, category, comments, and enclosure
- Include edge case tests for elements with attributes and empty values
- Improve test coverage for parsing both RSS 1.0 and RSS 2.0 versions
- Enhance test suite to ensure proper error handling for unknown elements
@sebastienrousseau
test(rssgen): ✅ enhance and complete unit tests for parser.rs
712a086 
- Add tests for `ParserConfig` with and without custom handlers
- Implement tests for `rdf:li` element handling in RSS 1.0 and non-RSS 1.0 cases
- Add tests for unknown channel elements in `parse_channel_element`
- Include tests for `process_text_event` and `process_cdata_event`
- Add edge case test for empty element name in `process_start_event`
- Ensure comprehensive test coverage for text and CDATA processing and custom handler application
@sebastienrousseau sebastienrousseau merged commit 0ac0ca1 into main Oct 15, 2024
14 checks passed
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.

2 participants

@sebastienrousseau