Skip to content

6609 – Update article-body__content to article-body-module__content#417

Merged
vasconsaurus merged 2 commits intodevelopfrom
6609-update-body-class-name
Oct 24, 2025
Merged

6609 – Update article-body__content to article-body-module__content#417
vasconsaurus merged 2 commits intodevelopfrom
6609-update-body-class-name

Conversation

@vasconsaurus
Copy link
Contributor

@vasconsaurus vasconsaurus commented Oct 24, 2025

Description

We seem to be getting: undefined method search' for nil:NilClass:00:00.
Reviewing both portuguese and spanish pages, it looks like the class article-body__content is now article-body-module__content.

References: CV2-6609

How has this been tested?

I was able to reproduce the error by mismatching the classname:

  1) Reuters instance parses a raw_claim_review
     Failure/Error: header = page.search('div[class^="article-body-module__content"]').first.search('h2[data-testid="Heading"]').last

     NoMethodError:
       undefined method `search' for nil:NilClass
     # ./lib/claim_review_parsers/reuters.rb:44:in `claim_result_from_subhead'
     # ./lib/claim_review_parsers/reuters.rb:51:in `claim_result_from_page'
     # ./lib/claim_review_parsers/reuters.rb:59:in `parse_raw_claim_review'
     # ./spec/lib/claim_review_parsers/reuters_test.rb:20:in `block (3 levels) in <top (required)>'
     # /usr/local/bundle/gems/webmock-3.18.1/lib/webmock/rspec.rb:37:in `block (2 levels) in <top (required)>'

@vasconsaurus vasconsaurus requested a review from caiosba October 24, 2025 18:26
@vasconsaurus vasconsaurus merged commit 850fa12 into develop Oct 24, 2025
5 checks passed
@vasconsaurus vasconsaurus deleted the 6609-update-body-class-name branch October 24, 2025 19:02
@vasconsaurus vasconsaurus changed the title Update article-body__content to article-body-module__content 6609 – Update article-body__content to article-body-module__content Oct 24, 2025
vasconsaurus added a commit that referenced this pull request Oct 28, 2025
…nt (#417)

We seem to be getting: `undefined method search' for nil:NilClass:00:00`. 
Reviewing both portuguese and spanish pages, it looks like the class `article-body__content` is now `article-body-module__content`.

References: CV2-6609

I was able to reproduce the error by mismatching the classname:

```
  1) Reuters instance parses a raw_claim_review
     Failure/Error: header = page.search('div[class^="article-body-module__content"]').first.search('h2[data-testid="Heading"]').last

     NoMethodError:
       undefined method `search' for nil:NilClass
     # ./lib/claim_review_parsers/reuters.rb:44:in `claim_result_from_subhead'
     # ./lib/claim_review_parsers/reuters.rb:51:in `claim_result_from_page'
     # ./lib/claim_review_parsers/reuters.rb:59:in `parse_raw_claim_review'
     # ./spec/lib/claim_review_parsers/reuters_test.rb:20:in `block (3 levels) in <top (required)>'
     # /usr/local/bundle/gems/webmock-3.18.1/lib/webmock/rspec.rb:37:in `block (2 levels) in <top (required)>'
```
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.

2 participants