Skip to content

Conversation

@sutaakar
Copy link
Contributor

Issue link

What changes have been made

Creating own copy of RayCluster CR for every negative unit test to make sure they don't affect each other.

Verification steps

Run unit tests.

Checks

  • I've made sure the tests are passing.
  • Testing Strategy
    • Unit tests
    • Manual tests
    • Testing is not required for this change

@openshift-ci openshift-ci bot requested review from Bobbins228 and jbusche July 23, 2024 07:34
@sutaakar sutaakar requested review from ChristianZaccaria and removed request for Bobbins228 and jbusche July 23, 2024 07:35
Copy link
Contributor

@ChristianZaccaria ChristianZaccaria left a comment

Choose a reason for hiding this comment

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

/lgtm thanks!

})

// Negative Test Cases
trueBool := true
Copy link
Contributor

@ChristianZaccaria ChristianZaccaria Jul 26, 2024

Choose a reason for hiding this comment

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

:) no comments

@openshift-ci
Copy link

openshift-ci bot commented Jul 26, 2024

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: ChristianZaccaria

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@ChristianZaccaria
Copy link
Contributor

/approved

@openshift-merge-bot openshift-merge-bot bot merged commit 2cd228e into project-codeflare:main Jul 26, 2024
@sutaakar sutaakar deleted the unit-test-fix branch July 26, 2024 07:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants