Merge pull request #3193 from stan-dev/fix/flaky-test-uninit-memory

Author: WardBrian

test/unit/math/prim/constraint/lb_transform_test.cpp

@@ -37,6 +37,7 @@ TEST(prob_transform, lb_constrain_matrix) {
   double lbd = 2.0;
 
   Eigen::VectorXd lb_bad(3);
+  lb_bad << 1.0, 2.0, 3.0;
 
   // matrix, real
   {
@@ -89,10 +90,11 @@ TEST(prob_transform, lb_constrain_std_vector) {
   Eigen::VectorXd lb(2);
   lb << 2.0, stan::math::NEGATIVE_INFTY;
   Eigen::VectorXd lb2(2);
-  lb << stan::math::NEGATIVE_INFTY, 1.0;
+  lb2 << stan::math::NEGATIVE_INFTY, 1.0;
   double lbd = 2.0;
 
   Eigen::VectorXd lb_bad(3);
+  lb_bad << 1.0, 2.0, 3.0;
 
   std::vector<Eigen::VectorXd> x_vec = {x, x};
   std::vector<Eigen::VectorXd> lb_vec = {lb, lb2};

test/unit/math/prim/constraint/lub_transform_test.cpp

@@ -134,7 +134,10 @@ TEST(prob_transform, lub_constrain_matrix) {
   double lbd = -2.0;
 
   Eigen::VectorXd ub_bad(3);
+  ub_bad << 1.0, 2.0, 3.0;
+
   Eigen::VectorXd lb_bad(3);
+  lb_bad << -1.0, -2.0, -3.0;
 
   // matrix, real, real
   {
@@ -330,7 +333,9 @@ TEST(prob_transform, lub_constrain_stdvec_matrix) {
   double lbd = -2.0;
 
   Eigen::VectorXd ub_bad(3);
+  ub_bad << 1.0, 2.0, 3.0;
   Eigen::VectorXd lb_bad(3);
+  lb_bad << -1.0, -2.0, -3.0;
 
   std::vector<Eigen::VectorXd> x_vec{x, x};
   std::vector<Eigen::VectorXd> ub_vec{ub, ub};

test/unit/math/prim/constraint/ub_transform_test.cpp

@@ -37,6 +37,7 @@ TEST(prob_transform, ub_constrain_matrix) {
   double ubd = 2.0;
 
   Eigen::VectorXd ub_bad(3);
+  ub_bad << 1.0, 2.0, 3.0;
 
   // matrix, real
   {
@@ -94,6 +95,7 @@ TEST(prob_transform, ub_constrain_std_vector) {
   double ubd = 2.0;
 
   Eigen::VectorXd ub_bad(3);
+  ub_bad << 1.0, 2.0, 3.0;
 
   std::vector<Eigen::VectorXd> x_vec = {x, 2 * x};
   std::vector<Eigen::VectorXd> ub_vec = {ub, ub2};

test/unit/math/prim/fun/fma_test.cpp

@@ -54,15 +54,15 @@ TEST(MathFunctions, fma_matrix) {
   Eigen::RowVectorXd y_rowvec(2);
   y_rowvec << 2.0, -3.0;
   Eigen::MatrixXd y_mat(2, 2);
-  x_mat << 1.0, 2.0, -1.0, 1.1;
+  y_mat << 1.0, 2.0, -1.0, 1.1;
 
   double z_scalar = 3.0;
   Eigen::VectorXd z_vec(2);
   z_vec << -3.0, 4.0;
   Eigen::RowVectorXd z_rowvec(2);
   z_rowvec << -3.0, 4.0;
   Eigen::MatrixXd zm(2, 2);
-  x_mat << 3.0, 4.0, -1.0, 1.1;
+  zm << 3.0, 4.0, -1.0, 1.1;
 
   EXPECT_MATRIX_EQ(add(elt_multiply(x_scalar, y_scalar), z_vec),
                    fma(x_scalar, y_scalar, z_vec));

test/unit/math/prim/fun/get_base1_lhs_test.cpp

@@ -7,6 +7,8 @@ TEST(MathMatrixPrimMat, get_base1_lhs_failing_in_26) {
   using Eigen::Matrix;
   using stan::math::get_base1_lhs;
   Matrix<double, Dynamic, Dynamic> y(2, 3);
+  y << 1, 2, 3, 4, 5, 6;
+
   EXPECT_THROW(get_base1_lhs(y, 3, 1, "y", 2), std::exception);
   EXPECT_THROW(get_base1_lhs(y, 1, 4, "y", 2), std::exception);
   for (int i = 1; i <= 2; ++i)

test/unit/math/prim/prob/multi_normal_prec_rng_test.cpp

@@ -14,6 +14,7 @@ TEST(ProbDistributionsMultiNormalPrec, vectorized) {
   Eigen::RowVectorXd mu_t(3);
   std::vector<Eigen::VectorXd> vec_mu(2);
   std::vector<Eigen::RowVectorXd> vec_mu_t(2);
+  mu << -1.0, 2.0, 4.0;
   vec_mu[0] = mu;
   vec_mu_t[0] = mu;
   mu << 2.0, -1.0, 4.0;

test/unit/math/rev/fun/scalar_seq_view_test.cpp

@@ -82,28 +82,28 @@ void expect_scalar_seq_view_adjoints(const C& v) {
 
 TEST(MathMetaRev, ScalarSeqViewVectorVar) {
   using stan::math::var;
-  Eigen::Matrix<var, -1, 1> A = Eigen::VectorXd(4);
+  Eigen::Matrix<var, -1, 1> A = Eigen::VectorXd::Ones(4);
   expect_scalar_seq_view_value(A);
   expect_scalar_seq_view_adjoints(A);
 }
 
 TEST(MathMetaRev, ScalarSeqViewRowVectorVar) {
   using stan::math::var;
-  Eigen::Matrix<var, 1, -1> A = Eigen::RowVectorXd(4);
+  Eigen::Matrix<var, 1, -1> A = Eigen::RowVectorXd::Ones(4);
   expect_scalar_seq_view_value(A);
   expect_scalar_seq_view_adjoints(A);
 }
 
 TEST(MathMetaRev, VarScalarSeqViewVector) {
   using stan::math::var_value;
-  var_value<Eigen::Matrix<double, -1, 1>> A = Eigen::VectorXd(4);
+  var_value<Eigen::Matrix<double, -1, 1>> A = Eigen::VectorXd::Ones(4);
   expect_scalar_seq_view_value(A);
   expect_scalar_seq_view_adjoints(A);
 }
 
 TEST(MathMetaRev, VarScalarSeqViewRowVector) {
   using stan::math::var_value;
-  var_value<Eigen::Matrix<double, 1, -1>> A = Eigen::RowVectorXd(4);
+  var_value<Eigen::Matrix<double, 1, -1>> A = Eigen::RowVectorXd::Ones(4);
   expect_scalar_seq_view_value(A);
   expect_scalar_seq_view_adjoints(A);
 }