cwltool: make coverage.html; make coverage-report fails

make coverage.html 
make coverage-report fails 

these commands fail due to the following error:

Finished processing dependencies for cwltool==1.0.20170309164828
export COVERAGE_PROCESS_START=/home/manu/github/cwltool/.coveragerc; \
       cd ; ./run_test.sh RUNNER=cwltool
/bin/sh: 2: ./run_test.sh: not found
Makefile:110: recipe for target '.coverage' failed
make: *** [.coverage] Error 127

related line of code in makefile: https://github.com/common-workflow-language/cwltool/blob/master/Makefile#L111

About this issue

  • Original URL
  • State: closed
  • Created 7 years ago
  • Comments: 22 (22 by maintainers)

Commits related to this issue

Most upvoted comments

@mr-c Working now. Need to remove hardcoded paths though

(venv) manu@hp:~/.../cwltool/common-workflow-language$ coverage report --omit="*/site-packages/*","*/schema_salad/*"
Name                                                         Stmts   Miss  Cover
--------------------------------------------------------------------------------
/home/manu/github/cwltool/cwltool.py                             5      0   100%
/home/manu/github/cwltool/cwltool/__init__.py                    2      0   100%
/home/manu/github/cwltool/cwltool/builder.py                   190    163    14%
/home/manu/github/cwltool/cwltool/cwlrdf.py                     84     71    15%
/home/manu/github/cwltool/cwltool/docker.py                     99     85    14%
/home/manu/github/cwltool/cwltool/docker_id.py                  46     32    30%
/home/manu/github/cwltool/cwltool/draft2tool.py                476    424    11%
/home/manu/github/cwltool/cwltool/errors.py                      4      0   100%
/home/manu/github/cwltool/cwltool/expression.py                174    144    17%
/home/manu/github/cwltool/cwltool/flatten.py                    19     16    16%
/home/manu/github/cwltool/cwltool/job.py                       314    274    13%
/home/manu/github/cwltool/cwltool/load_tool.py                 151    124    18%
/home/manu/github/cwltool/cwltool/main.py                      560    395    29%
/home/manu/github/cwltool/cwltool/mutation.py                   37     25    32%
/home/manu/github/cwltool/cwltool/pack.py                      125    112    10%
/home/manu/github/cwltool/cwltool/pathmapper.py                175    135    23%
/home/manu/github/cwltool/cwltool/process.py                   548    466    15%
/home/manu/github/cwltool/cwltool/resolver.py                   40     30    25%
/home/manu/github/cwltool/cwltool/sandboxjs.py                 211    185    12%
/home/manu/github/cwltool/cwltool/software_requirements.py      74     53    28%
/home/manu/github/cwltool/cwltool/stdfsaccess.py                51     25    51%
/home/manu/github/cwltool/cwltool/update.py                    377    329    13%
/home/manu/github/cwltool/cwltool/utils.py                     104     84    19%
/home/manu/github/cwltool/cwltool/workflow.py                  635    575     9%
--------------------------------------------------------------------------------
TOTAL                                                         4501   3747    17%

In summary, please don’t use the conformance tests inside cwltool/cwltool/schemas as they are out of date 😃

@mr-c thanks for pointing this out. I was using them yesterday while debugging my PR: https://github.com/common-workflow-language/cwltool/pull/337

Yes, we should explain how to run the conformance tests in the README – PR gladly accepted

I’m thinking of adding short description on running tox locally. And adding essential contents from https://github.com/common-workflow-language/cwltool/blob/master/cwltool/schemas/CONFORMANCE_TESTS.md