Back off Jooq's TransactionProvider on custom TransactionProvider

Closes gh-32899

Author: mhalbritter

spring-boot-project/spring-boot-autoconfigure/src/main/java/org/springframework/boot/autoconfigure/jooq/JooqAutoConfiguration.java

@@ -71,6 +71,7 @@ public DataSourceConnectionProvider dataSourceConnectionProvider(DataSource data
 
 	@Bean
 	@ConditionalOnBean(PlatformTransactionManager.class)
+	@ConditionalOnMissingBean(TransactionProvider.class)
 	public SpringTransactionProvider transactionProvider(PlatformTransactionManager txManager) {
 		return new SpringTransactionProvider(txManager);
 	}

spring-boot-project/spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/jooq/JooqAutoConfigurationTests.java

@@ -29,7 +29,9 @@
 import org.jooq.RecordMapperProvider;
 import org.jooq.RecordUnmapperProvider;
 import org.jooq.SQLDialect;
+import org.jooq.TransactionContext;
 import org.jooq.TransactionListenerProvider;
+import org.jooq.TransactionProvider;
 import org.jooq.TransactionalRunnable;
 import org.jooq.VisitListenerProvider;
 import org.jooq.impl.DataSourceConnectionProvider;
@@ -188,6 +190,16 @@ void relaxedBindingOfSqlDialect() {
 						.isEqualTo(SQLDialect.POSTGRES));
 	}
 
+	@Test
+	void transactionProviderBacksOffOnExistingTransactionProvider() {
+		this.contextRunner
+				.withUserConfiguration(JooqDataSourceConfiguration.class, CustomTransactionProviderConfiguration.class)
+				.run((context) -> {
+					TransactionProvider transactionProvider = context.getBean(TransactionProvider.class);
+					assertThat(transactionProvider).isInstanceOf(CustomTransactionProvider.class);
+				});
+	}
+
 	static class AssertFetch implements TransactionalRunnable {
 
 		private final DSLContext dsl;
@@ -239,6 +251,16 @@ DataSource jooqDataSource() {
 
 	}
 
+	@Configuration(proxyBeanMethods = false)
+	static class CustomTransactionProviderConfiguration {
+
+		@Bean
+		TransactionProvider transactionProvider() {
+			return new CustomTransactionProvider();
+		}
+
+	}
+
 	@Configuration(proxyBeanMethods = false)
 	static class TxManagerConfiguration {
 
@@ -259,4 +281,23 @@ public ExecuteListener provide() {
 
 	}
 
+	static class CustomTransactionProvider implements TransactionProvider {
+
+		@Override
+		public void begin(TransactionContext ctx) {
+
+		}
+
+		@Override
+		public void commit(TransactionContext ctx) {
+
+		}
+
+		@Override
+		public void rollback(TransactionContext ctx) {
+
+		}
+
+	}
+
 }