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

Tests: Update host test with ipa-join #77

Closed
wants to merge 2 commits into from

Conversation

mirielka
Copy link
Contributor

@mirielka mirielka commented Sep 13, 2016

Updating path to ipa-join command to allow execution of
test_xmlrpc/test_host::TestHostFalsePwdChange::test_join_host. Fixing
discrepancies in returned and checked attributes.

https://fedorahosted.org/freeipa/ticket/6326

@MartinBasti MartinBasti self-assigned this Sep 15, 2016
@MartinBasti
Copy link
Contributor

Please split this into 2 commits, it contains independent issues

  • wrong path to IPA join
  • extra attribute

It is not related to this PR, but for me it looks that this test is focused to ipa-join command and I don't see how this is related to API tests. I don't think that we can tests this from intree tests because it requires build first

Updating path to ipa-join command to allow execution of
test_xmlrpc/test_host::TestHostFalsePwdChange::test_join_host.

https://fedorahosted.org/freeipa/ticket/6326
Fixing discrepancies between returned and checked attributes in ipa-join host
test, that arose after recent changes in behavior.

https://fedorahosted.org/freeipa/ticket/6326
@MartinBasti MartinBasti added ack Pull Request approved, can be merged pushed Pull Request has already been pushed labels Sep 21, 2016
@mirielka mirielka deleted the master_skipped_host branch September 22, 2016 06:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ack Pull Request approved, can be merged pushed Pull Request has already been pushed
Projects
None yet
2 participants