Skip to content

Conversation

@mbrandenburger
Copy link
Member

@mbrandenburger mbrandenburger commented Oct 2, 2025

  • cleanup / update gh actions
  • cleanup makefile
  • move some integration tests to address changes in makefile
  • integration tests generate cmd folder now in out/cmd for each integration test.

@mbrandenburger mbrandenburger added this to the Q4 milestone Oct 2, 2025
@mbrandenburger mbrandenburger self-assigned this Oct 2, 2025
@mbrandenburger mbrandenburger added cleanup Something needs some cleanup testing All related to testing labels Oct 2, 2025
@mbrandenburger mbrandenburger force-pushed the itest-restruct branch 2 times, most recently from a33cb84 to 83e0585 Compare October 2, 2025 15:28
@mbrandenburger mbrandenburger requested a review from adecaro October 2, 2025 15:53
@mbrandenburger mbrandenburger marked this pull request as ready for review October 2, 2025 15:53
@mbrandenburger mbrandenburger marked this pull request as draft October 2, 2025 16:03
@mbrandenburger
Copy link
Member Author

mbrandenburger commented Oct 2, 2025

TODO: make clean should not delete cmd and testfolder in integration/fsc/pingpong

@mbrandenburger mbrandenburger marked this pull request as ready for review October 3, 2025 14:56
@mbrandenburger
Copy link
Member Author

TODO: make clean should not delete cmd and testfolder in integration/fsc/pingpong

As discussed, this PR changes NWO to place the generated cmd folders for into out/cmd for each integration test.
This allows to better differentiate between generate code and code we want to have checked in. integration/fsc/pingpong is an example, where we have both, cmd and testdata folders checked in.

Copy link
Contributor

@arner arner left a comment

Choose a reason for hiding this comment

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

Nice one!

Makefile Outdated
cd ./integration/$(firstword $(subst -, ,$*))/$(subst $(firstword $(subst -, ,$*))-,,$*); \
ginkgo $(GINKGO_TEST_OPTS) .

#cd ./integration/$(word 1,$(subst -, ,$(@:integration-tests-%=%)))/$(word 2,$(subst -, ,$(@:integration-tests-%=%))); \
Copy link
Contributor

Choose a reason for hiding this comment

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

can this be removed or explained?

Copy link
Member Author

Choose a reason for hiding this comment

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

good catch! removed

- cleanup / update gh actions
- cleanup makefile
- move some integration tests to adress changes in makefile

Signed-off-by: Marcus Brandenburger <bur@zurich.ibm.com>
Signed-off-by: Marcus Brandenburger <bur@zurich.ibm.com>
@mbrandenburger mbrandenburger merged commit 5ceb3a6 into hyperledger-labs:main Oct 7, 2025
32 of 33 checks passed
@mbrandenburger mbrandenburger deleted the itest-restruct branch October 7, 2025 09:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cleanup Something needs some cleanup testing All related to testing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants