Skip to content
This repository was archived by the owner on Jul 29, 2024. It is now read-only.

DeprecationWarning: os.tmpDir() is deprecated. Use os.tmpdir() instead. #4657

Closed
devoto13 opened this issue Jan 11, 2018 · 14 comments
Closed

Comments

@devoto13
Copy link
Contributor

Bug report

  • Node Version: 8.7.0
  • Protractor Version: 5.2.2
  • Angular Version: -
  • Browser(s): -
  • Operating System and Version macOS High Sierra
  1. Clone repo https://github.com/devoto13/protractor-warning
  2. npm install
  3. node --trace-warnings ./node_modules/.bin/protractor protractor.conf.js

Observe warning being printed to console:

DeprecationWarning: os.tmpDir() is deprecated. Use os.tmpdir() instead.

It does not break anything, but is annoying. I would like to get rid of it once and for all. It comes from the following dependency chain webdriver-js-extender -> selenium-webdriver@2.* -> tmp. One possible fix would be to update selenium-webdriver dependency for webdriver-js-extender or replace it with peer dependency. There is even a PR trying to do that - angular/webdriver-js-extender#12. However it didn't get any attention and @sjelin does not seem to be active on Github for almost a year. I just thought that maybe somebody from the Protractor team can give webdriver-js-extender some maintainer love :) Or if it is not possible maybe Protractor can consider dropping/replacing webdriver-js-extender.

@andreasmarkussen
Copy link

I have the exact same issue, which I think everyone has?

@jjdereu
Copy link

jjdereu commented Feb 28, 2018

+1

1 similar comment
@ksteuer
Copy link

ksteuer commented Mar 1, 2018

+1

@andreasmarkussen
Copy link

@juliemr - Are you aware what It takes to get some attention with this from the maintainers? And alternatively open up for additional maintainers?

@hassankaz
Copy link

Is there any workaround for this issue? This prevents me from running tests.

@qiyigg
Copy link
Contributor

qiyigg commented Mar 6, 2018

Sorry, I don't know much about webdriver-js-extender. Probably I can get some time to take a look at it this week, sorry for late response.
@hassankaz as far as I know, this shouldn't affect running tests, it just bring some annoying warnings, what error did you get?

@hassankaz
Copy link

Sorry my bad. My tests were not running for some other reason (webdriver-manager update needed to be run) . After I fixed the issue it's running again. thanks!

@heltonalves99
Copy link

heltonalves99 commented Mar 13, 2018

I have same issue. Any idea to fixes it ?

but my problem appears when I put shardTestFiles: true and maxInstances: 4 in protractor conf.

@devoto13
Copy link
Contributor Author

@qiyigg did you get a chance to look at webdriver-js-extender project yet?

@qiyigg
Copy link
Contributor

qiyigg commented Apr 14, 2018

@devoto13 Not yet. The thing is this project is maintained by another contributor who is not active for a long time, I did check with other contributors to see who is familiar with it, but unfortunately no one knows the detail. From what I know so far, probably we don't need this dependency any more, but it requires some investigation to remove it safely.

@devoto13
Copy link
Contributor Author

@qiyigg Thanks for an update!
Please let me know if there is something specific I can do to help move the investigation forward?

@garlapati99
Copy link

I am facing the same issue. how to resolve this
(node:21031) [DEP0022] DeprecationWarning: os.tmpDir() is deprecated. Use os.tmpdir() instead.
Thanks

@qiyigg
Copy link
Contributor

qiyigg commented Jul 16, 2018

I am working on a Protractor release today, this should be fixed after the new release.

@devoto13
Copy link
Contributor Author

This issues was fixed by 056eec2 and released in 5.4.0.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

8 participants