Merge pull request #386 from contentstack/fix/dx-3240

Fixed term move issue

Author: sunil-lakshman

lib/entity.js

@@ -313,7 +313,6 @@ export const move = (http, type, force = false, params = {}) => {
     try {
       let updateData = {}
       const json = cloneDeep(this)
-      delete json.parent_uid
       if (type) {
         updateData[type] = json
       } else {
@@ -328,7 +327,7 @@ export const move = (http, type, force = false, params = {}) => {
       if (force === true) {
         headers.params.force = true
       }
-      const response = await http.put(`${this.urlPath}/move`, updateData, headers)
+      const response = await http.put(`${this.urlPath}/move`, param, headers)
       if (response.data) {
         return new this.constructor(http, parseData(response, this.stackHeaders, this.content_type_uid, this.taxonomy_uid, http))
       } else {

test/sanity-check/api/terms-test.js

@@ -135,13 +135,14 @@ describe('Terms API Test', () => {
   })
 
   it('should move the term to parent uid passed', done => {
-    makeTerms(taxonomy.uid, childTerm2.term.uid).fetch()
+    const term = {
+      parent_uid: 'term_test_child1',
+      order: 1
+    }
+    makeTerms(taxonomy.uid, childTerm2.term.uid).move({ term, force: true })
       .then(async (term) => {
-        term.parent_uid = null
-        const moveTerm = await term.move({ force: true })
-        expect(moveTerm.parent_uid).to.be.equal(null)
+        expect(term.parent_uid).to.not.equal(null)
         done()
-        return moveTerm
       })
       .catch(done)
   })