Skip to content
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

LPD-28845 + LPD-31166 Pages API with REST BUilder Fix #563

Open
wants to merge 57 commits into
base: master
Choose a base branch
from

Conversation

ruben-pulido
Copy link
Owner

master

ruben-pulido and others added 30 commits July 24, 2024 00:05
…L since a StackOverflowError is happening in the generated testing code.

Without this commit the following tests fail:
- NavigationMenuResourceTest. testGraphQLDeleteNavigationMenu
- NavigationMenuResourceTest. testGraphQLGetNavigationMenu
- NavigationMenuResourceTest. testGraphQLGetNavigationMenuNotFound
- NavigationMenuResourceTest. testGraphQLGetSiteNavigationMenusPage
- NavigationMenuResourceTest. testGraphQLPostSiteNavigationMenu

testGraphQLDeleteNavigationMenu
java.lang.StackOverflowError
	at java.lang.Class.getDeclaredFields(Class.java:1915)
	at com.liferay.petra.reflect.ReflectionUtil.getDeclaredFields(ReflectionUtil.java:33)
	at com.liferay.headless.delivery.resource.v1_0.test.BaseNavigationMenuResourceTestCase.getDeclaredFields(BaseNavigationMenuResourceTestCase.java:1371)
	at com.liferay.headless.delivery.resource.v1_0.test.BaseNavigationMenuResourceTestCase.getGraphQLFields(BaseNavigationMenuResourceTestCase.java:1211)
	at com.liferay.headless.delivery.resource.v1_0.test.BaseNavigationMenuResourceTestCase.getGraphQLFields(BaseNavigationMenuResourceTestCase.java:1210)
	at com.liferay.headless.delivery.resource.v1_0.test.BaseNavigationMenuResourceTestCase.getGraphQLFields(BaseNavigationMenuResourceTestCase.java:1210)

testGraphQLGetNavigationMenu
java.lang.StackOverflowError
	at java.lang.reflect.Field.getAnnotation(Field.java:1120)
	at com.liferay.headless.delivery.resource.v1_0.test.BaseNavigationMenuResourceTestCase.getGraphQLFields(BaseNavigationMenuResourceTestCase.java:1199)
	at com.liferay.headless.delivery.resource.v1_0.test.BaseNavigationMenuResourceTestCase.getGraphQLFields(BaseNavigationMenuResourceTestCase.java:1210)
	at com.liferay.headless.delivery.resource.v1_0.test.BaseNavigationMenuResourceTestCase.getGraphQLFields(BaseNavigationMenuResourceTestCase.java:1210)

testGraphQLGetNavigationMenuNotFound
java.lang.StackOverflowError
	at java.lang.reflect.Field.getAnnotation(Field.java:1120)
	at com.liferay.headless.delivery.resource.v1_0.test.BaseNavigationMenuResourceTestCase.getGraphQLFields(BaseNavigationMenuResourceTestCase.java:1199)
	at com.liferay.headless.delivery.resource.v1_0.test.BaseNavigationMenuResourceTestCase.getGraphQLFields(BaseNavigationMenuResourceTestCase.java:1210)
	at com.liferay.headless.delivery.resource.v1_0.test.BaseNavigationMenuResourceTestCase.getGraphQLFields(BaseNavigationMenuResourceTestCase.java:1210)

testGraphQLGetSiteNavigationMenusPage
java.lang.StackOverflowError
	at java.lang.RuntimeException.<init>(RuntimeException.java:96)
	at com.liferay.petra.function.transform.TransformUtil.transform(TransformUtil.java:64)
	at com.liferay.headless.delivery.resource.v1_0.test.BaseNavigationMenuResourceTestCase.getDeclaredFields(BaseNavigationMenuResourceTestCase.java:1370)
	at com.liferay.headless.delivery.resource.v1_0.test.BaseNavigationMenuResourceTestCase.getGraphQLFields(BaseNavigationMenuResourceTestCase.java:1211)
	at com.liferay.headless.delivery.resource.v1_0.test.BaseNavigationMenuResourceTestCase.getGraphQLFields(BaseNavigationMenuResourceTestCase.java:1210)
	at com.liferay.headless.delivery.resource.v1_0.test.BaseNavigationMenuResourceTestCase.getGraphQLFields(BaseNavigationMenuResourceTestCase.java:1210)

