Towards #1290 automate one core issue regression test

Author: MeFisto94

jme3-core/build.gradle

@@ -14,11 +14,14 @@ sourceSets {
         java {
             srcDir 'src/test/java'
         }
+
+        System.setProperty "java.awt.headless", "true"
     }
 }
 
 dependencies {
     testCompile project(':jme3-testdata')
+    testRuntime project(':jme3-desktop')
 }
 
 task updateVersionPropertiesFile {
@@ -44,4 +47,4 @@ task updateVersionPropertiesFile {
     }
 }
 
-processResources.dependsOn updateVersionPropertiesFile
+processResources.dependsOn updateVersionPropertiesFile

jme3-core/src/test/java/com/jme3/test/PreventCoreIssueRegressions.java

@@ -0,0 +1,97 @@
+/*
+ * Copyright (c) 2019-2021 jMonkeyEngine
+ * All rights reserved.
+ *
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions are
+ * met:
+ *
+ * * Redistributions of source code must retain the above copyright
+ *   notice, this list of conditions and the following disclaimer.
+ *
+ * * Redistributions in binary form must reproduce the above copyright
+ *   notice, this list of conditions and the following disclaimer in the
+ *   documentation and/or other materials provided with the distribution.
+ *
+ * * Neither the name of 'jMonkeyEngine' nor the names of its contributors
+ *   may be used to endorse or promote products derived from this software
+ *   without specific prior written permission.
+ *
+ * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
+ * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED
+ * TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR
+ * PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT OWNER OR
+ * CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL,
+ * EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO,
+ * PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR
+ * PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF
+ * LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING
+ * NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS
+ * SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+ */
+package com.jme3.test;
+
+import com.jme3.app.LegacyApplication;
+import com.jme3.app.SimpleApplication;
+import com.jme3.app.state.AppState;
+import com.jme3.app.state.ScreenshotAppState;
+import com.jme3.asset.DesktopAssetManager;
+import com.jme3.input.InputManager;
+import com.jme3.input.dummy.DummyKeyInput;
+import com.jme3.input.dummy.DummyMouseInput;
+import com.jme3.renderer.Camera;
+import com.jme3.renderer.RenderManager;
+import com.jme3.renderer.ViewPort;
+import com.jme3.system.NullRenderer;
+import org.junit.Assert;
+import org.junit.Test;
+
+import java.lang.reflect.Field;
+import java.util.List;
+
+/**
+ * The Test Suite to prevent regressions from previously fixed Core issues
+ * @author Stephen Gold <[email protected]>
+ */
+public class PreventCoreIssueRegressions {
+
+    /**
+     * Test case for JME issue #1421: ScreenshotAppState never cleans up.
+     */
+    @Test
+    public void TestIssue1421() throws NoSuchFieldException, IllegalAccessException {
+        ScreenshotAppState screenshotAppState = new ScreenshotAppState("./", "screen_shot");
+
+        SimpleApplication app = new SimpleApplication(new AppState[]{}) {
+            @Override
+            public void simpleInitApp() { }
+        };
+
+        app.setAssetManager(new DesktopAssetManager(true));
+        Field f1 = LegacyApplication.class.getDeclaredField("inputManager");
+        f1.setAccessible(true);
+        f1.set(app, new InputManager(new DummyMouseInput(), new DummyKeyInput(), null, null));
+
+        Field f2 = LegacyApplication.class.getDeclaredField("renderManager");
+        f2.setAccessible(true);
+        f2.set(app, new RenderManager(new NullRenderer()));
+
+        app.getRenderManager().createPostView("null", new Camera(1, 1));
+
+        Assert.assertTrue(app.getStateManager().attach(screenshotAppState));
+        app.getStateManager().update(0f); // Causes SAS#initialize to be called
+
+        // Confirm that the SceneProcessor is attached.
+        List<ViewPort> vps = app.getRenderManager().getPostViews();
+        Assert.assertEquals(1, vps.size());
+        Assert.assertEquals(1, vps.get(0).getProcessors().size());
+        Assert.assertTrue(app.getInputManager().hasMapping("ScreenShot")); // Confirm that KEY_SYSRQ is mapped.
+        Assert.assertTrue(app.getStateManager().detach(screenshotAppState));
+
+        app.getStateManager().update(0f); // Causes SAS#cleanup to be called
+
+        // Check whether the SceneProcessor is still attached.
+        Assert.assertEquals(0, vps.get(0).getProcessors().size());
+        Assert.assertFalse(app.getInputManager().hasMapping("ScreenShot")); // Confirm that KEY_SYSRQ is unmapped.
+    }
+}