[MGNLREST-146] Implement REST integration tests Created: 07/Nov/17  Updated: 29/Oct/19  Resolved: 18/Jan/18

Status: Closed
Project: Magnolia REST Framework
Component/s: None
Affects Version/s: None
Fix Version/s: 2.0.2

Type: Task Priority: Neutral
Reporter: Mikaël Geljić Assignee: Ngoc Nguyenthanh
Resolution: Fixed Votes: 0
Labels: None
Remaining Estimate: 0d
Time Spent: 10d 6.5h
Original Estimate: 5d

Issue Links:
Cloners
dependency
relation
is related to MGNLTEST-62 Document the rest-integration-tests f... Closed
Template:
Acceptance criteria:
Empty
Task DoR:
Empty
Date of First Response:
Epic Link: REST Phase2
Sprint: Saigon 129, Saigon 130
Story Points: 8

 Description   

Implement REST integration tests according to outcome of the preceding research.

  • Specify which cases to test (ideally we test features individually)
    • Highlight redundancies with existing unit tests
  • Add a webapp artifact to the REST reactor
    • exclude it from releasing? are other test-webapps excluded from releases already?
  • Bootstrap dummy test content (under WEB-INF/bootstrap/common)
  • Evaluate whether or not to provide a "manual-tests" profile (effectively just booting tomcat via maven plugin)
    • may not even be mandatory, as the webapp is easily deployed from the IDE too


 Comments   
Comment by Mikaël Geljić [ 08/Jan/18 ]

Evaluate https://github.com/fabric8io/docker-maven-plugin vs. cargo

Comment by Ngoc Nguyenthanh [ 09/Jan/18 ]

Expectation

  • Have a dedicated module for doing the integration testing for REST module.
  • Integrated into the maven build. Able to run on local and on Jenkins

Detailing actions

Comment by Ngoc Nguyenthanh [ 19/Jan/18 ]

Reserved ports for rest-integration-test

    <containerHttpPort>8496</containerHttpPort>
    <cargoRmiPort>8495</cargoRmiPort>
    <cargoTomcatAjpPort>8494</cargoTomcatAjpPort>

May need to document in https://wiki.magnolia-cms.com/display/DEV/Integration+tests

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