updated automated tests to cater for the new page-width setting

readium · Mar 8, 2016 · cd8d330 · cd8d330
1 parent ba08b99
commit cd8d330
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 4 deletions.
diff --git a/src/js/ReaderSettingsDialog.js b/src/js/ReaderSettingsDialog.js
@@ -1,5 +1,6 @@
 define(['./ModuleConfig', 'hgn!readium_js_viewer_html_templates/settings-dialog.html', './ReaderSettingsDialog_Keyboard', 'i18nStrings', './Dialogs', 'Settings', './Keyboard'], function(moduleConfig, SettingsDialog, KeyboardSettings, Strings, Dialogs, Settings, Keyboard){
 
+    // change these values to affec the default state of the application's preferences at first-run.
     var defaultSettings = {
         fontSize: 100,
         syntheticSpread: "auto",

diff --git a/tests/tests.js b/tests/tests.js
@@ -291,17 +291,30 @@ console.log(config.browser.browserName);
 
     });
 
-    it('test margins', function(){
+    // it('test margins', function(){
+    //   return browser
+    //           .openSettingsDialog()
+    //           .elementByCss('#tab-butt-layout')
+    //           .click()
+    //           .elementByCss('#margin-size-input')
+    //           .isDisplayed()
+    //           .should.eventually.be.true
+    //           .elementByCss('#margin-size-input')
+    //           .getValue()
+    //           .should.become('60');
+
+    // });
+    it('test page width', function(){
       return browser
               .openSettingsDialog()
               .elementByCss('#tab-butt-layout')
               .click()
-              .elementByCss('#margin-size-input')
+              .elementByCss('#column-max-width-input')
               .isDisplayed()
               .should.eventually.be.true
-              .elementByCss('#margin-size-input')
+              .elementByCss('#column-max-width-input')
               .getValue()
-              .should.become('60');
+              .should.become('550');
 
     });