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

Lots of exceptions when a class is renamed #91

Open
guillep opened this issue Oct 2, 2018 · 2 comments
Open

Lots of exceptions when a class is renamed #91

guillep opened this issue Oct 2, 2018 · 2 comments

Comments

@guillep
Copy link

guillep commented Oct 2, 2018

I've renamed a class and suddenly I have a lot of exceptions

image

Version 0.3-alpha on Pharo7

@badetitou
Copy link
Owner

Oh... Ok
I know what it is.
SmartTest was doing a research of relative tests for a method (in the background)
And you modified a method during the SmartTest execution.

3 options.

  1. Desactivate SmT (but it's not the goal of having it)
  2. Making SmT faster
  3. Listen for the announcement of class rename (method rename etc.) and when it happens, stopping the current research

@guillep
Copy link
Author

guillep commented Oct 3, 2018

You're almost there. It's a concurrency problem. Your processing happens in a separate thread/process, so that means you have to ensure that your second process either:

  • handles graciously the errors
  • is programmed so this kind of error could not happen

As long as SmT uses concurrency, you will find this kind of problems.

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