Fixed IASP support#3258
Conversation
Signed-off-by: Seb Julliand <sebjulliand@gmail.com>
Signed-off-by: Seb Julliand <sebjulliand@gmail.com>
Signed-off-by: Seb Julliand <sebjulliand@gmail.com>
Signed-off-by: Seb Julliand <sebjulliand@gmail.com>
Signed-off-by: Seb Julliand <sebjulliand@gmail.com>
Signed-off-by: Seb Julliand <sebjulliand@gmail.com>
|
👋 A new build is available for this PR based on 6391717. |
| if (asp !== this.currentASP) { | ||
| await Promise.all([ | ||
| this.runSQL('connect reset').then(() => asp ? this.runSQL(`connect to ${asp}`) : undefined), | ||
| this.sendCommand({ command: 'cl "SETASPGRP *NONE"' }).then(() => asp ? this.sendCommand({ command: `cl "SETASPGRP ${asp}"` }) : undefined) |
There was a problem hiding this comment.
Command not qualified
this.sendCommand({ command: 'cl "QSYS/SETASPGRP *NONE"' }).then(() => asp ? this.sendCommand({ command: cl "QSYS/SETASPGRP ${asp}" }) : undefined)
buzzia2001
left a comment
There was a problem hiding this comment.
Ciao @sebjulliand
it seems to work, there is 1 command to qualify...
Also test cases related to iasp are ok
|
Thanks @buzzia2001 , I'll fix that soon. Something like dbName/iaspDevName maybe? |
So I'll try editing whataniasp and see what happens |
Changes
Fixes #3242
This PR fixes potential issues found when using IASP.
Using a profile, user can easily switch between iasps.
The iasp cannot be set on the default profile, it will always be assumed that the default asp will be set on the job description used by the user.
The library list validation method has been changed to use SQL instead of
librarylist.The ASP unit tests have been updated to support these changes. It is now required to set the iASP used for testing in the

.envfile:If it's not set, the tests are skipped.
Tests work fine:

How to test this PR
Checklist