fix code parser for := operator

R/utils-get_code_dependency.R

@@ -106,6 +106,7 @@ extract_calls <- function(pd) {
   calls <- Filter(function(call) !(nrow(call) == 1 && call$token == "';'"), calls)
   calls <- Filter(Negate(is.null), calls)
   calls <- fix_shifted_comments(calls)
+  calls <- remove_dt_assign(calls)
   fix_arrows(calls)
 }
 
@@ -144,6 +145,16 @@ fix_shifted_comments <- function(calls) {
   Filter(nrow, calls)
 }
 
+#' Fixes edge case of `:=` assignment operator being treated as assignment.
+#' @keywords internal
+#' @noRd
+remove_dt_assign <- function(calls) {
+  checkmate::assert_list(calls)
+  lapply(calls, function(call) {
+    call <- subset(call, !(token == "LEFT_ASSIGN" & text == ":="))
+  })
+}
+
 #' Fixes edge case of `<-` assignment operator being called as function,
 #' which is \code{`<-`(y,x)} instead of traditional `y <- x`.
 #' @keywords internal

tests/testthat/test-qenv_eval_code.R

@@ -105,6 +105,15 @@ testthat::test_that("eval_code accepts calls containing only comments and empty
   testthat::expect_identical(get_code(eval_code(qenv(), code)), code)
 })
 
+testthat::test_that("eval_code does not treat := as an assignment operator", {
+  code <- "
+    x <- 'name'
+    rlang::list2(!!x := 1)
+  "
+  q <- eval_code(qenv(), code)
+  testthat::expect_identical(get_code(q), code)
+})
+
 # comments ----------
 testthat::test_that("comments fall into proper calls", {
   # If comment is on top, it gets moved to the first call.