testGraphQLPostSiteNavigationMenu
java.lang.StackOverflowError
	at java.lang.RuntimeException.<init>(RuntimeException.java:96)
	at com.liferay.petra.function.transform.TransformUtil.transform(TransformUtil.java:64)
	at com.liferay.headless.delivery.resource.v1_0.test.BaseNavigationMenuResourceTestCase.getDeclaredFields(BaseNavigationMenuResourceTestCase.java:1370)
	at com.liferay.headless.delivery.resource.v1_0.test.BaseNavigationMenuResourceTestCase.getGraphQLFields(BaseNavigationMenuResourceTestCase.java:1211)
	at com.liferay.headless.delivery.resource.v1_0.test.BaseNavigationMenuResourceTestCase.getGraphQLFields(BaseNavigationMenuResourceTestCase.java:1210)
	at com.liferay.headless.delivery.resource.v1_0.test.BaseNavigationMenuResourceTestCase.getGraphQLFields(BaseNavigationMenuResourceTestCase.java:1210)
…g GraphQL since a StackOverflowError is happening in the generated testing code."

This reverts commit 7dd7dcd293c6b3eab67cb1196fdd7383b8c33f04.
This will simplify the process to update the expected files.

On a successful test run the files should be the same and git should not
report any changes.
@liferay-continuous-integration
Copy link
Collaborator

To conserve resources, the PR Tester does not automatically run for every pull.

If your code changes were already tested in another pull, reference that pull in this pull so the test results can be analyzed.

If your pull was never tested, comment "ci:test" to run the PR Tester for this pull.

Copy link

Please resolve conflicts so we can continue reviewing and processing.

Created by PR_Conflicts_Alert[bot]

17 similar comments
Copy link

Please resolve conflicts so we can continue reviewing and processing.

Created by PR_Conflicts_Alert[bot]

Copy link

Please resolve conflicts so we can continue reviewing and processing.

Created by PR_Conflicts_Alert[bot]

Copy link

Please resolve conflicts so we can continue reviewing and processing.

Created by PR_Conflicts_Alert[bot]

Copy link

Please resolve conflicts so we can continue reviewing and processing.

Created by PR_Conflicts_Alert[bot]

Copy link

Please resolve conflicts so we can continue reviewing and processing.

Created by PR_Conflicts_Alert[bot]

Copy link

Please resolve conflicts so we can continue reviewing and processing.

Created by PR_Conflicts_Alert[bot]

Copy link

Please resolve conflicts so we can continue reviewing and processing.

Created by PR_Conflicts_Alert[bot]

Copy link

Please resolve conflicts so we can continue reviewing and processing.

Created by PR_Conflicts_Alert[bot]

Copy link

Please resolve conflicts so we can continue reviewing and processing.

Created by PR_Conflicts_Alert[bot]

Copy link

Please resolve conflicts so we can continue reviewing and processing.

Created by PR_Conflicts_Alert[bot]

Copy link

Please resolve conflicts so we can continue reviewing and processing.

Created by PR_Conflicts_Alert[bot]

Copy link

Please resolve conflicts so we can continue reviewing and processing.

Created by PR_Conflicts_Alert[bot]

Copy link

Please resolve conflicts so we can continue reviewing and processing.

Created by PR_Conflicts_Alert[bot]

Copy link

Please resolve conflicts so we can continue reviewing and processing.

Created by PR_Conflicts_Alert[bot]

Copy link

Please resolve conflicts so we can continue reviewing and processing.

Created by PR_Conflicts_Alert[bot]

Copy link

Please resolve conflicts so we can continue reviewing and processing.

Created by PR_Conflicts_Alert[bot]

Copy link

Please resolve conflicts so we can continue reviewing and processing.

Created by PR_Conflicts_Alert[bot]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants