Skip to content

Conversation

mpkorstanje
Copy link
Contributor

@mpkorstanje mpkorstanje commented Sep 4, 2020

@coveralls
Copy link

coveralls commented Sep 4, 2020

Coverage Status

Coverage decreased (-0.02%) to 86.139% when pulling c05c2ab on notify-text-context-manager-about-test-method into 47e27a7 on main.

@mpkorstanje
Copy link
Contributor Author

To use TextContextManager.afterTestExecution(instance, method, exception) we need access to the scenario result. We can't access this because the object factory and backend are is isolated from the the scenarios execution. So we'll have to redesign the redesign the backend module with call backs in mind.

@mpkorstanje mpkorstanje closed this Sep 4, 2020
@mpkorstanje mpkorstanje deleted the notify-text-context-manager-about-test-method branch September 4, 2020 20:55
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

Successfully merging this pull request may close these issues.

2 participants