-
Notifications
You must be signed in to change notification settings - Fork 35
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
Fix tests broken in test_vlsir
after #185
#187
Comments
The problem happens only if running all the tests, not in isolation. Maybe some other test modifes the pdk? |
This seems to stem from the layer |
I think the problem comes from the meow plugin adding layers to the pdk that doon't have corresponding LayerViews. gplugins/gplugins/eme/meow_eme.py Lines 221 to 227 in a019e5a
Maybe a reasonable fix is to isolate the meow tests or somehow reload the pdk after running meow. |
i made a copy of the layers_stack inside the MEOW plugin, so tests are robust now and don't couple to each other through the layer_stack |
Describe the bug
The test in
gplugins/vlsir/tests/test_vlsir.py
fail withafter merging #185.
To Reproduce
Expected behavior
Succeed
Suggested fix
Maybe need to accept layers that don't exist in the active Pdk?
The text was updated successfully, but these errors were encountered: