Skip to content

Merge: fix pathing and add a basic integration test - #1050

Merged
josephjclark merged 3 commits into
openfn-merge-commandfrom
merge-fixes
Oct 7, 2025
Merged

josephjclark merged 3 commits into
openfn-merge-commandfrom
merge-fixes

Conversation

@josephjclark

Copy link
Copy Markdown
Collaborator

I know I said we wouldn't bother with integration tests here but I decided to add one

Then remembered why I wasn't keen: to get really good assertions here on UUID preservation is a lot of work. Gonna save that for the test repo - this just serves as a super basic functionality test

@github-project-automation github-project-automation Bot moved this to New Issues in Core Oct 7, 2025
@josephjclark
josephjclark merged commit 0ec5eb7 into openfn-merge-command Oct 7, 2025
6 checks passed
@josephjclark
josephjclark deleted the merge-fixes branch October 7, 2025 14:30
@github-project-automation github-project-automation Bot moved this from New Issues to Done in Core Oct 7, 2025
josephjclark added a commit that referenced this pull request Oct 7, 2025
* feat: init merge command

* tests: update

* feat: add removeUnmapped option

* feat: accept workflow-mappings as cli arguments

* tests: cli object parsing

* tests: merge handler test

* Merge: fix pathing and add a basic integration test (#1050)

* basic integration test

* remove log line

* remove more logging

* version: cli@1.16.0

---------

Co-authored-by: Joe Clark <jclark@openfn.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant