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

IT tests failing after executor refactoring #298

Closed
enridaga opened this issue Aug 31, 2022 · 6 comments
Closed

IT tests failing after executor refactoring #298

enridaga opened this issue Aug 31, 2022 · 6 comments

Comments

@enridaga
Copy link
Member

As the summary says

@enridaga
Copy link
Member Author

Apparently, after using the ondisk option, subsequent tests are failing. I isolated testIssue280 and testIssue284. 284 runs smoothly if executed alone. The test class we are focusing on now is IssuesTest2

@luigi-asprino
Copy link
Member

the BaseFacadeXGraphBuilder uses static fields

protected static Dataset dataset = null; // TODO making this static is a kludge maybe

maybe the problem depends on this?

@luigi-asprino
Copy link
Member

This

if(BaseFacadeXGraphBuilder.previousTDB2Path != "" && ONDISK_REUSE){

Shouldn't it be !BaseFacadeXGraphBuilder.previousTDB2Path.equals("")?

@luigi-asprino
Copy link
Member

This 8c6de26 removes the static fields from the class BaseFxGraphBuilder.
Tests in IssuesTest2 don't fail anymore

@luigi-asprino
Copy link
Member

We can close this issue if there are no other tests failing.

@luigi-asprino
Copy link
Member

I close this issue as it is completed via f1355c0

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

No branches or pull requests

2 participants