[MSHADE-382] Apply review suggestions

- remove unit test case
- change it verification script from .bsh to .groovy

Co-authored-by: Markus Karg <markus@headcrashing.eu>

src/it/projects/MSHADE-382_skip_execution/verify.bsh → src/it/projects/MSHADE-382_skip_execution/verify.groovy

@@ -16,11 +16,5 @@
  * specific language governing permissions and limitations
  * under the License.
  */
-import java.io.*;
 
-File originalJarFile = new File( basedir, "target/original-skip-execution-1.0.jar" );
-
-if ( originalJarFile.exists() )
-{
-    throw new IllegalStateException( "Shading was not skipped." );
-}
+assert !(new File( basedir, "target/original-skip-execution-1.0.jar" ).exists()) : "Shading was not skipped."

src/test/java/org/apache/maven/plugins/shade/mojo/ShadeMojoTest.java

@@ -62,43 +62,6 @@
 public class ShadeMojoTest
     extends PlexusTestCase
 {
-    public void testSkipShading() throws Exception
-    {
-        final ShadeMojo mojo = new ShadeMojo();
-
-        Exception thrown = null;
-        try
-        {
-            // execute without configuration expecting a failure
-            mojo.execute();
-        }
-        catch( Exception expected )
-        {
-            thrown = expected;
-        }
-
-        assertNotNull( thrown );
-
-        // set skip = true
-        Field skip = ShadeMojo.class.getDeclaredField( "skip" );
-        skip.setAccessible( true );
-        skip.set( mojo, Boolean.TRUE );
-
-        thrown = null;
-        try
-        {
-            // execute without configuration and skip
-            // should not cause any exceptions
-            mojo.execute();
-        }
-        catch( Exception unexpected )
-        {
-            thrown = unexpected;
-        }
-
-        assertNull( thrown );
-    }
-
     public void testManifestTransformerSelection() throws Exception
     {
         final ShadeMojo mojo = new ShadeMojo();