.

go/zodb/persistent.go

@@ -261,7 +261,7 @@ func (obj *Persistent) PDeactivate() {
 	if obj.state >= CHANGED {
 		return
 	}
-	if obj.oid == InvalidOid { // newly created not-yet committed object	// TODO tests
+	if obj.oid == InvalidOid { // newly created not-yet committed object
 		return
 	}
 

go/zodb/persistent_test.go

@@ -621,6 +621,18 @@ func testPersistentDB(t0 *testing.T, rawcache bool) {
 	t.checkObj(obj2, 102, at2, UPTODATE,    0, "kitty")
 
 
+	// newly created object should not go to ghost on deactivate
+	obj3 := NewMyObject(t.conn)
+	obj3.value = "new"
+	checkObj(t0, obj3, t.conn, InvalidOid, InvalidTid, UPTODATE, 0)
+	assert.Equal(obj3.value, "new")
+	t.PActivate(obj3)
+	checkObj(t0, obj3, t.conn, InvalidOid, InvalidTid, UPTODATE, 1)
+	obj3.PDeactivate()
+	checkObj(t0, obj3, t.conn, InvalidOid, InvalidTid, UPTODATE, 0)
+	assert.Equal(obj3.value, "new")
+
+
 	// finish tnx3 and txn2 - conn1 and conn2 go back to db pool
 	t.Abort()
 	t2.Abort()