mirror of
https://github.com/josdejong/mathjs.git
synced 2025-12-08 19:46:04 +00:00
* Add `.js` extension to source file imports * Specify package `exports` in `package.json` Specify package type as `commonjs` (It's good to be specific) * Move all compiled scripts into `lib` directory Remove ./number.js (You can use the compiled ones in `./lib/*`) Tell node that the `esm` directory is type `module` and enable tree shaking. Remove unused files from packages `files` property * Allow importing of package.json * Make library ESM first * - Fix merge conflicts - Refactor `bundleAny` into `defaultInstance.js` and `browserBundle.cjs` - Refactor unit tests to be able to run with plain nodejs (no transpiling) - Fix browser examples * Fix browser and browserstack tests * Fix running unit tests on Node 10 (which has no support for modules) * Fix node.js examples (those are still commonjs) * Remove the need for `browserBundle.cjs` * Generate minified bundle only * [Security] Bump node-fetch from 2.6.0 to 2.6.1 (#1963) Bumps [node-fetch](https://github.com/bitinn/node-fetch) from 2.6.0 to 2.6.1. **This update includes a security fix.** - [Release notes](https://github.com/bitinn/node-fetch/releases) - [Changelog](https://github.com/node-fetch/node-fetch/blob/master/docs/CHANGELOG.md) - [Commits](https://github.com/bitinn/node-fetch/compare/v2.6.0...v2.6.1) Signed-off-by: dependabot-preview[bot] <support@dependabot.com> Co-authored-by: dependabot-preview[bot] <27856297+dependabot-preview[bot]@users.noreply.github.com> * Cleanup console.log * Add integration tests to test the entry points (commonjs/esm, full/number only) * Create backward compatibility error messages in the files moved/removed since v8 * Describe breaking changes in HISTORY.md * Bump karma from 5.2.1 to 5.2.2 (#1965) Bumps [karma](https://github.com/karma-runner/karma) from 5.2.1 to 5.2.2. - [Release notes](https://github.com/karma-runner/karma/releases) - [Changelog](https://github.com/karma-runner/karma/blob/master/CHANGELOG.md) - [Commits](https://github.com/karma-runner/karma/compare/v5.2.1...v5.2.2) Signed-off-by: dependabot-preview[bot] <support@dependabot.com> Co-authored-by: dependabot-preview[bot] <27856297+dependabot-preview[bot]@users.noreply.github.com> Co-authored-by: Lee Langley-Rees <lee@greenimp.co.uk> Co-authored-by: dependabot-preview[bot] <27856297+dependabot-preview[bot]@users.noreply.github.com>
86 lines
3.0 KiB
JavaScript
86 lines
3.0 KiB
JavaScript
// Only use native node.js API's and references to ./lib here, this file is not transpiled!
|
|
const assert = require('assert')
|
|
const { createSnapshotFromFactories, validateBundle } = require('../../lib/cjs/utils/snapshot')
|
|
const factoriesAny = require('../../lib/cjs/factoriesAny')
|
|
const version = require('../../package.json').version
|
|
const embeddedDocs = require('../../lib/cjs/expression/embeddedDocs/embeddedDocs')
|
|
|
|
const { expectedInstanceStructure } = createSnapshotFromFactories(factoriesAny)
|
|
|
|
describe('lib/browser', function () {
|
|
it('should load lib/browser/math.js', function () {
|
|
const math = require('../../lib/browser/math.js')
|
|
|
|
assert.strictEqual(math.add(2, 3), 5)
|
|
assert.strictEqual(math.version, version)
|
|
})
|
|
|
|
it('should have all expected functions in lib/browser/main.js', function () {
|
|
// snapshot testing
|
|
const math = require('../../lib/browser/math.js')
|
|
|
|
// don't output all warnings "math.foo.bar is move to math.bar, ..."
|
|
const originalWarn = console.warn
|
|
console.warn = (...args) => {
|
|
if (args.join(' ').indexOf('is moved to') === -1) {
|
|
originalWarn.apply(console, args)
|
|
}
|
|
}
|
|
|
|
validateBundle(expectedInstanceStructure, math)
|
|
|
|
console.warn = originalWarn
|
|
})
|
|
|
|
it('should contain embedded docs for every function', function () {
|
|
const math = require('../../lib/browser/math.js')
|
|
|
|
// names to ignore
|
|
const ignore = [
|
|
// functions not supported or relevant for the parser:
|
|
'chain', 'print',
|
|
'compile', 'parse', 'parser', // TODO: add embedded docs for compile, parse, and parser?
|
|
'reviver', 'replacer', // TODO: add embedded docs for reviver and replacer?
|
|
'apply', // FIXME: apply is not supported right now because of security concerns
|
|
'addScalar', 'divideScalar', 'multiplyScalar', 'equalScalar'
|
|
]
|
|
|
|
// test whether all functions are documented
|
|
const missing = []
|
|
Object.keys(math.expression.mathWithTransform).forEach(function (prop) {
|
|
const obj = math[prop]
|
|
if (math.typeOf(obj) !== 'Object') {
|
|
try {
|
|
if (ignore.indexOf(prop) === -1) {
|
|
math.help(prop).toString()
|
|
}
|
|
} catch (err) {
|
|
missing.push(prop)
|
|
}
|
|
}
|
|
})
|
|
|
|
// test whether there is documentation for non existing functions
|
|
const redundant = Object.keys(embeddedDocs).filter(function (prop) {
|
|
// TODO: figure out why a property embeddedDocs is exported too
|
|
return prop !== 'embeddedDocs' && math[prop] === undefined
|
|
})
|
|
|
|
if (missing.length > 0 || redundant.length > 0) {
|
|
let message = 'Validation failed: not all functions have embedded documentation. '
|
|
|
|
if (missing.length > 0) {
|
|
message += 'Undocumented functions: ' + missing.join(', ') + '. '
|
|
}
|
|
|
|
if (redundant.length > 0) {
|
|
message += 'Documentation for non-existing functions: ' + redundant.join(', ') + '. '
|
|
}
|
|
|
|
message += 'Embedded documentation for the expression parser is defined in src/expression/embeddedDocs.'
|
|
|
|
throw Error(message)
|
|
}
|
|
})
|
|
})
|