fix: move this.displayCodeHighlights to CanvasDrawer

Author: aminya

lib/minimap-element.js

@@ -137,10 +137,7 @@ class MinimapElement {
     * @access private
     */
     this.textOpacity = undefined
-    /**
-    * @access private
-    */
-    this.displayCodeHighlights = undefined
+
     /**
     * @access private
     */
@@ -262,7 +259,7 @@ class MinimapElement {
       }),
 
       atom.config.observe('minimap.displayCodeHighlights', (displayCodeHighlights) => {
-        this.displayCodeHighlights = displayCodeHighlights
+        this.CanvasDrawer.displayCodeHighlights = displayCodeHighlights
 
         if (this.attached) { this.requestForcedUpdate() }
       }),
@@ -889,7 +886,7 @@ class MinimapElement {
    *                                        highlights or not
    */
   setDisplayCodeHighlights (displayCodeHighlights) {
-    this.displayCodeHighlights = displayCodeHighlights
+    this.CanvasDrawer.displayCodeHighlights = displayCodeHighlights
     if (this.attached) { this.requestForcedUpdate() }
   }
 

lib/mixins/canvas-drawer.js

@@ -21,6 +21,8 @@ export default class CanvasDrawer {
   constructor() {
     // set in setModel of minimapElement
     this.minimap = undefined
+    // set by minimapElement
+    this.displayCodeHighlights = undefined
   }
   /**
    * Initializes the canvas elements needed to perform the `Minimap` rendering.

spec/minimap-element-spec.js

@@ -1827,7 +1827,7 @@ describe('MinimapElement', () => {
           })
 
           it('toggles the code highlights on the minimap element', () => {
-            expect(minimapElement.displayCodeHighlights).toBeTruthy()
+            expect(minimapElement.CanvasDrawer.displayCodeHighlights).toBeTruthy()
           })
 
           it('requests an update', () => {
@@ -1996,14 +1996,14 @@ describe('MinimapElement', () => {
           describe('on the code highlight item', () => {
             let [initial] = []
             beforeEach(() => {
-              initial = minimapElement.displayCodeHighlights
+              initial = minimapElement.CanvasDrawer.displayCodeHighlights
               atom.commands.dispatch(quickSettingsElement, 'core:move-down')
               atom.commands.dispatch(quickSettingsElement, 'core:move-down')
               atom.commands.dispatch(quickSettingsElement, 'core:confirm')
             })
 
             it('toggles the code highlights on the minimap element', () => {
-              expect(minimapElement.displayCodeHighlights).toEqual(!initial)
+              expect(minimapElement.CanvasDrawer.displayCodeHighlights).toEqual(!initial)
             })
           })