freeipa

#9332 Extend negative test coverage for automember
Closed: fixed by frenaud. Opened by mbhalodi.

Closed: fixed
Reopen Issue

Description

Revisit the automember-cli bash test written in downstream and port the remaining (like the negative tests) scenarios to upstream.

A few bash test examples which can be ported are:
- ipaautomember_addAutomemberCondition_negative_badgroup
- ipaautomember_addAutomemberCondition_negative_badtype
- ipaautomember_addAutomemberCondition_negative_badkey
And many more.

PR: https://github.com/freeipa/freeipa/pull/6718

master:

  • 4119e4e79905b911cfcb001bf75838732c6c472a ipatests: add missing automember-cli tests

The above commit adds most of the missing tests. We still need new code in test_xmlrpc for the ipa automember-remove-condition commands.

I don't see removeAutomemberCondition (def remove_condition) in upstream defined like in base.py or anywhere in the plugins, so it's not been ported yet!

FYI I have already marked those (ipa automember-remove-condition) tests as remaining earlier only while I was revisiting them one by one from downstream.

Note: To write the test for this, I need to write the remove condition function first, will update you soon on this!

ipa-4-9:

  • 34c1574bed9fe6d35ea6a9e04f4e2e148fec9788 ipatests: add missing automember-cli tests

ipa-4-10:

  • 6db9bbd85a837950d9244502507535c1f79ab64a ipatests: add missing automember-cli tests

master:

  • ef0ae496b7e0923de9ce499ce2401318c7fb8ace ipatests: add remove automember condition tests

ipa-4-10:

  • 846c267f58ecfa4fc1a1a3be91c404e58074b1b3 ipatests: add remove automember condition tests

ipa-4-9:

  • 2c3e799fba866a7207291dd6c88ba10f915d3cf7 ipatests: add remove automember condition tests

Metadata Update from @frenaud:
- Issue close_status updated to: fixed
- Issue status updated to: Closed (was: Open)
Metadata
None
None
None
None
None
Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.
Red Hat Logo

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.