pr: apply suggestions

Author: averissimo

NEWS.md

@@ -4,7 +4,9 @@
 
 * `get_code()` was extended with `names` parameter and allows the code extraction to be limited to objects stored in
 `qenv` but limited to `names`.
-* `qenv` inherits from the `environment` class, allowing to use `names()`, `as.environment()`, R operators (`[[` and `$`) and other functions on `qenv` objects.
+* `qenv` inherits from the `environment` class, allowing to use `ls()`, `names()`, `as.environment()` and other functions on `qenv` objects.
+* `join()` method is deprecated, please use `c()` instead
+* `get_var()` method is deprecated, please use `get`, `[[` or `$` instead.
 
 # teal.code 0.5.0
 

tests/testthat/test-qenv_constructor.R

@@ -60,8 +60,3 @@ testthat::describe("parent of qenv environment is the parent of .GlobalEnv", {
     testthat::expect_identical(parent.env(q), parent.env(.GlobalEnv))
   })
 })
-
-testthat::test_that("qenv environment is locked", {
-  q <- qenv()
-  testthat::expect_error(q@.xData$x <- 1, "cannot add bindings to a locked environment")
-})