refactor: remove unnecessary asynchronicity from the addPlaceholderList method (#38)

Author: Hesam Bahrami

src/main.js

@@ -359,9 +359,9 @@ class NestedSort {
     this.placeholderInUse.style.minHeight = `${this.draggedNode.offsetHeight}px`
   }
 
-  async addPlaceholderList() {
+  addPlaceholderList() {
     this.getPlaceholderList()
-    await this.targetedNode.appendChild(this.placeholderInUse)
+    this.targetedNode.appendChild(this.placeholderInUse)
     this.animatePlaceholderList()
   }
 

test/main.test.js

@@ -884,13 +884,13 @@ describe('NestedSort', () => {
   })
 
   describe('addPlaceholderList method', () => {
-    it('should append the placeholderInUse property to the targetedNode and call the animatePlaceholderList method', async () => {
+    it('should append the placeholderInUse property to the targetedNode and call the animatePlaceholderList method', () => {
       const ns = initDataDrivenList()
       ns.targetedNode = document.querySelector('li[data-id="1"]')
       ns.targetedNode.appendChild = jest.fn()
       ns.animatePlaceholderList = jest.fn()
 
-      await ns.addPlaceholderList()
+      ns.addPlaceholderList()
 
       expect(ns.targetedNode.appendChild).toHaveBeenCalledTimes(1)
       expect(ns.targetedNode.appendChild).toHaveBeenCalledWith(ns.placeholderInUse)