| Commit message (Collapse) | Author | Age | Files | Lines |
... | |
|
|
|
|
|
| |
Recent ipaldap work has made LDAPEntry incompatible with python-ldap's
LDIFWriter.
Convert entry to dict before printing debug output.
|
| |
|
|
|
|
| |
Part of: https://fedorahosted.org/freeipa/ticket/3833
|
|
|
|
|
|
|
|
|
|
|
| |
This is achieved by storing both decoded and encoded attribute values in
LDAPEntry and synchronizing changes between them whenever an attribute is
accessed.
Added a new property "raw" to LDAPEntry. It provides a dictionary-like
object which can be used to directly access encoded attribute values.
https://fedorahosted.org/freeipa/ticket/3521
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Adds support for host definition by a environment variables of the
following form:
ROLE_<keyword>_envX, where X is the number of the environment
for which host referenced by a role <keyword> should be defined.
Adds a required_extra_roles attribute to the IntegrationTest class,
which can test developer use to specify the extra roles that this
particular test requires. If not all required extra roles are
available, the test will be skipped.
All extra (and static) roles are accessible to the IntegrationTests
via the host_by_role method, which returns a host of given role.
Part of: https://fedorahosted.org/freeipa/ticket/3833
|
|
|
|
| |
Part of: https://fedorahosted.org/freeipa/ticket/3833
|
|
|
|
| |
Part of: https://fedorahosted.org/freeipa/ticket/3833
|
|
|
|
| |
Part of: https://fedorahosted.org/freeipa/ticket/3833
|
|
|
|
|
|
| |
- was broken by navigation and application controller refactoring
https://fedorahosted.org/freeipa/ticket/4003
|
| |
|
|
|
|
|
|
| |
When the directory directly under root (e.g. /etc) did not exist,
mkdir_recursive failed.
Fix the issue.
|
|
|
|
|
|
|
| |
The rename tests use names that were not being cleaned up when the
tests fail. Add cleanup steps for them.
Also, use --force so system permissions are removed as well.
|
| |
|
|
|
|
| |
https://fedorahosted.org/freeipa/ticket/3819
|
|
|
|
| |
Part of: https://fedorahosted.org/freeipa/ticket/3834
|
|
|
|
| |
Part of: https://fedorahosted.org/freeipa/ticket/3834
|
|
|
|
| |
Part of: https://fedorahosted.org/freeipa/ticket/3834
|
|
|
|
| |
Part of: https://fedorahosted.org/freeipa/ticket/3834
|
|
|
|
| |
Part of: https://fedorahosted.org/freeipa/ticket/3834
|
|
|
|
| |
Part of: https://fedorahosted.org/freeipa/ticket/3834
|
|
|
|
| |
Part of: https://fedorahosted.org/freeipa/ticket/3834
|
|
|
|
|
|
|
|
|
|
|
| |
The integration tests must wait for replication to happen before checking
results. In some cases, the tests have failed because the checks that
detect completed replication were insufficient.
This fixes the code to:
- Wait for replication to be completed on both servers
- In the case of an error, continue waiting -- it might be the case that
the DS is temporarily unreachable
|
|
|
|
|
| |
This makes the tests shorter, more descriptive, and easier to change
e.g. when new attributes are added.
|
|
|
|
|
|
|
|
|
|
| |
The --pin and --dirman-password options simplified ipa-certinstall
usage. Use them in tests.
Also add tests for the old way of calling the command.
https://fedorahosted.org/freeipa/ticket/3869
http://www.freeipa.org/page/V3/ipa-server-certinstall_CLI_cleanup
|
|
|
|
| |
The test installed the HTTP cert instead of the DS one.
|
|
|
|
|
|
|
|
|
|
|
| |
When trying to create a new ordered test case by inheriting
from already defined test case, by overriding few of its methods,
the execution order of the tests is as follows:
- first all non-overriden test methods from the parent test class
- then all overriden tests methods
This patch makes sure that methods are executed in the logical order,
that is, the order defined in the parent class.
|
|
|
|
|
| |
The StrEnum and Int tests are restructured to allow sharing the tests.
Individual *Enum tests are separated into methods.
|
|
|
|
|
| |
Int already needed to take both int and long. This makes the functionality
available for all Param classes.
|
|
|
|
| |
Options should be Param subclasses.
|
| |
|
|
|
|
|
| |
When reporting argument errors, show also a context -- what is processed,
what is the name of the command.
|
|
|
|
|
|
| |
If an exception is expected, but another one is raised, the CLITestContext
raised a generic AssertionError.
Pass through the original exception instead
|
| |
|
|
|
|
|
|
|
|
|
|
|
| |
To differentiate between individual tests in BeakerLib output,
the argument needs to be added to the test name. Since Nose
doesn't provide a way to get the argument in a plugin,
a `test_argument` attribute must be added to the test function
to support this, simlarly to how `description` is used to set
individual "docstrings".
Add test_argument to the generated tests in the CA-less suite.
|
|
|
|
|
|
|
| |
Ordered test generators were not announced in plugin hooks, so
e.g. the Beakerlib or collect plugin did not announce them.
Exclude test generators from ordering.
|
| |
|
| |
|
|
|
|
|
|
|
|
|
|
| |
This adds a transport that uses /usr/bin/ssh calls to communicate
with remote hosts.
This transport is a bit slower and buffers output more than paramiko,
so it is only used if paramiko is not available, or forced with an
environment variable.
https://fedorahosted.org/freeipa/ticket/3890
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This will make it possible to use a different mechanism for cases like
- Paramiko is not available
- Hosts without SSH servers (e.g. Windows)
Add BaseHost, Transport & Command base classes that define the interface
and common functionality, and Host, ParamikoTransport & SSHCommand with
specific details.
The {get,put}_file_contents methods are left on Host for convenience;
all other Transport methods must be now accessed through the transport.
Part of the work for https://fedorahosted.org/freeipa/ticket/3890
|
|
|
|
|
|
| |
Map type radio used old way of defining that its value should not be used in add command. Recent patch related to 'enable' attribute hardened/fixed the behavior of radio widgets so they are disabled in UI as well when enabled==false. Automount did not reflect this change.
https://fedorahosted.org/freeipa/ticket/3954
|
|
|
|
|
|
|
|
| |
Old host entries are missing object class with krbticketflags attribute. Therefore UI does not receive attrlevelrights for it. This OC is added when ipakrbokasdelegate is set.
This patch adds the usual hack for such cases.
https://fedorahosted.org/freeipa/ticket/3940
|
|
|
|
|
|
|
|
|
| |
Updating a CIDict with data like {'A': 1, 'a': 2} would lead to data
loss since only one of the items would get to the CIDict.
This can result in non-obvious bugs similar to this one in python-ldap:
https://bugzilla.redhat.com/show_bug.cgi?id=1007820
Raise an error in this case; any resolution must be done by the caller.
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Make the CIDict interface match standard dict (except view* methods).
Add __contains__, __iter__, clear.
Add keyword and iterable support for __init__, update.
Also add values() and itervalues(). Previously the dict versions were
used; the new ones guarantee that the order matches keys().
Mark view* methods as not implemented.
CIDict.copy() now returns a CIDict.
Test the above additions, and fromkeys() which worked but wasn't tested.
|
| |
|
|
|
|
|
|
|
|
| |
Nose doesn't pick up directories that don't begin with 'test'.
Rename ipatests/test_ipaserver/install to test_install so that it's run.
Also, merge test_ipautil.py from ipapython/test into tests/test_ipapython,
so the whole test suite is in one place.
|
|
|
|
| |
https://fedorahosted.org/freeipa/ticket/3907
|
|
|
|
|
|
|
|
| |
The search had an incorrect scope and therefore it didn't find any RUV.
This issue prevented removing of replica.
https://fedorahosted.org/freeipa/ticket/3876
|
|
|
|
|
|
|
| |
Add integration tests for the forced client re-enrollment feature:
http://www.freeipa.org/page/V3/Forced_client_re-enrollment#Test_Plan
https://fedorahosted.org/freeipa/ticket/3832
|
|
|
|
|
|
| |
Add man pages for ipa-run-tests, ipa-test-task, and ipa-test-config.
https://fedorahosted.org/freeipa/ticket/3855 (part 5)
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Differences from the test plan at
http://www.freeipa.org/index.php?title=V3/CA-less_install&oldid=6669 are:
- The following tests are included in all applicable positive
install tests, rather than being standalone test cases:
- Verify CA certificate stored in LDAP
- Verify CA PEM file created by IPA server install
- Verify that IPA server install does not configure certmonger
- Verify CA PEM file created by IPA replica install
- Verify that IPA replica install does not configure certmonger
- Verify CA PEM file created by IPA client install
- PKI setup is done only once for each test class
- Master installation is done once for the IPA command tests, and
once for the certinstall tests
- Certificates are compared after base64 decoding to avoid failures
from formatting mismatches
- Minor changes necessary for automation (e.g. adding --unattended
and --password options, correcting error messages)
- Web UI tests are not included here
https://fedorahosted.org/freeipa/ticket/3830
|