Skip to content
Please note that GitHub no longer supports your web browser.

We recommend upgrading to the latest Google Chrome or Firefox.

Learn more
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

Travis CI is for some reason failing to download files needed by `test_remote.R` specifically in the MacOS build. #246

Closed
bokov opened this issue Nov 4, 2019 · 2 comments · Fixed by #247
Labels
bug

Comments

@bokov
Copy link
Contributor

@bokov bokov commented Nov 4, 2019

Please specify whether your issue is about:

  • a possible bug
  • a question about package functionality
  • a suggested code or documentation change, improvement to the code, or feature request

If you are reporting (1) a bug or (2) a question about code, please supply:

Make a commit and view results on travis-ci.org:

── 1. Error: Import Remote GitHub File (@test_remote.R#16)  ────────────────────

Could not resolve host: raw.githubusercontent.com

1: remote_to_local(rfile) at testthat/test_remote.R:16

2: curl::curl_fetch_memory(file)

── 2. Error: Import Remote File from Shortened URL (@test_remote.R#26)  ────────

Could not resolve host: goo.gl

1: expect_true(inherits(import(shorturl), "data.frame"), label = "Import remote file") at testthat/test_remote.R:26

2: quasi_label(enquo(object), label, arg = "object")

3: eval_bare(get_expr(quo), get_env(quo))

4: inherits(import(shorturl), "data.frame")

5: import(shorturl)

6: remote_to_local(file, format = format)

7: curl::curl_fetch_memory(file)

── 3. Error: Import from Google Sheets (@test_remote.R#31)  ────────────────────

Could not resolve host: docs.google.com

1: expect_true(inherits(import(googleurl1), "data.frame"), label = "Import google sheets (specified sheet)") at testthat/test_remote.R:31

2: quasi_label(enquo(object), label, arg = "object")

3: eval_bare(get_expr(quo), get_env(quo))

4: inherits(import(googleurl1), "data.frame")

5: import(googleurl1)

6: remote_to_local(file, format = format)

7: curl::curl_fetch_memory(file)
@bokov

This comment has been minimized.

Copy link
Contributor Author

@bokov bokov commented Nov 4, 2019

Running these tests locally works (though I don't use a Mac)

@bokov

This comment has been minimized.

Copy link
Contributor Author

@bokov bokov commented Nov 4, 2019

bokov added a commit to bokov/rio that referenced this issue Nov 8, 2019
…e8.3 setting, letting the current default (and newer) version of xcode be used instead. Also took the before_install script back out because now the data.table binary is again available. Fixes leeper#246. Fixes leeper#242
@leeper leeper added the bug label Nov 10, 2019
bokov added a commit to bokov/rio that referenced this issue Dec 4, 2019
leeper added a commit that referenced this issue Dec 17, 2019
@leeper leeper closed this in #247 Dec 17, 2019
leeper added a commit that referenced this issue Dec 17, 2019
…e8.3 setting, letting the current default (and newer) version of xcode be used instead. Also took the before_install script back out because now the data.table binary is again available. Fixes #246. Fixes #242
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.