Unverified Commit b2705dc4 authored by Nikita Titov's avatar Nikita Titov Committed by GitHub
Browse files

[R-package][tests] remove unused lines in tests for the speedup (#3783)

* remove unused params from tests for the speedup

* Update test_lgb.Booster.R
parent ed651e86
...@@ -1735,7 +1735,6 @@ test_that("lgb.train() w/ linear learner fails already-constructed dataset with ...@@ -1735,7 +1735,6 @@ test_that("lgb.train() w/ linear learner fails already-constructed dataset with
data = dtrain data = dtrain
, nrounds = 10L , nrounds = 10L
, params = modifyList(params, list(linear_tree = TRUE)) , params = modifyList(params, list(linear_tree = TRUE))
, valids = list("train" = dtrain)
) )
}, regexp = "Cannot change linear_tree after constructed Dataset handle") }, regexp = "Cannot change linear_tree after constructed Dataset handle")
}) })
...@@ -1862,7 +1861,6 @@ test_that("lgb.train() works with linear learners and data where a feature has o ...@@ -1862,7 +1861,6 @@ test_that("lgb.train() works with linear learners and data where a feature has o
data = dtrain data = dtrain
, nrounds = 10L , nrounds = 10L
, params = modifyList(params, list(linear_tree = TRUE)) , params = modifyList(params, list(linear_tree = TRUE))
, valids = list("train" = dtrain)
) )
expect_true(lgb.is.Booster(bst_linear)) expect_true(lgb.is.Booster(bst_linear))
}) })
......
...@@ -188,7 +188,6 @@ test_that("boosters with linear models at leaves can be written to text file and ...@@ -188,7 +188,6 @@ test_that("boosters with linear models at leaves can be written to text file and
data = dtrain data = dtrain
, nrounds = 10L , nrounds = 10L
, params = params , params = params
, valids = list("train" = dtrain)
) )
expect_true(lgb.is.Booster(bst)) expect_true(lgb.is.Booster(bst))
...@@ -204,8 +203,8 @@ test_that("boosters with linear models at leaves can be written to text file and ...@@ -204,8 +203,8 @@ test_that("boosters with linear models at leaves can be written to text file and
bst2 <- lgb.load( bst2 <- lgb.load(
filename = model_file filename = model_file
) )
pred2 <- predict(bst2, X) preds2 <- predict(bst2, X)
expect_identical(preds, pred2) expect_identical(preds, preds2)
}) })
...@@ -792,7 +791,6 @@ test_that("boosters with linear models at leaves can be written to RDS and re-lo ...@@ -792,7 +791,6 @@ test_that("boosters with linear models at leaves can be written to RDS and re-lo
data = dtrain data = dtrain
, nrounds = 10L , nrounds = 10L
, params = params , params = params
, valids = list("train" = dtrain)
) )
expect_true(lgb.is.Booster(bst)) expect_true(lgb.is.Booster(bst))
...@@ -806,6 +804,6 @@ test_that("boosters with linear models at leaves can be written to RDS and re-lo ...@@ -806,6 +804,6 @@ test_that("boosters with linear models at leaves can be written to RDS and re-lo
# load the booster and make predictions...should be the same # load the booster and make predictions...should be the same
bst2 <- readRDS.lgb.Booster(file = model_file) bst2 <- readRDS.lgb.Booster(file = model_file)
pred2 <- predict(bst2, X) preds2 <- predict(bst2, X)
expect_identical(preds, pred2) expect_identical(preds, preds2)
}) })
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment