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

Failing component test #13173

Open
weiquu opened this issue Sep 2, 2024 · 0 comments
Open

Failing component test #13173

weiquu opened this issue Sep 2, 2024 · 0 comments
Labels
a-Testing Testing-related traits such as efficiency, robustness, coverage p.High Significant impact; would like to do in the next few releases

Comments

@weiquu
Copy link
Contributor

weiquu commented Sep 2, 2024

The SearchAccountRequestsActionTest has suddenly started failing... did a quick check and couldn't find any recent code change that caused it. The tests are failing due to this error: java.lang.IllegalArgumentException: UUID string too large, from the same method. Example stack trace:

java.lang.IllegalArgumentException: UUID string too large
	at java.util.UUID.fromString(UUID.java:199)
	at teammates.storage.sqlsearch.AccountRequestSearchManager.getEntityFromDocument(AccountRequestSearchManager.java:39)
	at teammates.storage.sqlsearch.AccountRequestSearchManager.getEntityFromDocument(AccountRequestSearchManager.java:19)
	at teammates.storage.sqlsearch.SearchManager.convertDocumentToEntities(SearchManager.java:220)
	at teammates.storage.sqlsearch.AccountRequestSearchManager.searchAccountRequests(AccountRequestSearchManager.java:56)
	at teammates.storage.sqlapi.AccountRequestsDb.searchAccountRequestsInWholeSystem(AccountRequestsDb.java:187)
	at teammates.sqllogic.core.AccountRequestsLogic.searchAccountRequestsInWholeSystem(AccountRequestsLogic.java:182)
	at teammates.sqllogic.api.Logic.searchAccountRequestsInWholeSystem(Logic.java:1712)
	at teammates.ui.webapi.SearchAccountRequestsAction.execute(SearchAccountRequestsAction.java:24)
	at teammates.ui.webapi.SearchAccountRequestsAction.execute(SearchAccountRequestsAction.java:16)
	at teammates.ui.webapi.BaseActionTest.getJsonResult(BaseActionTest.java:571)
	at teammates.ui.webapi.BaseActionTest.getJsonResult(BaseActionTest.java:561)
	at teammates.ui.webapi.SearchAccountRequestsActionTest.testExecute_searchName_shouldSucceed(SearchAccountRequestsActionTest.java:94)
	...

Might be related to this PR, but needs more investigation: #12950. Could also be a setup issue on the database or data bundle side.

Putting as p.High since we should really ensure all tests are passing...

@weiquu weiquu added p.High Significant impact; would like to do in the next few releases a-Testing Testing-related traits such as efficiency, robustness, coverage labels Sep 2, 2024
This was referenced Sep 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a-Testing Testing-related traits such as efficiency, robustness, coverage p.High Significant impact; would like to do in the next few releases
Projects
None yet
Development

No branches or pull requests

1 participant