Skip to content

Commit

Permalink
no symbols in browser
Browse files Browse the repository at this point in the history
  • Loading branch information
David Mark Clements committed May 11, 2019
1 parent 2b3bbf1 commit 8f9ff4f
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 13 deletions.
1 change: 0 additions & 1 deletion browser.js
Original file line number Diff line number Diff line change
Expand Up @@ -162,7 +162,6 @@ pino.levels = {
}

pino.stdSerializers = stdSerializers
pino.symbols = require('./lib/symbols')

function set (opts, logger, level, fallback) {
var proto = Object.getPrototypeOf(logger)
Expand Down
12 changes: 0 additions & 12 deletions test/browser.test.js
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,6 @@
const test = require('tape')
const fresh = require('import-fresh')
const pinoStdSerializers = require('pino-std-serializers')
const pinoSymbols = require('../lib/symbols')
const pino = require('../browser')

levelTest('fatal')
Expand Down Expand Up @@ -125,17 +124,6 @@ test('exposes err stdSerializer', ({ end, ok }) => {
end()
})

test('exposes real symbols', ({ end, ok, same }) => {
ok(pino.symbols)
// confirm every symbol is present
for (let symbol in pinoSymbols) {
ok(pino.symbols[symbol], `pino.symbols.${symbol}`)
}
// confirm real symbols are used
same(pino.symbols, pinoSymbols)
end()
})

consoleMethodTest('error')
consoleMethodTest('fatal', 'error')
consoleMethodTest('warn')
Expand Down

0 comments on commit 8f9ff4f

Please sign in to comment.