fix: controller crash when provenance is set only on the global repo

[savitaashture]

This commit fixes the controller crash by adding
a nil pointer check before accessing the elements of an object. Issue: https://issues.redhat.com/browse/SRVKP-5571

Verification:

Created Global Repo like below

apiVersion: pipelinesascode.tekton.dev/v1alpha1
kind: Repository
metadata:  
  name: pipelines-as-code  
  namespace: openshift-pipelines
spec:  
  settings:    
    pipelinerun_provenance: "default_branch"

created local Repo like below

apiVersion: pipelinesascode.tekton.dev/v1alpha1
kind: Repository
metadata:  
  name: test  
  namespace: test
spec:  
  settings: {}
  url: https://github.com/savitaashture/test-pac

Signed-off-by: Savita Ashture [email protected]

Changes

Submitter Checklist

• 📝 Please ensure your commit message is clear and informative. For guidance on crafting effective commit messages, refer to the How to write a git commit message guide. We prefer the commit message to be included in the PR body itself rather than a link to an external website (ie: Jira ticket).

• ♽ Before submitting a PR, run make test lint to avoid unnecessary CI processing. For an even more efficient workflow, consider installing pre-commit and running pre-commit install in the root of this repository.

• ✨ We use linters to maintain clean and consistent code. Please ensure you've run make lint before submitting a PR. Some linters offer a --fix mode, which can be executed with the command make fix-linters (ensure markdownlint and golangci-lint tools are installed first).

• 📖 If you're introducing a user-facing feature or changing existing behavior, please ensure it's properly documented.

• 🧪 While 100% coverage isn't a requirement, we encourage unit tests for any code changes where possible.

• 🎁 If feasible, please check if an end-to-end test can be added. See README for more details.

• 🔎 If there's any flakiness in the CI tests, don't necessarily ignore it. It's better to address the issue before merging, or provide a valid reason to bypass it if fixing isn't possible (e.g., token rate limitations).

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 64.63%. Comparing base (9ca0a1a) to head (ca4fc85).
Report is 1 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1742      +/-   ##
==========================================
+ Coverage   64.61%   64.63%   +0.01%     
==========================================
  Files         146      146              
  Lines       11271    11271              
==========================================
+ Hits         7283     7285       +2     
+ Misses       3456     3454       -2     
  Partials      532      532              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[enarha]

/lgtm

[savitaashture]

I have corrected e2e

But why linter is failing i need to check

[savitaashture]

/test linters

[savitaashture]

@zakisk @enarha CI is green
PTAL Thank you

[enarha]

I like it better with the updated test. Thanks.

[savitaashture]

/test go-testing