| # |
Jun 14th 2018, 12:32 |
 spriz |
at least using `vendor/bin/phpunit src/to/file --filter=testAdd |
| # |
Jun 14th 2018, 12:31 |
spriz |
only the failing one - the other passing ones does pass if they’re ran isolated |
| # |
Jun 14th 2018, 12:31 |
lorenzo |
if you tun that test alone, does it fail too? |
| # |
Jun 14th 2018, 12:30 |
spriz |
worst part is: this is the controller action that fails: `public function index(){return $this->Crud->execute();}` :,) :,) |
| # |
Jun 14th 2018, 12:30 |
spriz |
if I put that debug in the AppController it spits our right number for the other test methods |
| # |
Jun 14th 2018, 12:29 |
spriz |
testing same controller |
| # |
Jun 14th 2018, 12:29 |
spriz |
not the other tests in same file |
| # |
Jun 14th 2018, 12:29 |
spriz |
oh - the worst part is btw that’s it’s only seem to be for one test |
| # |
Jun 14th 2018, 12:27 |
spriz |
what actually breaks is that the controller goes crazy on a `->firstOrFail()` since it cannot find the specified id (which is in our fixtures only) |
| # |
Jun 14th 2018, 12:26 |
spriz |
only thing calling alias is the migrations plugin in a trait, which we’re not running though |
| # |
Jun 14th 2018, 12:26 |
spriz |
if I do a `debug($this->Users->find()->count())` in the Controller it’s testing I get the number of users in my dev env rather than from fixtures |
| # |
Jun 14th 2018, 12:25 |
lorenzo |
how do you know it is using the default? |
| # |
Jun 14th 2018, 12:25 |
lorenzo |
is something else calling alias again? |
| # |
Jun 14th 2018, 12:25 |
lorenzo |
do you have a test connection declared? |
| # |
Jun 14th 2018, 12:24 |
spriz |
that’s where I ran out of ideas |
| # |
Jun 14th 2018, 12:24 |
spriz |
yurp |
| # |
Jun 14th 2018, 12:23 |
lorenzo |
is this line being executed? https://github.com/cakephp/cakephp/blob/master/src/TestSuite/Fixture/FixtureManager.php#L128 |
| # |
Jun 14th 2018, 12:23 |
lorenzo |
heh |
| # |
Jun 14th 2018, 12:23 |
spriz |
wow, I tried to copy just the “<listener” line - great job @spriz :,) |
| # |
Jun 14th 2018, 12:22 |
spriz |
<listeners> <listener class=“\Cake\TestSuite\Fixture\FixtureInjector” file=“./vendor/cakephp/cakephp/src/TestSuite/Fixture/FixtureInjector.php”> <arguments> <object class=“\Cake\TestSuite\Fixture\FixtureManager”/> </arguments> </listener> </listeners> |
| # |
Jun 14th 2018, 12:22 |
spriz |
seems ok @lorenzo ``` |
| # |
Jun 14th 2018, 12:21 |
spriz |
It’s 3.5.x btw |
| # |
Jun 14th 2018, 12:21 |
lorenzo |
faulty phpunit.xml that has no fixtures lisetner? |
| # |
Jun 14th 2018, 12:20 |
spriz |
Anyone have suggestions on why a cake app “all of a sudden” is looking in the `default` datasource rather than the `test`? :thinking_face: |
| # |
Jun 14th 2018, 11:45 |
josbeir |
that would indeed be less work for the end user |
| # |
Jun 14th 2018, 11:45 |
lorenzo |
right @josbeir |
| # |
Jun 14th 2018, 11:35 |
josbeir |
$query->func()->concat => \ORM\blabla\concat() or something :P |
| # |
Jun 14th 2018, 11:34 |
josbeir |
probably more for the expression functions and stuff ? |
| # |
Jun 14th 2018, 11:30 |
neon1024 |
Won’t they all need to take a query instance? |
| # |
Jun 14th 2018, 11:29 |
josbeir |
i'm all for that |
| # |
Jun 14th 2018, 11:28 |
lorenzo |
I’m actually planning to introduce a bunch of namespaces functions for the ORM, to make query building easier |
| # |
Jun 14th 2018, 11:27 |
neon1024 |
@lorenzo I see, yeah, it’s an interesting idea |
| # |
Jun 14th 2018, 11:27 |
josbeir |
its a subject that will probably give heated discussions if it was implemented in a framework like cake |
| # |
Jun 14th 2018, 11:26 |
lorenzo |
Static is the same idea, but again placed into a class for no good reason other than grouping and auto loading |
| # |
Jun 14th 2018, 11:26 |
lorenzo |
You can do that with composer, though |
| # |
Jun 14th 2018, 11:26 |
lorenzo |
Only if you include the file @neon1024 |
| # |
Jun 14th 2018, 11:25 |
neon1024 |
Are namespace functions always available? |
| # |
Jun 14th 2018, 11:25 |
neon1024 |
How does this differ from static? |
| # |
Jun 14th 2018, 11:24 |
lorenzo |
State would need to be passed as arguments, forcing you to basically come up naturally with the idea of dependency injection |
| # |
Jun 14th 2018, 11:24 |
LubosR |
josbeir, ok :) thanks anyway |
| # |
Jun 14th 2018, 11:24 |
josbeir |
yeah i just remembered @lorenzo but its pretty verbose, not sure if its a better way to handle the problem |