Creación de fixtures y definición básica del servicio de Agent

No es necesario hacer shift luego del filter para obtener el primer elemento, mas cuando se sabe que solo hay un elemento en ese array como en este caso que se esta buscando por id.

JS tiene una funcion que se llama find() la cual retorna el primer elemento que sea verdadero, y ademas deja de buscar cuando encuentra dicho elemento, lo cual es mucho mas eficiente que usar filter + shift. Quedaria asi con find

byUuid: uuid => agents.find(agent => agent.uuid === uuid),
byId: id => agents.find(agent => agent.id === id )

Para crear un nuevo objeto con nueva referencia, hay formas mucho mas faciles con ECMASCRIPT 5 que el Object.asign

Puedes simplemente hacer {…agent, id: 2, uuid: ‘yyy-yyy-yyw’, connectedD: false, username: ‘test’}
Con …agent, estas heredando todas sus propiedades y se sobrescriben con lo k pongas a continuacion.

Para los que tengan el error

✖ Internal error
Error: Cannot apply enhancement-only precompilation, possible bad usage:

La forma de solucionarlo es la siguiente:

intalar @babel/register ( npm i -D @babel/register )

seguido de esto modificamos el package.json

para que quede de la siguiente forma:

"scripts": {
    "lint": "standard",
    "setup": "DEBUG=sens:* node setup.js",
    "test": "DEBUG=sens:* nyc --reporter=lcov ava"
  },
  "ava": {
    "files": [
        "tests/**-tests.js"
      ],
      "require": [
        "@babel/register"
      ]
    },

Podemos hacer la función extend como un arrow function single line, me parece util compartirlo.

const extend = (obj, values) => Object.assign({}, obj, { ...values });

El arrow retorna el object assign con el spread de los valores que queremos modificar.

Hay que definir la funcion antes de usarla dentro del array de agents, porque al no usar function js no hace hoisting.

No entiendo la importancia de las pruebas. Se ve interesante, pero no entiendo para qué tenemos que programar pruebas. No se supone que si el código está malo, revienta y ya?. Nos damos cuenta porque no hace lo que debería y durante el desarrollo procedemos a rastrear para resolver el problema. Estamos probando el código que nosotros escribimos, no las librerías (este concepto lo entiendo) pero ha sido un tiempo importante el que ha pasado solo montando las pruebas. Sin embargo esta práctica es usada y he escuchado que hay departamentos de pruebas en empresas grandes, entonces al parecer es muy importante; necesito entender la importancia de esto!

Creo que esta sintaxis es válida desde ECMAScript 2016:

const agents = [
  agent,
  { ...agent, id: 2, uuid: 'yyy-yyy-yyw', connected: false, username: 'test' },
  { ...agent, id: 3, uuid: 'yyy-yyy-yyx' },
  { ...agent, id: 4, uuid: 'yyy-yyy-yyz', username: 'test' }
]

Con esto se ahorran la función y la creación de instancias innecesarias.

✌

No entiendo mucho esto de las pruebas!

Ya me siento super perdido. 😦

El método **assign **se utiliza para copiar los valores de todas las propiedades enumerables de uno o más objetos fuente a un objeto destino,

interesante la función de extend que creó Julian. Para saber más de assign dejo el link https://developer.mozilla.org/es/docs/Web/JavaScript/Referencia/Objetos_globales/Object/assign

Cuando la aplicación es mas grande, estos fixtures contienen arboles de objetos dentro de el. Object.assign solo desconecta la referencia del objeto en general, pero si hay atributos que contienen referencias a otros objetos, estas referencias seguirán intactas, y si objetos a de eso atributos puede que tengan otros atributos apuntando a otras referencias, y así hasta el infinito y mas allá jajajaja.

En fin, hay que buscar estrategias de clonación “profunda” o lo que llamamos “recursiva”, lodash tiene una utilidad super chevere para eso:

const cloneDeep = require('lodash/cloneDeep')

...

module.exports = {
    single: () => cloneDeep(agent),
    all: () => cloneDeep(agents),
    connected: () => cloneDeep(agents.filter(a => a.connected)),
    platziAgents: () => cloneDeep(agents.filter(a => a.username === 'platzi')),
    byUuid: uuid => cloneDeep(agents.filter(a => a.uuid === uuid).shift()),
    byId: id => cloneDeep(agents.filter(a => a.id === id).shift()),
}