(refactor): unify payments-stripe to use getStripeInstanceForShop, an…

…d pull out a STRIPE_PACKAGE_NAME constant

Signed-off-by: Alex Rattray <[email protected]>

imports/plugins/included/payments-stripe/client/settings/stripe.js

@@ -3,14 +3,15 @@ import { AutoForm } from "meteor/aldeed:autoform";
 import { Reaction, i18next } from "/client/api";
 import { Packages } from "/lib/collections";
 import { StripePackageConfig } from "../../lib/collections/schemas";
+import { STRIPE_PACKAGE_NAME } from "../../lib/constants";
 
 /**
  * @summary get Stripe Package record
  * @returns {Object} Package data
  */
 function packageData() {
   return Packages.findOne({
-    name: "reaction-stripe",
+    name: STRIPE_PACKAGE_NAME,
     shopId: Reaction.getShopId()
   });
 }

imports/plugins/included/payments-stripe/lib/constants.js

@@ -0,0 +1,2 @@
+
+export const STRIPE_PACKAGE_NAME = "reaction-stripe";

imports/plugins/included/payments-stripe/register.js

@@ -5,10 +5,11 @@ import stripeCapturePayment from "./server/no-meteor/util/stripeCapturePayment";
 import stripeCreateAuthorizedPayment from "./server/no-meteor/util/stripeCreateAuthorizedPayment";
 import stripeCreateRefund from "./server/no-meteor/util/stripeCreateRefund";
 import stripeListRefunds from "./server/no-meteor/util/stripeListRefunds";
