P281424/AMR
1
0
Fork 0
mirror of https://github.com/msberends/AMR.git synced 2024-12-25 18:06:12 +01:00
Code Releases Activity

fix clipboard check

Browse Source
This commit is contained in:
dr. M.S. (Matthijs) Berends 2018-04-30 16:54:37 +02:00
parent 8fe70d5652
commit 6fa93fc286
2 changed files with 10 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
R/clipboard.R
View File

@ -53,7 +53,7 @@ clipboard_import <- function(sep = '\t',
FALSE
})
if (import_tbl == FALSE) {
if (all(import_tbl == FALSE)) {
cat("No clipboard content found.")
if (Sys.info()['sysname'] %like% "Linux") {
cat(" These functions do not work without X11 installed.")
@ -127,11 +127,11 @@ clipboard_export <- function(x,
}
is_Windows <- function() {
all(Sys.info()['sysname'] %like% "Windows")
Sys.info()['sysname'] %like% "Windows"
}
check_xclip <- function() {
if (!isTRUE(file.exists(Sys.which("xclip")[1L]))) {
if (all(Sys.info()['sysname'] %like% "Linux")) {
if (Sys.info()['sysname'] %like% "Linux") {
stop("Please install Linux package xclip first.")
} else {
stop("Please install package xclip first (use `brew install xclip` on macOS).")

14
tests/testthat/test-clipboard.R
View File

@ -1,12 +1,12 @@
context("clipboard.R")
test_that("clipboard works", {
t1 <<- AMR::antibiotics # why is the <<- needed? Won't work without it...
clipboard_export(t1, info = FALSE)
t2 <- clipboard_import()
if (is.null(t1) | is.null(t2)) {
t1 <- TRUE
t2 <- TRUE
if (grepl(Sys.info()['sysname'], "windows", ignore.case = TRUE)) {
t1 <<- AMR::antibiotics # why is the <<- needed? Won't work without it...
clipboard_export(t1, info = FALSE)
t2 <- clipboard_import()
expect_equal(t1, t2)
} else {
expect_equal(TRUE, TRUE)
}
expect_equal(t1, t2)
})