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

Mirror unit test #379

Draft
wants to merge 4 commits into
base: unstable
Choose a base branch
from
Draft

Mirror unit test #379

wants to merge 4 commits into from

Conversation

zbeekman
Copy link
Member

No description provided.

@codecov
Copy link

codecov bot commented Sep 21, 2020

Codecov Report

Merging #379 into unstable will decrease coverage by 0.45%.
The diff coverage is 66.66%.

Impacted file tree graph

@@             Coverage Diff              @@
##           unstable     #379      +/-   ##
============================================
- Coverage     68.39%   67.93%   -0.46%     
============================================
  Files            92       92              
  Lines          7995     7996       +1     
  Branches       1374     1374              
============================================
- Hits           5468     5432      -36     
- Misses         2030     2070      +40     
+ Partials        497      494       -3     
Flag Coverage Δ
#CI 67.93% <66.66%> (-0.46%) ⬇️
#unittests 67.93% <66.66%> (-0.46%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
packages/taucmdr/cf/software/installation.py 61.51% <ø> (-1.36%) ⬇️
packages/taucmdr/cf/software/tau_installation.py 55.19% <0.00%> (-0.58%) ⬇️
...kages/taucmdr/cf/software/binutils_installation.py 55.81% <100.00%> (+0.51%) ⬆️
packages/taucmdr/util.py 62.34% <0.00%> (-6.62%) ⬇️

@github-actions
Copy link

Pylint Output

Report

8103 statements analysed.

Statistics by type

type number old number difference %documented %badname
module 93 93 = 100.00 0.00
class 116 116 = 99.14 0.86
method 591 591 = 98.82 1.02
function 116 116 = 94.83 0.86

Raw metrics

type number % previous difference
code 9807 51.62 9804 +3.00
docstring 4822 25.38 4817 +5.00
comment 2841 14.95 2841 =
empty 1527 8.04 1526 +1.00

Duplication

now previous difference
nb duplicated lines 0 0 =
percent duplicated lines 0.000 0.000 =

Messages by category

type number previous difference
convention 19 20 -1.00
refactor 15 15 =
warning 20 20 =
error 0 0 =

% errors / warnings by module

module error warning refactor convention
taucmdr.model.experiment 0.00 45.00 0.00 5.26
taucmdr.util 0.00 5.00 20.00 5.26
taucmdr.model.trial 0.00 5.00 13.33 0.00
taucmdr.model.project 0.00 5.00 13.33 0.00
taucmdr.model.target 0.00 5.00 6.67 0.00
taucmdr.cf.storage.local_file 0.00 5.00 6.67 0.00
taucmdr.cf.software.papi_installation 0.00 5.00 0.00 10.53
taucmdr.cli.commands.trial.renumber 0.00 5.00 0.00 0.00
taucmdr.cli.commands.target.edit 0.00 5.00 0.00 0.00
taucmdr.cli.commands.target.create 0.00 5.00 0.00 0.00
taucmdr.cli.commands.target.copy 0.00 5.00 0.00 0.00
taucmdr.init 0.00 5.00 0.00 0.00
taucmdr.cf.software.tau_installation 0.00 0.00 20.00 26.32
taucmdr.cf.compiler.init 0.00 0.00 6.67 5.26
taucmdr.cli.commands.select 0.00 0.00 6.67 0.00
taucmdr.cli.init 0.00 0.00 6.67 0.00
taucmdr.mvc.model 0.00 0.00 0.00 10.53
taucmdr.mvc.controller 0.00 0.00 0.00 10.53
taucmdr.progress 0.00 0.00 0.00 5.26
taucmdr.model.application 0.00 0.00 0.00 5.26
taucmdr.error 0.00 0.00 0.00 5.26
taucmdr.cli.commands.experiment.edit 0.00 0.00 0.00 5.26
taucmdr.cf.storage.levels 0.00 0.00 0.00 5.26

Messages

message id occurrences
missing-docstring 14
arguments-differ 12
inconsistent-return-statements 10
invalid-name 4
useless-super-delegation 3
too-many-nested-blocks 2
too-many-locals 2
unused-argument 1
too-many-public-methods 1
redefined-builtin 1
raising-format-tuple 1
logging-not-lazy 1
fixme 1
bad-continuation 1

Your code has been rated at 9.83/10 (previous run: 9.83/10, +0.00)

Per-file output (click to expand)
************* Module taucmdr
W: 42, 4: Redefining built-in 'basestring' (redefined-builtin)
************* Module taucmdr.error
C: 76, 4: Missing method docstring (missing-docstring)
************* Module taucmdr.progress
C:178, 4: Missing method docstring (missing-docstring)
************* Module taucmdr.cf.software.papi_installation
C: 49, 0: Wrong continued indentation (add 7 spaces).
         'http://fs.paratools.com/tau-mirror/papi-5.5.1.tar.gz']}
         ^      | (bad-continuation)
