Skip to content

Commit 203749c

Browse files
committed
Update test to expect a deferral instead of undefined
1 parent 65740bc commit 203749c

File tree

1 file changed

+6
-4
lines changed
  • dev/tests/js/jasmine/tests/app/code/Magento/Checkout/frontend/js/model/cart/totals-processor

1 file changed

+6
-4
lines changed

dev/tests/js/jasmine/tests/app/code/Magento/Checkout/frontend/js/model/cart/totals-processor/default.test.js

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -119,14 +119,15 @@ define([
119119
'data_id': 1
120120
})
121121
);
122+
var deferral = new $.Deferred();
122123
spyOn(mocks['Magento_Checkout/js/model/cart/cache'], 'get');
123124
spyOn(mocks['mage/storage'], 'post').and.callFake(function () {
124125
data.shippingMethodCode = mocks['Magento_Checkout/js/model/quote'].shippingMethod()['method_code'];
125126
data.shippingCarrierCode = mocks['Magento_Checkout/js/model/quote'].shippingMethod()['carrier_code'];
126127

127-
return new $.Deferred().resolve(result);
128+
return deferral.resolve(result);
128129
});
129-
expect(defaultProcessor.estimateTotals(address)).toBeUndefined();
130+
expect(defaultProcessor.estimateTotals(address)).toBe(deferral);
130131
expect(mocks['Magento_Checkout/js/model/quote'].setTotals).toHaveBeenCalledWith(totals);
131132
expect(mocks['Magento_Checkout/js/model/totals'].isLoading.calls.argsFor(0)[0]).toBe(true);
132133
expect(mocks['Magento_Checkout/js/model/totals'].isLoading.calls.argsFor(1)[0]).toBe(false);
@@ -143,10 +144,11 @@ define([
143144
})
144145
);
145146
spyOn(mocks['Magento_Checkout/js/model/cart/cache'], 'get');
147+
var deferral = new $.Deferred();
146148
spyOn(mocks['mage/storage'], 'post').and.callFake(function () {
147-
return new $.Deferred().reject('Error Message');
149+
return deferral.reject('Error Message');
148150
});
149-
expect(defaultProcessor.estimateTotals(address)).toBeUndefined();
151+
expect(defaultProcessor.estimateTotals(address)).toBe(deferral);
150152
expect(mocks['Magento_Checkout/js/model/totals'].isLoading.calls.argsFor(0)[0]).toBe(true);
151153
expect(mocks['Magento_Checkout/js/model/totals'].isLoading.calls.argsFor(1)[0]).toBe(false);
152154
expect(mocks['mage/storage'].post).toHaveBeenCalled();

0 commit comments

Comments
 (0)