Re: [DISCUSS] Smoke tests

2017-04-04 Thread Clebert Suconic
> Unless of course it's written up as a proper example with a proper readme >> > with context and is appropriately named like all the other examples. >> What >> > I don't want to do is confuse users by having integration tests mixed in >> > with examples aimed

Re: [DISCUSS] Smoke tests

2017-04-03 Thread Martyn Taylor
ests mixed in > > with examples aimed for end users. It's just going to cause confusion. > > > > On Mon, Apr 3, 2017 at 10:24 AM, nigro_franz > wrote: > > > > > Agree! It is a good idea indeed! > > > > > > > > > > > >

Re: [DISCUSS] Smoke tests

2017-04-03 Thread Clebert Suconic
d users. It's just going to cause confusion. > > On Mon, Apr 3, 2017 at 10:24 AM, nigro_franz wrote: > > > Agree! It is a good idea indeed! > > > > > > > > -- > > View this message in context: http://activemq.2283324.n4. > > nabble.com/DISC

Re: [DISCUSS] Smoke tests

2017-04-03 Thread Clebert Suconic
n't want to do is confuse users by having integration tests mixed in > with examples aimed for end users. It's just going to cause confusion. > > On Mon, Apr 3, 2017 at 10:24 AM, nigro_franz wrote: > > > Agree! It is a good idea indeed! > > > > > > >

Re: [DISCUSS] Smoke tests

2017-04-03 Thread Martyn Taylor
in context: http://activemq.2283324.n4. > nabble.com/DISCUSS-Smoke-tests-tp4724485p4724492.html > Sent from the ActiveMQ - Dev mailing list archive at Nabble.com. >

Re: [DISCUSS] Smoke tests

2017-04-03 Thread nigro_franz
Agree! It is a good idea indeed! -- View this message in context: http://activemq.2283324.n4.nabble.com/DISCUSS-Smoke-tests-tp4724485p4724492.html Sent from the ActiveMQ - Dev mailing list archive at Nabble.com.

Re: [DISCUSS] Smoke tests

2017-04-03 Thread Clebert Suconic
It should still be part of the release. It's a valid test anyoene can run. We just make it clear. On Mon, Apr 3, 2017 at 4:32 AM Martyn Taylor wrote: > Good idea Clebert. We probably don't want to include these smoke tests as > part of the release, so can you ensure you don't include it in

Re: [DISCUSS] Smoke tests

2017-04-03 Thread Martyn Taylor
Good idea Clebert. We probably don't want to include these smoke tests as part of the release, so can you ensure you don't include it in the examples pom :) Cheers On Mon, Apr 3, 2017 at 1:56 AM, Clebert Suconic wrote: > We always had the examples on Artemis playing a double thingy... being >

[DISCUSS] Smoke tests

2017-04-02 Thread Clebert Suconic
We always had the examples on Artemis playing a double thingy... being both an educational and documentational resource, and that has also saved us a few times from making changes that would invalidate configurations and the manual. However, to replicate ARTEMIS-1089 now, I couldn't easily produc