Resolve "Follow-up from Add docker support for interactive web terminal"
requested to merge 3639-follow-up-from-resolve-add-docker-support-for-interactive-web-terminal into master
What does this MR do?
Follow up for !1008 (merged) discussions
Why was this MR needed?
- There is a test that was not needed
- Missing assertions
Are there points in the code the reviewer needs to double check?
Does this MR meet the acceptance criteria?
-
Documentation created/updated -
Added tests for this feature/bug -
In case of conflicts with master
- branch was rebased
What are the relevant issue numbers?
Closes #3639 (closed)
Edited by Steve Xuereb