+import { STRIPE_PACKAGE_NAME } from "./lib/constants";
 
 Reaction.registerPackage({
   label: "Stripe",
-  name: "reaction-stripe",
+  name: STRIPE_PACKAGE_NAME,
   icon: "fa fa-cc-stripe",
   autoEnable: true,
   graphQL: {

imports/plugins/included/payments-stripe/server/no-meteor/util/getStripeInstanceForShop.js

@@ -1,26 +1,27 @@
 import ReactionError from "@reactioncommerce/reaction-error";
+import Logger from "@reactioncommerce/logger";
 import getStripeInstance from "./getStripeInstance";
+import getStripePackageForShop from "./getStripePackageForShop";
 
-const PACKAGE_NAME = "reaction-stripe";
 
 /**
- * @summary Given a shop ID, gets the Stripe package data for that shop and an instance
- *   of the Stripe API configured with that shop's API key.
+ * @summary Given a shop ID, gets an instance of the Stripe API configured with that shop's API key.
  * @param {Object} context The context object, with `collections.Packages` on it
  * @param {String} shopId The shop ID
  * @returns {Object} The Stripe SDK object
  */
 export default async function getStripeInstanceForShop(context, shopId) {
-  const { collections } = context;
-  const { Packages } = collections;
+  const stripePkg = await getStripePackageForShop(context, shopId);
+  const stripePkgSettings = (stripePkg || {}).settings || {};
 
-  const stripePkg = await Packages.findOne({
-    name: PACKAGE_NAME,
-    shopId
-  });
-
-  const stripeApiKey = stripePkg && stripePkg.settings && stripePkg.settings.api_key;
+  const stripeApiKey = stripePkgSettings.api_key;
   if (!stripeApiKey) {
+    const stripeAccessToken = (stripePkgSettings.connectAuth || {}).access_token;
+    if (stripeAccessToken) {
+      Logger.warn("Using a Stripe access_token instead of an api_key is deprecated. Please set an API Key.");
+      return getStripeInstance(stripeAccessToken);
+    }
+
     throw new ReactionError("not-configured", "Stripe is not configured properly. Please set an API Key.");
   }
 

imports/plugins/included/payments-stripe/server/no-meteor/util/getStripePackageForShop.js

@@ -0,0 +1,16 @@
+import { PACKAGE_NAME } from "/imports/plugins/included/payments-stripe/lib/constants";
+
+/**
+ * @summary Given a shop ID, gets the Stripe package data for that shop.
+ * @param {Object} context The context object, with `collections.Packages` on it
+ * @param {String} shopId The shop ID
+ * @returns {Object} The Stripe package data.
+ */
+export default async function getStripePackageForShop(context, shopId) {
+  const { Packages } = context.collections;
+
+  return Packages.findOne({
+    name: PACKAGE_NAME,
+    shopId
+  });
+}

imports/plugins/included/payments-stripe/server/no-meteor/util/stripeCaptureCharge.js

@@ -1,7 +1,6 @@
 import Logger from "@reactioncommerce/logger";
 import formatForStripe from "./formatForStripe";
-import getStripeApiKey from "./getStripeApiKey";
-import getStripeInstance from "./getStripeInstance";
+import getStripeInstanceForShop from "./getStripeInstanceForShop";
 
 /**
  * @summary Capture the results of a previous charge
@@ -16,8 +15,7 @@ export default async function stripeCaptureCharge(context, payment) {
     amount: formatForStripe(payment.amount)
   };
 
-  const stripeKey = await getStripeApiKey(context, payment.paymentPluginName, payment.shopId);
-  const stripe = getStripeInstance(stripeKey);
+  const stripe = await getStripeInstanceForShop(context, payment.shopId);
 
   try {
     const captureResult = await stripe.charges.capture(payment.transactionId, captureDetails);

imports/plugins/included/payments-stripe/server/no-meteor/util/stripeCapturePayment.test.js

@@ -3,7 +3,7 @@ import nock from "nock";
 import mockContext from "/imports/test-utils/helpers/mockContext";
 import stripeCapturePayment from "./stripeCapturePayment";
 
-jest.mock("./getStripeApiKey", () => jest.fn().mockImplementation(() => "STRIPE_API_KEY"));
+jest.mock("./getStripeInstanceForShop", () => jest.fn().mockImplementation(() => require("stripe")("STRIPE_API_KEY")));
 
 const stripeCaptureResult = {
   id: "ch_17hZ4wBXXkbZQs3xL5JhlSgS",

imports/plugins/included/payments-stripe/server/no-meteor/util/stripeCreateAuthorizedPayment.js

@@ -1,8 +1,8 @@
 import Random from "@reactioncommerce/random";
+import { STRIPE_PACKAGE_NAME } from "../lib/constants";
 import getStripeInstanceForShop from "../util/getStripeInstanceForShop";
 
 const METHOD = "credit";
-const PACKAGE_NAME = "reaction-stripe";
 const PAYMENT_METHOD_NAME = "stripe_card";
 
 // NOTE: The "processor" value is lowercased and then prefixed to various payment Meteor method names,
@@ -95,7 +95,7 @@ export default async function stripeCreateAuthorizedPayment(context, input) {
     method: METHOD,
     mode: "authorize",
     name: PAYMENT_METHOD_NAME,
-    paymentPluginName: PACKAGE_NAME,
+    paymentPluginName: STRIPE_PACKAGE_NAME,
     processor: PROCESSOR,
     riskLevel: riskLevelMap[charge.outcome && charge.outcome.risk_level] || "normal",
     shopId,

imports/plugins/included/payments-stripe/server/no-meteor/util/stripeCreateRefund.js

@@ -1,6 +1,5 @@
 import Logger from "@reactioncommerce/logger";
-import getStripeApiKey from "./getStripeApiKey";
-import getStripeInstance from "./getStripeInstance";
+import getStripeInstanceForShop from "./getStripeInstanceForShop";
 import formatForStripe from "./formatForStripe";
 
 /**
@@ -16,8 +15,7 @@ import formatForStripe from "./formatForStripe";
 export default async function stripeCreateRefund(context, paymentMethod, amount) {
   let result;
   try {
-    const stripeKey = await getStripeApiKey(context, paymentMethod.paymentPluginName, paymentMethod.shopId);
-    const stripe = getStripeInstance(stripeKey);
+    const stripe = await getStripeInstanceForShop(context, paymentMethod.shopId);
 
     const refundResult = await stripe.refunds.create({ charge: paymentMethod.transactionId, amount: formatForStripe(amount) });
     Logger.debug(refundResult);

imports/plugins/included/payments-stripe/server/no-meteor/util/stripeCreateRefund.test.js

@@ -3,7 +3,7 @@ import nock from "nock";
 import mockContext from "/imports/test-utils/helpers/mockContext";
 import stripeCreateRefund from "./stripeCreateRefund";
 
-jest.mock("./getStripeApiKey", () => jest.fn().mockImplementation(() => "STRIPE_API_KEY"));
+jest.mock("./getStripeInstanceForShop", () => jest.fn().mockImplementation(() => require("stripe")("STRIPE_API_KEY")));
 
 test("should call StripeApi.methods.createRefund with the proper parameters and return saved = true", async () => {
   const paymentMethod = {

imports/plugins/included/payments-stripe/server/no-meteor/util/stripeListRefunds.js

@@ -1,6 +1,5 @@
 import Logger from "@reactioncommerce/logger";
-import getStripeApiKey from "./getStripeApiKey";
-import getStripeInstance from "./getStripeInstance";
+import getStripeInstanceForShop from "./getStripeInstanceForShop";
 
 /**
  * @name stripeListRefunds
@@ -12,8 +11,7 @@ import getStripeInstance from "./getStripeInstance";
  * @private
  */
 export default async function stripeListRefunds(context, paymentMethod) {
-  const stripeKey = await getStripeApiKey(context, paymentMethod.paymentPluginName, paymentMethod.shopId);
-  const stripe = getStripeInstance(stripeKey);
+  const stripe = await getStripeInstanceForShop(context, paymentMethod.shopId);
 
   let refundListResults;
   try {

imports/plugins/included/payments-stripe/server/no-meteor/util/stripeListRefunds.test.js

@@ -3,7 +3,7 @@ import nock from "nock";
 import mockContext from "/imports/test-utils/helpers/mockContext";
 import stripeListRefunds from "./stripeListRefunds";
 
-jest.mock("./getStripeApiKey", () => jest.fn().mockImplementation(() => "STRIPE_API_KEY"));
+jest.mock("./getStripeInstanceForShop", () => jest.fn().mockImplementation(() => require("stripe")("STRIPE_API_KEY")));
 
 test("should call StripeApi.methods.listRefunds with the proper parameters and return a properly" +
 "formatted list of refunds", async () => {