Skip to content

fix LicensesIT and add to list of tests, fix output of "set license active" API #8501#8502

Merged
kcondon merged 5 commits intodevelopfrom
8501-licenses-it
Mar 17, 2022
Merged

fix LicensesIT and add to list of tests, fix output of "set license active" API #8501#8502
kcondon merged 5 commits intodevelopfrom
8501-licenses-it

Conversation

@pdurbin
Copy link
Member

@pdurbin pdurbin commented Mar 17, 2022

What this PR does / why we need it:

A few reasons:

  • LicensesIT isn't being executed by Jenkins (or docker-aio).
  • LicensesIT isn't passing
  • The output from the "set license active" API is missing a space ("ID 5set" should be "ID 5 set")

Which issue(s) this PR closes:

Closes #8501

Special notes for your reviewer:

None.

Suggestions on how to test this:

  • Let Jenkins run the test suite.
  • Confirm that a space has been added to the "set license active" API (see above).

Does this PR introduce a user interface change? If mockups are available, please link/include them here:

No.

Is there a release notes update needed for this change?:

No.

Additional documentation:

None.

@pdurbin pdurbin changed the title add LicensesIT to list of tests #8501 fix LicensesIT and add to list of tests, fix output of "set license active" API #8501 Mar 17, 2022
@pdurbin pdurbin removed their assignment Mar 17, 2022
@kcondon kcondon self-assigned this Mar 17, 2022
@kcondon kcondon merged commit 0738520 into develop Mar 17, 2022
@kcondon kcondon deleted the 8501-licenses-it branch March 17, 2022 20:08
@pdurbin pdurbin added this to the 5.10 milestone Mar 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

LicensesIT not executed by Jenkins (missing from tests/integration-tests.txt)

3 participants