Deprecate Spring context configuration by more than one class

[brasmusson]

Summary

Deprecate Spring context configuration by more than one class

Details

Print a warning if more than one glue class that attempts to configure the Spring context is found (that is have a @ContextConfiguration or @ContextHierarcy annotation.

Motivation and Context

We have decided to only allow one glue class to configure the Spring context, as this is a non-backward compatible change, it will be introduced in v3.0.0 (#1246). Until then a deprecation warning should be printed.

How Has This Been Tested?

The automated test suite has been updated to verify this behavior.

Types of changes

• Bug fix (non-breaking change which fixes an issue).
• New feature (non-breaking change which adds functionality).
• Breaking change (fix or feature that would cause existing functionality to not work as expected).

Checklist:

• I've added tests for my code.
• My change requires a change to the documentation.
• I have updated the documentation accordingly.

[coveralls]

Coverage increased (+0.01%) to 81.259% when pulling 0d0baef on spring-context-deprecation-warning into 9a951ec on master.

[mpkorstanje]

I think your editor is configured to use tabs instead of spaces.

[brasmusson]

Removed all tabs.

[coveralls]

Coverage increased (+0.01%) to 81.259% when pulling 83da61a on spring-context-deprecation-warning into 9a951ec on master.

[mpkorstanje]

Hi all. I'm new to Cucumber and I've seen most of the tutorials on Cucumber-Spring-Boot stack recommending that each step class should extend some base SpringBootIntegrationTest class, which configures context. How should this work now when this change is introduced? Is there any material explaining best new approach?

Your step definitions no longer have to extend from that class. You can either pick any one of your step definition classes and put the spring configuration on that one. Or you can use a dedicated class like
CucumberContextConfiguration. As long as the class has at least one step def it will be discovered.

[mpkorstanje]

Is there any material explaining best new approach?

The example project is your best guess at the moment.

https://github.com/cucumber/cucumber-jvm/tree/master/examples/spring-txn

[tobamaestro]

Thanks. I've found it in the meantime and I removed my previous question, but I guess it's good you replied if anyone else wonders about the same thing again.

[lock]

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.