Uploaded image for project: 'WildFly'
  1. WildFly
  2. WFLY-15720

Rename or remove AssumeTestGroupUtil.assumeElytronProfileEnabled()

    XMLWordPrintable

Details

    Description

      AssumeTestGroupUtil.assumeElytronProfileEnabled() is misnamed because it throws AssumptionViolatedException if the -Delytron profile is used, i.e. if the condition stated in the method name is true. That's backwards and isn't how other methods like assumeSecurityManagerDisabled and assumeJDKVersionBefore/After work.

      So either rename to assumeElytronProfileDisabled() to reflect how it's actually used, or remove it altogether because the whole -Delytron thing is cruft in WildFly main.

      Attachments

        Activity

          People

            bstansbe@redhat.com Brian Stansberry
            bstansbe@redhat.com Brian Stansberry
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: