excluded test-loadAnnot.R from coverage analysis

mschilli87 commented 7 years ago

I have no clue why @lintr-bot keeps spamming about the disabled camel_case_linter. Also covr doesn't seem to get triggerd by command line merges, so you might want to open a pull request before merging to master.

mschilli87 commented 7 years ago

@retaj: Please don't merge this. I think covr anyways checks only the R/ directory but I want to make sure to trigger a coverage check.

lintr-bot commented 7 years ago

tests/testthat.R:2:9: style: Variable and function names should be all lowercase.

library(ciRcus)
        ^~~~~~

tests/testthat/test-loadAnnot.R:87:25: style: Variable and function names should be all lowercase.

  expect_equal(rownames(colData(se.sorted)),   colData(se.sorted)$sample)
                        ^~~~~~~

tests/testthat/test-loadAnnot.R:87:48: style: Variable and function names should be all lowercase.

  expect_equal(rownames(colData(se.sorted)),   colData(se.sorted)$sample)
                                               ^~~~~~~

tests/testthat/test-loadAnnot.R:88:25: style: Variable and function names should be all lowercase.

  expect_equal(rownames(colData(se.unsorted)), colData(se.unsorted)$sample)
                        ^~~~~~~

tests/testthat/test-loadAnnot.R:88:48: style: Variable and function names should be all lowercase.

  expect_equal(rownames(colData(se.unsorted)), colData(se.unsorted)$sample)
                                               ^~~~~~~