canjs / can-map-define

Define rich attribute behavior
https://canjs.com/doc/can-map-define.html
MIT License
3 stars 2 forks source link

getOwnEnumerableKeys doesn't account for default behavior #88

Closed m-mujica closed 6 years ago

m-mujica commented 6 years ago
var VM = CanMap.extend({
  define: {
    "*": {
      serialize: false
    },
    notEnumerable: {
      value: "no"
    },
    enumerableProp: {
      serialize: true,
      value: "yes"
    },
    notEnumerable2: {
      serialize: false,
      value: "maybe"
    }
  }
});

var vm = new VM();

// throws!!!!
assert.deepEqual(
  canReflect.getOwnEnumerableKeys(vm), 
  ["enumerableProp"],
  "vm.getOwnEnumerableKeys()"
);