Revert "Chore: Temporarily allow draw annotations to be enabled via c…

…onfig (#416)" (#434)
box · Oct 13, 2017 · 09c279d · 09c279d
1 parent 86072bd
commit 09c279d
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 25 deletions.
diff --git a/src/lib/annotations/BoxAnnotations.js b/src/lib/annotations/BoxAnnotations.js
@@ -9,8 +9,7 @@ const ANNOTATORS = [
         NAME: 'Document',
         CONSTRUCTOR: DocAnnotator,
         VIEWER: ['Document', 'Presentation'],
-        // Temporarily include draw annotations for demo purposes
-        TYPE: [TYPES.point, TYPES.highlight, TYPES.highlight_comment, TYPES.draw]
+        TYPE: [TYPES.point, TYPES.highlight, TYPES.highlight_comment]
     },
     {
         NAME: 'Image',

diff --git a/src/lib/annotations/__tests__/BoxAnnotations-test.js b/src/lib/annotations/__tests__/BoxAnnotations-test.js
@@ -131,29 +131,6 @@ describe('lib/annotators/BoxAnnotations', () => {
             });
             expect(stubs.instantiateControllers).to.be.called;
         });
-
-        // Temporary test for demo purposes
-        it('should remove draw annotations unless explicitly enabled', () => {
-            let config = {
-                enabled: true,
-                drawEnabled: true,
-                disabledTypes: ['point']
-            };
-            const docAnnotator = {
-                NAME: 'Document',
-                VIEWER: ['Document'],
-                TYPE: ['point', 'highlight', 'draw']
-            };
-
-            loader.annotators = [docAnnotator];
-
-            annotator = loader.determineAnnotator('Document', {}, config);
-            expect(annotator.TYPE.includes('draw')).to.be.true;
-
-            config.drawEnabled = undefined;
-            let annotator = loader.determineAnnotator('Document', {}, config);
-            expect(annotator.TYPE.includes('draw')).to.be.false;
-        });
     });
 
     describe('instantiateControllers()', () => {