Hello Florin,

I thank you for the review and suggestions=) I will modify it and 
re-submit it.

Best,
Yoshi

Am 31.05.2013 09:59, schrieb Florin Strugariu:
>
> In tests/desktop/test_search.py:
>
> > +        #1. start on the home page
> > +        home_pg = PageProvider(mozwebqa).home_page()
> > +
> > +        #2. type "Firefox crashed"
> > +        #3. hit Enter
> > +        search_pg = SearchPage(mozwebqa)
> > +        search_pg.do_search_on_search_query(self.forum_search_term+"ed")
> > +
> > +        #4. In the results list there are two types of results:
> > +        #   Forum and KB. Click on a forum result.
> > +        #   (Url is in the forum of /questions/[some number])
> > +        #5. A complete forum thread should be displayed.
> > +        Assert.true(search_pg.is_result_present, "result page is not present.")
> > +        result_thread_title = search_pg.get_result_question_text()
> > +        Assert.contains(self.forum_search_term,result_thread_title)
> > +        search_pg.click_question_link(self.forum_search_term)
>
> could we add a assert after this... just to make sure that it goes to 
> the right page
>
> —
> Reply to this email directly or view it on GitHub 
> <https://github.com/mozilla/sumo-tests/pull/200/files#r4473058>.
>