Skip to content
This repository was archived by the owner on Dec 19, 2023. It is now read-only.

Commit 8726ca4

Browse files
committed
refactored schema directive
1 parent 3ec6820 commit 8726ca4

File tree

3 files changed

+14
-34
lines changed

3 files changed

+14
-34
lines changed

graphql-spring-boot-autoconfigure/src/main/java/com/oembedler/moon/graphql/boot/SchemaDirective.java

Lines changed: 13 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -2,14 +2,22 @@
22

33
import graphql.schema.idl.SchemaDirectiveWiring;
44

5-
public interface SchemaDirective {
5+
public class SchemaDirective {
66

7-
String getName();
7+
private final String name;
8+
private final SchemaDirectiveWiring directive;
89

9-
SchemaDirectiveWiring getDirective();
10+
public SchemaDirective(String name, SchemaDirectiveWiring directive) {
11+
this.name = name;
12+
this.directive = directive;
13+
}
14+
15+
public String getName() {
16+
return name;
17+
}
1018

11-
static SchemaDirective create(String name, SchemaDirectiveWiring directive) {
12-
return new SchemaDirectiveImpl(name, directive);
19+
public SchemaDirectiveWiring getDirective() {
20+
return directive;
1321
}
1422

1523
}

graphql-spring-boot-autoconfigure/src/main/java/com/oembedler/moon/graphql/boot/SchemaDirectiveImpl.java

Lines changed: 0 additions & 25 deletions
This file was deleted.

graphql-spring-boot-autoconfigure/src/test/java/com/oembedler/moon/graphql/boot/test/graphqlJavaTools/GraphQLToolsDirectiveTest.java

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -7,14 +7,11 @@
77
import graphql.schema.GraphQLObjectType;
88
import graphql.schema.idl.SchemaDirectiveWiring;
99
import graphql.schema.idl.SchemaDirectiveWiringEnvironment;
10-
import graphql.schema.idl.SchemaParser;
1110
import org.junit.Test;
1211
import org.springframework.context.annotation.Bean;
1312
import org.springframework.context.annotation.Configuration;
1413
import org.springframework.stereotype.Component;
1514

16-
import static org.junit.Assert.assertNotNull;
17-
1815
public class GraphQLToolsDirectiveTest extends AbstractAutoConfigurationTest {
1916

2017
public GraphQLToolsDirectiveTest() {
@@ -39,7 +36,7 @@ String schemaLocationTest(String id) {
3936

4037
@Bean
4138
public SchemaDirective uppercaseDirective() {
42-
return SchemaDirective.create("uppercase", new SchemaDirectiveWiring() {
39+
return new SchemaDirective("uppercase", new SchemaDirectiveWiring() {
4340
@Override
4441
public GraphQLObjectType onObject(SchemaDirectiveWiringEnvironment<GraphQLObjectType> environment) {
4542
return null;

0 commit comments

Comments
 (0)