trying to fix Travis instability

[Turini]

using CDI.current() and providing a scope for Context

[Turini]

@renanigt can you test it in your environment? just run mvn test

[renanigt]

@Turini, before this commit I got many errors, but now I got just 2 errors:

Tests in error: 
  shouldNotRunVRaptorStackIfVRaptorRequestStartedEventNotFired(br.com.caelum.vraptor.ioc.RequestStartedFactoryTest): WELD-001303: No active contexts for scope type javax.enterprise.context.RequestScoped
  shouldRunVRaptorStackIfVRaptorRequestStartedEventIsFired(br.com.caelum.vraptor.ioc.RequestStartedFactoryTest): WELD-001303: No active contexts for scope type javax.enterprise.context.RequestScoped

[Turini]

please, can you create a simple test and run this code?

System.out.println(junit.runner.Version.id());

[renanigt]

@Turini, the output was 4.11.

[renanigt]

Running into eclipse, the error doesn't occurs.

[garcia-jj]

@Turini Can you force Travis build many times (something like 10 times) to make sure that all tests pass ok? I think that Travis are more important to solve than other environments because Travis errors bothering us, and hidden some test failures when we commit something.

Another point: whats the Maven versions used by Travis? Have you the same version? Because can be a Maven issue.

[renanigt]

I agree that Travis is most important than other enviroments, but IMHO we need to continue trying to fix it in all enviroment.

If in some enviroments the tests doesn't pass, it discourages users to send a PR.

If pass only in Travis, we need to send a PR to know if tests are passing or not.

[Turini]

Hi guys, sorry my delay (really busy week!). Comments inline:

Can you force Travis build many times (something like 10 times) to
make sure that all tests pass ok?

Already done, and its working always on Travis in this branch. I'll merge it
to see if master and another branches will work too (without close the issue)

Another point: whats the Maven versions used by Travis? Have you the
same version? Because can be a Maven issue.

Yes, I've built an aws ec2 instance to reproduce travis environment...

[garcia-jj]

@renanigt My point is to solve Trais too soon as possible, and if tests are fail for some users, we can fix later.

[renanigt]

True @garcia-jj !

Thanks. 😃

[Turini]

guys, master is now green on travis!
(the other current PR's isn't... because does not have this code).

[garcia-jj]

I did tests locally before push, so my pull requests are fine to merge. But if you prefer I can rebase my branches from master.

[Turini]

no need :) we can see how travis will react in the next PR's

[garcia-jj]

Ohh, yeahhh. Now Travis are smiling to us. ☕ Green lights back.

[Turini]

great!