Skip to content
This repository has been archived by the owner on Apr 7, 2022. It is now read-only.

[1LP][RFR] Added automation coverage for test_keypair_ownership_association #9992

Merged
merged 1 commit into from
Mar 26, 2020

Conversation

mmojzis
Copy link
Contributor

@mmojzis mmojzis commented Mar 17, 2020

No description provided.

@dajoRH dajoRH added the lint-ok label Mar 17, 2020
@mmojzis mmojzis force-pushed the test_keypair_ownership_coverage branch from f4ba2c4 to 2a9ee40 Compare March 23, 2020 12:14
Copy link
Contributor

@john-dupuy john-dupuy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

1 tiny comment, but LGTM

def test_keypair_ownership_association(appliance, keypair):
"""
Bugzilla: 1713400
Bugzilla: 1713400, 1747179
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In the rest of the codebase, this is formatted like:

Suggested change
Bugzilla: 1713400, 1747179
Bugzilla:
1713400
1747179

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fixed

@john-dupuy john-dupuy changed the title [RFR] Added automation coverage for test_keypair_ownership_association [1LP][WIPTEST] Added automation coverage for test_keypair_ownership_association Mar 23, 2020
@john-dupuy john-dupuy added the blockers-only To be applied on PRs which contain blocker modifications ONLY label Mar 23, 2020
@mmojzis mmojzis force-pushed the test_keypair_ownership_coverage branch from 2a9ee40 to e0974f6 Compare March 26, 2020 15:14
@mmojzis mmojzis changed the title [1LP][WIPTEST] Added automation coverage for test_keypair_ownership_association [1LP][RFR] Added automation coverage for test_keypair_ownership_association Mar 26, 2020
@mshriver mshriver self-assigned this Mar 26, 2020
@mshriver mshriver merged commit b4184cb into ManageIQ:master Mar 26, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
blockers-only To be applied on PRs which contain blocker modifications ONLY lint-ok
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants