Skip to content

Dir not deleted on graceful change with Node.js ≥ 12 #209

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
2 of 9 tasks
guimard opened this issue Jan 19, 2020 · 1 comment
Closed
2 of 9 tasks

Dir not deleted on graceful change with Node.js ≥ 12 #209

guimard opened this issue Jan 19, 2020 · 1 comment

Comments

@guimard
Copy link

guimard commented Jan 19, 2020

Operating System

  • Linux
  • Windows 7
  • Windows 10
  • MacOS
  • other:

NodeJS Version

  • 0.x
  • 4.x
  • 6.x
  • 7.x
  • other: 12

Tmp Version

TBD: 0.1.0

Expected Behavior

TBD: remove tmp directory on graceful cleanup

Experienced Behavior

TBD: did not, proven by tests :

  341 passing (3s)
  2 failing

  1) tmp
       #dirSync()
         when running standard outband tests
           on graceful cleanup:

      AssertionError [ERR_ASSERTION]: /tmp/tmp-895811RisWt1aul3Ir should not exist
      + expected - actual

      -false
      +true
      
      at Object.assertDoesNotExist (test/assertions.js:67:10)
      at test/dir-sync-test.js:56:24
      at _close (test/child-process.js:58:7)
      at Socket._stderrEnd (test/child-process.js:80:5)
      at Pipe.<anonymous> (net.js:659:12)

  2) tmp
       #dir()
         when running standard outband tests
           on graceful cleanup:

      AssertionError [ERR_ASSERTION]: /tmp/tmp-895860AHeHf1UUAj7m should not exist
      + expected - actual

      -false
      +true
      
      at Object.assertDoesNotExist (test/assertions.js:67:10)
      at test/dir-test.js:65:24
      at _close (test/child-process.js:58:7)
      at Socket._stderrEnd (test/child-process.js:80:5)
      at Pipe.<anonymous> (net.js:659:12)
@silkentrance
Copy link
Collaborator

Fixed with #198.

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

No branches or pull requests

2 participants