Uploaded image for project: 'JBoss Enterprise Application Platform'
  1. JBoss Enterprise Application Platform
  2. JBEAP-9070

Add assumes to failing tests in AS TS with Elytron profile

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Obsolete
    • Icon: Major Major
    • None
    • 7.1.0.DR12
    • Test Suite
    • None

      Add assumes for failing tests in AS TS with Elytron profile (-Delytron) similar to the ones for invocation which sharethe implementation in DisableInvocationTestUtil.

      The class could be reused, but maybe it's more clear when we use a new one (to stay clear in naming).

      Related HipChat discussion:

      [2:27 PM] Josef Cacek: @KabirKhan @ctomc Does it make sense to introduce similar assumptions for -Delytron as the ones used for invocation?
      https://github.com/wildfly/wildfly/blob/096f7168a7af81c864f38383d09afef6a2ea766d/testsuite/shared/sr...
      [2:28 PM] Josef Cacek: to have passing testsuite for PRs?
      [2:35 PM] Tomaz Cerar: @JosefCacek are there tests in elytron TS module that already pass?
      [2:35 PM] Josef Cacek: @ctomc Yes, some credential store tests and Basic authentication tests.
      [2:36 PM] Tomaz Cerar: in that case, sure it is probably fine
      [2:38 PM] Josef Cacek: @ctomc The question was not related to elytron module (which runs already always), but to elytron profile in the integration module.
      [2:38 PM] Josef Cacek: which we would like to run with Pull requests once it's green.
      [2:39 PM] Josef Cacek: - and we can help to make it green for now with a new Assumption tool.
      [2:40 PM] Tomaz Cerar: yeah for that asumptions make sense
      [2:45 PM] Josef Cacek: @ctomc OK, thanks.
      [3:10 PM] Kabir Khan: @JosefCacek @ctomc +1 - please use that same util class
      [3:10 PM] Kabir Khan: DisableInvocationTestUtil in testsuite/shared
      [3:10 PM] Kabir Khan: hmm, perhaps not since probably different tests are affected
      [3:11 PM] Kabir Khan: perhaps create a similar one, so we can search easily for assumed one
      [3:11 PM] Josef Cacek: @KabirKhan we could just introduce a new methods in the same class.
      [3:11 PM] Kabir Khan: ah yes :)
      

              josef.cacek@gmail.com Josef Cacek (Inactive)
              josef.cacek@gmail.com Josef Cacek (Inactive)
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Created:
                Updated:
                Resolved: