Fix linting

Author: ehmicky

src/helpers/plugin.test.js

@@ -9,6 +9,7 @@ const validateContext = (context) => {
 }
 
 const addInstancesData = (info, originalInfo) =>
+  // eslint-disable-next-line fp/no-mutating-methods
   Object.defineProperty(
     info,
     'instancesData',

src/options/clone.js

@@ -32,6 +32,7 @@ const deepCloneObject = (object) => {
   for (const key of Reflect.ownKeys(object)) {
     const descriptor = Object.getOwnPropertyDescriptor(object, key)
     const childCopy = deepClone(descriptor.value)
+    // eslint-disable-next-line fp/no-mutating-methods
     Object.defineProperty(copy, key, { ...descriptor, value: childCopy })
   }
 

src/options/clone.test.js

@@ -14,6 +14,7 @@ each(ErrorSubclasses, ({ title }, ErrorClass) => {
 
   test(`Options can be non-enumerable | ${title}`, (t) => {
     const { options } = new ErrorClass('test', {
+      // eslint-disable-next-line fp/no-mutating-methods
       prop: Object.defineProperty({}, 'one', {
         value: true,
         enumerable: false,

src/plugins/info/main.js

@@ -46,6 +46,7 @@ export const getPluginInfo = ({
 // `instancesData` is internal, undocumented and non-enumerable.
 // It is only needed in very specific plugins like `modern-errors-serialize`
 const setInstancesData = (info) => {
+  // eslint-disable-next-line fp/no-mutating-methods
   Object.defineProperty(info, 'instancesData', {
     value: instancesData,
     enumerable: false,

src/plugins/info/main.test.js

@@ -23,6 +23,7 @@ each(
     test(`plugin.properties|instanceMethods|staticMethods cannot modify ErrorClasses | ${title}`, (t) => {
       const { ErrorClasses: ErrorClassesInfo } = getInfo(ErrorClass)
       const { length } = ErrorClassesInfo
+      // eslint-disable-next-line fp/no-mutating-methods
       ErrorClassesInfo.push(true)
       t.is(ErrorClass.getProp().ErrorClasses.length, length)
     })

src/plugins/properties/assign.test.js

@@ -47,6 +47,7 @@ each(ErrorSubclasses, ({ title }, ErrorClass) => {
 
   test(`plugin.properties() non-enumerable properties can be assigned | ${title}`, (t) => {
     const cause = new ErrorClass('test')
+    // eslint-disable-next-line fp/no-mutating-methods
     Object.defineProperty(cause, 'nonEnumProp', {
       value: true,
       enumerable: false,
@@ -86,6 +87,7 @@ each(ErrorSubclasses, ({ title }, ErrorClass) => {
 
   test(`plugin.properties() keeps descriptors | ${title}`, (t) => {
     const cause = new ErrorClass('test')
+    // eslint-disable-next-line fp/no-mutating-methods
     Object.defineProperty(cause, 'prop', {
       value: false,
       enumerable: false,

src/subclass/check.test.js

@@ -5,6 +5,7 @@ import { ErrorClasses } from '../helpers/main.test.js'
 
 class NullClass {}
 
+// eslint-disable-next-line fp/no-mutating-methods
 Object.setPrototypeOf(NullClass, null)
 
 each(
@@ -69,6 +70,7 @@ each(ErrorClasses, ['', null], ({ title }, ErrorClass, invalidPrototype) => {
     const custom = () => {}
     // eslint-disable-next-line fp/no-mutation
     custom.prototype = invalidPrototype
+    // eslint-disable-next-line fp/no-mutating-methods
     Object.setPrototypeOf(custom, ErrorClass)
     t.throws(ErrorClass.subclass.bind(undefined, 'TestError', { custom }))
   })

src/subclass/normalize.test.js

@@ -38,6 +38,7 @@ each(ErrorClasses, ({ title }, ErrorClass) => {
   test(`ErrorClass.normalize() normalizes known errors | ${title}`, (t) => {
     const error = new ErrorClass('test')
     const { name } = error
+    // eslint-disable-next-line fp/no-mutating-methods
     Object.defineProperty(error, 'name', {
       value: name,
       enumerable: true,

src/utils/descriptors.js

@@ -1,5 +1,6 @@
 // Most error core properties are not enumerable
 export const setNonEnumProp = (object, propName, value) => {
+  // eslint-disable-next-line fp/no-mutating-methods
   Object.defineProperty(object, propName, {
     value,
     enumerable: false,