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

Update ASP configuration and tests #2483

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Conversation

worksofliam
Copy link
Contributor

@worksofliam worksofliam commented Jan 25, 2025

Replace the old ASP configuration with a new API and implement tests for basic ASP functionality. This change enhances the handling of ASPs throughout the application.

This means that Code for IBM i will always use the ASP that is defined as part of their user profile / job description.

The downside about this PR is I actually have no systems to test it on. So I kind of buggered up there. But, I have started on some test cases which will only run when an ASP is defined.

  • Finish off test cases

Signed-off-by: worksofliam <mrliamallan@live.co.uk>
Signed-off-by: worksofliam <mrliamallan@live.co.uk>
Signed-off-by: worksofliam <mrliamallan@live.co.uk>
Signed-off-by: worksofliam <mrliamallan@live.co.uk>
@worksofliam worksofliam temporarily deployed to testing_environment January 25, 2025 19:42 — with GitHub Actions Inactive
Copy link
Contributor

👋 A new build is available for this PR based on 6e87a85.

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