-
Notifications
You must be signed in to change notification settings - Fork 64
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add Support to Generate GraphQL Schema for Local Service Variable Declaration #1704
Add Support to Generate GraphQL Schema for Local Service Variable Declaration #1704
Conversation
.../src/main/java/io/ballerina/stdlib/graphql/compiler/LocalLevelServiceObjectAnalysisTask.java
Outdated
Show resolved
Hide resolved
...test/resources/ballerina_sources/generator_tests/25_local_graphql_service_object/service.bal
Outdated
Show resolved
Hide resolved
...test/resources/ballerina_sources/generator_tests/25_local_graphql_service_object/service.bal
Outdated
Show resolved
Hide resolved
.../src/main/java/io/ballerina/stdlib/graphql/compiler/LocalLevelServiceObjectAnalysisTask.java
Outdated
Show resolved
Hide resolved
.../src/main/java/io/ballerina/stdlib/graphql/compiler/LocalLevelServiceObjectAnalysisTask.java
Outdated
Show resolved
Hide resolved
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## master #1704 +/- ##
============================================
- Coverage 86.47% 86.32% -0.16%
- Complexity 762 771 +9
============================================
Files 129 130 +1
Lines 9094 9153 +59
Branches 3563 3569 +6
============================================
+ Hits 7864 7901 +37
- Misses 923 948 +25
+ Partials 307 304 -3 ☔ View full report in Codecov by Sentry. |
...test/resources/ballerina_sources/generator_tests/25_local_graphql_service_object/service.bal
Outdated
Show resolved
Hide resolved
compiler-plugin/src/main/java/io/ballerina/stdlib/graphql/compiler/GraphqlCodeModifier.java
Outdated
Show resolved
Hide resolved
.../src/main/java/io/ballerina/stdlib/graphql/compiler/LocalLevelServiceObjectAnalysisTask.java
Outdated
Show resolved
Hide resolved
.../src/main/java/io/ballerina/stdlib/graphql/compiler/LocalLevelServiceObjectAnalysisTask.java
Outdated
Show resolved
Hide resolved
.../src/main/java/io/ballerina/stdlib/graphql/compiler/LocalLevelServiceObjectAnalysisTask.java
Outdated
Show resolved
Hide resolved
.../src/main/java/io/ballerina/stdlib/graphql/compiler/LocalLevelServiceObjectAnalysisTask.java
Outdated
Show resolved
Hide resolved
...c/main/java/io/ballerina/stdlib/graphql/compiler/schema/generator/GraphqlSourceModifier.java
Outdated
Show resolved
Hide resolved
...test/resources/ballerina_sources/generator_tests/25_local_graphql_service_object/service.bal
Outdated
Show resolved
Hide resolved
...resources/ballerina_sources/generator_tests/25_local_service_object_declarations/service.bal
Outdated
Show resolved
Hide resolved
...es/ballerina_sources/generator_tests/26_object_field_service_object_declarations/service.bal
Outdated
Show resolved
Hide resolved
.../test/resources/ballerina_sources/generator_tests/27_inline_service_declarations/service.bal
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shall we add test in the ballerina-tests
module, so that we can test the runtime as well?
|
Purpose
Fixes: #3317
Examples
This improvement enables to generate GraphQL Schema for the following locally defined
backendService
service object.Checklist
commons
package changes (if there are any, please update the GraphQL version in GraphQL tools and Ballerina dev tools)compiler
package changes (if there are any, please update the GraphQL version in Ballerina dev tools)