rewrite dropdown unit tests

build/build-plugins.js

@@ -36,7 +36,7 @@ const bsPlugins = {
   Button: path.resolve(__dirname, '../js/src/button/button.js'),
   Carousel: path.resolve(__dirname, '../js/src/carousel/carousel.js'),
   Collapse: path.resolve(__dirname, '../js/src/collapse/collapse.js'),
-  Dropdown: path.resolve(__dirname, '../js/src/dropdown.js'),
+  Dropdown: path.resolve(__dirname, '../js/src/dropdown/dropdown.js'),
   Modal: path.resolve(__dirname, '../js/src/modal.js'),
   Popover: path.resolve(__dirname, '../js/src/popover.js'),
   ScrollSpy: path.resolve(__dirname, '../js/src/scrollspy.js'),

js/index.esm.js

@@ -9,7 +9,7 @@ import Alert from './src/alert/alert'
 import Button from './src/button/button'
 import Carousel from './src/carousel/carousel'
 import Collapse from './src/collapse/collapse'
-import Dropdown from './src/dropdown'
+import Dropdown from './src/dropdown/dropdown'
 import Modal from './src/modal'
 import Popover from './src/popover'
 import ScrollSpy from './src/scrollspy'

js/index.umd.js

@@ -9,7 +9,7 @@ import Alert from './src/alert/alert'
 import Button from './src/button/button'
 import Carousel from './src/carousel/carousel'
 import Collapse from './src/collapse/collapse'
-import Dropdown from './src/dropdown'
+import Dropdown from './src/dropdown/dropdown'
 import Modal from './src/modal'
 import Popover from './src/popover'
 import ScrollSpy from './src/scrollspy'

js/src/dropdown/dropdown.js

@@ -12,12 +12,12 @@ import {
   makeArray,
   noop,
   typeCheckConfig
-} from './util/index'
+} from '../util/index'
-import Data from './dom/data'
+import Data from '../dom/data'
-import EventHandler from './dom/event-handler'
+import EventHandler from '../dom/event-handler'
-import Manipulator from './dom/manipulator'
+import Manipulator from '../dom/manipulator'
 import Popper from 'popper.js'
-import SelectorEngine from './dom/selector-engine'
+import SelectorEngine from '../dom/selector-engine'
 
 /**
  * ------------------------------------------------------------------------
@@ -289,15 +289,9 @@ class Dropdown {
   }
 
   _getMenuElement() {
-    if (!this._menu) {
+    const parent = Dropdown._getParentFromElement(this._element)
-      const parent = Dropdown._getParentFromElement(this._element)
 
-      if (parent) {
+    return SelectorEngine.findOne(Selector.MENU, parent)
-        this._menu = SelectorEngine.findOne(Selector.MENU, parent)
-      }
-    }
-
-    return this._menu
   }
 
   _getPlacement() {
@@ -545,7 +539,7 @@ EventHandler
  * ------------------------------------------------------------------------
  * add .dropdown to jQuery only if jQuery is present
  */
-
+/* istanbul ignore if */
 if (typeof $ !== 'undefined') {
   const JQUERY_NO_CONFLICT = $.fn[NAME]
   $.fn[NAME] = Dropdown._jQueryInterface

js/tests/karma.conf.js

@@ -9,6 +9,7 @@ const {
 } = require('./browsers')
 const babel = require('rollup-plugin-babel')
 const istanbul = require('rollup-plugin-istanbul')
+const resolve = require('rollup-plugin-node-resolve')
 
 const { env } = process
 const browserStack = env.BROWSER === 'true'
@@ -65,7 +66,8 @@ const rollupPreprocessor = {
       plugins: [
         '@babel/plugin-proposal-object-rest-spread'
       ]
-    })
+    }),
+    resolve()
   ],
   output: {
     format: 'iife',