Add failing unit test for accessing member of null object

lefticus · lefticus · commit 4a70ffe59921 · 2014-09-08T18:23:53.000-06:00
diff --git a/src/test_module.cpp b/src/test_module.cpp
@@ -51,12 +51,17 @@ class TestMoreDerivedType : public TestDerivedType
 
 std::shared_ptr<TestBaseType> derived_type_factory()
 {
-  return std::shared_ptr<TestBaseType>(new TestDerivedType());
+  return std::make_shared<TestDerivedType>();
 }
 
 std::shared_ptr<TestBaseType> more_derived_type_factory()
 {
-  return std::shared_ptr<TestBaseType>(new TestMoreDerivedType());
+  return std::make_shared<TestBaseType>();
+}
+
+std::shared_ptr<TestBaseType> null_factory()
+{
+  return std::shared_ptr<TestBaseType>();
 }
 
 std::string hello_world()
@@ -111,6 +116,7 @@ CHAISCRIPT_MODULE_EXPORT  chaiscript::ModulePtr create_chaiscript_module_test_mo
 
   m->add(chaiscript::fun(&derived_type_factory), "derived_type_factory");
   m->add(chaiscript::fun(&more_derived_type_factory), "more_derived_type_factory");
+  m->add(chaiscript::fun(&null_factory), "null_factory");
 
   m->add(chaiscript::fun(&TestDerivedType::func), "func");
 
diff --git a/unittests/null_object_access.chai b/unittests/null_object_access.chai
@@ -0,0 +1,13 @@
+load_module("test_module")
+
+auto o = null_factory();
+
+try {
+  o.func();
+} catch (e) {
+  exit(0);
+}
+
+assert_true(false);
+
+

Original file line number	Diff line number	Diff line change
`@@ -51,12 +51,17 @@ class TestMoreDerivedType : public TestDerivedType`
`51`	`51`
`52`	`52`	`std::shared_ptr<TestBaseType> derived_type_factory()`
`53`	`53`	`{`
`54`		`- return std::shared_ptr<TestBaseType>(new TestDerivedType());`
	`54`	`+ return std::make_shared<TestDerivedType>();`
`55`	`55`	`}`
`56`	`56`
`57`	`57`	`std::shared_ptr<TestBaseType> more_derived_type_factory()`
`58`	`58`	`{`
`59`		`- return std::shared_ptr<TestBaseType>(new TestMoreDerivedType());`
	`59`	`+ return std::make_shared<TestBaseType>();`
	`60`	`+}`
	`61`	`+`
	`62`	`+std::shared_ptr<TestBaseType> null_factory()`
	`63`	`+{`
	`64`	`+ return std::shared_ptr<TestBaseType>();`
`60`	`65`	`}`
`61`	`66`
`62`	`67`	`std::string hello_world()`
`@@ -111,6 +116,7 @@ CHAISCRIPT_MODULE_EXPORT chaiscript::ModulePtr create_chaiscript_module_test_mo`
`111`	`116`
`112`	`117`	`m->add(chaiscript::fun(&derived_type_factory), "derived_type_factory");`
`113`	`118`	`m->add(chaiscript::fun(&more_derived_type_factory), "more_derived_type_factory");`
	`119`	`+ m->add(chaiscript::fun(&null_factory), "null_factory");`
`114`	`120`
`115`	`121`	`m->add(chaiscript::fun(&TestDerivedType::func), "func");`
`116`	`122`