Add have_rich_text

[vsppedro]

Hello, first of all, thank you all for maintaining and improving this gem.

So, I was wondering if there it's a possibility to add something like this:

it { is_expected.to have_rich_text(:body) }

To test ActionText with RSpec. Example:

class Post < ApplicationRecord
  has_rich_text :body
end

I would love to work on this if by chance this suggestion is accepted.

Thank you for your time.

EDIT: For now I'm using a custom matcher:

RSpec::Matchers.define :have_rich_text do |expected|
  match do |actual|
    actual.send(expected).class == ActionText::RichText
  end
end

[guialbuk]

Hi @vsppedro

Your contribution would be appreciated. Feel free to submit a PR, or even a draft one to receive feedback during your progress.

And feel free to ask any questions 🙂

[vsppedro]

I will close this issue for now because the PR with the solution has been merged! 🎉

[O330oei]

🤪😎🤪 Sent Samsung S8 Droid

…

________________________________ From: Pedro Paiva <notifications@github.com> Sent: Thursday, January 23, 2020 8:48:39 AM To: thoughtbot/shoulda-matchers <shoulda-matchers@noreply.github.com> Cc: Subscribed <subscribed@noreply.github.com> Subject: Re: [thoughtbot/shoulda-matchers] Add `have_rich_text` (#1252) I'll close this issue because the PR with the solution was merged! 🎉 — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub<#1252?email_source=notifications&email_token=AOD4J2UXI7BHXY67YPKB3CTQ7HC6PA5CNFSM4JLOXUKKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEJYAVYQ#issuecomment-577768162>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AOD4J2QWR43EZ4HQJ5JTAHDQ7HC6PANCNFSM4JLOXUKA>.