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

Rename or remove AssumeTestGroupUtil.assumeElytronProfileEnabled()

XMLWordPrintable

      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.

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

                Created:
                Updated:
                Resolved: