Skip to content

Fix instrumentation tests by properly resetting fs.statSync#10777

Draft
pichfl wants to merge 1 commit intoember-cli:masterfrom
pichfl:mainmatter/fix-instrumentation-tests
Draft

Fix instrumentation tests by properly resetting fs.statSync#10777
pichfl wants to merge 1 commit intoember-cli:masterfrom
pichfl:mainmatter/fix-instrumentation-tests

Conversation

@pichfl
Copy link
Contributor

@pichfl pichfl commented Aug 21, 2025

Fixes the following error that popped up with [email protected]:

 1) models/instrumentation.js
       ._enableFSMonitorIfInstrumentationEnabled
         "before each" hook for "if instrumentation is enabled, monitor":

      AssertionError: expected [Function] to equal [Function statSync]
      + expected - actual

      -{
      -  "__restore": [Function]
      -}
      +[Function]

      at Context.<anonymous> (tests/unit/models/instrumentation-test.js:38:30)
      at process.processImmediate (node:internal/timers:505:21)

@pichfl pichfl marked this pull request as draft August 21, 2025 11:06
@pichfl
Copy link
Contributor Author

pichfl commented Aug 21, 2025

Scratch this, [email protected] breaks heimdall-fs-monitor

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant