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

" Test the output for present when Queue Manager does exist " - Failing #42

Open
stewmcl opened this issue Jun 8, 2023 · 2 comments
Open
Assignees
Labels
bug Something isn't working

Comments

@stewmcl
Copy link
Contributor

stewmcl commented Jun 8, 2023

TASK [set queue manager running when it is in present state] ***********************************
ok: [fyre]

TASK [dump test output] ************************************************************************
ok: [fyre] => {
"msg": {
"changed": false,
"failed": false,
"failed_when_result": false,
"msg": "IBM MQ queue manager 'qm1_present_start' started",
"output": "",
"rc": 0,
"state": ""
}
}

TASK [Test the output for present when Queue Manager does exist] *******************************
fatal: [fyre]: FAILED! => {
"assertion": "testout_2.state == 'running'",
"changed": false,
"evaluated_to": false,
"msg": "Assertion failed"
}

Appears to be that the state is not being correctly set

@stewmcl stewmcl added the bug Something isn't working label Jun 8, 2023
@stewmcl stewmcl self-assigned this Jun 8, 2023
@stewmcl stewmcl changed the title Test: set queue manager running when it is in present state failing " Test the output for present when Queue Manager does exist " - Failing Jun 8, 2023
@stewmcl
Copy link
Contributor Author

stewmcl commented Jun 14, 2023

This failing test is a sympton of a larger problem, that being the handling of the results in the queue manager module.
There's too much of a mix of using the results dictionary and manually setting the parameters for the module.exit_json function. As such, this issue should be solved in #44

@stewmcl stewmcl removed their assignment Jul 12, 2023
@JamRamPage JamRamPage self-assigned this May 1, 2024
@JamRamPage
Copy link
Contributor

I think #105 should fix this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

When branches are created from issues, their pull requests are automatically linked.

2 participants