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

WIP - Remove logging - Don't merge #13203

Closed
wants to merge 2 commits into from
Closed

Conversation

danielb987
Copy link
Contributor

No description provided.

@danielb987 danielb987 added the New PR Newly created PR that is less than 1 day old label Jun 20, 2024
@danielb987 danielb987 self-assigned this Jun 20, 2024
Copy link

mergeable bot commented Jun 20, 2024

Thanks for the PR. Please consider adding a release note in the help/en/releasenotes/current-draft-note.shtml file.

@bobjacobsen
Copy link
Member

It still fails the same way with this branch.

If I disable the testTurnoutStatePersistencePy test in java/test/jmri/jmrit/jython/SampleScriptTest.java, it goes back to running OK. But I haven't been able to figure out that routine causes the test run to just stop silently.

Comment on lines +123 to +128
// Uncomment the line below and you will get the summary
// if (1==1) return;

InstanceManager.getDefault(ShutDownManager.class).shutdown();

if (1==1) return;
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@bobjacobsen
Note these lines. If you uncomment line 124, you will get the summary. So it seems to me that ShutDownManager.shutdown() is the culprit. But I don't know why. Maybe a mockup of the shutdown manager fails?

@danielb987
Copy link
Contributor Author

Closing this since it's not needed anymore. See #13204.

@danielb987 danielb987 closed this Jun 21, 2024
@danielb987 danielb987 deleted the TestScript branch June 21, 2024 01:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
New PR Newly created PR that is less than 1 day old
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants