Skip to content

Commit cbffd97

Browse files
anandkaranubcanandkaranubc
authored
test: add tests for C99 edge cases
PR-URL: #6519 Reviewed-by: Athan Reines <[email protected]>
1 parent 0d59526 commit cbffd97

File tree

3 files changed

+90
-3
lines changed

3 files changed

+90
-3
lines changed

‎lib/node_modules/@stdlib/math/base/special/cabsf/lib/main.js‎

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,6 @@ var imag = require( '@stdlib/complex/float32/imag' );
4040
* // returns ~5.83
4141
*/
4242
function cabsf( z ) {
43-
// TODO: consider whether to use C99 rules for special cases involving infinities and nans (see https://github.com/python/cpython/blob/f4c03484da59049eb62a9bf7777b963e2267d187/Objects/complexobject.c#L191)
4443
return hypotf( real( z ), imag( z ) );
4544
}
4645

‎lib/node_modules/@stdlib/math/base/special/cabsf/test/test.js‎

Lines changed: 45 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,8 @@
2323
var tape = require( 'tape' );
2424
var isnanf = require( '@stdlib/math/base/assert/is-nanf' );
2525
var EPS = require( '@stdlib/constants/float32/eps' );
26+
var PINF = require( '@stdlib/constants/float32/pinf' );
27+
var NINF = require( '@stdlib/constants/float32/ninf' );
2628
var absf = require( '@stdlib/math/base/special/absf' );
2729
var Complex64 = require( '@stdlib/complex/float32/ctor' );
2830
var cabsf = require( './../lib' );
@@ -67,7 +69,49 @@ tape( 'the function computes the absolute value of a complex number', function t
6769
t.end();
6870
});
6971

70-
tape( 'if either the real or imaginary component is `NaN`, the function returns `NaN`', function test( t ) {
72+
tape( 'if either the real or imaginary component is `+infinity`, the function returns `+infinity`', function test( t ) {
73+
var v;
74+
75+
v = cabsf( new Complex64( PINF, 3.0 ) );
76+
t.strictEqual( v, PINF, 'returns expected value' );
77+
78+
v = cabsf( new Complex64( 5.0, PINF ) );
79+
t.strictEqual( v, PINF, 'returns expected value' );
80+
81+
v = cabsf( new Complex64( PINF, PINF ) );
82+
t.strictEqual( v, PINF, 'returns expected value' );
83+
84+
v = cabsf( new Complex64( NaN, PINF ) );
85+
t.strictEqual( v, PINF, 'returns expected value' );
86+
87+
v = cabsf( new Complex64( PINF, NaN ) );
88+
t.strictEqual( v, PINF, 'returns expected value' );
89+
90+
t.end();
91+
});
92+
93+
tape( 'if either the real or imaginary component is `-infinity`, the function returns `+infinity`', function test( t ) {
94+
var v;
95+
96+
v = cabsf( new Complex64( NINF, 3.0 ) );
97+
t.strictEqual( v, PINF, 'returns expected value' );
98+
99+
v = cabsf( new Complex64( 5.0, NINF ) );
100+
t.strictEqual( v, PINF, 'returns expected value' );
101+
102+
v = cabsf( new Complex64( NINF, NINF ) );
103+
t.strictEqual( v, PINF, 'returns expected value' );
104+
105+
v = cabsf( new Complex64( NaN, NINF ) );
106+
t.strictEqual( v, PINF, 'returns expected value' );
107+
108+
v = cabsf( new Complex64( NINF, NaN ) );
109+
t.strictEqual( v, PINF, 'returns expected value' );
110+
111+
t.end();
112+
});
113+
114+
tape( 'if either the real or imaginary component is `NaN` but not `+-infinity`, the function returns `NaN`', function test( t ) {
71115
var v;
72116

73117
v = cabsf( new Complex64( NaN, 3.0 ) );

‎lib/node_modules/@stdlib/math/base/special/cabsf/test/test.native.js‎

Lines changed: 45 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,8 @@ var resolve = require( 'path' ).resolve;
2424
var tape = require( 'tape' );
2525
var isnanf = require( '@stdlib/math/base/assert/is-nanf' );
2626
var EPS = require( '@stdlib/constants/float32/eps' );
27+
var PINF = require( '@stdlib/constants/float32/pinf' );
28+
var NINF = require( '@stdlib/constants/float32/ninf' );
2729
var absf = require( '@stdlib/math/base/special/absf' );
2830
var Complex64 = require( '@stdlib/complex/float32/ctor' );
2931
var tryRequire = require( '@stdlib/utils/try-require' );
@@ -76,7 +78,49 @@ tape( 'the function computes the absolute value of a complex number', opts, func
7678
t.end();
7779
});
7880

79-
tape( 'if either the real or imaginary component is `NaN`, the function returns `NaN`', opts, function test( t ) {
81+
tape( 'if either the real or imaginary component is `+infinity`, the function returns `+infinity`', opts, function test( t ) {
82+
var v;
83+
84+
v = cabsf( new Complex64( PINF, 3.0 ) );
85+
t.strictEqual( v, PINF, 'returns expected value' );
86+
87+
v = cabsf( new Complex64( 5.0, PINF ) );
88+
t.strictEqual( v, PINF, 'returns expected value' );
89+
90+
v = cabsf( new Complex64( PINF, PINF ) );
91+
t.strictEqual( v, PINF, 'returns expected value' );
92+
93+
v = cabsf( new Complex64( NaN, PINF ) );
94+
t.strictEqual( v, PINF, 'returns expected value' );
95+
96+
v = cabsf( new Complex64( PINF, NaN ) );
97+
t.strictEqual( v, PINF, 'returns expected value' );
98+
99+
t.end();
100+
});
101+
102+
tape( 'if either the real or imaginary component is `-infinity`, the function returns `+infinity`', opts, function test( t ) {
103+
var v;
104+
105+
v = cabsf( new Complex64( NINF, 3.0 ) );
106+
t.strictEqual( v, PINF, 'returns expected value' );
107+
108+
v = cabsf( new Complex64( 5.0, NINF ) );
109+
t.strictEqual( v, PINF, 'returns expected value' );
110+
111+
v = cabsf( new Complex64( NINF, NINF ) );
112+
t.strictEqual( v, PINF, 'returns expected value' );
113+
114+
v = cabsf( new Complex64( NaN, NINF ) );
115+
t.strictEqual( v, PINF, 'returns expected value' );
116+
117+
v = cabsf( new Complex64( NINF, NaN ) );
118+
t.strictEqual( v, PINF, 'returns expected value' );
119+
120+
t.end();
121+
});
122+
123+
tape( 'if either the real or imaginary component is `NaN` but not `+-infinity`, the function returns `NaN`', opts, function test( t ) {
80124
var v;
81125

82126
v = cabsf( new Complex64( NaN, 3.0 ) );

0 commit comments

Comments
 (0)