Improving tests

Author: David-Desmaisons

src/vuedraggable.js

@@ -121,7 +121,7 @@ const props = {
 const draggableComponent = {
   name: "draggable",
 
-  inheritAttrs: true,
+  inheritAttrs: false,
 
   props,
 

tests/unit/vuedraggable.spec.js

@@ -11,15 +11,19 @@ let wrapper;
 let vm;
 let props;
 let items;
-let html;
+let element;
 
 describe("draggable.vue", () => {
   beforeEach(() => {
+    Sortable.mockClear();
     items = ["a", "b", "c"];
     wrapper = shallowMount(draggable, {
       propsData: {
         list: items
       },
+      attrs: {
+        sortableOption: "value"
+      },
       slots: {
         default: items.map(item => `<div>${item}</div>`),
         header: "<header/>",
@@ -28,13 +32,17 @@ describe("draggable.vue", () => {
     });
     vm = wrapper.vm;
     props = vm.$options.props;
-    html = wrapper.element;
+    element = wrapper.element;
   });
 
-  it("should instantiate without error", () => {
+  it("instantiate without error", () => {
     expect(wrapper).not.toBeUndefined();
   });
 
+  it("has draggable name", () => {
+    expect(vm.name).not.toBe("draggable");
+  });
+
   test.each([
     ["options", { type: Object }],
     ["list", {
@@ -117,4 +125,14 @@ describe("draggable.vue", () => {
   it("keeps a reference to Sortable instance", () => {
     expect(vm._sortable).toBe(SortableFake);
   })
+
+  it("creates sortable instance with options", () => {
+    expect(Sortable.mock.calls.length).toBe(1);
+    const parameters = Sortable.mock.calls[0];
+    expect(parameters[0]).toBe(element);
+    expect(parameters[1]).toMatchObject({
+      draggable: ">*",
+      sortableOption: "value"
+    });
+  })
 });