Andal.LND/edx-platform
6
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

Merge pull request #8113 from edx/ahsan/ECOM-1302-Continue-Payment-button-not-accessible

Browse Source 
Continue to Payment button is not accessible
This commit is contained in:
Ahsan Ulhaq
2015-05-27 02:02:36 -04:00
parent 29c86032ec 1d5a310f5d
commit 0a43ebf191
2 changed files with 3 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
lms/static/js/spec/verify_student/make_payment_step_view_spec.js
View File

@@ -103,9 +103,10 @@ define([
expect($el.length).toEqual(_.size(buttons));
_.each(buttons, function( expectedText, expectedId ) {
var buttonEl = $( '#' + expectedId );
buttonEl.removeAttr('disabled');
expect( buttonEl.length ).toEqual( 1 );
expect( buttonEl[0] ).toHaveClass( 'payment-button' );
expect( buttonEl[0].text ).toEqual( expectedText );
expect( buttonEl[0] ).toHaveText( expectedText );
buttonEl[0].click();
expect(requests[requests.length - 1].requestBody.split('&')).toContain('processor=' + expectedId);
});

2
lms/static/js/verify_student/views/make_payment_step_view.js
View File

@@ -58,7 +58,7 @@ var edx = edx || {};
_getPaymentButtonHtml: function(processorName) {
var self = this;
return _.template(
'<a class="next action-primary payment-button" id="<%- name %>" tab-index="0"><%- text %></a> '
'<button class="next action-primary payment-button" id="<%- name %>" ><%- text %></button> '
)({name: processorName, text: self._getPaymentButtonText(processorName)});
},