Test improvements (#103)

* Fixed duplication kve bug

* Some extra testing stuff

* A few fixes

app/__mocks__/dummy.js

@@ -0,0 +1 @@
+module.exports = {};

app/__tests__/renderer.test.js

@@ -0,0 +1,7 @@
+import * as renderer from '../renderer';
+
+describe('imports', () => {
+  it('ui module should import successfully', () => {
+    expect(renderer).toBeDefined();
+  });
+});

app/ui/components/keyvalueeditor/Row.js

@@ -305,9 +305,11 @@ function isAbove (monitor, component) {
   const hoveredNode = ReactDOM.findDOMNode(component);
 
   const hoveredTop = hoveredNode.getBoundingClientRect().top;
+  const height = hoveredNode.clientHeight;
   const draggedTop = monitor.getSourceClientOffset().y;
 
-  return hoveredTop > draggedTop;
+  // NOTE: Not quite sure why it's height / 3 (seems to work)
+  return hoveredTop > draggedTop - (height / 3);
 }
 
 const dragTarget = {

app/ui/css/components/keyvalueeditor.less

@@ -12,8 +12,8 @@
     flex-direction: row;
     width: 100%;
     padding-left: @padding-md;
-    margin-top: @padding-xs;
-    margin-bottom: @padding-xs;
+    padding-top: @padding-xs;
+    padding-bottom: @padding-xs;
     box-sizing: border-box;
 
     &.key-value-editor__row-wrapper--disabled .input,

package.json

@@ -40,14 +40,18 @@
     "setupFiles": [
       "./__jest__/setup.js"
     ],
+    "moduleNameMapper": {
+      "\\.(css|less|png)$": "<rootDir>/__mocks__/dummy.js"
+    },
     "rootDir": "app",
     "verbose": false,
     "collectCoverage": false,
     "collectCoverageFrom": [
       "**/*.js",
+      "!**/*.min.js",
       "!**/main.development.js",
       "!**/static/**",
-      "!**/ui/**",
+      "!**/ui/components/**",
       "!**/__fixtures__/**",
       "!**/__tests__/**",
       "!**/__jest__/**",