Start extracting service policy boilerplate to common function

We would like to use the Cockatiel library in our service classes to
ensure that requests are retried using the circuit breaker pattern. Some
of our service classes do this already, but we are copying and pasting
the code around. This commit extracts some of the boilerplate code to a
new function in the `@metamask/controller-utils` package,
`createServicePolicy` so that we no longer have to do this. Because it
would require an enormous amount of tests, this commit does not include
a complete version of `createServicePolicy`, notably omitting
`maxRetries` option as well as a way to handle slow requests; those will
be added later.

packages/controller-utils/CHANGELOG.md

@@ -7,6 +7,10 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+### Added
+
+- Add `createServicePolicy` function to assist with reducing boilerplate for service classes ([#5053](https://github.com/MetaMask/core/pull/5053))
+
 ## [11.4.5]
 
 ### Changed

packages/controller-utils/jest.config.js

@@ -18,7 +18,7 @@ module.exports = merge(baseConfig, {
   coverageThreshold: {
     global: {
       branches: 78.12,
-      functions: 85.41,
+      functions: 84.61,
       lines: 87.3,
       statements: 86.5,
     },

packages/controller-utils/package.json

@@ -55,6 +55,7 @@
     "@types/bn.js": "^5.1.5",
     "bignumber.js": "^9.1.2",
     "bn.js": "^5.2.1",
+    "cockatiel": "^3.1.2",
     "eth-ens-namehash": "^2.0.8",
     "fast-deep-equal": "^3.1.3"
   },
@@ -65,6 +66,7 @@
     "deepmerge": "^4.2.2",
     "jest": "^27.5.1",
     "nock": "^13.3.1",
+    "sinon": "^9.2.4",
     "ts-jest": "^27.1.4",
     "typedoc": "^0.24.8",
     "typedoc-plugin-missing-exports": "^2.0.0",