Skip to content
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

Fix warnings in the tests #883

Closed
kossnocorp opened this issue Sep 20, 2018 · 6 comments
Closed

Fix warnings in the tests #883

kossnocorp opened this issue Sep 20, 2018 · 6 comments

Comments

@kossnocorp
Copy link
Member

kossnocorp commented Sep 20, 2018

  1. Buffer related:
(node:4314) [DEP0005] DeprecationWarning: Buffer() is deprecated due to security and usability issues. Please use the Buffer.alloc(), Buffer.allocUnsafe(), or Buffer.from() methods instead.
  1. os.tmpDir related:
(node:75285) [DEP0022] DeprecationWarning: os.tmpDir() is deprecated. Use os.tmpdir() instead.
  1. find related
find: warning: you have specified the -mindepth option after a non-option argument -type, but options are not positional (-mindepth affects tests specified before it as well as those specified after it).  Please specify options before other arguments.
@kossnocorp kossnocorp changed the title Fix deprecation warnings in the tests Fix warnings in the tests Sep 20, 2018
@frytyler
Copy link

frytyler commented Oct 2, 2018

Hey I'll take a look at this and see if I can clean up the warnings.

@frytyler
Copy link

frytyler commented Oct 2, 2018

Hey @kossnocorp I've been trying to reproduce the issue by running yarn test or yarn test --single-run and I don't see those warnings. I do however see 11 failing tests. Not sure if I'm not seeing the warnings because of the failures though.

Are you able to help me out here? Do I need to run a specific suite?

@frytyler
Copy link

frytyler commented Oct 2, 2018

@kossnocorp oops, I was on the master branch which I didn't realize is v2. I just switched to v1 started to see the warnings.

But now I have a question. These warnings seem to be related to out of date packages. Will v2 bring those packages up to date? Or should I try to update the ones causing the issues.

@kossnocorp
Copy link
Member Author

@frytyler this issue is related only to master branch. We have the warnings in Node.js 10:

As far I as I remember I've fixed os.tmpDir updating Karma.

@piotrl
Copy link
Contributor

piotrl commented Jun 3, 2019

I analysed log, it seem deprecation warning about Buffer comes from yarn - we're using it in version 1.3.2, and since 1.10.0 the warning is gone.

I tested it on local machine with yarn 1.12.3 and I confirm - there is no deprecation warning anymore.
Therefore updating yarn on CI should do the trick.

@kossnocorp
Copy link
Member Author

@piotrl thank you, I'm on it!

kossnocorp added a commit that referenced this issue Jun 7, 2019
This should fix the warnings, see: #883 (comment)
elmomalmo pushed a commit to elmomalmo/date-fns that referenced this issue Jul 12, 2019
* Update Yarn to the latest version
* Fix find warning
* Fix yarn warning about passing args with --
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants