Skip to content

Error and Error Handling Changes#53

Closed
ElijahSwiftIBM wants to merge 1 commit into
devfrom
feature/handle-empty-return
Closed

Error and Error Handling Changes#53
ElijahSwiftIBM wants to merge 1 commit into
devfrom
feature/handle-empty-return

Conversation

@ElijahSwiftIBM

Copy link
Copy Markdown
Collaborator

-Standardize SecurityRequestError Structure, merging IRRSMO00 Errors and that of RACF Errors with the smae internal structure in the result dictionary
-Add an install script that defines IRR.IRRSMO00.PRECHECK with UACC of none and/or checks if the profile exists and the current user's access -Added NullResponseError
-Added Unit Testing for Install script and null response error

💡

Issue: #44 #48 #52

💻 What does this address?

Addresses listed issues in pyRACF's error handling and associated processing.

  1. Adds new error for Null Response from IRRSMO00
  2. Standardizes SecurityRequestError structure
  3. Adds "install script" to define/check authorizations for IRR.IRRSMO00.PRECHECK

📟 Implementation Details

  1. Checked for null string response from IRRSMO00 and throw new error if so
  2. Define new methods of SecurityRequestError to restructure IRRSMO00 response XML with IRRSMO00 error to more closely align with RACF-error structure
  3. Added "Scripts" folder and defined externally available function that checks active user's access to IRR.IRRSMO00.PRECHECK and creates the resource if possible.

📋 Is there a test case?

Designed new test cases for new error and install script under "common" test cases. Tested new SecurityRequestError functions with existing user and group error tests

-Standardize SecurityRequestError Structure, merging IRRSMO00 Errors and that of RACF Errors with the smae internal structure in the result dictionary
-Add an install script that defines IRR.IRRSMO00.PRECHECK with UACC of none and/or checks if the profile exists and the current user's access
-Added NullResponseError
-Added Unit Testing for Install script and null response error

Signed-off-by: Elijah Swift <elijah.swift@ibm.com>
@ElijahSwiftIBM

Copy link
Copy Markdown
Collaborator Author

This PR was closed as all included features are a part of #57

@ElijahSwiftIBM ElijahSwiftIBM deleted the feature/handle-empty-return branch November 26, 2023 02:15
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.

1 participant