[ 
https://issues.apache.org/jira/browse/OPENJPA-2996?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=18115772#comment-18115772
 ] 

ASF subversion and git services commented on OPENJPA-2996:
----------------------------------------------------------

Commit 2f0c9165813a5002600450f239324354b6a83e4c in openjpa's branch 
refs/heads/master from Richard Zowalla
[ https://gitbox.apache.org/repos/asf?p=openjpa.git;h=2f0c91658 ]

[OPENJPA-2996] Exercise the lazy not-loaded path in load state tests

testNotLoadedLazy duplicated testNotLoadedEager in TestPersistenceUnitUtil,
TestPersistenceUtil and TestProviderUtil. Also attach and persist the
RelEntity built by createLazyEntity so testLoadingLazyAttributeByName
really loads a non-empty relation.


> testNotLoadedLazy duplicates eager check, lazy path untested
> ------------------------------------------------------------
>
>                 Key: OPENJPA-2996
>                 URL: https://issues.apache.org/jira/browse/OPENJPA-2996
>             Project: OpenJPA
>          Issue Type: Sub-task
>          Components: jpa
>    Affects Versions: 4.2.0
>            Reporter: Maxim Solodovnik
>            Priority: Major
>             Fix For: 4.2.0
>
>
> Discussion thread: 
> https://github.com/apache/openjpa/pull/144#discussion_r3683007607
> **(low)** In this rewrite, testNotLoadedLazy still calls 
> `verifyIsLoadedEagerState(false)` (duplicate of testNotLoadedEager), so the 
> lazy not-loaded path (`verifyIsLoadedLazyState(false)`) remains untested. 
> Also createLazyEntity builds a RelEntity that is never persisted, which the 
> new testLoadingLazyAttributeByName relies on. 



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to