Skip to content

Tests are failing #123

Closed
Closed

Description

@RyanZim
  1 test failed [21:00:18]



  error › invalid --config
  /home/ryan/postcss-cli/test/error.js:44

   43:   .then(({ err, code }) => {                         
   44:     t.is(code, 1, 'expected non-zero error code')    
   45:     t.regex(err, /ENOENT: no such file or directory/)

  Actual:

    0

  Expected:

    1

  expected non-zero error code

Attn: @michael-ciniawsky

I suspect this has something to do with the latest patch version of cosmiconfig, will investigate later if no one else beats me to it.

Activity

  1. michael-ciniawsky commented on Apr 29, 2017

    @michael-ciniawsky
    Contributor

    This error /ENOENT: no such file or directory/ was removed with the latest cosmiconfig update

    cosmiconfig/cosmiconfig#63
    cosmiconfig/cosmiconfig#64

  2. RyanZim commented on Apr 29, 2017

    @RyanZim
    CollaboratorAuthor

    OK, thanks for doing the lookup!

    It seems that we should still error out somehow if the config fails to load.

  3. michael-ciniawsky commented on Apr 29, 2017

    @michael-ciniawsky
    Contributor

    Yep, but I don't now how yet tbh 😛 Now they shifted the responsibility for all their crap on us again, suckers.... 😛 I also need to add something like resolve-from to postcss-load-config, because currently, if the CLI is installed globally it fails atm, (it doesn't require() the plugins from process.cwd()) I fix it soon when time :)

  4. RyanZim commented on Apr 29, 2017

    @RyanZim
    CollaboratorAuthor

    I also need to add something like resolve-from to postcss-load-config, because currently, if the CLI is installed globally it fails atm, (it doesn't require() the plugins from process.cwd()) I fix it soon when time :)

    Ugh; that's why it's good to use devDependencies + npm scripts

  5. michael-ciniawsky commented on Apr 29, 2017

    @michael-ciniawsky
    Contributor

    Yep I also prefer it that way, but the fix is easy and it's nothing urgent 😛

  6. RyanZim commented on Feb 10, 2018

    @RyanZim
    CollaboratorAuthor

    @michael-ciniawsky Is this worth fixing, or do we just remove that test and leave things as they are?

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

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions