You are passing a ``NullProgressMonitor`` only in tests, so I'm wondering if it is a good idea to have a default argument (test code should not be leaking into the application code, and I feel this is what is happening here). Any strong feeling about removing the default argument?