Uploaded image for project: 'SwitchYard'
  1. SwitchYard
  2. SWITCHYARD-427

SwitchYardFacet should validate config xml

    Details

    • Type: Bug
    • Status: Closed (View Workflow)
    • Priority: Major
    • Resolution: Rejected
    • Affects Version/s: 0.2
    • Fix Version/s: None
    • Component/s: configuration, tooling
    • Labels:
      None
    • Estimated Difficulty:
      Low

      Description

      Similar to how the ConfigureMojo of the Maven SwitchYard plugin validates the outputted model configuration xml by default, but has an option to disable it, the SwitchYardFacet should also validate it's outputted xml, most likely triggered by writeSwitchYardConfig(). It is important that this can be disabled, though, in case the user plans on post-processing the xml to make it valid. Here is how the ConfigureMojo does it: http://goo.gl/Kgebe

        Gliffy Diagrams

          Activity

          Hide
          kcbabo Keith Babo added a comment -

          Do we need to do this again in the forge tooling? Config is already validated every time you build an application or run a test. I think it's definitely a good idea to have unit tests for the forge tooling which test for schema validity, but not sure this is necessary when users actually use the tooling.

          Show
          kcbabo Keith Babo added a comment - Do we need to do this again in the forge tooling? Config is already validated every time you build an application or run a test. I think it's definitely a good idea to have unit tests for the forge tooling which test for schema validity, but not sure this is necessary when users actually use the tooling.
          Hide
          kcbabo Keith Babo added a comment -

          Bumping from 0.3 unless we determine this is a critical need for the current release.

          Show
          kcbabo Keith Babo added a comment - Bumping from 0.3 unless we determine this is a critical need for the current release.
          Hide
          dward David Ward added a comment -

          After more consideration, I agree with Keith that this is not necessary.

          Show
          dward David Ward added a comment - After more consideration, I agree with Keith that this is not necessary.

            People

            • Assignee:
              dward David Ward
              Reporter:
              dward David Ward
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development