Drop duplicate isPrimitive() in harness/testTypedArray.js

[ivankra]

It's already defined in harness/assert.js, included in every test.

Some engines/runtimes e.g. deno complain about redeclarations in the same scope and abort execution.

[ljharb]

this doesn't seem like a duplicate - where's the second function definition?

[linusg]

test262/harness/assert.js

Line 104 in 4a1e962

function isPrimitive(value) {

[ljharb]

ah ok, it's a global. makes sense

[ptomato]

Can we add isPrimitive to the defines in assert.js and expect that tests might use it? (Or, if not, move it inside the compareArray function body)

[ivankra]

Added it to defines in assert.js.

[ptomato]

Yes, confirmed that we do want to expose isPrimitive for tests.