Require yii only once

[naffiq]

Hi!
Thanks for the great solution, but we encountered a problem, when we try to create couple of Environment classes.
The error we were getting is Fatal error: Cannot declare class Yii, because the name is already in use, and after some debugging we've figured out, that Environment class uses require($this->yiiPath) instead of require_once($this->yiiPath).
If this change wouldn't break anything, I would highly appreciate merging it to master and releasing new version with this "fix" so we can get rid of overriden version of this class in our project

[naffiq]

Is there any chance of this request being merged or should I close it?

[cebe]

This looks suspicious, why are you calling setup() twice?

[naffiq]

@cebe
I was running tests with Codeception and have following config in codecept.yml file:

modules:
    config:
        Yii2:
            configFile: 'config/codeception.php'
            cleanup: false

And my config/codeception.php file is:

<?php
require_once(__DIR__ . '/../vendor/autoload.php');
require_once(__DIR__ . '/../components/rocket/RFEnvironment.php');
$env = new \app\components\RFEnvironment( // My overriden Environment class
    [
        dirname(__DIR__) . '/config'
    ],
    'test'
);
$env->setup();
return $env->web;

Maybe I structured the whole thing wrong, but requiring Yii once solved the problem.
If this change impacts others' code I'm ok with overriding Environment class for my case until I find better structure for my project.

Would be really grateful for helping me with configuring testing environment.