Do not claim ownership of .java source files

BeanShell is a very close approximation to Java syntax. But it does not
handle everything you can write into a .java file. And it certainly does
not actually try to compile and run it with javac. It is wrong for
BeanShell to snarf up these files, and in particular it clashes with the
SciJava scripting-java component, which *does* actually use javac.

Author: ctrueden

src/main/java/org/scijava/plugins/scripting/beanshell/BeanshellScriptLanguage.java

@@ -31,6 +31,9 @@
 
 package org.scijava.plugins.scripting.beanshell;
 
+import java.util.List;
+import java.util.stream.Collectors;
+
 import javax.script.ScriptEngine;
 
 import org.scijava.plugin.Plugin;
@@ -44,6 +47,7 @@
  * An adapter of the {@link BshScriptEngineFactory} to the SciJava scripting interface.
  * 
  * @author Mark Hiner
+ * @author Curtis Rueden
  * @author Johannes Schindelin
  * @see ScriptEngine
  */
@@ -54,6 +58,8 @@ public BeanshellScriptLanguage() {
 		super(new BshScriptEngineFactory());
 	}
 
+	// -- ScriptLanguage methods --
+
 	@Override
 	public Object decode(final Object object) {
 		if (object instanceof Primitive) {
@@ -66,4 +72,29 @@ public Object decode(final Object object) {
 		return object;
 	}
 
+	// -- ScriptEngineFactory methods --
+
+	// NB: BeanShell must *not* claim ownership of .java files!
+	// It clashes with the scripting-java script language.
+
+	@Override
+	public List<String> getExtensions() {
+		return super.getExtensions().stream()//
+			.filter(extension -> !"java".equals(extension))//
+			.collect(Collectors.toList());
+	}
+
+	@Override
+	public List<String> getMimeTypes() {
+		return super.getMimeTypes().stream()//
+			.filter(mimeType -> !"application/x-java-source".equals(mimeType))//
+			.collect(Collectors.toList());
+	}
+
+	@Override
+	public List<String> getNames() {
+		return super.getNames().stream()//
+			.filter(name -> !"java".equalsIgnoreCase(name))//
+			.collect(Collectors.toList());
+	}
 }

src/test/java/org/scijava/plugins/scripting/beanshell/BeanshellTest.java

@@ -33,7 +33,8 @@
 
 import static org.junit.Assert.assertEquals;
 
-import java.io.IOException;
+import java.util.Arrays;
+import java.util.List;
 import java.util.concurrent.ExecutionException;
 
 import javax.script.Bindings;
@@ -137,4 +138,17 @@ public void testNull() throws InterruptedException, ExecutionException {
 		final Object result = m.getReturnValue();
 		assertEquals(expected, result);
 	}
+
+	@Test
+	public void testExtensionsMimeTypesNames() {
+		// NB: We *don't* want BeanShell to claim ownership of .java files!
+		final ScriptLanguage lang = scriptService.getLanguageByExtension("bsh");
+		final List<String> expectedExtensions = Arrays.asList("bsh");
+		assertEquals(expectedExtensions, lang.getExtensions());
+		final List<String> expectedMimeTypes = Arrays.asList(
+			"application/x-beanshell", "application/x-bsh");
+		assertEquals(expectedMimeTypes, lang.getMimeTypes());
+		final List<String> expectedNames = Arrays.asList("beanshell", "bsh");
+		assertEquals(expectedNames, lang.getNames());
+	}
 }