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

Engine Reload is not functional #11

Open
CharliePoole opened this issue May 4, 2020 · 5 comments
Open

Engine Reload is not functional #11

CharliePoole opened this issue May 4, 2020 · 5 comments
Assignees
Labels
Feature A new feature. High Priority High priority issue
Milestone

Comments

@CharliePoole
Copy link
Contributor

NUnit Compatibility Issue

Category: Functionality
Visibility: Users
Resolution: Resolved: We should implement this in one project and then port it to the other.

Description

Since the Console runner never used it, some function was left out of the engines' Reload API, which makes it unusable by the GUI. The GUI instead simply unloads and loads the assemblies. However, this isn't fully satisfactory because information displayed is lost in the process when a new package is created.

@ChrisMaddock
Copy link

Did you implement this in the TestCentric engine Charlie, or is it still to do? I'm aware you have an old PR open on the NUnit engine for this.

IMO, this is a TODO for the NUnit Engine, which has been a lower priority due to the lack of active users of this functionality.

@CharliePoole
Copy link
Contributor Author

Still a TODO for me. I compensated in various ways, but I'd rather use Reload from the engine.

@CharliePoole CharliePoole self-assigned this Dec 8, 2020
@CharliePoole
Copy link
Contributor Author

Based on prior discussion, marking this resolved as needing implementation in both projects.

@CharliePoole CharliePoole transferred this issue from TestCentric/testcentric-gui Oct 30, 2021
@CharliePoole
Copy link
Contributor Author

We are no longer trying to maintain NUnit compatibility but the reload functionality may still need some work.

@CharliePoole CharliePoole added Feature A new feature. High Priority High priority issue and removed NUnit Compatibility labels Aug 7, 2023
@CharliePoole CharliePoole added this to the 2.0.0-beta5 milestone Dec 28, 2023
@CharliePoole
Copy link
Contributor Author

It's actually unclear if this is still a problem so I'm dropping it from the beta5 release for now. We can test further with the GUI when it's using the beta5 engine.

@CharliePoole CharliePoole removed this from the 2.0.0-beta5 milestone Feb 29, 2024
@CharliePoole CharliePoole added this to the 2.0.0-beta7 milestone Oct 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature A new feature. High Priority High priority issue
Projects
None yet
Development

No branches or pull requests

2 participants