Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Integration Tests not Running against new PRs #420

Closed
vinamra28 opened this issue Jul 15, 2020 · 2 comments · Fixed by #421
Closed

Integration Tests not Running against new PRs #420

vinamra28 opened this issue Jul 15, 2020 · 2 comments · Fixed by #421
Assignees
@piyush-garg @PuneetPunamiya
Labels
area/testing Issues or PRs related to testing kind/bug Categorizes issue or PR as related to a bug.

Comments

@vinamra28
Copy link
Member

vinamra28 commented Jul 15, 2020 •
edited
Loading

Expected Behavior

While raising a new PR, integration tests should run and then should give the result whether passed or failed.

Actual Behavior

currently all tests are passing because of this new catalog reorganisation. example integration test logs for PR #391 -> logs
#352 -> logs

Steps to Reproduce the Problem

  1. Raise a new PR to add a task along with corresponding tests according to the latest catalog structure
  2. View the integration test logs

Additional Info

This is happening because the catalog is re-organised and structure of tasks have also changed
@vinamra28
Copy link
Member Author

/kind bug
/area testing

@tekton-robot tekton-robot added kind/bug Categorizes issue or PR as related to a bug. area/testing Issues or PRs related to testing labels Jul 15, 2020
@piyush-garg
Copy link
Contributor

/assign
/assign @PuneetPunamiya

piyush-garg added a commit to piyush-garg/catalog that referenced this issue Jul 16, 2020
@piyush-garg
Update e2e scipts according to new structure
b3bb1ae 
This will update the e2e scripts to run tests according to
new catalog structure after reorg

Fix tektoncd#420
@piyush-garg piyush-garg mentioned this issue Jul 16, 2020
Merged
9 tasks
piyush-garg added a commit to piyush-garg/catalog that referenced this issue Jul 16, 2020
@piyush-garg
Update e2e scipts according to new structure
3bffa60 
This will update the e2e scripts to run tests according to
new catalog structure after reorg

Ignore cloudevent test as it is not working for now

Fix tektoncd#420
piyush-garg added a commit to piyush-garg/catalog that referenced this issue Jul 16, 2020
@piyush-garg
Update e2e scipts according to new structure
830c5b1 
This will update the e2e scripts to run tests according to
new catalog structure after reorg

Fix tektoncd#420
tekton-robot pushed a commit that referenced this issue Jul 16, 2020
@piyush-garg @tekton-robot
Update e2e scipts according to new structure
479c6d9 
This will update the e2e scripts to run tests according to
new catalog structure after reorg

Fix #420
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@piyush-garg piyush-garg

@PuneetPunamiya PuneetPunamiya

Labels
area/testing Issues or PRs related to testing kind/bug Categorizes issue or PR as related to a bug.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Update e2e scipts according to new structure piyush-garg/catalog
4 participants
@piyush-garg @vinamra28 @PuneetPunamiya @tekton-robot