Okay, I will revert these changes. Why is it better to do it directly in
functional tests?


On Wed, Jun 12, 2013 at 4:30 PM, Joas Schilling <notifications@github.com>wrote:

> In tests/functional/report_post_captcha.php:
>
> > +{
> > +	public function test_user_report_post()
> > +	{
> > +		$this->login();
> > +		$crawler = self::request('GET', 'report.php?f=2&p=1');
> > +		$this->assertNotContains($this->lang('CONFIRM_CODE'), $crawler->filter('html')->text());
> > +	}
> > +
> > +	public function test_guest_report_post()
> > +	{
> > +		$this->enable_reporting_guest();
> > +		$crawler = self::request('GET', 'report.php?f=2&p=1');
> > +		$this->assertContains($this->lang('CONFIRM_CODE'), $crawler->filter('html')->text());
> > +	}
> > +
> > +	protected function enable_reporting_guest()
>
> Please revert it after the test.
> Also im not sure. Would be better, to do it directly (not via going to the
> page)
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/phpbb/phpbb3/pull/1474/files#r4654119>
> .
>