I hope these are correct - I aligned them to the HTML structure that I see in 
the current version of Whimsy.

It seems that the tests are not always run - is there a CI environment that is 
doing this, or would it be worth me looking into adding it to Jenkins?

Cheers,
Brett

> On 23 Jul 2017, at 10:49 pm, [email protected] wrote:
> 
> This is an automated email from the ASF dual-hosted git repository.
> 
> brett pushed a commit to branch master
> in repository https://gitbox.apache.org/repos/asf/whimsy.git
> 
> commit dad6ca8e0fe4393388521972026a2e17b47a658f
> Author: Brett Porter <[email protected]>
> AuthorDate: Sun Jul 23 22:45:56 2017 +1000
> 
>    update tests for layout changes
> ---
> www/board/agenda/spec/other_views_spec.rb | 10 +++++-----
> 1 file changed, 5 insertions(+), 5 deletions(-)
> 
> diff --git a/www/board/agenda/spec/other_views_spec.rb 
> b/www/board/agenda/spec/other_views_spec.rb
> index f363b4d..7ce38df 100644
> --- a/www/board/agenda/spec/other_views_spec.rb
> +++ b/www/board/agenda/spec/other_views_spec.rb
> @@ -16,7 +16,7 @@ feature 'other reports' do
>     visit '/2015-01-21/comments'
> 
>     # unseen items
> -    expect(page).to have_selector 'h4 a', text: 'Curator'
> +    expect(page).to have_selector 'a.h4', text: 'Curator'
>     expect(page).to have_selector 'pre', 
>       text: /last PMC member and committer additions/
> 
> @@ -68,20 +68,20 @@ feature 'other reports' do
>     visit '/2015-01-21/shepherd/Sam'
> 
>     # action items
> -    expect(page).to have_selector 'pre.report span', 
> +    expect(page).to have_selector 'pre.report', 
>       text: '* Sam: pursue a report for Abdera'
>     expect(page).to have_selector 'pre.report em', 
>       text: "Clarification provided in this month's report."
> 
>     # committee reports
> -    expect(page).to have_selector 'h3.reviewed 
> a[href="shepherd/queue/Flink"]',
> +    expect(page).to have_selector 'a.reviewed[href="shepherd/queue/Flink"]',
>       text: 'Flink'
> -    expect(page).to have_selector 'a.default', text: 'Airavata'
> +    expect(page).to have_selector 'a.missing', text: 'Airavata'
>     expect(page).to have_selector 'h4', text: 'Comments'
>     expect(page).to have_selector 'pre.comment span', 
>       text: 'cm: great report!'
>     expect(page).to have_selector 'h4', text: 'Action Items'
> -    expect(page).to have_selector 'pre.report span', 
> +    expect(page).to have_selector 'pre.report', 
>       text: '* Chris: Please clarify what "voted on" means'
>     expect(page).to have_selector 'button[data-attach=AY]', text: 'flag'
>     expect(page).to have_selector '.shepherd button', text: 'send email'
> 
> -- 
> To stop receiving notification emails like this one, please contact
> "[email protected]" <[email protected]>.

Reply via email to