Hello.

I tried to come up a patch for this issue but I found out something interesting. In short, I believe these issue is already covered by https://github.com/shelljs/shelljs/blob/master/test/ls.js#L333. The report of codecov might be false negative.

I did the following things:

It's kind of silly but I injected console.log in lines between https://github.com/shelljs/shelljs/blob/master/src/ls.js#L82 and https://github.com/shelljs/shelljs/blob/master/src/ls.js#L86, then ran unit test locally. I did see these console.log got printed.
Ran test locally via npm run test and the coverage report is a little bit different from https://codecov.io/gh/shelljs/shelljs/src/master/src/ls.js .
My local coverage report looks like this: (on Mac OSX 10.11.6 with node v8.10.0)

----------------|----------|----------|----------|----------|-------------------|
File            |  % Stmts | % Branch |  % Funcs |  % Lines | Uncovered Line #s |
----------------|----------|----------|----------|----------|-------------------|
All files       |    95.81 |    89.21 |    99.24 |     96.6 |                   |
 shelljs        |      100 |      100 |      100 |      100 |                   |
  commands.js   |      100 |      100 |      100 |      100 |                   |
  global.js     |      100 |      100 |      100 |      100 |                   |
  plugin.js     |      100 |      100 |      100 |      100 |                   |
  shell.js      |      100 |      100 |      100 |      100 |                   |
 shelljs/src    |    95.74 |    89.21 |    99.22 |    96.54 |                   |
  cat.js        |      100 |      100 |      100 |      100 |                   |
  cd.js         |      100 |     87.5 |      100 |      100 |                35 |
  chmod.js      |    89.41 |    90.12 |      100 |    89.41 |... 81,183,187,192 |
  common.js     |    97.94 |    94.07 |    96.55 |     98.4 |       260,327,358 |
  cp.js         |    89.84 |    87.76 |      100 |    91.13 |... 76,177,178,179 |
  dirs.js       |    97.18 |    93.88 |      100 |    97.18 |             27,29 |
  echo.js       |      100 |    83.33 |      100 |      100 |                43 |
  error.js      |      100 |      100 |      100 |      100 |                   |
  exec-child.js |      100 |      100 |      100 |      100 |                   |
  exec.js       |    93.51 |    86.49 |      100 |     97.1 |           132,133 |
  find.js       |    95.45 |     62.5 |      100 |    95.45 |                34 |
  grep.js       |      100 |      100 |      100 |      100 |                   |
  head.js       |      100 |      100 |      100 |      100 |                   |
  ln.js         |    96.88 |    79.17 |      100 |    96.88 |                53 |
  ls.js         |    97.87 |       90 |      100 |    97.87 |                64 |
...
----------------|----------|----------|----------|----------|-------------------|

For ls.js, there is only https://github.com/shelljs/shelljs/blob/master/src/ls.js#64 is not covered, because this test is not run on Windows.
In order to see more detail, I change reporter type to html

node_modules/.bin/nyc --reporter=html --reporter=lcov node_modules/.bin/ava test/*.js

I put the html format report here.

I read through unit test and source code of ls.js and I think this issue is probably covered by https://github.com/shelljs/shelljs/blob/master/test/ls.js#L333 already.

Anyway, I'll try to find a linux machine and run test coverage to see if I can reproduce the same result.

Description

Metadata

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

Issue actions