Remove copy & Copyable

Part of emberjs#19617

Author: mixonic

packages/@ember/-internals/runtime/index.js

@@ -1,7 +1,6 @@
 export { default as Object, FrameworkObject } from './lib/system/object';
 export { default as RegistryProxyMixin } from './lib/mixins/registry_proxy';
 export { default as ContainerProxyMixin } from './lib/mixins/container_proxy';
-export { default as copy } from './lib/copy';
 export { default as compare } from './lib/compare';
 export { default as isEqual } from './lib/is-equal';
 export {
@@ -19,7 +18,6 @@ export { default as ArrayProxy } from './lib/system/array_proxy';
 export { default as ObjectProxy } from './lib/system/object_proxy';
 export { default as CoreObject } from './lib/system/core_object';
 export { default as ActionHandler } from './lib/mixins/action_handler';
-export { default as Copyable } from './lib/mixins/copyable';
 export { default as Enumerable } from './lib/mixins/enumerable';
 export { default as _ProxyMixin, contentFor as _contentFor } from './lib/mixins/-proxy';
 export { default as Observable } from './lib/mixins/observable';

packages/@ember/-internals/runtime/lib/copy.js

@@ -3,15 +3,13 @@ import EmberArray, { A as emberA, MutableArray } from '../../lib/mixins/array';
 import { generateGuid, guidFor } from '@ember/-internals/utils';
 import {
   get,
-  set,
   computed,
   addArrayObserver,
   removeArrayObserver,
   arrayContentWillChange,
   arrayContentDidChange,
 } from '@ember/-internals/metal';
 import EmberObject from '../../lib/system/object';
-import Copyable from '../../lib/mixins/copyable';
 import { moduleFor } from 'internal-test-helpers';
 
 export function newFixture(cnt) {
@@ -147,42 +145,6 @@ class NativeArrayHelpers extends AbstractArrayHelper {
   }
 }
 
-class CopyableNativeArray extends AbstractArrayHelper {
-  newObject() {
-    return emberA([generateGuid()]);
-  }
-
-  isEqual(a, b) {
-    if (!(a instanceof Array)) {
-      return false;
-    }
-
-    if (!(b instanceof Array)) {
-      return false;
-    }
-
-    if (a.length !== b.length) {
-      return false;
-    }
-
-    return a[0] === b[0];
-  }
-}
-
-class CopyableArray extends AbstractArrayHelper {
-  newObject() {
-    return CopyableObject.create();
-  }
-
-  isEqual(a, b) {
-    if (!(a instanceof CopyableObject) || !(b instanceof CopyableObject)) {
-      return false;
-    }
-
-    return get(a, 'id') === get(b, 'id');
-  }
-}
-
 class ArrayProxyHelpers extends AbstractArrayHelper {
   newObject(ary) {
     return ArrayProxy.create({ content: emberA(super.newObject(ary)) });
@@ -271,21 +233,6 @@ const TestMutableArray = EmberObject.extend(MutableArray, {
   },
 });
 
-const CopyableObject = EmberObject.extend(Copyable, {
-  id: null,
-
-  init() {
-    this._super(...arguments);
-    set(this, 'id', generateGuid());
-  },
-
-  copy() {
-    let ret = CopyableObject.create();
-    set(ret, 'id', get(this, 'id'));
-    return ret;
-  },
-});
-
 class MutableArrayHelpers extends NativeArrayHelpers {
   newObject(ary) {
     return TestMutableArray.create(super.newObject(ary));
@@ -316,15 +263,11 @@ export function runArrayTests(name, Tests, ...types) {
         case 'MutableArray':
           moduleFor(`MutableArray: ${name}`, Tests, MutableArrayHelpers);
           break;
-        case 'CopyableArray':
-          moduleFor(`CopyableArray: ${name}`, Tests, CopyableArray);
-          break;
-        case 'CopyableNativeArray':
-          moduleFor(`CopyableNativeArray: ${name}`, Tests, CopyableNativeArray);
-          break;
         case 'NativeArray':
           moduleFor(`NativeArray: ${name}`, Tests, NativeArrayHelpers);
           break;
+        default:
+          throw new Error(`runArrayTests passed unexpected type ${type}`);
       }
     });
   } else {

packages/@ember/-internals/runtime/lib/mixins/copyable.js

@@ -1,3 +1,2 @@
 export { getCachedValueFor as cacheFor } from '@ember/-internals/metal';
-export { copy } from '@ember/-internals/runtime';
 export { guidFor } from '@ember/-internals/utils';

packages/@ember/-internals/runtime/tests/copyable-array/copy-test.js

@@ -41,10 +41,8 @@ import {
   RegistryProxyMixin,
   ContainerProxyMixin,
   compare,
-  copy,
   isEqual,
   Array as EmberArray,
-  Copyable,
   MutableEnumerable,
   MutableArray,
   TargetActionSupport,
@@ -397,8 +395,6 @@ Ember.Object = EmberObject;
 Ember._RegistryProxyMixin = RegistryProxyMixin;
 Ember._ContainerProxyMixin = ContainerProxyMixin;
 Ember.compare = compare;
-Ember.copy = copy;
-Ember.isEqual = isEqual;
 
 /**
 @module ember
@@ -430,7 +426,6 @@ Ember.ObjectProxy = ObjectProxy;
 Ember.ActionHandler = ActionHandler;
 Ember.CoreObject = CoreObject;
 Ember.NativeArray = NativeArray;
-Ember.Copyable = Copyable;
 Ember.MutableEnumerable = MutableEnumerable;
 Ember.MutableArray = MutableArray;
 Ember.Evented = Evented;

packages/@ember/-internals/runtime/tests/core/copy_test.js

@@ -351,7 +351,6 @@ let allExports = [
 
   // @ember/object/internals
   ['cacheFor', '@ember/object/internals', 'cacheFor'],
-  ['copy', '@ember/object/internals', 'copy'],
   ['guidFor', '@ember/object/internals', 'guidFor'],
 
   // @ember/object/mixin
@@ -546,7 +545,6 @@ let allExports = [
   ['Comparable', '@ember/-internals/runtime'],
   ['ActionHandler', '@ember/-internals/runtime'],
   ['NativeArray', '@ember/-internals/runtime'],
-  ['Copyable', '@ember/-internals/runtime'],
   ['MutableEnumerable', '@ember/-internals/runtime'],
   EMBER_MODERNIZED_BUILT_IN_COMPONENTS
     ? null