`assert_dom_equal` should default `expected` to `document_root_element`

[chancancode]

This seems consistent with the assert_select API... 👍 / 👎 ?

[rafaelfranca]

👍

[kaspth]

Sounds good to me, yeah 👍

[kaspth]

Hmm, on second thought. Why is it expected, we want to default document_root_element to and not actual?

Also unsure if having an *_equal assertion without 2 args would read. Consider:

assert_equal 'cheese'

How about this?

def assert_dom(expected, message = nil)
  assert_dom_equal expected, document_root_element, message
end

[chancancode]

Eh I totally meant actual, not expected

…

On Thu, Mar 9, 2017 at 11:39 AM Kasper Timm Hansen ***@***.***> wrote: Hmm, on second thought. Why is it expected, we want to default document_root_element to and not actual? Also unsure if having an *_equal assertion without 2 args would read. Consider: assert_equal 'cheese' How about this? def assert_dom(expected, message = nil) assert_dom_equal expected, document_root_element, messageend — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#60 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AADaFfWTXl58w8KTXnetQirHlWQNpxgSks5rkFVggaJpZM4MKr_E> .

[kaspth]

Ha, okay, then it makes more sense 😄