-
Notifications
You must be signed in to change notification settings - Fork 15
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Pricing tests #24
Pricing tests #24
Conversation
f076f3e
to
6b372ed
Compare
6b372ed
to
c3089c9
Compare
545f49d
to
24869db
Compare
8ab5115
to
635d184
Compare
Pull Request Test Coverage Report for Build e94e558897e70d953ee086f40a93ab8318ac0190-PR-24
💛 - Coveralls |
e23cbda
to
c6a66b3
Compare
035d6fb
to
2771a76
Compare
724f7bc
to
bf8cc6b
Compare
test/use-checkout-pricing.test.js
Outdated
cb(); | ||
return checkoutPricingReturn; | ||
}), | ||
giftcard: jest.fn(() => checkoutPricingReturn), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this needs to be giftCard
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The method right now is giftcard
coming back from PRICING_METHODS, I know there's a PR to deprecate this in favor of giftCard
, should we switch to use this method instead? recurly/recurly-js#580
bf8cc6b
to
e459e32
Compare
e459e32
to
d55f53f
Compare
These tests only cover the
subscriptions
interface insideuseCheckoutPricing
and base functionality.After this is merged, I'll open up additional PRs to cover the rest of the interface.
Totally open to feedback and willing to change anything.