[jira] [Commented] (DELTASPIKE-1007) test @Transactional and @TransactionScoped support for @Repository

2018-04-23 Thread Thomas Andraschko (JIRA)

[ 
https://issues.apache.org/jira/browse/DELTASPIKE-1007?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16448071#comment-16448071
 ] 

Thomas Andraschko commented on DELTASPIKE-1007:
---

We maybe don't have tests for Transactional but we have many many tests for 
interceptors. I think it's ok to close. WDYT [~gpetracek]?

> test @Transactional and @TransactionScoped support for @Repository
> --
>
> Key: DELTASPIKE-1007
> URL: https://issues.apache.org/jira/browse/DELTASPIKE-1007
> Project: DeltaSpike
>  Issue Type: Task
>  Components: Data-Module, Tests
>Affects Versions: 1.5.1
>Reporter: Gerhard Petracek
>Assignee: Harald Wellmann
>Priority: Major
>
> currently it's tested manually, but we should add a test to the test-suite.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)


[jira] [Commented] (DELTASPIKE-1007) test @Transactional and @TransactionScoped support for @Repository

2015-12-21 Thread Gerhard Petracek (JIRA)

[ 
https://issues.apache.org/jira/browse/DELTASPIKE-1007?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15066250#comment-15066250
 ] 

Gerhard Petracek commented on DELTASPIKE-1007:
--

[~hwellmann] thx for helping with this topic, but this ticket was intended for 
@org.apache.deltaspike.jpa.api.transaction.Transactional and 
@org.apache.deltaspike.jpa.api.transaction.TransactionScoped -> i'm not sure 
that we should add a whole module for it. the intention was just to test that 
the @org.apache.deltaspike.jpa.api.transaction.Transactional gets called as 
expected.

> test @Transactional and @TransactionScoped support for @Repository
> --
>
> Key: DELTASPIKE-1007
> URL: https://issues.apache.org/jira/browse/DELTASPIKE-1007
> Project: DeltaSpike
>  Issue Type: Task
>  Components: Data-Module, Tests
>Affects Versions: 1.5.1
>Reporter: Gerhard Petracek
>Assignee: Harald Wellmann
> Fix For: 1.5.3
>
>
> currently it's tested manually, but we should add a test to the test-suite.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (DELTASPIKE-1007) test @Transactional and @TransactionScoped support for @Repository

2015-12-21 Thread Harald Wellmann (JIRA)

[ 
https://issues.apache.org/jira/browse/DELTASPIKE-1007?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15066274#comment-15066274
 ] 

Harald Wellmann commented on DELTASPIKE-1007:
-

Well, the description was a bit vague... My tests do cover both flavours of 
{{\@Transactional}}. There are {{Jta*}} tests using the JTA 1.2 annotations 
(providing regression tests for DELTASPIKE-420) and {{DeltaSpike*}} tests using 
the DeltaSpike JPA annotations.

A separate test module seems to be easiest way to handle multiple versions of 
the JTA API. Moving these new tests into the Data {{impl}} module would require 
upgrading the JTA dependency to 1.2, which might lead to incompatibility with 
Java EE 6.

> test @Transactional and @TransactionScoped support for @Repository
> --
>
> Key: DELTASPIKE-1007
> URL: https://issues.apache.org/jira/browse/DELTASPIKE-1007
> Project: DeltaSpike
>  Issue Type: Task
>  Components: Data-Module, Tests
>Affects Versions: 1.5.1
>Reporter: Gerhard Petracek
>Assignee: Harald Wellmann
> Fix For: 1.5.3
>
>
> currently it's tested manually, but we should add a test to the test-suite.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Commented] (DELTASPIKE-1007) test @Transactional and @TransactionScoped support for @Repository

2015-12-21 Thread Gerhard Petracek (JIRA)

[ 
https://issues.apache.org/jira/browse/DELTASPIKE-1007?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15066541#comment-15066541
 ] 

Gerhard Petracek commented on DELTASPIKE-1007:
--

well, we have to test mainly our logic and in this case just if interceptors 
like the tx-interceptor works with the data-module.
imo it doesn't make sense to test the same again (since 
@javax.transaction.Transactional) is technically just an other interceptor you 
can use.
we also don't test data-repos with other ee-interceptors, because they should 
just work.
-> from the conceptional point of view we currently have an own module just to 
have a "redundant" test.

> test @Transactional and @TransactionScoped support for @Repository
> --
>
> Key: DELTASPIKE-1007
> URL: https://issues.apache.org/jira/browse/DELTASPIKE-1007
> Project: DeltaSpike
>  Issue Type: Task
>  Components: Data-Module, Tests
>Affects Versions: 1.5.1
>Reporter: Gerhard Petracek
>Assignee: Harald Wellmann
> Fix For: 1.5.3
>
>
> currently it's tested manually, but we should add a test to the test-suite.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)