Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Merged
merged 5 commits into from
Mar 17, 2022

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