[Bug 1442516] Review Request: nodejs-cache-base - Basic object cache with get, set, del, and has methods

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



https://bugzilla.redhat.com/show_bug.cgi?id=1442516

Robert-André Mauchin <zebob.m@xxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |zebob.m@xxxxxxxxx



--- Comment #1 from Robert-André Mauchin <zebob.m@xxxxxxxxx> ---
A lot of tests fail:

  26 passing (43ms)
  18 failing
  1) cache-base constructor: should set :
     AssertionError: undefined == 1
      at Context.<anonymous> (test/test.js:32:14)
  2) cache-base get/set: set() - add: should set a new property with the given
value:
     AssertionError: undefined == 1
      at Context.<anonymous> (test/test.js:50:16)
  3) cache-base get/set: set() - update: should update an existing property
with the given value:
     AssertionError: undefined == 2
      at Context.<anonymous> (test/test.js:57:16)
  4) cache-base get/set: set() - update: should get the given property:
     AssertionError: undefined == 'b'
      at Context.<anonymous> (test/test.js:62:16)
  5) cache-base .union() should union a string value:
     TypeError: union-value expects the first argument to be an object.
      at unionValue (/usr/lib/node_modules/union-value/index.js:10:11)
      at Cache.union (index.js:119:5)
      at Context.<anonymous> (test/test.js:69:11)
  6) cache-base .union() should union multiple string values:
     TypeError: union-value expects the first argument to be an object.
      at unionValue (/usr/lib/node_modules/union-value/index.js:10:11)
      at Cache.union (index.js:119:5)
      at Context.<anonymous> (test/test.js:74:11)
  7) cache-base .union() should union multiple arrays:
     TypeError: union-value expects the first argument to be an object.
      at unionValue (/usr/lib/node_modules/union-value/index.js:10:11)
      at Cache.union (index.js:119:5)
      at Context.<anonymous> (test/test.js:81:11)
  8) cache-base .union() should union nested string values:
     TypeError: union-value expects the first argument to be an object.
      at unionValue (/usr/lib/node_modules/union-value/index.js:10:11)
      at Cache.union (index.js:119:5)
      at Context.<anonymous> (test/test.js:88:11)
  9) cache-base .union() should union and uniquify arrays:
     TypeError: union-value expects the first argument to be an object.
      at unionValue (/usr/lib/node_modules/union-value/index.js:10:11)
      at Cache.union (index.js:119:5)
      at Context.<anonymous> (test/test.js:95:11)
  10) cache-base .set() should set a value:
     AssertionError: undefined == 'b'
      at Context.<anonymous> (test/test.js:105:14)
  11) cache-base .set() should set properties on the `cache` object.:
     AssertionError: undefined == 'b'
      at Context.<anonymous> (test/test.js:110:14)
  12) cache-base .set() should allow an object to be set directly.:
     AssertionError: undefined == 'y'
      at Context.<anonymous> (test/test.js:115:14)
  13) cache-base .set() should set nested properties on the `cache` object.:
     TypeError: Cannot read property 'd' of undefined
      at Context.<anonymous> (test/test.js:121:32)
  14) cache-base .set() should return `this` for chaining:
     AssertionError: undefined == 'bb'
      at Context.<anonymous> (test/test.js:130:14)
  15) cache-base .get() should get a value:
     AssertionError: undefined == 'b'
      at Context.<anonymous> (test/test.js:147:14)
  16) cache-base .get() should get a nested property value:
     TypeError: Cannot read property 'b' of undefined
      at Context.<anonymous> (test/test.js:152:25)
  17) events get should emit the value with "get" events:
     AssertionError: undefined == 'b'
      at Cache.<anonymous> (test/test.js:208:16)
      at Cache.Emitter.emit
(/usr/lib/node_modules/component-emitter/index.js:134:20)
      at Cache.get (index.js:150:10)
      at Context.<anonymous> (test/test.js:212:11)
  18) events has should emit the value with "has" events:
      AssertionError: false == true
      + expected - actual
      +true
      -false

      at Cache.<anonymous> (test/test.js:235:16)
      at Cache.Emitter.emit
(/usr/lib/node_modules/component-emitter/index.js:134:20)
      at Cache.has (index.js:178:10)
      at Context.<anonymous> (test/test.js:239:11)

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
_______________________________________________
package-review mailing list -- package-review@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to package-review-leave@xxxxxxxxxxxxxxxxxxxxxxx




[Index of Archives]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite Conditions]     [KDE Users]

  Powered by Linux