Skip to content
This repository was archived by the owner on Mar 10, 2025. It is now read-only.

Commit ad4f51e

Browse files
committed
More tests fixes
1 parent f1d02b6 commit ad4f51e

File tree

4 files changed

+27
-28
lines changed

4 files changed

+27
-28
lines changed

core/src/test/groovy/org/grails/gorm/graphql/MongoSchemaSpec.groovy

+6
Original file line numberDiff line numberDiff line change
@@ -15,9 +15,15 @@ import org.grails.gorm.graphql.domain.general.GeneralPackage
1515
import org.grails.gorm.graphql.testing.GraphQLSchemaSpec
1616
import org.grails.gorm.graphql.types.GraphQLTypeManager
1717
import spock.lang.AutoCleanup
18+
import spock.lang.Ignore
1819
import spock.lang.Shared
1920
import spock.lang.Specification
2021

22+
/**
23+
* Ignored due to https://github.com/fakemongo/fongo/issues/367.
24+
* Fongo not compatible with the mongo driver being used
25+
*/
26+
@Ignore
2127
class MongoSchemaSpec extends Specification implements GraphQLSchemaSpec {
2228

2329
@Shared @AutoCleanup MongoDatastore mongoDatastore

core/src/test/groovy/org/grails/gorm/graphql/SchemaSpec.groovy

+18-27
Original file line numberDiff line numberDiff line change
@@ -44,6 +44,10 @@ class SchemaSpec extends Specification implements GraphQLSchemaSpec {
4444
}.join('').replace('Output', '')
4545
}
4646

47+
private String normalizeType(String prefix, GraphQLPropertyType type) {
48+
prefix + normalizeType(type)
49+
}
50+
4751
void "test ComplexOperation"() {
4852
given:
4953
GraphQLObjectType type = schema.getType('AwesomeType')
@@ -134,16 +138,12 @@ class SchemaSpec extends Specification implements GraphQLSchemaSpec {
134138
type.getFieldDefinition('parentCircular').type == schema.getType('DebugCircular')
135139
}
136140

137-
@Unroll
138-
void "test !ToOne#type"() {
141+
void "test !ToOne"() {
139142
expect:
140-
schema.getType('ToOne' + type) == null
141-
142-
where:
143-
type << [GraphQLPropertyType.CREATE_NESTED,
144-
GraphQLPropertyType.CREATE_EMBEDDED,
145-
GraphQLPropertyType.UPDATE_NESTED,
146-
GraphQLPropertyType.UPDATE_EMBEDDED].collect{ normalizeType(it) }
143+
schema.getType(normalizeType('ToOne', GraphQLPropertyType.CREATE_NESTED)) == null
144+
schema.getType(normalizeType('ToOne', GraphQLPropertyType.CREATE_EMBEDDED)) == null
145+
schema.getType(normalizeType('ToOne', GraphQLPropertyType.UPDATE_NESTED)) == null
146+
schema.getType(normalizeType('ToOne', GraphQLPropertyType.UPDATE_EMBEDDED)) == null
147147
}
148148

149149
void "test ToOne"() {
@@ -179,28 +179,19 @@ class SchemaSpec extends Specification implements GraphQLSchemaSpec {
179179
//everything else is a scalar.. not worth testing every property
180180
}
181181

182-
@Unroll
183-
void "test !One#type"() {
182+
void "test !One"() {
184183
expect:
185-
schema.getType('One' + type) == null
186-
187-
where:
188-
type << [GraphQLPropertyType.CREATE,
189-
GraphQLPropertyType.CREATE_EMBEDDED,
190-
GraphQLPropertyType.UPDATE,
191-
GraphQLPropertyType.UPDATE_EMBEDDED].collect{ normalizeType(it) }
184+
schema.getType(normalizeType('One', GraphQLPropertyType.CREATE)) == null
185+
schema.getType(normalizeType('One', GraphQLPropertyType.CREATE_EMBEDDED)) == null
186+
schema.getType(normalizeType('One', GraphQLPropertyType.UPDATE)) == null
187+
schema.getType(normalizeType('One', GraphQLPropertyType.UPDATE_EMBEDDED)) == null
192188
}
193189

194-
@Unroll
195-
void "test One#type"() {
190+
void "test One"() {
196191
expect:
197-
schema.getType('One' + type) != null
198-
199-
where:
200-
type << [GraphQLPropertyType.OUTPUT,
201-
GraphQLPropertyType.CREATE_NESTED,
202-
GraphQLPropertyType.UPDATE_NESTED].collect{ normalizeType(it) }
203-
192+
schema.getType(normalizeType('One', GraphQLPropertyType.OUTPUT)) != null
193+
schema.getType(normalizeType('One', GraphQLPropertyType.CREATE_NESTED)) != null
194+
schema.getType(normalizeType('One', GraphQLPropertyType.UPDATE_NESTED)) != null
204195
}
205196

206197
void "test CircularOneCreateNested"() {

core/src/test/groovy/org/grails/gorm/graphql/interceptor/manager/DefaultGraphQLInterceptorManagerSpec.groovy

+2
Original file line numberDiff line numberDiff line change
@@ -59,6 +59,8 @@ class DefaultGraphQLInterceptorManagerSpec extends Specification {
5959

6060
class OrderedBaseGraphQLInterceptor extends BaseGraphQLFetcherInterceptor implements Ordered {
6161

62+
int order
63+
6264
OrderedBaseGraphQLInterceptor(int order) {
6365
this.order = order
6466
}

gradle.properties

+1-1
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ grailsVersion=4.0.0.M2
1111
grailsDocsVersion=3.3.2
1212
groovyVersion=2.5.6
1313
slf4jVersion=1.7.22
14-
spockVersion=1.2-groovy-2.5
14+
spockVersion=1.3-groovy-2.5
1515
micronautVersion=1.0.3
1616
graphqlJavaVersion=11.0
1717
codenarcVersion=1.2.1

0 commit comments

Comments
 (0)