Skip to content
This repository has been archived by the owner on Nov 11, 2024. It is now read-only.

Refactoring of framework test runner so it can have a single unit test running through it all #618

Merged
merged 10 commits into from
Aug 20, 2024

Conversation

techcobweb
Copy link
Contributor

@techcobweb techcobweb commented Aug 15, 2024

Signed-off-by: Mike Cobbett [email protected]

Why ?

The java test runner is a huge piece of logic with no unit tests.

  • Put a unit test around it and run through the golden path at least.
  • Add mocks as required
  • Do minimal changes needed to get a unit test going, so we can refactor later.

@galasa-team
Copy link

Build successful

1 similar comment
@galasa-team
Copy link

Build successful

@galasa-team
Copy link

Build successful

Signed-off-by: Mike Cobbett <[email protected]>
@galasa-team
Copy link

Build successful

@techcobweb techcobweb requested a review from eamansour August 20, 2024 08:34
@galasa-team
Copy link

Build successful

@galasa-team
Copy link

Build successful

@galasa-team
Copy link

Build failed, see http://localhost:8001/api/v1/namespaces/tekton-pipelines/services/tekton-dashboard:http/proxy/#/namespaces/galasa-build/pipelineruns/repo-framework-pr-618-lrrm2 for details. If you are unable to do so, please contact a member of the Galasa team.

@galasa-team
Copy link

Build successful

@techcobweb techcobweb merged commit 39ea101 into main Aug 20, 2024
7 checks passed
@techcobweb techcobweb deleted the mcobbett-overrides branch August 20, 2024 15:40
@galasa-team
Copy link

Build successful

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants