Skip to content

Commit a913eb7

Browse files
petricargPetrica Chiriac
and
Petrica Chiriac
authored
GitHub nodejs issues #15 (#16)
* Fix error handling in performRebillUpdate() issues #15 * Change VERSION_NUMBER to K2.5 * Update expireYear to 2025 --------- Co-authored-by: Petrica Chiriac <petrica@localhost>
1 parent c1dd418 commit a913eb7

File tree

4 files changed

+24
-5
lines changed

4 files changed

+24
-5
lines changed

config/rgProperties.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@
1010
"LIVE_HOST_16": "gateway-16.rocketgate.com",
1111
"LIVE_HOST_17": "gateway-17.rocketgate.com",
1212
"TEST_HOST": "dev-gateway.rocketgate.com",
13-
"VERSION_NUMBER": "K2.4"
13+
"VERSION_NUMBER": "K2.5"
1414
},
1515
"responseSettings": {
1616
"VERSION_INDICATOR": "version",

lib/gatewayService.js

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -642,13 +642,14 @@ function GatewayService(options) {
642642
callback(results, request, response);
643643
});
644644
} else {
645-
performTransaction(request, response, function(results) {
645+
performTransaction(request, response, function(results, request, response) {
646646
if (results) {
647647
performConfirmation(request, response, function(results, request, response) {
648648
callback(results, request, response);
649649
});
650+
} else {
651+
callback(results, request, response);
650652
}
651-
callback(results, request, response);
652653
});
653654
}
654655
}

test/unit/buildPaymentLink.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ describe('perform auth only', function() {
1616
request.amount = "9.99";
1717
request.cardNo = "4111-1111-1111-1111";
1818
request.expireMonth = "02";
19-
request.expireYear = "2025";
19+
request.expireYear = "2029";
2020
request.cvv2 = "999";
2121
request.customerFirstName = "Joe";
2222
request.customerLastName = "JSTester";

test/unit/performRebillUpdate.js

Lines changed: 19 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@ describe('perform rebill update', function() {
3333
request.amount = "5.55";
3434
request.cardNo = "4111-1111-1111-1111";
3535
request.expireMonth = "02";
36-
request.expireYear = "2020";
36+
request.expireYear = "2029";
3737
request.cvv2 = "999";
3838

3939
service.setTestMode(true, function(err, result) {
@@ -98,6 +98,24 @@ describe('perform rebill update', function() {
9898
request.username = "node_added_username";
9999
request.customerPassword = "node_added_password";
100100

101+
service.performRebillUpdate(request, {}, function(results, request, response) {
102+
results.should.equal(true);
103+
response[responseSettings.RESPONSE_CODE].should.equal('0');
104+
done();
105+
});
106+
});
107+
});
108+
it ('should perform rebill update with amount', function (done) {
109+
service.performPurchase(request, {}, function(results, request, response) {
110+
// perform purchase then update
111+
request = new Request();
112+
request.merchantID = 1;
113+
request.merchantPassword = "testpassword";
114+
request.merchantCustomerID = time + ".JSTest";
115+
request.merchantInvoiceID = time + ".Test";
116+
request.amount = "4.55";
117+
request.rebillFrequency = 'MONTHLY';
118+
101119
service.performRebillUpdate(request, {}, function(results, request, response) {
102120
results.should.equal(true);
103121
response[responseSettings.RESPONSE_CODE].should.equal('0');

0 commit comments

Comments
 (0)