Skip to content

Commit

Permalink
chore: ignore freqz in the doc test for the time being (see #2988)
Browse files Browse the repository at this point in the history
  • Loading branch information
josdejong committed Jul 13, 2023
1 parent 6169f0a commit ab409df
Showing 1 changed file with 4 additions and 3 deletions.
7 changes: 4 additions & 3 deletions test/node-tests/doc.test.js
Original file line number Diff line number Diff line change
Expand Up @@ -99,7 +99,7 @@ const knownProblems = new Set([
'mod', 'invmod', 'floor', 'fix', 'expm1', 'exp', 'dotPow', 'dotMultiply',
'dotDivide', 'divide', 'ceil', 'cbrt', 'add', 'usolveAll', 'usolve', 'slu',
'rationalize', 'qr', 'lusolve', 'lup', 'lsolveAll', 'lsolve', 'derivative',
'symbolicEqual', 'map', 'schur', 'sylvester'
'symbolicEqual', 'map', 'schur', 'sylvester', 'freqz'
])

function maybeCheckExpectation (name, expected, expectedFrom, got, gotFrom) {
Expand Down Expand Up @@ -132,8 +132,9 @@ function checkExpectation (want, got) {
if (typeof want === 'number' && typeof got === 'number' && want !== got) {
console.log(` Note: return value ${got} not exactly as expected: ${want}`)
return approx.equal(got, want, 1e-9)
} if (typeof want !== 'undefined') {
approx.deepEqual(got, want)
}
if (typeof want !== 'undefined') {
return approx.deepEqual(got, want)
} else {
// don't check if we don't know what the result is supposed to be
}
Expand Down

0 comments on commit ab409df

Please sign in to comment.