fix hasBeenCalled function

[nadiapadalka]

Added functionality to compare values inside deepEquals function, and tests for testing

[Nowele]

yarn run-s clean build test

 PASS  test/e2e/parameterized/parameterized.test.js
 PASS  test/e2e/simple-suites/simple-suites.test.js
 PASS  test/e2e/assert/assert.test.js
 PASS  test/e2e/before-after/before-after.test.js
 PASS  test/e2e/selection/selection.test.js
 PASS  test/runner/file-match.test.ts (7.907 s)
 PASS  test/reporter/jest-reporter.test.ts (8.38 s)

Test Suites: 7 passed, 7 total
Tests:       33 passed, 33 total
Snapshots:   0 total
Time:        8.965 s

[olenahuchko]

Your fix looks great to me, also verified with some other test cases:

mockFunc = _brs_.mockFunction("testHasBeenCalledWith", sub (value as boolean) : end sub)
m.node.callFunc("testHasBeenCalledWith", true)
m.assert.hasBeenCalledWith(mockFunc, [false], "") ' - failed 🟢 

m.assert.deepEquals([true], [false], "")  ' - failed 🟢 

m.assert.deepEquals({value: true}, {value: false}, "") ' - passed 🔴 

m.assert.deepEquals(true, false, "") ' - passed 🔴 

The last two are not related to hasBeenCalled, but it would be great to get it fixed as well here or maybe in the separate ticket

[nadiapadalka]

Your fix looks great to me, also verified with some other test cases:

mockFunc = _brs_.mockFunction("testHasBeenCalledWith", sub (value as boolean) : end sub)
m.node.callFunc("testHasBeenCalledWith", true)
m.assert.hasBeenCalledWith(mockFunc, [false], "") ' - failed 🟢 

m.assert.deepEquals([true], [false], "")  ' - failed 🟢 

m.assert.deepEquals({value: true}, {value: false}, "") ' - passed 🔴 

m.assert.deepEquals(true, false, "") ' - passed 🔴 

The last two are not related to hasBeenCalled, but it would be great to get it fixed as well here or maybe in the separate ticket

Yes, let's create a separate ticket for deepEquals

[nadiapadalka]

merging this in as this PR has 2 approvals