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

Only allow exact AT versions for recommended test plan versions #1087

Merged
merged 30 commits into from
May 8, 2024

Conversation

alflennik
Copy link
Contributor

Implements restrictions to prevent reports being created with a minimum AT version once the test plan version has reached the recommended phase.

alflennik and others added 25 commits April 3, 2024 13:37
…ress #792

* Add resolver for finding firstRequiredAtVersion for a RECOMMENDED TestPlanVersion, given an atId

* Update tests

* Fix graphql call when including "firstRequiredAtVersion" under "testPlanVersions"

* Update description of firstRequiredAtVersion

* Rename resolver
…anRun

# Conflicts:
#	server/tests/integration/graphql.test.js
@alflennik alflennik requested a review from howard-e May 2, 2024 21:41
Copy link
Contributor

@howard-e howard-e left a comment

Choose a reason for hiding this comment

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

Looks good! Confirmed the minimum version selection isn't allowed, 1 more step along!

onSelect={exactOrMinimum =>
onSelect={exactOrMinimum => {
if (
selectedTestPlanVersion.phase ===
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
selectedTestPlanVersion.phase ===
selectedTestPlanVersion?.phase ===

@@ -336,6 +342,7 @@ const ManageTestQueue = ({
};

const onAtChange = e => {
setShowMinimumAtVersionErrorMessage(false);
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: for consistency, can you move this a line below?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

haha ok, I'll grant you it's an improvement

@alflennik alflennik merged commit 6a98bc7 into trends May 8, 2024
2 checks passed
@alflennik alflennik deleted the minimum-exact-phase branch May 8, 2024 20:09
@howard-e howard-e mentioned this pull request Jun 20, 2024
howard-e added a commit that referenced this pull request Jun 20, 2024
This includes work to support #791 and #792.

Includes the following changes:
* #1055
* #1001
* #1065
* #1052 
* #1087
* #1098 
* #1092
* #1131
* #1124

---------

Co-authored-by: Howard Edwards <[email protected]>
Co-authored-by: Paul Clue <[email protected]>
Co-authored-by: alflennik <[email protected]>
howard-e added a commit that referenced this pull request Jun 24, 2024
Includes the following changes:
* #1123, addresses #791 and #792 with:
  * #1055
  * #1001
  * #1065
  * #1052 
  * #1087
  * #1098 
  * #1092
  * #1131
  * #1124
* #1128, addresses #1100
* #1102, addresses #957
* #1132
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.

2 participants