[MAGNOLIA-5157] Further simplify test-webapp Created: 04/Jul/13  Updated: 10/Mar/21  Resolved: 10/Mar/21

Status: Closed
Project: Magnolia
Component/s: integration tests
Affects Version/s: None
Fix Version/s: None

Type: Improvement Priority: Neutral
Reporter: Magnolia International Assignee: Unassigned
Resolution: Outdated Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Issue Links:
supersession
supersedes MAGNOLIA-5155 Remove redundancy of magnolia-test-pu... Closed
Template:
Acceptance criteria:
Empty
Task DoD:
[ ]* Doc/release notes changes? Comment present?
[ ]* Downstream builds green?
[ ]* Solution information and context easily available?
[ ]* Tests
[ ]* FixVersion filled and not yet released
[ ]  Architecture Decision Record (ADR)

 Description   

Further simplifications - to be tested/validated:

  • use magnoliaAuthor and magnoliaPublic for context paths - this way we can entirely remove the src folder of test-webapp, and just keep a pom !
  • remove misleading defaults (context path, hostname, etc) from AbstractMagnoliaIntegrationTest.Instance#AUTHOR - enforce the use of properties. IntelliJ passes them properly (from the maven surefire config), needs to be checked with Eclipse. This will avoid running into confusing issues where a default host/port/path is used and the application is deployed somewhere else. It's one of the many steps that will help adding IntTests to more modules.

Generated at Mon Feb 12 04:02:31 CET 2024 using Jira 9.4.2#940002-sha1:46d1a51de284217efdcb32434eab47a99af2938b.