Question

I am trying to test a class that manages data access in the database (you know, CRUD, essentially). The DB library we’re using happens to have an API wherein you first get the table object by a static call:

function getFoo($id) {   $MyTableRepresentation = DB_DataObject::factory('mytable');   $MyTableRepresentation->get($id);   ... do some stuff   return $somedata } 

…you get the idea.

We’re trying to test this method, but mocking the DataObject stuff so that (a) we don’t need an actual db connection for the test, and (b) we don’t even need to include the DB_DataObject lib for the test.

However, in PHPUnit I can’t seem to get $this->getMock() to appropriately set up a static call. I have…

        $DB_DataObject = $this->getMock('DB_DataObject', array('factory')); 

…but the test still says unknown method ‘factory’. I know it’s creating the object, because before it said it couldn’t find DB_DataObject. Now it can. But, no method?

What I really want to do is to have two mock objects, one for the table object returned as well. So, not only do I need to specify that factory is a static call, but also that it returns some specified other mock object that I’ve already set up.

I should mention as a caveat that I did this in SimpleTest a while ago (can’t find the code) and it worked fine.

What gives?

[UPDATE]

I am starting to grasp that it has something to do with expects()

Answer 1

I agree with both of you that it would be better not to use a static call. However, I guess I forgot to mention that DB_DataObject is a third party library, and the static call is their best practice for their code usage, not ours. There are other ways to use their objects that involve constructing the returned object directly. It just leaves those darned include/require statements in whatever class file is using that DB_DO class. That sucks because the tests will break (or just not be isolated) if you’re meanwhile trying to mock a class of the same name in your test–at least I think.