Openid 20210628 #3
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Adds a test for fetching the size attribute of two replicas of a given data object to see that they are different.
Replicas can have different sizes than the size in the catalog for a given data object (stale replicas). Therefore, this change adds a size attribute to iRODSReplica to allow users to get the size of a particular replica.
Escape all StringProperty values for packStruct. HTML escaping was already in place for a few spots, so these have been removed.
MySQL needs uppercased table and column names, so the TestSpecificQuery tests have been made to accommodate this requirement.
The implemented PAM protocol is based on the following steps 1 . AUTH_PLUG_REQ_AN is sent by passing a context containing a_user, a_pw and a_ttl (defaulted to 60) 2 . irods server responds with a temporary password 3 . A a new connection is created (a combination of disconnect + connect). 4 . The temporary password is used to login as a native protocol To use PAM set authentication_scheme='pam' when creating the iRODSSession object iRODSSession(user='yourPAMuser', password='yourPAMpassword, authentication_scheme='pam', [...]) As a plus: - created a generic AuthPluginMessage to replace the GSIAuthMessage - added more constants for better readability
Use start=1 on enumerate
Add example for searching metadata
Finding out the full resource hierarchy is important for dealing with replicas. In the past, we had a naming scheme that allowed us to work around it, but now that naming scheme doesn't provide what we need. Including resc_hier in the replicas information allows us to do this the right way
Also: In cleanup( ), the session pool's connections will now be properly shut down and the pool itself re-initialized to contain no active or idle connections. This is important in case the session is used again, as happens in certain tests (see the admin_test) which deliberately do this kind of refresh to update information from the catalog. When called from iRODSSession's __del__ destructor, an exception to that rule is made, and the pool reinitialization does not happen.
Tests were failing against 4-2-stable and master branches of iRODS server because the PUT test during server installation lasted long enough to satisfy the client's HEARTBEAT requirement, but the new default is to shut down the running server just after installation. So instead of HEARTBEAT we now use communication on a specified (but easily reconfigurable) port 8888 to signal the client that the server is up "for good."
Two tests querying for unicode object names got minor changes / refactor.
…robably worth exposing.
Expose creation time of collection
martingolas
pushed a commit
that referenced
this pull request
Dec 2, 2022
- irods.test.helpers.make_session uses kwargs. (Passing along all options to the iRODSSession constructor.) - Clarify usage and prerequisites for manual SSL and PAM tests. - Skip SSL tests under topology (when server is not co-resident). - keep test suite passing, and reasonably quiet, on Python 3
martingolas
pushed a commit
that referenced
this pull request
Dec 2, 2022
The test routine test_object_names_with_nonprintable_chars( ) failed when running under docker compose (under the Jenkins client dot) due to an attempt to register it from a local directory not shared by the server node.
martingolas
pushed a commit
that referenced
this pull request
Dec 2, 2022
- Correct DEST_RESC (vs. RESC) oversights in irods/irods#5848 related tests. - Correct tests failing for QUASI_XML under Python2. - instances of fix Python 2 / unicode mismatch causing tests to fail.
martingolas
pushed a commit
that referenced
this pull request
Dec 2, 2022
martingolas
pushed a commit
that referenced
this pull request
Dec 2, 2022
martingolas
pushed a commit
that referenced
this pull request
Dec 2, 2022
martingolas
pushed a commit
that referenced
this pull request
Dec 2, 2022
- don't use f"" style strings
- getblocking socket method doesn't exist before Python3.7
- skip test that makes logging explode in client dot
- in test, do not assert that a connection throws an exception on
a second call to disconnect().
martingolas
pushed a commit
that referenced
this pull request
Dec 2, 2022
martingolas
pushed a commit
that referenced
this pull request
Dec 2, 2022
martingolas
pushed a commit
that referenced
this pull request
Dec 2, 2022
martingolas
pushed a commit
that referenced
this pull request
Dec 2, 2022
martingolas
pushed a commit
that referenced
this pull request
Dec 2, 2022
PRC post PUT replication still does emulate the PUT api's after-the-fact replication, and for test_repave_replicas to pass post-4.2.11, it was merely necessary to remove REG_CHKSUM from the keywords active for REPL.
martingolas
pushed a commit
that referenced
this pull request
Dec 2, 2022
martingolas
pushed a commit
that referenced
this pull request
Dec 2, 2022
including: - create "server_python_version" variable - distinguish "client_python_version" - fix Python REP installer
martingolas
pushed a commit
that referenced
this pull request
Dec 2, 2022
Where needed for some tests in login_auth_tests.py to pass, the user alissa was automated. Also, some cleanup was done around the module.
martingolas
pushed a commit
that referenced
this pull request
Dec 2, 2022
martingolas
pushed a commit
that referenced
this pull request
Dec 2, 2022
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.