Merge pull request #1 from mcwhittemore/master

Merge from upstream

.circleci/config.yml

@@ -0,0 +1,39 @@
+# Javascript Node CircleCI 2.0 configuration file
+#
+# Check https://circleci.com/docs/2.0/language-javascript/ for more details
+#
+version: 2
+jobs:
+  build:
+    docker:
+      # specify the version you desire here
+      - image: circleci/node:7.10
+
+      # Specify service dependencies here if necessary
+      # CircleCI maintains a library of pre-built images
+      # documented at https://circleci.com/docs/2.0/circleci-images/
+      # - image: circleci/mongo:3.4.4
+
+    working_directory: ~/repo
+
+    steps:
+      - checkout
+
+      # Download and cache dependencies
+      - restore_cache:
+          keys:
+          - v1-dependencies-{{ checksum "package.json" }}
+          # fallback to using the latest cache if no exact match is found
+          - v1-dependencies-
+
+      - run: yarn install
+
+      - save_cache:
+          paths:
+            - node_modules
+          key: v1-dependencies-{{ checksum "package.json" }}
+
+      # run tests!
+      - run: yarn test
+
+

index.js

@@ -40,7 +40,7 @@ var sizeOfValue = module.exports.sizeOfValue = function(v) {
   // (length of attribute name) + sum (size of nested elements) + (3 bytes) . 
   // The size of an empty List or Map is 
   // (length of attribute name) + (3 bytes).
-  else if (typeof v === 'object') {
+  else if (typeof v === 'object' && !(v instanceof Buffer)) {
     s += 3
     // List value
     // Calulates size of elements by 

package.json

@@ -3,7 +3,7 @@
     "mocha": "^3.1.2"
   },
   "name": "dyno-item-size",
-  "version": "0.1.0",
+  "version": "0.3.0",
   "description": "compute the size of an item the same way dynamodb does",
   "main": "index.js",
   "directories": {

test/basic.test.js

@@ -2,7 +2,7 @@ var dynoItemSize = require('..');
 
 var items = {
   string: { content: 'string', size: 12 },
-  number: { content: 27, size: 11 },
+  number: { content: 27, size: 10 },
   buffer: { content: new Buffer('hi', 'utf8'), size: 8 }
 };