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

Unified Diff: test/test_browser_charm_details.js

Issue 8268043: Render the configuration tab in charm details.
Patch Set: Render the configuration tab in charm details. Created 11 years, 12 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
« no previous file with comments | « app/subapps/browser/views/charm.js ('k') | test/test_model.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: test/test_browser_charm_details.js
=== modified file 'test/test_browser_charm_details.js'
--- test/test_browser_charm_details.js 2013-04-02 14:38:12 +0000
+++ test/test_browser_charm_details.js 2013-04-02 17:24:31 +0000
@@ -204,6 +204,28 @@
'<h1>README Header</h1>');
});
+ it('should display the config data in the config tab', function() {
+ var view = new CharmView({
+ charm: new models.BrowserCharm({
+ files: [],
+ id: 'precise/ceph-9',
+ options: {
+ 'client-port': {
+ 'default': 9160,
+ 'description': 'Port for client communcation',
+ 'type': 'int'
+ }
+ }
+ })
+ });
+ view.render(node);
+
+ var dds = Y.all('#bws_configuration dd');
+ dds.size().should.eql(2);
+ dds.pop().get('text').should.eql('Default: 9160');
+ dds.pop().get('text').should.eql('Port for client communcation');
+ });
+
it('_buildQAData properly summerizes the scores', function() {
var view = new CharmView({
charm: new models.BrowserCharm({
« no previous file with comments | « app/subapps/browser/views/charm.js ('k') | test/test_model.js » ('j') | no next file with comments »

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