The fact that this new shell (created after lockShells was called) should be locked is non-obvious and untested, so you might want to add a test that shows that behavior. Also, this variable is never used (as I understand, it's created so that there exists more than one shell?) so you might want to throw in a quick comment explaining why it's here.