feat(payments-next): use LoggerService within TypeCachable decorators

Because:

* The LoggerService provides much more useful data for debugging issues, and was not being used in the TypeCachable decorator

This commit:

* Adds the requirement for the LoggerService to be present on the consuming classes of the TypeCachable decorators
* Moves the cacheManager excludeContext call out of the strapi client and into the type-cachable barrel file, as it's now needed in several places
* Updates the consuming classes to pass the LoggerService through to the decorator

Closes #PAY-3571

Author: david1alvarez

libs/payments/cart/src/lib/tax.service.spec.ts

@@ -1,5 +1,6 @@
 import { faker } from '@faker-js/faker';
 import { Test } from '@nestjs/testing';
+import { MockLoggerProvider } from '@fxa/shared/log';
 
 import {
   AccountCustomerManager,
@@ -50,6 +51,7 @@ describe('TaxService', () => {
         MockStatsDProvider,
         MockStripeConfigProvider,
         StripeClient,
+        MockLoggerProvider,
         MockAccountDatabaseNestFactory,
         AccountCustomerManager,
         CustomerManager,
@@ -139,7 +141,7 @@ describe('TaxService', () => {
 
       jest
         .spyOn(accountCustomerManager, 'getAccountCustomerByUid')
-        .mockRejectedValue(new AccountCustomerNotFoundError('not found'));
+        .mockRejectedValue(new AccountCustomerNotFoundError('not found', new Error()));
       jest.spyOn(geodbManager, 'getTaxAddress').mockReturnValue(geoAddress);
 
       const result = await taxService.getTaxAddress(ip, uid);

libs/payments/customer/src/lib/customer.manager.spec.ts

@@ -4,6 +4,7 @@
 
 import { faker } from '@faker-js/faker';
 import { Test } from '@nestjs/testing';
+import { MockLoggerProvider } from '@fxa/shared/log';
 
 import {
   StripeClient,
@@ -28,6 +29,7 @@ describe('CustomerManager', () => {
         StripeClient,
         CustomerManager,
         MockStatsDProvider,
+        MockLoggerProvider,
       ],
     }).compile();
 

libs/payments/customer/src/lib/customerSession.manager.spec.ts

@@ -3,6 +3,7 @@
  * file, You can obtain one at http://mozilla.org/MPL/2.0/. */
 
 import { Test } from '@nestjs/testing';
+import { MockLoggerProvider } from '@fxa/shared/log';
 
 import { CustomerSessionManager } from './customerSession.manager';
 import {
@@ -24,6 +25,7 @@ describe('CustomerSessionManager', () => {
         CustomerSessionManager,
         StripeClient,
         MockStatsDProvider,
+        MockLoggerProvider,
       ],
     }).compile();
 

libs/payments/customer/src/lib/invoice.manager.spec.ts

@@ -3,6 +3,7 @@
  * file, You can obtain one at http://mozilla.org/MPL/2.0/. */
 
 import { Test } from '@nestjs/testing';
+import { MockLoggerProvider } from '@fxa/shared/log';
 
 import {
   StripeClient,
@@ -63,6 +64,7 @@ describe('InvoiceManager', () => {
         MockStripeConfigProvider,
         InvoiceManager,
         MockStatsDProvider,
+        MockLoggerProvider,
       ],
     }).compile();
 

libs/payments/customer/src/lib/paymentIntent.manager.spec.ts

@@ -3,6 +3,7 @@
  * file, You can obtain one at http://mozilla.org/MPL/2.0/. */
 
 import { Test } from '@nestjs/testing';
+import { MockLoggerProvider } from '@fxa/shared/log';
 
 import { PaymentIntentManager } from './paymentIntent.manager';
 import {
@@ -24,6 +25,7 @@ describe('PaymentIntentManager', () => {
         PaymentIntentManager,
         StripeClient,
         MockStatsDProvider,
+        MockLoggerProvider,
       ],
     }).compile();
 

libs/payments/customer/src/lib/paymentMethod.manager.spec.ts

@@ -4,6 +4,7 @@
 
 import { faker } from '@faker-js/faker';
 import { Test } from '@nestjs/testing';
+import { MockLoggerProvider } from '@fxa/shared/log';
 import { PaymentMethodManager } from './paymentMethod.manager';
 import {
   MockPaypalClientConfigProvider,
@@ -44,6 +45,7 @@ describe('PaymentMethodManager', () => {
         PaypalCustomerManager,
         StripeClient,
         MockStatsDProvider,
+        MockLoggerProvider,
       ],
     }).compile();
 

libs/payments/customer/src/lib/price.manager.spec.ts

@@ -3,6 +3,7 @@
  * file, You can obtain one at http://mozilla.org/MPL/2.0/. */
 
 import { Test } from '@nestjs/testing';
+import { MockLoggerProvider } from '@fxa/shared/log';
 
 import {
   StripeClient,
@@ -29,6 +30,7 @@ describe('PriceManager', () => {
         StripeClient,
         PriceManager,
         MockStatsDProvider,
+        MockLoggerProvider,
       ],
     }).compile();
 

libs/payments/customer/src/lib/product.manager.spec.ts

@@ -3,6 +3,7 @@
  * file, You can obtain one at http://mozilla.org/MPL/2.0/. */
 
 import { Test } from '@nestjs/testing';
+import { MockLoggerProvider } from '@fxa/shared/log';
 
 import {
   StripeClient,
@@ -24,6 +25,7 @@ describe('ProductManager', () => {
         StripeClient,
         ProductManager,
         MockStatsDProvider,
+        MockLoggerProvider,
       ],
     }).compile();
 

libs/payments/customer/src/lib/promotionCode.manager.spec.ts

@@ -4,6 +4,7 @@
 
 import { faker } from '@faker-js/faker';
 import { Test } from '@nestjs/testing';
+import { MockLoggerProvider } from '@fxa/shared/log';
 import { Stripe } from 'stripe';
 
 import { ProductManager } from './product.manager';
@@ -71,6 +72,7 @@ describe('PromotionCodeManager', () => {
         PromotionCodeManager,
         StripeClient,
         SubscriptionManager,
+        MockLoggerProvider,
       ],
     }).compile();
 

libs/payments/customer/src/lib/setupIntent.manager.spec.ts

@@ -3,6 +3,7 @@
  * file, You can obtain one at http://mozilla.org/MPL/2.0/. */
 
 import { Test } from '@nestjs/testing';
+import { MockLoggerProvider } from '@fxa/shared/log';
 
 import { SetupIntentManager } from './setupIntent.manager';
 import {
@@ -25,6 +26,7 @@ describe('SetupIntentManager', () => {
         SetupIntentManager,
         StripeClient,
         MockStatsDProvider,
+        MockLoggerProvider,
       ],
     }).compile();