Fix a bug in Msn Filetransfer preventing NullPointerException. And tests to recreate needed contacts for filetransfer tests.

src/net/java/sip/communicator/impl/protocol/msn/OperationSetFileTransferMsnImpl.java

@@ -94,7 +94,7 @@ public FileTransfer sendFile(   Contact toContact,
                     file);
 
         if(ft == null)
-            new IllegalStateException(
+            throw new IllegalStateException(
                 "A problem occured sending file, contact not found or offline");
 
         FileTransferImpl outFileTransfer = new FileTransferImpl(

test/net/java/sip/communicator/slick/protocol/msn/TestOperationSetFileTransferImpl.java

@@ -143,8 +143,10 @@ public void prepareContactList()
         throws Exception
     {
         // clear the provider the first time we run a filetransfer list
-        if(getContact1() == null && getContact2() == null)
+        if(contact1 == null && contact2 == null)
+        {
             fixture.clearProvidersLists();
+        }
 
         if(getContact1() == null)
         {