Skip to content

Conversation

@dnwe
Copy link
Collaborator

@dnwe dnwe commented Dec 27, 2024

To avoid re-running all the unittests when running the integration (FV) tests, add a go:build directive of '!integration' to all unittest files.

@dnwe dnwe added the chore label Dec 27, 2024
@dnwe dnwe requested review from hindessm and prestona December 27, 2024 14:56
dnwe added 4 commits December 27, 2024 23:03
To avoid re-running all the unittests when running the integration (FV)
tests, add a go:build directive of '!integration' to all unittest files.

Signed-off-by: Dominic Evans <dominic.evans@uk.ibm.com>
Move unittest and functional test shared code into helper files

Signed-off-by: Dominic Evans <dominic.evans@uk.ibm.com>
Signed-off-by: Dominic Evans <dominic.evans@uk.ibm.com>
Currently gotestsum just outputs the package-level metrics. For FV it
would be useful to see the per-func metrics for the main package.

Signed-off-by: Dominic Evans <dominic.evans@uk.ibm.com>
@dnwe dnwe changed the base branch from main to dnwe/go December 27, 2024 22:04
Base automatically changed from dnwe/go to main January 6, 2025 09:41
Copy link
Collaborator

@hindessm hindessm left a comment

Choose a reason for hiding this comment

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

LGTM

@dnwe dnwe merged commit d6ea1b0 into main Jan 6, 2025
14 checks passed
@dnwe dnwe deleted the dnwe/unittests branch January 6, 2025 09:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3 participants