Index: src/main/java/org/apache/maven/plugin/idea/IdeaModuleMojo.java
===================================================================
--- src/main/java/org/apache/maven/plugin/idea/IdeaModuleMojo.java	(revision 723786)
+++ src/main/java/org/apache/maven/plugin/idea/IdeaModuleMojo.java	(working copy)
@@ -947,6 +947,13 @@
         {
             return null;
         }
+        boolean isTestSources = false;
+        if ("sources".equals(classifier) && "test-jar".equals(artifact.getType()))
+        {
+            // strip the tests suffix too
+            delIndex -= "-tests".length();
+            isTestSources = true;
+        }
 
         List remoteRepos = executedProject.getRemoteArtifactRepositories();
         try
@@ -956,7 +963,7 @@
                                                                                         artifact.getVersion(),
                                                                                         artifact.getType(),
                                                                                         classifier );
-            String dstFilename = basePath.substring( 0, delIndex ) + '-' + classifier + ".jar";
+            String dstFilename = basePath.substring( 0, delIndex ) + '-' + (isTestSources ? "test-" : "") + classifier + ".jar";
             File dstFile = new File( dstFilename );
             classifiedArtifact.setFile( dstFile );
             //this check is here because wagonManager does not seem to check if the remote file is newer