Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(425)

Unified Diff: test/test_endpoints.js

Issue 13775044: Remove the serviceInspector feature flag.
Patch Set: Remove the serviceInspector feature flag. Created 11 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Please Sign in to add in-line comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: test/test_endpoints.js
=== modified file 'test/test_endpoints.js'
--- test/test_endpoints.js 2013-09-17 13:27:43 +0000
+++ test/test_endpoints.js 2013-09-18 14:38:50 +0000
@@ -343,12 +343,13 @@
conn = new utils.SocketStub();
env = juju.newEnvironment({conn: conn});
env.connect();
+ destroyMe.push(env);
rharding 2013/09/19 15:06:28 the env wasn't getting destroyed. This led to the
jeff.pihach 2013/09/19 15:36:06 Nice find
gary.poster 2013/09/19 15:51:29 +1
app = new Y.juju.App({env: env, consoleEnabled: true});
+ app.showView(new Y.View());
destroyMe.push(app);
- app.showView(new Y.View());
controller = app.endpointsController;
+ controller.endpointsMap = {};
destroyMe.push(controller);
- controller.endpointsMap = {};
});
afterEach(function() {

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b