Ignore non-graphql files (#1502)

* Ignore non-GraphQL files

* Ignore non-GraphQL files

* Add query to schemas for testing

* Fix lint error

* Use filename instead of toString()

* Rename variable

* Cleanup code

---------

Co-authored-by: Grace <gding@netflix.com>

graphql-dgs/src/main/kotlin/com/netflix/graphql/dgs/internal/DgsSchemaProvider.kt

@@ -547,7 +547,8 @@ class DgsSchemaProvider(
         }
 
         schemas += metaInfSchemas
-        return schemas
+
+        return schemas.filter { it.filename.endsWith(".graphql", true) || it.filename.endsWith(".graphqls", true) }
     }
 
     companion object {

graphql-dgs/src/test/kotlin/com/netflix/graphql/dgs/DgsSchemaProviderTest.kt

@@ -190,6 +190,22 @@ internal class DgsSchemaProviderTest {
         ).findSchemaFiles()
     }
 
+    @Test
+    fun findSchemaFilesIgnoreNonGraphQLFiles() {
+        val schemaFiles = schemaProvider(
+            schemaLocations = listOf("classpath*:location3/**/*.graphql*")
+        ).findSchemaFiles()
+        assertEquals("location3-schema1.graphql", schemaFiles[0].filename)
+        assertEquals("location3-schema2.graphqls", schemaFiles[1].filename)
+
+        // Check that the .graphqlconfig file has been ignored
+        val schemaFilesNames: MutableList<String> = mutableListOf()
+        for (schemaFile in schemaFiles) {
+            schemaFilesNames.add(schemaFile.filename)
+        }
+        assert(!schemaFilesNames.contains("location3-ignore.graphqlconfig"))
+    }
+
     @Test
     fun addFetchers() {
         val fetcher = object : Any() {

graphql-dgs/src/test/resources/location3/location3-ignore.graphqlconfig

@@ -0,0 +1,3 @@
+type Query {
+  hello(name:String): String
+}

graphql-dgs/src/test/resources/location3/location3-schema1.graphql

@@ -0,0 +1,3 @@
+type Query {
+    hello(name:String): String
+}

graphql-dgs/src/test/resources/location3/location3-schema2.graphqls

@@ -0,0 +1,3 @@
+type Query {
+    hello(name:String): String
+}