Issue97 jeremypng

[jeremypng]

Let me know if you guys want anything else changed here. This code works in my testing.

update: fixed spacing, hopefully passes CI this time

[sonarqubecloud]

SonarCloud Quality Gate failed.

0 Bugs
0 Vulnerabilities
0 Security Hotspots
2 Code Smells

No Coverage information
46.3% Duplication

[jasonking3]

@jeremypng - I think we could pass the SonarCloud check if we just called get_action_status from waitfor_action_completion rather than duplicating all that code. Other than that, I think we are good. I like the change to also look at property names.

[sonarqubecloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
0.0% Duplication

[jeremypng]

Ok, I brought this one up to date w/ master, and removed the duplicated code in waitfor_action_completion. Are we good to go?

[sonarqubecloud]

SonarCloud Quality Gate failed.   

0 Bugs
0 Vulnerabilities
8 Security Hotspots
2 Code Smells

No Coverage information
0.0% Duplication