views:

1529

answers:

4

Hi guys, this kind of follows on from another question of mine.

Basically, once I have the code to access the file (will review the answers there in a minute) what would be the best way to test it?

I am thinking create method which just spawns lots of BackgroundWorker's or something and tells them to all load/save the file, and test with varying file/object sizes. Then, get a response back from the threads to see if it failed/succeeded/made the world implode etc.

Can you guys offer any suggestions on the best way to approach this? As I said before, this is all kinda new to me :)

Many thanks.

Edit

Following ajmastrean's post:

I am using a console app to test with Debug.Asserts :)


Update

I originally rolled with using BackgroundWorker to deal with the threading (since I am used to that from Windows dev) I soon realised that when I was performing tests where multiple ops (threads) needed to complete before continuing, I realised it was going to be a bit of a hack to get it to do this.

I then followed up on ajmastrean's post and realised I should really be using the Thread class for working with concurrent operations. I will now refactor using this method (albeit a different approach). Thanks!

A: 

@Rob

Are you using the Visual Studio test tools? or a third party tool, like NUnit or MbUnit? There are some very different patterns for multi-threaded tests for each.

And for that matter, are you unit testing or just putting a console app together for test purposes?

Anthony Mastrean
A: 

Your idea should work fine. Basically you just want to spawn a bunch of threads, and make sure the ones writing the file take long enough to do it to actually make the readers wait. If all of your threads return without error, and without blocking forever, then the test succeeds.

Eric Z Beard
+1  A: 

In .NET, 'ThreadPool' and 'Thread' threads won't return to you without setting up a 'ManualResetEvent' or 'AutoResetEvent'. I find these overkill for a quick test method (not to mention kind of complicated to create, set, and manage). Background worker is a also a bit complex with the callbacks and such.

Something I have found that works is 1. Create an array of threads. 2. Setup the ThreadStart method of each thread. 3. Start each thread. 4. Join on all threads (blocks the current thread until all other threads complete or abort).

public static void MultiThreadedTest()
{
    Thread[] threads = new Thread[count];

    foreach(Thread thread in threads)
    {
        thread = new Thread(DoSomeWork());
    }

    foreach(Thread thread in threads)
    {
        thread.Start();
    }

    foreach(Thread thread in threads)
    {
        thread.Join();
    }
}
Anthony Mastrean
Accepted because this deals with concurrency which is an issue with the testing since the main app thread terminates before BackgroundWorkers have completed their operation (and I got tired of adding Thread.Sleep!)
Rob Cooper
A: 

@ajmastrean, since unit test result must be predictable we need to synchronize threads somehow. I can't see a simple way to do it without using events.

I found that ThreadPool.QueueUserWorkItem gives me an easy way to test such use cases

 ThreadPool.QueueUserWorkItem(x => { 
    File.Open(fileName, FileMode.Open);
    event1.Set(); // Start 2nd tread;
    event2.WaitOne(); // Blocking the file;
});
ThreadPool.QueueUserWorkItem(x => { 
    try
    {
        event1.WaitOne(); // Waiting until 1st thread open file
        File.Delete(fileName); // Simulating conflict
    }
    catch (IOException e)
    {
        Debug.Write("File access denied");
    }
});
aku