Skip to content

Commit

Permalink
attempt to fix dependencies on DiagrammeRsvg et al.
Browse files Browse the repository at this point in the history
  • Loading branch information
rempsyc committed Jun 9, 2024
1 parent 7ffa7fe commit 243a49a
Show file tree
Hide file tree
Showing 3 changed files with 10 additions and 2 deletions.
3 changes: 2 additions & 1 deletion R/nice_lavaanPlot.R
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,8 @@ nice_lavaanPlot <- function(
graph_options = c(rankdir = "LR"), ...) {
insight::check_if_installed(
c(
"lavaanPlot", "DiagrammeRsvg", "rsvg", "png", "webshot"
"lavaanPlot"#, #"DiagrammeRsvg",
# "rsvg", "png", "webshot"
),
reason = "for this function."
)
Expand Down
4 changes: 3 additions & 1 deletion tests/testthat/test-cfa_fit_plot.R
Original file line number Diff line number Diff line change
Expand Up @@ -27,6 +27,7 @@ fit2 <- cfa(model, data, estimator = estimator)

test_that("cfa_fit_plot comparison to cfa", {
skip_if_not_installed("lavaanPlot")
skip_if_not_installed("DiagrammeRsvg")
fit1 <- cfa_fit_plot(model, data)
expect_equal(
summary(fit1),
Expand All @@ -37,6 +38,7 @@ test_that("cfa_fit_plot comparison to cfa", {
test_that("cfa_fit_plot save as PDF", {
skip_on_cran()
skip_if_not_installed("lavaanPlot")
skip_if_not_installed("DiagrammeRsvg")
fit3 <- cfa_fit_plot(model, data, save.as.pdf = TRUE, file.name = "cfaplot")
expect_equal(
summary(fit3),
Expand All @@ -49,6 +51,7 @@ fit5 <- cfa(model, data, estimator = estimator)

test_that("cfa_fit_plot remove items", {
skip_if_not_installed("lavaanPlot")
skip_if_not_installed("DiagrammeRsvg")
fit4 <- cfa_fit_plot(model, data, print = FALSE, remove.items = c("x1"))
expect_equal(
summary(fit4),
Expand All @@ -62,5 +65,4 @@ test_that("cfa_fit_plot save as PDF error", {
)
})


setwd(.old_wd)
5 changes: 5 additions & 0 deletions tests/testthat/test-nice_lavaanPlot.R
Original file line number Diff line number Diff line change
Expand Up @@ -45,6 +45,7 @@ fit.sem2 <- sem(HS.model2, data)

test_that("nice_lavaanPlot on CFA", {
skip_if_not_installed("lavaanPlot")
skip_if_not_installed("DiagrammeRsvg")
expect_s3_class(
nice_lavaanPlot(fit.cfa),
c("grViz", "htmlwidget")
Expand All @@ -53,6 +54,7 @@ test_that("nice_lavaanPlot on CFA", {

test_that("nice_lavaanPlot on SEM", {
skip_if_not_installed("lavaanPlot")
skip_if_not_installed("DiagrammeRsvg")
expect_s3_class(
nice_lavaanPlot(fit.sem),
c("grViz", "htmlwidget")
Expand All @@ -61,6 +63,7 @@ test_that("nice_lavaanPlot on SEM", {

test_that("nice_lavaanPlot on lavaan", {
skip_if_not_installed("lavaanPlot")
skip_if_not_installed("DiagrammeRsvg")
expect_s3_class(
nice_lavaanPlot(fit.lavaan),
c("grViz", "htmlwidget")
Expand All @@ -69,8 +72,10 @@ test_that("nice_lavaanPlot on lavaan", {

test_that("nice_lavaanPlot different sem model", {
skip_if_not_installed("lavaanPlot")
skip_if_not_installed("DiagrammeRsvg")
expect_s3_class(
nice_lavaanPlot(fit.sem2),
c("grViz", "htmlwidget")
)
})

0 comments on commit 243a49a

Please sign in to comment.