Skip to content
This repository has been archived by the owner on Oct 10, 2023. It is now read-only.

Update git test #198

Merged
merged 15 commits into from
May 9, 2023
Merged

Update git test #198

merged 15 commits into from
May 9, 2023

Conversation

thepetk
Copy link
Collaborator

@thepetk thepetk commented May 2, 2023

What does this PR do?

After the migration of Alizer test resources to their new Repository: alizer-test-resources it updates the flow of git_test_recongizer.go in order to apply tests independently. More detailed:

  • Updates the git_test.json in order to have the correct resources and not using the git repos.
  • Updates the flow of TestExternalRepos in order to clone the alizer-test-resources and loop through its different directories.

Which issue(s) this PR fixes:

Fixes #155

@thepetk thepetk requested a review from mike-hoang May 3, 2023 10:44
@thepetk thepetk mentioned this pull request May 4, 2023
@thepetk
Copy link
Collaborator Author

thepetk commented May 4, 2023

@mike-hoang I have added a test case for #200 (I've imported the hypershift project with this PR).

I'm putting WIP because:

@thepetk thepetk changed the title Update git test WIP: Update git test May 4, 2023
thepetk and others added 7 commits May 4, 2023 15:38
Signed-off-by: thepetk <thepetk@gmail.com>
Signed-off-by: thepetk <thepetk@gmail.com>
Signed-off-by: thepetk <thepetk@gmail.com>
Co-authored-by: Michael Hoang <35011707+mike-hoang@users.noreply.github.com>
Signed-off-by: thepetk <thepetk@gmail.com>
Signed-off-by: thepetk <thepetk@gmail.com>
@thepetk thepetk force-pushed the update_git_test branch from 60eb291 to 3b8591f Compare May 4, 2023 14:41
Signed-off-by: thepetk <thepetk@gmail.com>
@thepetk thepetk requested a review from mike-hoang May 5, 2023 08:31
Signed-off-by: thepetk <thepetk@gmail.com>
@thepetk thepetk changed the title WIP: Update git test Update git test May 8, 2023
@thepetk
Copy link
Collaborator Author

thepetk commented May 8, 2023

@mike-hoang all sub-tasks have been completed and I think we are ready now :)

@thepetk thepetk merged commit 3b9a354 into redhat-developer:main May 9, 2023
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.

Create test resources for go library and avoid using external repos
2 participants