Commit a31f9cc

dreab8mbellade

authored and

committed

Fix MutationDelegateTest, MutationDelegateJoinedInheritanceTest and MutationDelegateIdentityTest update query assertions

1 parent 100114e commit a31f9ccCopy full SHA for a31f9cc

File tree

-6

lines changed

-6

lines changed

Lines changed: 2 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -139,7 +139,7 @@ public void testUpdateGeneratedValuesAndIdentity(SessionFactoryScope scope) {`
`139`	`139`	`assertThat( entity.getUpdateDate() ).isNotNull();`
`140`	`140`
`141`	`141`	`inspector.assertIsSelect( 0 );`
`142`		`- assertThat( inspector.getSqlQueries().get( 0 ) ).contains( "update" );`
	`142`	`+ assertThat( inspector.getSqlQueries().get( 1 ) ).contains( "update" );`
`143`	`143`	`inspector.assertExecutedCount(`
`144`	`144`	`delegate != null && delegate.supportsArbitraryValues() ? 2 : 3`
`145`	`145`	`);`
`@@ -188,7 +188,7 @@ public void testInsertGeneratedValuesAndIdentityAndRowId(SessionFactoryScope sco`
`188`	`188`
`189`	`189`	`assertThat( entity.getUpdateDate() ).isNotNull();`
`190`	`190`
`191`		`- assertThat( inspector.getSqlQueries().get( 0 ) ).contains( "update" );`
	`191`	`+ assertThat( inspector.getSqlQueries().get( 0 ) ).contains( "update" );`
`192`	`192`	`inspector.assertNumberOfOccurrenceInQueryNoSpace( 0, "id_column", shouldHaveRowId ? 0 : 1 );`
`193`	`193`	`} );`
`194`	`194`	`scope.inSession( session -> assertThat( session.find(`

Lines changed: 2 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -146,7 +146,7 @@ public void testUpdateBaseEntity(SessionFactoryScope scope) {`
`146`	`146`	`assertThat( entity.getUpdateDate() ).isNotNull();`
`147`	`147`
`148`	`148`	`inspector.assertIsSelect( 0 );`
`149`		`- assertThat( inspector.getSqlQueries().get( 0 ) ).contains( "update" );`
	`149`	`+ assertThat( inspector.getSqlQueries().get( 1 ) ).contains( "update" );`
`150`	`150`	`inspector.assertExecutedCount(`
`151`	`151`	`delegate != null && delegate.supportsArbitraryValues() ? 2 : 3`
`152`	`152`	`);`
`@@ -174,7 +174,7 @@ public void testUpdateChildEntity(SessionFactoryScope scope) {`
`174`	`174`	`assertThat( entity.getChildUpdateDate() ).isNotNull();`
`175`	`175`
`176`	`176`	`inspector.assertIsSelect( 0 );`
`177`		`- assertThat( inspector.getSqlQueries().get( 0 ) ).contains( "update" );`
	`177`	`+ assertThat( inspector.getSqlQueries().get( 1 ) ).contains( "update" );`
`178`	`178`	`// Note: this is a current restriction, mutation delegates only retrieve generated values`
`179`	`179`	`// on the "root" table, and we expect other values to be read through a subsequent select`
`180`	`180`	`inspector.assertIsSelect( 2 );`

Lines changed: 2 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -88,7 +88,7 @@ public void testUpdateGeneratedValues(SessionFactoryScope scope) {`
`88`	`88`	`assertThat( entity.getUpdateDate() ).isNotNull();`
`89`	`89`
`90`	`90`	`inspector.assertIsSelect( 0 );`
`91`		`- assertThat( inspector.getSqlQueries().get( 0 ) ).contains( "update" );`
	`91`	`+ assertThat( inspector.getSqlQueries().get( 1 ) ).contains( "update" );`
`92`	`92`	`inspector.assertExecutedCount(`
`93`	`93`	`delegate != null && delegate.supportsArbitraryValues() ? 2 : 3`
`94`	`94`	`);`
`@@ -133,7 +133,7 @@ public void testGeneratedValuesAndRowId(SessionFactoryScope scope) {`
`133`	`133`	`session.flush();`
`134`	`134`
`135`	`135`	`assertThat( entity.getUpdateDate() ).isNotNull();`
`136`		`- assertThat( inspector.getSqlQueries().get( 0 ) ).contains( "update" );`
	`136`	`+ assertThat( inspector.getSqlQueries().get( 0 ) ).contains( "update" );`
`137`	`137`	`inspector.assertNumberOfOccurrenceInQueryNoSpace( 0, "id_column", shouldHaveRowId ? 0 : 1 );`
`138`	`138`	`} );`
`139`	`139`	`scope.inSession( session -> assertThat( session.find( ValuesAndRowId.class, 1 ).getUpdateDate() ).isNotNull() );`

Comments

(0)