Skip to content

#4191 - FieldVisibilitySchemaTransformation reproduction #4192

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.

Sign up for GitHub

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

Jump to bottom
Open
bbakerman wants to merge 3 commits into
base: master
Choose a base branch
from
Open

#4191 - FieldVisibilitySchemaTransformation reproduction #4192

bbakerman wants to merge 3 commits into from
+435,123 −17

Conversation

@bbakerman
Copy link
Member

@bbakerman bbakerman commented Dec 17, 2025

#4191

Reproduction - currently not failing locally. It needs lots of memory so this might fail in automated testing

@github-actions
Copy link
Contributor

github-actions bot commented Dec 17, 2025 •
edited
Loading

Test Results

  257 files   -  70    257 suites   - 70   10m 25s ⏱️ + 5m 17s
4 705 tests  - 516  4 695 ✅  - 518  9 💤 +1  1 ❌ +1 
4 794 runs   - 516  4 784 ✅  - 518  9 💤 +1  1 ❌ +1 

For more details on these failures, see this check.

Results for commit 3a91b63. ± Comparison against base commit 021fe1b.

This pull request removes 714 and adds 174 tests. Note that renamed tests count towards both. 
	?

	, expected: combo-\"\\\b\f\n\r\t, #4]
                __schema { types { fields { args { type { name fields { name }}}}}}
                __schema { types { fields { type { name fields { name }}}}}
                __schema { types { inputFields { type { inputFields { name }}}}}
                __schema { types { interfaces { fields { type { interfaces { name } } } } } }
                __schema { types { name} }
                __type(name : "t") { name }
                a1: __schema { types { name} }
                a1: __type(name : "t") { name }
…

Gradle Test Executor 4 ‑ failed to execute tests
graphql.AssertTest ‑ assertFalse with different number of error args but false does not throw assertions [toRun: <graphql.AssertTest$__spock_feature_0_21prov0_closure23@2849434b delegate=graphql.AssertTest@4af70944 owner=graphql.AssertTest@4af70944 thisObject=graphql.AssertTest@4af70944 resolveStrategy=0 directive=0 parameterTypes=[class java.lang.Object] maximumNumberOfParameters=1 bcw=null>, expectedMessage: error arg1, #0]
graphql.AssertTest ‑ assertFalse with different number of error args but false does not throw assertions [toRun: <graphql.AssertTest$__spock_feature_0_21prov0_closure24@60bbacfc delegate=graphql.AssertTest@4af70944 owner=graphql.AssertTest@4af70944 thisObject=graphql.AssertTest@4af70944 resolveStrategy=0 directive=0 parameterTypes=[class java.lang.Object] maximumNumberOfParameters=1 bcw=null>, expectedMessage: error arg1 arg2, #1]
graphql.AssertTest ‑ assertFalse with different number of error args but false does not throw assertions [toRun: <graphql.AssertTest$__spock_feature_0_21prov0_closure25@65b97f47 delegate=graphql.AssertTest@4af70944 owner=graphql.AssertTest@4af70944 thisObject=graphql.AssertTest@4af70944 resolveStrategy=0 directive=0 parameterTypes=[class java.lang.Object] maximumNumberOfParameters=1 bcw=null>, expectedMessage: error arg1 arg2 arg3, #2]
graphql.AssertTest ‑ assertFalse with different number of error args throws assertions [toRun: <graphql.AssertTest$__spock_feature_0_20prov0_closure20@75ae4a1f delegate=graphql.AssertTest@4af70944 owner=graphql.AssertTest@4af70944 thisObject=graphql.AssertTest@4af70944 resolveStrategy=0 directive=0 parameterTypes=[class java.lang.Object] maximumNumberOfParameters=1 bcw=null>, expectedMessage: error arg1, #0]
graphql.AssertTest ‑ assertFalse with different number of error args throws assertions [toRun: <graphql.AssertTest$__spock_feature_0_20prov0_closure21@70228253 delegate=graphql.AssertTest@4af70944 owner=graphql.AssertTest@4af70944 thisObject=graphql.AssertTest@4af70944 resolveStrategy=0 directive=0 parameterTypes=[class java.lang.Object] maximumNumberOfParameters=1 bcw=null>, expectedMessage: error arg1 arg2, #1]
graphql.AssertTest ‑ assertFalse with different number of error args throws assertions [toRun: <graphql.AssertTest$__spock_feature_0_20prov0_closure22@63c12e52 delegate=graphql.AssertTest@4af70944 owner=graphql.AssertTest@4af70944 thisObject=graphql.AssertTest@4af70944 resolveStrategy=0 directive=0 parameterTypes=[class java.lang.Object] maximumNumberOfParameters=1 bcw=null>, expectedMessage: error arg1 arg2 arg3, #2]
graphql.AssertTest ‑ assertNotNull with different number of  error args throws assertions [toRun: <graphql.AssertTest$__spock_feature_0_5prov0_closure3@2b03d52f delegate=graphql.AssertTest@4af70944 owner=graphql.AssertTest@4af70944 thisObject=graphql.AssertTest@4af70944 resolveStrategy=0 directive=0 parameterTypes=[class java.lang.Object] maximumNumberOfParameters=1 bcw=null>, expectedMessage: error arg1, #0]
graphql.AssertTest ‑ assertNotNull with different number of  error args throws assertions [toRun: <graphql.AssertTest$__spock_feature_0_5prov0_closure4@1f11f64e delegate=graphql.AssertTest@4af70944 owner=graphql.AssertTest@4af70944 thisObject=graphql.AssertTest@4af70944 resolveStrategy=0 directive=0 parameterTypes=[class java.lang.Object] maximumNumberOfParameters=1 bcw=null>, expectedMessage: error arg1 arg2, #1]
graphql.AssertTest ‑ assertNotNull with different number of  error args throws assertions [toRun: <graphql.AssertTest$__spock_feature_0_5prov0_closure5@216e0771 delegate=graphql.AssertTest@4af70944 owner=graphql.AssertTest@4af70944 thisObject=graphql.AssertTest@4af70944 resolveStrategy=0 directive=0 parameterTypes=[class java.lang.Object] maximumNumberOfParameters=1 bcw=null>, expectedMessage: error arg1 arg2 arg3, #2]
…
This pull request removes 1 skipped test and adds 1 skipped test. Note that renamed tests count towards both. 
graphql.util.TreeParallelTraverserTest ‑ test parallel traversing

graphql.schema.transform.FieldVisibilitySchemaTransformationTest ‑ issue 4191 - specific problem on field visibility - smaller reproduction
This pull request skips 1 test. 
graphql.schema.fetching.LambdaFetchingSupportTest ‑ different class loaders induce certain behaviours

♻️ This comment has been updated with latest results.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@bbakerman