search

stianlagstad / chimeraviz

chimeraviz is an R package that automates the creation of chimeric RNA visualizations.
36 stars 14 forks source link

Improve test coverage #58

Closed stianlagstad closed 5 years ago

lintr-bot commented 5 years ago

tests/testthat/test_utilities.R:301:1: style: lines should not be more than 80 characters.

​    system.file("extdata", "fusion5267and11759reads.bam", package = "chimeraviz")
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

tests/testthat/test_utilities.R:349:1: style: lines should not be more than 80 characters.

​    system.file("extdata", "fusion5267and11759reads.bedGraph", package = "chimeraviz")
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

tests/testthat/test_utilities.R:372:23: style: Variable and function names should not be longer than 30 characters.

​  argument_checker <- .is_either_bamfile_or_bedgraphfile_valid(
                      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

tests/testthat/test_utilities.R:375:1: style: lines should not be more than 80 characters.

​    system.file("extdata", "fusion5267and11759reads.bedGraph", package = "chimeraviz")
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

tests/testthat/test_utilities.R:380:23: style: Variable and function names should not be longer than 30 characters.

​  argument_checker <- .is_either_bamfile_or_bedgraphfile_valid(
                      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

tests/testthat/test_utilities.R:382:1: style: lines should not be more than 80 characters.

​    system.file("extdata", "fusion5267and11759reads.bam", package = "chimeraviz"),
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

tests/testthat/test_utilities.R:390:23: style: Variable and function names should not be longer than 30 characters.

​  argument_checker <- .is_either_bamfile_or_bedgraphfile_valid(
                      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

tests/testthat/test_utilities.R:392:1: style: lines should not be more than 80 characters.

​    system.file("extdata", "fusion5267and11759reads.bam", package = "chimeraviz"),
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

tests/testthat/test_utilities.R:393:1: style: lines should not be more than 80 characters.

​    system.file("extdata", "fusion5267and11759reads.bedGraph", package = "chimeraviz")
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

tests/testthat/test_utilities.R:400:23: style: Variable and function names should not be longer than 30 characters.

​  argument_checker <- .is_either_bamfile_or_bedgraphfile_valid(
                      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
lintr-bot commented 5 years ago

tests/testthat/test_utilities.R:380:23: style: Variable and function names should not be longer than 30 characters.

​  argument_checker <- .is_either_bamfile_or_bedgraphfile_valid(
                      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

tests/testthat/test_utilities.R:392:23: style: Variable and function names should not be longer than 30 characters.

​  argument_checker <- .is_either_bamfile_or_bedgraphfile_valid(
                      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

tests/testthat/test_utilities.R:406:23: style: Variable and function names should not be longer than 30 characters.

​  argument_checker <- .is_either_bamfile_or_bedgraphfile_valid(
                      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

tests/testthat/test_utilities.R:424:23: style: Variable and function names should not be longer than 30 characters.

​  argument_checker <- .is_either_bamfile_or_bedgraphfile_valid(
                      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
lintr-bot commented 5 years ago

tests/testthat/test_utilities.R:380:1: style: lines should not be more than 80 characters.

​  argument_checker <- .is_either_bamfile_or_bedgraphfile_valid( # Exclude Linting
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

tests/testthat/test_utilities.R:380:23: style: Variable and function names should not be longer than 30 characters.

​  argument_checker <- .is_either_bamfile_or_bedgraphfile_valid( # Exclude Linting
                      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

tests/testthat/test_utilities.R:392:1: style: lines should not be more than 80 characters.

​  argument_checker <- .is_either_bamfile_or_bedgraphfile_valid( # Exclude Linting
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

tests/testthat/test_utilities.R:392:23: style: Variable and function names should not be longer than 30 characters.

​  argument_checker <- .is_either_bamfile_or_bedgraphfile_valid( # Exclude Linting
                      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

tests/testthat/test_utilities.R:406:1: style: lines should not be more than 80 characters.

​  argument_checker <- .is_either_bamfile_or_bedgraphfile_valid( # Exclude Linting
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

tests/testthat/test_utilities.R:406:23: style: Variable and function names should not be longer than 30 characters.

​  argument_checker <- .is_either_bamfile_or_bedgraphfile_valid( # Exclude Linting
                      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

tests/testthat/test_utilities.R:424:1: style: lines should not be more than 80 characters.

​  argument_checker <- .is_either_bamfile_or_bedgraphfile_valid( # Exclude Linting
^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

tests/testthat/test_utilities.R:424:23: style: Variable and function names should not be longer than 30 characters.

​  argument_checker <- .is_either_bamfile_or_bedgraphfile_valid( # Exclude Linting
                      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
codecov-io commented 5 years ago

Codecov Report

Merging #58 into master will increase coverage by 1.59%. The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##           master     #58      +/-   ##
=========================================
+ Coverage    65.5%   67.1%   +1.59%     
=========================================
  Files          21      21              
  Lines        4514    4514              
=========================================
+ Hits         2957    3029      +72     
+ Misses       1557    1485      -72
Impacted Files Coverage Δ
R/utilities.R 80.16% <0%> (+14.57%) :arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update b554d48...453ee4f. Read the comment docs.

codecov-io commented 5 years ago

Codecov Report

Merging #58 into master will increase coverage by 1.59%. The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##           master     #58      +/-   ##
=========================================
+ Coverage    65.5%   67.1%   +1.59%     
=========================================
  Files          21      21              
  Lines        4514    4514              
=========================================
+ Hits         2957    3029      +72     
+ Misses       1557    1485      -72
Impacted Files Coverage Δ
R/utilities.R 80.16% <0%> (+14.57%) :arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update b554d48...453ee4f. Read the comment docs.