I agree, will add a commit for this.


On Sun, Apr 28, 2013 at 11:41 AM, Jeff Welch <notifications@github.com>wrote:

> In Tests/_files/BeforeClassAndAfterClassTest.php:
>
> > +<?php
> > +class BeforeClassAndAfterClassTest extends PHPUnit_Framework_TestCase
> > +{
> > +    public static $beforeClassWasRun = 0;
> > +    public static $afterClassWasRun = 0;
> > +
> > +    public static function resetProperties()
> > +    {
> > +        self::$beforeClassWasRun = 0;
> > +        self::$afterClassWasRun = 0;
> > +    }
> > +
> > +    /**
> > +     * @beforeClass
> > +     */
> > +    public function initialClassSetup()
>
> This method should be static. I believe this is what is causing the
> travis failure. Since this is easy to miss and hard to debug, maybe we
> should warn the user if the @beforeClass or @afterClass is not static.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/sebastianbergmann/phpunit/pull/898/files#r3991062>
> .
>



-- 
Giorgio Sironi (@giorgiosironi)
http://giorgiosironi.com <http://giorgiosironi.blogspot.com/>