Fix wrong transaction requirement check for lock

FYI @gavinking, this reverts a part of what you have done in #8866 which causes JPA TCK test failure of ee.jakarta.tck.persistence.core.entityManager2.Client2.lockTransactionRequiredExceptionTest

Author: beikov

hibernate-core/src/main/java/org/hibernate/internal/SessionImpl.java

@@ -2570,7 +2570,7 @@ public <T> T find(EntityGraph<T> entityGraph, Object primaryKey, FindOption... o
 	}
 
 	private void checkTransactionNeededForLock(LockMode lockMode) {
-		if ( !LockMode.PESSIMISTIC_READ.greaterThan( lockMode ) ) {
+		if ( lockMode.greaterThan( LockMode.READ ) ) {
 			checkTransactionNeededForUpdateOperation();
 		}
 	}

hibernate-core/src/test/java/org/hibernate/orm/test/jpa/transaction/FlushAndTransactionTest.java

@@ -48,9 +48,8 @@ public void testAlwaysTransactionalOperations() {
 		catch (TransactionRequiredException e) {
 			//success
 		}
-		em.lock( book, LockModeType.READ );
 		try {
-			em.lock( book, LockModeType.PESSIMISTIC_READ );
+			em.lock( book, LockModeType.READ );
 			fail( "lock has to be inside a Tx" );
 		}
 		catch (TransactionRequiredException e) {