fix: issue with LoadConfig getting deprecated

There was an issue where deprecated rules where not being returned like
they where suppose to.

test/findUnused.js

@@ -18,6 +18,6 @@ describe('loadConfig', () => {
 
   it('finds unused rules that are deprecated', () => {
     const unusedRules = findUnused(config, true)
-    assert.equal(unusedRules.size, 264)
+    assert.equal(unusedRules.size, 263)
   })
 })

test/fixtures/.eslintrc.js

@@ -1,6 +1,7 @@
 module.exports = {
   rules: {
     semi: ['error', 'never'],
-    curly: 'error'
+    curly: 'error',
+    'valid-jsdoc': 'off'
   }
 }

test/load-config.js

@@ -25,7 +25,14 @@ describe('Load Config', () => {
   it('should have only currentRules', () => {
     const { currentRules } = config
 
-    assert.hasAllKeys(currentRules, ['curly', 'semi'])
-    assert.equal(currentRules.size, 2)
+    assert.hasAllKeys(currentRules, ['curly', 'semi', 'valid-jsdoc'])
+    assert.equal(currentRules.size, 3)
+  })
+
+  it('should return deprecated rules', () => {
+    const { deprecated } = config
+
+    assert.equal(deprecated.size, 1)
+    assert.equal(deprecated.has('valid-jsdoc'), true)
   })
 })

utils/load-config.js

@@ -30,16 +30,18 @@ class LoadConfig {
 
     this.currentRules = new Map()
 
-    Object.keys(rules).forEach((rule) => {
+    Object.keys(rules).forEach(rule => {
       this.currentRules.set(rule, rules[rule])
     })
   }
 
   get deprecated() {
     const deprecatedRules = new Map()
 
-    this.currentRules.forEach((definition, rule) => {
-      if (rule.meta && rule.meta.deprecated) {
+    this.currentRules.forEach((value, rule) => {
+      const definition = this.rules.get(rule)
+
+      if (definition && definition.meta && definition.meta.deprecated) {
         deprecatedRules.set(rule, definition)
       }
     })