Skip to content

Commit

Permalink
fix(tests): comment out skipped tests
Browse files Browse the repository at this point in the history
This commit works around an apparent bug in mocha version 3.x that
throws function name errors on test that are skipped.  The skipped tests
have been commented out and will be re-enabled as soon as mocha gets
around to providing a solution.
  • Loading branch information
Jonathan Niles committed Aug 1, 2016
1 parent 6a575e6 commit f2f3118
Show file tree
Hide file tree
Showing 4 changed files with 8 additions and 4 deletions.
2 changes: 1 addition & 1 deletion .travis.yml
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ before_install:
- export CHROME_BIN=/usr/bin/google-chrome
- sudo apt-get update -qq
- sudo apt-get install -qq mysql-server-5.6 mysql-client-5.6 mysql-client-core-5.6
- sudo apt-get install wkhtmltopdf
- sudo apt-get install -qq wkhtmltopdf
- sh -e /etc/init.d/xvfb start
- wget https://dl.google.com/linux/direct/google-chrome-stable_current_amd64.deb
- sudo dpkg -i google-chrome*.deb
Expand Down
2 changes: 2 additions & 0 deletions client/test/e2e/cash/cash.spec.js
Original file line number Diff line number Diff line change
Expand Up @@ -145,6 +145,7 @@ describe('Cash Payments Module', function () {
});

/** @todo - once invoice posting is figured out, this test should be uncommented and work */
/*
it.skip('should make a payment against previous invoices', function () {
var gridId = 'debtorInvoicesGrid';
Expand Down Expand Up @@ -187,6 +188,7 @@ describe('Cash Payments Module', function () {
// dismiss the modal
element(by.css('[data-modal-action="dismiss"]')).click();
});
*/
});

describe('Cash Transfer ', function (){
Expand Down
2 changes: 1 addition & 1 deletion client/test/e2e/patient/invoice/invoice.spec.js
Original file line number Diff line number Diff line change
Expand Up @@ -114,5 +114,5 @@ describe('Patient Invoice', function () {
components.notification.hasDanger();
});

it('can calculate totals correctly');
//it('can calculate totals correctly');
});
6 changes: 4 additions & 2 deletions client/test/e2e/permissions/permissions.spec.js
Original file line number Diff line number Diff line change
Expand Up @@ -64,7 +64,8 @@ describe('Permissions Module', function () {
FU.validation.error('PermissionsCtrl.user.passwordVerify');
});

it.skip('edits the previously created user', function (done) {
/*
it('edits the previously created user', function (done) {
// use the UI grid to select the previously created user
//gridUtils.
Expand All @@ -84,5 +85,6 @@ describe('Permissions Module', function () {
// submit the user
element(by.id('submitCreate')).click();
});
});
*/
});

0 comments on commit f2f3118

Please sign in to comment.