fix(Popover): remove old arrow markup, correct enabled className

eddywashere · eddywashere · commit b9d3ea7574e4 · 2016-11-28T00:09:24.000-08:00
diff --git a/src/Popover.js b/src/Popover.js
@@ -23,7 +23,10 @@ const defaultProps = {
 
 const defaultTetherConfig = {
   classPrefix: 'bs-tether',
-  classes: { element: 'popover', enabled: 'open' },
+  classes: {
+    element: false,
+    enabled: 'in'
+  },
   constraints: [
     { to: 'scrollParent', attachment: 'together none' },
     { to: 'window', attachment: 'together none' }
@@ -63,7 +66,7 @@ class Popover extends React.Component {
 
     return (
       <TetherContent
-        arrow="popover"
+        className="popover"
         tether={tetherConfig}
         tetherRef={this.props.tetherRef}
         isOpen={this.props.isOpen}
diff --git a/src/__tests__/Popover.spec.js b/src/__tests__/Popover.spec.js
@@ -68,6 +68,7 @@ describe('Popover', () => {
     expect(isOpen).toBe(false);
 
     expect(wrapper.find(TetherContent).length).toBe(0);
+    expect(document.body.querySelectorAll('.popover.in').length).toBe(0);
     expect(document.getElementsByClassName('popover').length).toBe(0);
     expect(document.getElementsByClassName('popover-inner').length).toBe(0);
     expect(document.getElementsByClassName('popover-title').length).toBe(0);
@@ -80,6 +81,7 @@ describe('Popover', () => {
 
     expect(isOpen).toBe(true);
     expect(wrapper.find(TetherContent).length).toBe(1);
+    expect(document.body.querySelectorAll('.popover.in').length).toBe(1);
     expect(document.getElementsByClassName('popover').length).toBe(1);
     expect(document.getElementsByClassName('popover-inner').length).toBe(1);
     expect(document.getElementsByClassName('popover-title').length).toBe(1);
