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

Cleanup work for launching single task executions. #102

Merged
merged 6 commits into from
Oct 8, 2020

Conversation

schottra
Copy link
Contributor

@schottra schottra commented Oct 6, 2020

  • Updated EntityDetails to pass the correct props when launching a Task
  • Updated RelaunchExecutionForm and underlying hooks/functions to generate the correct initial parameters based on whether the source execution was a single task or a workflow.
  • Updated header of Execution Details page to render the correct parent link based on what type of execution we are viewing (workflow vs. single task)
  • Updated Relaunch flow to set the referenceExecution value when relaunching
  • Updated ExecutionMetadata component to show referenceExecution values when they exist.

@schottra schottra marked this pull request as draft October 6, 2020 23:47
@codecov-io
Copy link

codecov-io commented Oct 7, 2020

Codecov Report

Merging #102 into single-task-execution will increase coverage by 0.58%.
The diff coverage is 89.01%.

Impacted file tree graph

@@                    Coverage Diff                    @@
##           single-task-execution     #102      +/-   ##
=========================================================
+ Coverage                  68.58%   69.16%   +0.58%     
=========================================================
  Files                        382      383       +1     
  Lines                       6254     6304      +50     
  Branches                     973      987      +14     
=========================================================
+ Hits                        4289     4360      +71     
+ Misses                      1965     1944      -21     
Impacted Files Coverage Δ
src/components/Launch/LaunchForm/launchMachine.ts 86.53% <ø> (ø)
src/components/Launch/LaunchForm/types.ts 100.00% <ø> (ø)
src/models/Execution/api.ts 95.00% <ø> (ø)
src/components/Entities/EntityDetails.tsx 44.11% <20.00%> (-4.16%) ⬇️
...Launch/LaunchForm/useMappedExecutionInputValues.ts 72.72% <72.72%> (ø)
...ExecutionDetails/ExecutionDetailsAppBarContent.tsx 95.65% <100.00%> (ø)
.../Executions/ExecutionDetails/ExecutionMetadata.tsx 100.00% <100.00%> (ø)
...cutions/ExecutionDetails/RelaunchExecutionForm.tsx 100.00% <100.00%> (+46.66%) ⬆️
...omponents/Executions/ExecutionDetails/constants.ts 100.00% <100.00%> (ø)
...rc/components/Executions/ExecutionDetails/utils.ts 100.00% <100.00%> (ø)
... and 6 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b71b307...b7e8e03. Read the comment docs.

@schottra schottra marked this pull request as ready for review October 7, 2020 22:55
Copy link

@podehnal podehnal left a comment

Choose a reason for hiding this comment

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

👍

@schottra schottra merged commit c32b653 into single-task-execution Oct 8, 2020
schottra added a commit that referenced this pull request Oct 8, 2020
* refactor: make workflow details generic so it can be used for tasks (#96)

* refactor: make workflow details generic so it can be used for tasks

* chore: cleanup and moving tests over

* feat: adding route and navigation to the task details page (#97)

* Refactor Launch form to use a state machine (#99)

* refactor: filling out details of the state machine for launch

* refactor: checkpoint

* refactor: mostly finished wiring of machine to component state

* refactor: more work to get form component migrated to using machine

* refactor: cleaning up state for selectors

* fix: type error due to patch version upgrade

* refactor: trying a flat state structure

* fix: getting all tests passing again

* chore: cleanup and docs

* chore: pull request feedback

* refactor: Make launch state and components generic (#100)

* refactor: splitting launch machine into two separate types

* refactor: move shared state out to component

* refactor: use composition to create workflow form

* refactor: update usage of LaunchWorkflowForm -> LaunchForm

* chore: cleanup and fix tests

* feat: Add Task support to Launch form (#101)

* feat: add task support in launch components

* test: updating launch form tests to handle task cases

* fix: remaining broken tests

* Cleanup work for launching single task executions. (#102)

* feat: add support for launch tasks in entity details view

* fix: correctly map initial parameters when relaunching

* fix: correct parent name and back link in execution details page

* fix: pass through referenceExecution when relaunching

* test: check rendering of referenceExecution

* test: adding tests for relaunch flow
@schottra schottra deleted the single-task-cleanup branch December 4, 2020 20:34
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.

3 participants