Skip to content

Commit

Permalink
rm failing csvy tests for CRAN
Browse files Browse the repository at this point in the history
  • Loading branch information
leeper committed Mar 29, 2018
1 parent 2d7be1c commit af8e331
Show file tree
Hide file tree
Showing 12 changed files with 173 additions and 135 deletions.
4 changes: 4 additions & 0 deletions .Rbuildignore
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,10 @@
^inst/standarderrors\.pdf$
^figure$
^figure/.+$
^man-roxygen$
^man-roxygen/.+$
^revdep$
^revdep/.+$
^cache/.+$
^docs$
^docs/.+$
Expand Down
4 changes: 2 additions & 2 deletions DESCRIPTION
Original file line number Diff line number Diff line change
@@ -1,8 +1,8 @@
Package: rio
Type: Package
Title: A Swiss-Army Knife for Data I/O
Version: 0.5.9
Date: 2018-02-01
Version: 0.5.10
Date: 2018-03-29
Authors@R: c(person("Jason", "Becker", role = "ctb", email = "[email protected]"),
person("Chung-hong", "Chan", role = "aut", email = "[email protected]"),
person("Geoffrey CH", "Chan", role = "ctb", email = "[email protected]"),
Expand Down
4 changes: 4 additions & 0 deletions NEWS.md
Original file line number Diff line number Diff line change
@@ -1,3 +1,7 @@
# rio 0.5.10

* Removed some csvy-related tests, which were failing on CRAN.

# rio 0.5.9

* Removed longstanding warnings from the tests of `export()` to fixed-width format.
Expand Down
6 changes: 3 additions & 3 deletions README.Rmd
Original file line number Diff line number Diff line change
Expand Up @@ -175,11 +175,11 @@ The core advantage of **rio** is that it makes assumptions that the user is prob

## Package Installation

[![CRAN Version](http://www.r-pkg.org/badges/version/rio)](https://cran.r-project.org/package=rio)
![Downloads](http://cranlogs.r-pkg.org/badges/rio)
[![CRAN Version](https://www.r-pkg.org/badges/version/rio)](https://cran.r-project.org/package=rio)
![Downloads](https://cranlogs.r-pkg.org/badges/rio)
[![Travis-CI Build Status](https://travis-ci.org/leeper/rio.png?branch=master)](https://travis-ci.org/leeper/rio)
[![Appveyor Build status](https://ci.appveyor.com/api/projects/status/40ua5l06jw0gjyjb?svg=true)](https://ci.appveyor.com/project/leeper/rio)
[![codecov.io](http://codecov.io/github/leeper/rio/coverage.svg?branch=master)](http://codecov.io/github/leeper/rio?branch=master)
[![codecov.io](https://codecov.io/github/leeper/rio/coverage.svg?branch=master)](https://codecov.io/github/leeper/rio?branch=master)

The package is available on [CRAN](https://cran.r-project.org/package=rio) and can be installed directly in R using `install.packages()`. You may want to run `install_formats()` after the first installation.

Expand Down
254 changes: 139 additions & 115 deletions README.md

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion docs/articles/index.html

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion docs/articles/rio.html

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 2 additions & 2 deletions docs/authors.html

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

9 changes: 9 additions & 0 deletions docs/news/index.html

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion docs/reference/index.html

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

15 changes: 6 additions & 9 deletions tests/testthat/test_format_csvy.R
Original file line number Diff line number Diff line change
@@ -1,18 +1,15 @@
context("CSVY imports/exports")
require("datasets")

tmp <- tempfile(fileext = ".csvy")

test_that("Export to CSVY", {
suppressWarnings(expect_true(export(iris, "iris.csvy") %in% dir()))
unlink("iris.csvy")
suppressWarnings(expect_true(file.exists(export(iris, tmp))))
})

test_that("Import from CSVY", {
suppressWarnings(d <- import(system.file("examples", "example.csvy", package = "csvy")))
suppressWarnings(d <- import(tmp))
expect_true(inherits(d, "data.frame"))

suppressWarnings(d2 <- import(system.file("examples", "example2.csvy", package = "csvy")))
expect_true(all(c("title", "units", "source") %in% names(attributes(d2))))

suppressWarnings(d3 <- import(system.file("examples", "example3.csvy", package = "csvy")))
expect_true(identical(dim(d3), c(2L, 3L)))
})

unlink(tmp)
2 changes: 1 addition & 1 deletion tests/testthat/test_remote.R
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ test_that("Import Remote Stata File", {
})

test_that("Import Remote GitHub File", {
rfile <- "https://raw.githubusercontent.com/leeper/csvy/master/inst/examples/example.csvy"
rfile <- "https://raw.githubusercontent.com/leeper/rio/master/inst/examples/no_header.csv"
rfile_imported <- try(import(rfile))
if (!inherits(rfile_imported, "try-error")) {
expect_true(inherits(rfile_imported, "data.frame"), label = "Import remote file")
Expand Down

0 comments on commit af8e331

Please sign in to comment.