W: 69, 4: Parameters differ from overridden '_prepare_src' method (arguments-differ)
C: 77, 8: Variable name "cc" doesn't conform to u'[a-z_][a-z0-9_]{2,30}$' pattern (invalid-name)
************* Module taucmdr.util
R:143, 0: Either all return statements in a function should return an expression, or none of them should. (inconsistent-return-statements)
R:423, 0: Either all return statements in a function should return an expression, or none of them should. (inconsistent-return-statements)
W:465,18: Unused argument 'label' (unused-argument)
R:714, 4: Either all return statements in a function should return an expression, or none of them should. (inconsistent-return-statements)
C:734, 4: Missing function docstring (missing-docstring)
************* Module taucmdr.cf.storage.levels
C: 64, 0: Missing function docstring (missing-docstring)
************* Module taucmdr.cf.storage.local_file
W: 80, 4: Parameters differ from overridden 'write' method (arguments-differ)
R:218, 4: Either all return statements in a function should return an expression, or none of them should. (inconsistent-return-statements)
************* Module taucmdr.cf.compiler
R:300, 8: Too many nested blocks (6/5) (too-many-nested-blocks)
C:662, 4: Missing method docstring (missing-docstring)
************* Module taucmdr.mvc.controller
C: 54, 0: Missing function docstring (missing-docstring)
C: 55, 4: Missing function docstring (missing-docstring)
************* Module taucmdr.mvc.model
C: 75, 4: Missing method docstring (missing-docstring)
C:349,16: Missing function docstring (missing-docstring)
************* Module taucmdr.cli
R:230, 0: Either all return statements in a function should return an expression, or none of them should. (inconsistent-return-statements)
************* Module taucmdr.cf.software.tau_installation
C:800,45: Variable name "f" doesn't conform to u'[a-z_][a-z0-9_]{2,30}$' pattern (invalid-name)
C:900, 4: Missing method docstring (missing-docstring)
R:927, 4: Either all return statements in a function should return an expression, or none of them should. (inconsistent-return-statements)
R:1679, 4: Either all return statements in a function should return an expression, or none of them should. (inconsistent-return-statements)
C:1828, 4: Missing method docstring (missing-docstring)
C:1853, 8: Variable name "p" doesn't conform to u'[a-z_][a-z0-9_]{2,30}$' pattern (invalid-name)
C:1854, 8: Variable name "m" doesn't conform to u'[a-z_][a-z0-9_]{2,30}$' pattern (invalid-name)
R:169, 0: Too many public methods (26/25) (too-many-public-methods)
************* Module taucmdr.cli.commands.select
R:118, 4: Either all return statements in a function should return an expression, or none of them should. (inconsistent-return-statements)
************* Module taucmdr.cli.commands.trial.renumber
W: 84, 8: Specify string format arguments as logging function parameters (logging-not-lazy)
************* Module taucmdr.cli.commands.experiment.edit
C: 39, 0: Missing class docstring (missing-docstring)
************* Module taucmdr.cli.commands.target.copy
W: 59,12: Parameters differ from overridden '__call__' method (arguments-differ)
************* Module taucmdr.cli.commands.target.edit
W: 61,12: Parameters differ from overridden '__call__' method (arguments-differ)
************* Module taucmdr.cli.commands.target.create
W: 68,12: Parameters differ from overridden '__call__' method (arguments-differ)
************* Module taucmdr.model.application
C: 50, 0: Missing function docstring (missing-docstring)
************* Module taucmdr.model.project
W: 98, 4: Parameters differ from overridden 'delete' method (arguments-differ)
R:142, 4: Too many local variables (21/20) (too-many-locals)
R:150, 8: Too many nested blocks (8/5) (too-many-nested-blocks)
************* Module taucmdr.model.target
W:673, 0: FIXME: not implemented (fixme)
R:670, 4: Either all return statements in a function should return an expression, or none of them should. (inconsistent-return-statements)
************* Module taucmdr.model.trial
R:353, 4: Either all return statements in a function should return an expression, or none of them should. (inconsistent-return-statements)
R:529, 4: Too many local variables (24/20) (too-many-locals)
W:587,24: Exception arguments suggest string formatting might be intended (raising-format-tuple)
************* Module taucmdr.model.experiment
W:109, 4: Parameters differ from overridden 'one' method (arguments-differ)
W:113, 4: Parameters differ from overridden 'all' method (arguments-differ)
W:117, 4: Parameters differ from overridden 'count' method (arguments-differ)
W:123, 4: Parameters differ from overridden 'search' method (arguments-differ)
W:127, 4: Parameters differ from overridden 'exists' method (arguments-differ)
W:134, 4: Useless super delegation in method 'create' (useless-super-delegation)
W:137, 4: Useless super delegation in method 'update' (useless-super-delegation)
W:140, 4: Useless super delegation in method 'unset' (useless-super-delegation)
W:143, 4: Parameters differ from overridden 'delete' method (arguments-differ)
C:432, 4: Missing method docstring (missing-docstring)
Stderror
Using config file /home/runner/work/taucmdr/taucmdr/pylintrc

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

Successfully merging this pull request may close these issues.

1 participant