Sequelize: Create two referenced Rows in same Transaction - node.js

Question

I want to create two rows.
First I want to create a tenant and after I want to creae a user who has a reference to the tenant.
I want to do this in one transaction (register).
The tenant would be created, but when sequelize try to create the user I get an error:
Failing row contains
Routine: ExecConstraints
My DB-Mappings:
const TenantMapping = sequelize.define('tenant', {
id: { type: DataTypes.NUMBER, primaryKey: true, autoIncrement: true },
label: { type: DataTypes.STRING, allowNull: false },
name: { type: DataTypes.STRING },
postOfficeBox: { type: DataTypes.STRING },
street: { type: DataTypes.STRING },
houseNo: { type: DataTypes.STRING },
zipCode: { type: DataTypes.STRING, validate: { max: 10 } },
city: { type: DataTypes.STRING },
phone: { type: DataTypes.STRING },
mobilePhone: { type: DataTypes.STRING },
email: { type: DataTypes.STRING, allowNull: false },
website: { type: DataTypes.STRING },
birth: { type: DataTypes.DATE, allowNull: false },
death: { type: DataTypes.DATE }
}, {
...getSequelizeTableSettings({ schema: 'auth' })
});
const UserMapping = sequelize.define('user', {
id: { type: DataTypes.NUMBER, primaryKey: true, autoIncrement: true },
tenantId: { type: DataTypes.NUMBER, allowNull: false },
email: { type: DataTypes.STRING, allowNull: false },
password: { type: DataTypes.STRING, allowNull: false },
role: { type: DataTypes.STRING, allowNull: false },
isActivated: { type: DataTypes.BOOLEAN, allowNull: false },
birth: { type: DataTypes.DATE, allowNull: false },
death: { type: DataTypes.DATE }
}, {
...getSequelizeTableSettings({ schema: 'auth' })
});
My Controller
const response = await sequelize.transaction(async (transaction) => {
try {
const { tenantLabel, email, password } = req.body;
const tenant = await this.tenantRepository.create({
tenantLabel,
email
}, {
transaction
});
const user = await this.userRepository.create({
email,
password,
tenantId: tenant.id,
role: UserRole.ADMIN
}, {
transaction
});
await transaction.commit();
res.status(200).json({
tenant,
user
});
} catch (exception) {
res.status(400).send({
...exception
});
}
});
If I create the tenant and user in two different transactions - it works fine.
What is wrong?

Answer 1

I use PostgreSQL
{
"name": "SequelizeDatabaseError",
"parent": {
"length": 257,
"name": "error",
"severity": "ERROR",
"code": "23502",
"detail": "Failing row contains (22, null, email#mail.com, null, null, f, 2022-01-22 12:07:33.899, null).",
"schema": "auth",
"table": "user",
"column": "tenant_id",
"file": "execMain.c",
"line": "1965",
"routine": "ExecConstraints",
"sql": "INSERT INTO \"auth\".\"user\" (\"id\",\"email\",\"birth\") VALUES (DEFAULT,$1,$2) RETURNING \"id\",\"tenant_id\",\"email\",\"password\",\"role\",\"is_activated\",\"birth\",\"death\";",
"parameters": [
"email#mail.com",
"2022-01-22 12:07:33.899 +00:00"
]
},
"original": {
"length": 257,
"name": "error",
"severity": "ERROR",
"code": "23502",
"detail": "Failing row contains (22, null, email#mail.com, null, null, f, 2022-01-22 12:07:33.899, null).",
"schema": "auth",
"table": "user",
"column": "tenant_id",
"file": "execMain.c",
"line": "1965",
"routine": "ExecConstraints",
"sql": "INSERT INTO \"auth\".\"user\" (\"id\",\"email\",\"birth\") VALUES (DEFAULT,$1,$2) RETURNING \"id\",\"tenant_id\",\"email\",\"password\",\"role\",\"is_activated\",\"birth\",\"death\";",
"parameters": [
"email#mail.com",
"2022-01-22 12:07:33.899 +00:00"
]
},
"sql": "INSERT INTO \"auth\".\"user\" (\"id\",\"email\",\"birth\") VALUES (DEFAULT,$1,$2) RETURNING \"id\",\"tenant_id\",\"email\",\"password\",\"role\",\"is_activated\",\"birth\",\"death\";",
"parameters": [
"email#mail.com",
"2022-01-22 12:07:33.899 +00:00"
]
}
I log my object with I want to create a new row before to try create with sequelize.
console.log('create new something', JSON.stringify(object));
const response = await this.sequelize.create(object, transaction);
In the console I have this:
create new something {"email":"email#mail.com","password":"$2b$04$2vDzrLah9FjhxG.nlIJjluByE28O6bjyIr4s7LXqoFgGwyjHODJMG","role":"ADMIN","tenantId":22,"isActivated":false}
Thats correct. But look in the error message. Sequelize try to create a new user only with email - no password, no role, no tenantId, no isActivated...

Answer 2

TenantRepository:
export class TenantRepository extends Repository<TenantResource> {
constructor() {
super(DB_MAPPINGS.TenantMapping, { identifier: 'id' });
}
public async create(
{ tenantLabel, email }: { tenantLabel: string; email: string; },
{ transaction }: { transaction: Transaction }
): Promise<TenantResource> {
return await this.CREATE({
label: tenantLabel,
email
},
{
transaction
}
);
}
}
UserRepostory:
export class UserRepository extends Repository<UserResource> {
constructor() {
super(DB_MAPPINGS.UserMapping, { identifier: 'id' });
}
public async create(
{ email, password, role, tenantId }: { email: string; password: string; role: UserRole; tenantId: number; },
{ transaction }: { transaction: Transaction }
): Promise<UserResource> {
return await this.CREATE({
email,
password: await this.encryptPassword({ password }),
role,
tenantId,
isActivated: false
},
{
transaction
}
);
}
}
Repository:
export class Repository<T> extends RepositoryHelper<T> {
public async CREATE(
object: Partial<T>,
{ transaction }: { transaction: Transaction }
): Promise<T> {
const response = await this.sequelize.create(object, transaction);
return response.toJSON();
}
}

Answer 3

I found my problem. I have to pass the transaction.
export class Repository<T> extends RepositoryHelper<T> {
public async CREATE(
object: Partial<T>,
{ transaction }: { transaction: Transaction }
): Promise<T> {
const response = await this.sequelize.create(object, { transaction });
return response.toJSON();
}
}