Skip to content

Commit

Permalink
Merge pull request #97 from Rapporteket/mock-shiny-session
Browse files Browse the repository at this point in the history
allow mock shiny session for dev contexts
  • Loading branch information
areedv authored Nov 4, 2021
2 parents 2a36f62 + 2aebc47 commit f2eba5c
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 2 deletions.
2 changes: 1 addition & 1 deletion DESCRIPTION
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
Package: rapbase
Type: Package
Title: Base Functions and Resources for Rapporteket
Version: 1.20.0
Version: 1.20.1
Authors@R: c(
person(given = "Are",
family = "Edvardsen",
Expand Down
4 changes: 4 additions & 0 deletions NEWS.md
Original file line number Diff line number Diff line change
@@ -1,3 +1,7 @@
# rapbase 1.20.1

* Fix unit testing issue by also allowing "MockShinySession" as an attribute of the shiny session object

# rapbase 1.20.0

* Navbar user information widget provided as a shiny module
Expand Down
2 changes: 1 addition & 1 deletion R/UserInfo.R
Original file line number Diff line number Diff line change
Expand Up @@ -80,7 +80,7 @@ userInfo <- function(entity, shinySession = NULL, devContexts = c("DEV"),
stop("Session information is empty! Eventually, that will come bite you")
}

if (!any(c("ShinySession", "session_proxy") %in%
if (!any(c("ShinySession", "session_proxy", "MockShinySession") %in%
attributes(shinySession)$class)) {
stop(paste(
"Got no object of class 'ShinySession' or 'session_proxy'!",
Expand Down

0 comments on commit f2eba5c

Please sign in to comment.