fix: add buffer and allow browser tests

Add buffer dependency related to this ipfs/js-ipfs#2924

Remove dependency on path and fs and allow for browser tests.

bench/index.js

@@ -1,6 +1,6 @@
 'use strict'
 
-const Buffer = require('safe-buffer').Buffer
+const { Buffer } = require('buffer')
 const Benchmark = require('benchmark')
 if (typeof window !== 'undefined') {
   window.Benchmark = Benchmark

package.json

@@ -8,13 +8,13 @@
     "url": "https://github.com/ipfs/protons"
   },
   "dependencies": {
+    "buffer": "^5.5.0",
     "protocol-buffers-schema": "^3.3.1",
-    "safe-buffer": "^5.1.1",
     "signed-varint": "^2.0.1",
     "varint": "^5.0.0"
   },
   "devDependencies": {
-    "aegir": "^20.4.1",
+    "aegir": "^21.3.0",
     "benchmark": "^2.1.4",
     "protocol-buffers": "^4.1.0",
     "protons": "^1.0.0",

src/compile/encode.js

@@ -1,5 +1,5 @@
 'use strict'
-
+const { Buffer } = require('buffer')
 var defined = require('./utils').defined
 var varint = require('varint')
 

src/compile/encodings.js

@@ -2,7 +2,7 @@
 
 var varint = require('varint')
 var svarint = require('signed-varint')
-var Buffer = require('safe-buffer').Buffer
+const { Buffer } = require('buffer')
 
 var encoder = function (type, encode, decode, encodingLength) {
   encode.bytes = decode.bytes = 0

src/index.js

@@ -1,6 +1,7 @@
 'use strict'
 
 var schema = require('protocol-buffers-schema')
+const { Buffer } = require('buffer')
 var compile = require('./compile')
 
 var flatten = function (values) {

test/basic.js

@@ -1,11 +1,9 @@
 'use strict'
 
 var tape = require('tape')
-var fs = require('fs')
 var protobufNpm = require('protocol-buffers')
 var protobuf = require('../')
-var path = require('path')
-var proto = fs.readFileSync(path.join(__dirname, '/test.proto'))
+var proto = require('./test.proto.js')
 var Basic = protobuf(proto).Basic
 var BasicNpm = protobufNpm(proto).Basic
 

test/booleans.js

@@ -1,10 +1,9 @@
 'use strict'
 
 var tape = require('tape')
-var fs = require('fs')
-var path = require('path')
 var protobuf = require('../src')
-var Booleans = protobuf(fs.readFileSync(path.join(__dirname, '/test.proto'))).Booleans
+var proto = require('./test.proto.js')
+var Booleans = protobuf(proto).Booleans
 
 tape('booleans encode + decode', function (t) {
   var b1 = Booleans.encode({

test/bytes.js

@@ -1,10 +1,8 @@
 'use strict'
 
 var tape = require('tape')
-var fs = require('fs')
-var path = require('path')
 var protobuf = require('../src')
-var Bytes = protobuf(fs.readFileSync(path.join(__dirname, '/test.proto'))).Bytes
+var Bytes = protobuf(require('./test.proto.js')).Bytes
 
 tape('bytes encode + decode', function (t) {
   var b1 = Bytes.encode({

test/custom-types.js

@@ -1,10 +1,8 @@
 'use strict'
 
 var tape = require('tape')
-var fs = require('fs')
-var path = require('path')
 var protobuf = require('../src')
-var CustomType = protobuf(fs.readFileSync(path.join(__dirname, '/test.proto'))).CustomType
+var CustomType = protobuf(require('./test.proto.js')).CustomType
 
 tape('custom types encode + decode', function (t) {
   var b1 = CustomType.encode({

test/defaults.js

@@ -1,10 +1,8 @@
 'use strict'
 
 var tape = require('tape')
-var fs = require('fs')
-var path = require('path')
 var protobuf = require('../')
-var Defaults = protobuf(fs.readFileSync(path.join(__dirname, '/test.proto'))).Defaults
+var Defaults = protobuf(require('./test.proto.js')).Defaults
 
 tape('defaults decode', function (t) {
   var o1 = Defaults.decode(Buffer.alloc(0)) // everything default

test/enums.js

@@ -1,10 +1,8 @@
 'use strict'
 
 var tape = require('tape')
-var fs = require('fs')
-var path = require('path')
 var protobuf = require('../')
-var messages = protobuf(fs.readFileSync(path.join(__dirname, '/test.proto')))
+var messages = protobuf(require('./test.proto.js'))
 
 tape('enums', function (t) {
   var e = messages.FOO

test/float.js

@@ -1,10 +1,8 @@
 'use strict'
 
 var tape = require('tape')
-var fs = require('fs')
-var path = require('path')
 var protobuf = require('../')
-var Float = protobuf(fs.readFileSync(path.join(__dirname, '/test.proto'))).Float
+var Float = protobuf(require('./test.proto.js')).Float
 
 tape('float encode + decode', function (t) {
   var arr = new Float32Array(3)

test/integers.js

@@ -1,10 +1,8 @@
 'use strict'
 
 var tape = require('tape')
-var fs = require('fs')
-var path = require('path')
 var protobuf = require('../')
-var Integers = protobuf(fs.readFileSync(path.join(__dirname, '/test.proto'))).Integers
+var Integers = protobuf(require('./test.proto.js')).Integers
 
 tape('integers encode + decode', function (t) {
   var b1 = Integers.encode({

test/map.js

@@ -1,10 +1,8 @@
 'use strict'
 
 var tape = require('tape')
-var fs = require('fs')
-var path = require('path')
 var protobuf = require('../')
-var Map = protobuf(fs.readFileSync(path.join(__dirname, '/test.proto'))).Map
+var Map = protobuf(require('./test.proto.js')).Map
 
 tape('map encode + decode', function (t) {
   var b1 = Map.encode({

test/nested.js

@@ -1,10 +1,8 @@
 'use strict'
 
 var tape = require('tape')
-var fs = require('fs')
-var path = require('path')
 var protobuf = require('../')
-var Nested = protobuf(fs.readFileSync(path.join(__dirname, '/test.proto'))).Nested
+var Nested = protobuf(require('./test.proto.js')).Nested
 
 tape('nested encode', function (t) {
   var b1 = Nested.encode({

test/notpacked.js

@@ -1,10 +1,8 @@
 'use strict'
 
 var tape = require('tape')
-var fs = require('fs')
-var path = require('path')
 var protobuf = require('../')
-var proto = fs.readFileSync(path.join(__dirname, '/test.proto'))
+var proto = require('./test.proto.js')
 var NotPacked = protobuf(proto).NotPacked
 var FalsePacked = protobuf(proto).FalsePacked
 

test/oneof.js

@@ -1,10 +1,8 @@
 'use strict'
 
 var tape = require('tape')
-var fs = require('fs')
-var path = require('path')
 var protobuf = require('../')
-var proto = protobuf(fs.readFileSync(path.join(__dirname, '/test.proto')))
+var proto = protobuf(require('./test.proto.js'))
 var Property = proto.Property
 var PropertyNoOneof = proto.PropertyNoOneof
 

test/optional.js

@@ -1,10 +1,8 @@
 'use strict'
 
 var tape = require('tape')
-var fs = require('fs')
 var protobuf = require('../')
-var path = require('path')
-var proto = fs.readFileSync(path.join(__dirname, '/test.proto'))
+var proto = require('./test.proto.js')
 var Optional = protobuf(proto).Optional
 
 tape('optional encode + decode has zero value', function (t) {

test/packed.js

@@ -1,10 +1,8 @@
 'use strict'
 
 var tape = require('tape')
-var fs = require('fs')
-var path = require('path')
 var protobuf = require('../')
-var Packed = protobuf(fs.readFileSync(path.join(__dirname, '/test.proto'))).Packed
+var Packed = protobuf(require('./test.proto.js')).Packed
 
 tape('Packed encode', function (t) {
   var b1 = Packed.encode({

test/repeated.js

@@ -1,10 +1,8 @@
 'use strict'
 
 var tape = require('tape')
-var fs = require('fs')
-var path = require('path')
 var protobuf = require('../')
-var Repeated = protobuf(fs.readFileSync(path.join(__dirname, '/test.proto'))).Repeated
+var Repeated = protobuf(require('./test.proto.js')).Repeated
 
 tape('repeated encode', function (t) {
   var b1 = Repeated.encode({

test/strings.js

@@ -1,10 +1,8 @@
 'use strict'
 
 var tape = require('tape')
-var fs = require('fs')
-var path = require('path')
 var protobuf = require('../src')
-var Strings = protobuf(fs.readFileSync(path.join(__dirname, '/test.proto'))).Strings
+var Strings = protobuf(require('./test.proto.js')).Strings
 
 tape('strings encode + decode', function (t) {
   var b1 = Strings.encode({

test/test.proto → test/test.proto.js

@@ -1,4 +1,5 @@
-message Basic {
+'use strict'
+module.exports = `message Basic {
   required double num = 1;
   required bytes payload = 2;
 }
@@ -129,4 +130,4 @@ message ComplexProperty {
     int32 int_value = 9;
     string string_value = 10;
   }
-}
+}`

test/utf-8.js

@@ -1,10 +1,8 @@
 'use strict'
 
 var tape = require('tape')
-var fs = require('fs')
-var path = require('path')
 var protobuf = require('../')
-var UTF8 = protobuf(fs.readFileSync(path.join(__dirname, '/test.proto'))).UTF8
+var UTF8 = protobuf(require('./test.proto.js')).UTF8
 
 tape('strings can be utf-8', function (t) {
   var ex = {