Skip to content
Permalink
Browse files

DEV: correct spec making bad assumptions

bio_cooked is not meant to be touched directly, on save we "cook" the raw
bio.
  • Loading branch information...
SamSaffron committed Jun 12, 2019
1 parent 0a1b16b commit 89c4332ac137de3aa5734c727f8dc2314a1ec2a4
Showing with 3 additions and 2 deletions.
  1. +3 −2 spec/requests/groups_controller_spec.rb
@@ -292,7 +292,7 @@ def expect_type_to_return_right_groups(type, expected_group_ids)
end

it 'should respond to HTML' do
group.update_attribute(:bio_cooked, 'testing group bio')
group.update!(bio_raw: 'testing **group** bio')

get "/groups/#{group.name}.html"

@@ -302,8 +302,9 @@ def expect_type_to_return_right_groups(type, expected_group_ids)
property: 'og:title', content: group.name
})

# note this uses an excerpt so it strips html
expect(response.body).to have_tag(:meta, with: {
property: 'og:description', content: group.bio_cooked
property: 'og:description', content: 'testing group bio'
})
end

1 comment on commit 89c4332

@discoursereviewbot

This comment has been minimized.

Copy link

commented on 89c4332 Jun 12, 2019

SamSaffron posted:

@majakomel can you add a quick spec to test the clearing case

Please sign in to comment.
You can’t perform that action at this time.