No longer throw an exception if both plugin yml and groovy exist at runtime because it should be prevented in GrailsPluginGradlePlugin

Author: jameskleeh

grails-core/src/main/groovy/org/grails/plugins/AbstractGrailsPlugin.java

@@ -125,19 +125,14 @@ public boolean isEnabled(String[] profiles) {
     }
 
     protected Resource readPluginConfiguration(Class<?> pluginClass) {
-        Resource ymlUrlResource = getConfigurationResource(pluginClass, PLUGIN_YML_PATH);
-        Resource groovyUrlResource = getConfigurationResource(pluginClass, PLUGIN_GROOVY_PATH);
+        Resource ymlResource = getConfigurationResource(pluginClass, PLUGIN_YML_PATH);
+        Resource groovyResource = getConfigurationResource(pluginClass, PLUGIN_GROOVY_PATH);
 
-        Boolean groovyUrlResourceExists = groovyUrlResource != null && groovyUrlResource.exists();
-
-        if(ymlUrlResource != null && ymlUrlResource.exists()) {
-            if (groovyUrlResourceExists) {
-                throw new RuntimeException("A plugin may define a plugin.yml or a plugin.groovy, but not both");
-            }
-            return ymlUrlResource;
+        if(ymlResource != null && ymlResource.exists()) {
+            return ymlResource;
         }
-        if(groovyUrlResourceExists) {
-            return groovyUrlResource;
+        if(groovyResource != null && groovyResource.exists()) {
+            return groovyResource;
         }
         return null;
     }

grails-core/src/test/groovy/org/codehaus/groovy/grails/plugins/BinaryPluginSpec.groovy

@@ -51,15 +51,16 @@ class BinaryPluginSpec extends Specification {
             cssResource == null
     }
 
-    def "Test plugin with both plugin.yml and plugin.groovy throws exception"() {
+    //This should never occur because the GrailsPluginGradlePlugin.groovy should catch it
+    def "Test plugin with both plugin.yml and plugin.groovy, yml is used"() {
         when:
         def descriptor = new BinaryGrailsPluginDescriptor(new ByteArrayResource(testBinary.getBytes('UTF-8')), ['org.codehaus.groovy.grails.plugins.TestBinaryResource'])
         MockConfigBinaryGrailsPlugin.YAML_EXISTS = true
         MockConfigBinaryGrailsPlugin.GROOVY_EXISTS = true
-        new MockConfigBinaryGrailsPlugin(descriptor)
+        def binaryPlugin = new MockConfigBinaryGrailsPlugin(descriptor)
 
         then:
-        thrown(RuntimeException)
+        binaryPlugin.propertySource.getProperty('foo') == "bar"
     }
 
     def "Test plugin with only plugin.yml"() {