René's URL Explorer Experiment

{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"`fs.utimes` doesn't support `number` or `string` on Windows","articleBody":"\u003c!--\r\nThank you for reporting an issue.\r\n\r\nThis issue tracker is for bugs and issues found within Node.js core.\r\nIf you require more general support please file an issue on our help\r\nrepo. https://github.com/nodejs/help\r\n\r\n\r\nPlease fill in as much of the template below as you're able.\r\n\r\nVersion: output of `node -v`\r\nPlatform: output of `uname -a` (UNIX), or version and 32 or 64-bit (Windows)\r\nSubsystem: if known, please specify affected core module name\r\n-->\r\n\r\n* **Version**: v13.14.0 (unable to install v14 on windows 7)\r\n* **Platform**: Microsoft Windows Server 2019 (GH Actions Windows CI) & 32 bit Windows 7\r\n* **Subsystem**: fs\r\n\r\n### What steps will reproduce the bug?\r\n\r\n```js\r\nconst fs = require('fs');\r\n\r\nfs.utimesSync(__filename, Date.now(), Date.now());\r\n```\r\n\r\n```js\r\nconst fs = require('fs');\r\n\r\nfs.utimesSync(__filename, Date.now().toString(), Date.now().toString());\r\n```\r\n\r\n\u003c!--\r\nEnter details about your bug, preferably a simple code snippet that can be\r\nrun using `node` directly without installing third-party dependencies.\r\n-->\r\n\r\n### How often does it reproduce? Is there a required condition?\r\n\r\nAlways\r\n\r\n### What is the expected behavior?\r\n\r\nIt should work same as Linux and Mac\r\n\r\n\u003c!--\r\nIf possible please provide textual output instead of screenshots.\r\n-->\r\n\r\n### What do you see instead?\r\n\r\n```\r\n  ● does not re-run tests when only access time is modified\r\n\r\n    EINVAL: invalid argument, utime 'C:\\Users\\RUNNER~1\\AppData\\Local\\Temp\\watch_mode_no_access\\foo.js'\r\n\r\n      61 |   const modulePath = path.join(DIR, 'foo.js');\r\n      62 |   const stat = fs.lstatSync(modulePath);\r\n    > 63 |   fs.utimesSync(modulePath, stat.atime.getTime(), stat.mtime.getTime());\r\n         |      ^\r\n      64 | \r\n      65 |   await testRun.waitUntil(({stderr}) => numberOfTestRuns(stderr) === 2);\r\n      66 | \r\n\r\n      at Object.utimesSync (e2e/__tests__/watchModeNoAccess.test.ts:63:6)\r\n```\r\n\r\nThat's from this CI run, where the same test passes on both Linux and Mac: https://github.com/facebook/jest/runs/641141754\r\n\r\n\u003c!--\r\nIf possible please provide textual output instead of screenshots.\r\n-->\r\n\r\n### Additional information\r\n\r\n\u003c!--\r\nTell us anything else you think we should know.\r\n-->\r\n\r\nAccording to the docs, `utimes` can take `number | string | Date`: https://nodejs.org/api/fs.html#fs_fs_utimes_path_atime_mtime_callback. Seems like Windows only supports `Date`?\r\n\r\nThis is the same as #5561.","author":{"url":"https://github.com/SimenB","@type":"Person","name":"SimenB"},"datePublished":"2020-05-04T06:57:38Z","url":"https://github.com/nodejs/node/issues/33227"}