Merge pull request #4400 from roystgnr/get_uninitialized_info

Fix System::get_info() on uninitialized systems

Author: roystgnr

src/systems/system.C

@@ -1881,18 +1881,23 @@ std::string System::get_info() const
 
   oss << '\n';
 
-  oss << "    n_dofs()="             << this->n_dofs()             << '\n';
-  dof_id_type local_dofs = this->n_local_dofs();
-  oss << "    n_local_dofs()="       << local_dofs                 << '\n';
-  this->comm().max(local_dofs);
-  oss << "    max(n_local_dofs())="       << local_dofs                 << '\n';
+  if (this->is_initialized())
+    {
+      oss << "    n_dofs()="             << this->n_dofs()             << '\n';
+      dof_id_type local_dofs = this->n_local_dofs();
+      oss << "    n_local_dofs()="       << local_dofs                 << '\n';
+      this->comm().max(local_dofs);
+      oss << "    max(n_local_dofs())="       << local_dofs                 << '\n';
 #ifdef LIBMESH_ENABLE_CONSTRAINTS
-  oss << "    n_constrained_dofs()=" << this->n_constrained_dofs() << '\n';
-  oss << "    n_local_constrained_dofs()=" << this->n_local_constrained_dofs() << '\n';
-  dof_id_type local_unconstrained_dofs = this->n_local_dofs() - this->n_local_constrained_dofs();
-  this->comm().max(local_unconstrained_dofs);
-  oss << "    max(local unconstrained dofs)=" << local_unconstrained_dofs << '\n';
+      oss << "    n_constrained_dofs()=" << this->n_constrained_dofs() << '\n';
+      oss << "    n_local_constrained_dofs()=" << this->n_local_constrained_dofs() << '\n';
+      dof_id_type local_unconstrained_dofs = this->n_local_dofs() - this->n_local_constrained_dofs();
+      this->comm().max(local_unconstrained_dofs);
+      oss << "    max(local unconstrained dofs)=" << local_unconstrained_dofs << '\n';
 #endif
+    }
+  else
+    oss << "    (still uninitialized)\n";
 
   oss << "    " << "n_vectors()="  << this->n_vectors()  << '\n';
   oss << "    " << "n_matrices()="  << this->n_matrices()  << '\n';

tests/systems/systems_test.C

@@ -445,6 +445,8 @@ public:
 
   CPPUNIT_TEST( test100KVariables );
 
+  CPPUNIT_TEST( testUninitializedInfo );
+
   CPPUNIT_TEST( testPostInitAddVector );
   CPPUNIT_TEST( testAddVectorProjChange );
   CPPUNIT_TEST( testAddVectorTypeChange );
@@ -661,6 +663,17 @@ public:
     CPPUNIT_ASSERT_EQUAL(sys.get_dof_map().n_dofs(888), dof_id_type(5));
   }
 
+  void testUninitializedInfo()
+  {
+    Mesh mesh(*TestCommWorld);
+    EquationSystems es(mesh);
+    ExplicitSystem & sys = simpleSetup(mesh, es);
+
+    auto info = sys.get_info();
+
+    CPPUNIT_ASSERT (info.find("uninitialized") != std::string::npos);
+  }
+
 
   void testPostInitAddVector()
   {