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.

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

Add illegal version tests to SupportedVersionTest #3239

Merged
merged 6 commits into from
Mar 14, 2025

Conversation

ohadzeliger
Copy link
Contributor

This resolves #3238
Adding a few tests that include illegal version clauses to the end-to-end YAML SupportedVersionTest

@ohadzeliger ohadzeliger self-assigned this Mar 10, 2025
@ohadzeliger ohadzeliger added the testing improvement Change that improves our testing label Mar 10, 2025
@ohadzeliger ohadzeliger requested a review from ScottDugas March 10, 2025 19:21
@ohadzeliger ohadzeliger changed the base branch from main to 4.0.561-release March 11, 2025 22:03
@ohadzeliger ohadzeliger changed the base branch from 4.0.561-release to main March 11, 2025 22:03
# See the License for the specific language governing permissions and
# limitations under the License.

# This test should be fail
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
# This test should be fail
# This test should fail

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

@ohadzeliger ohadzeliger merged commit 0dd025e into FoundationDB:main Mar 14, 2025
5 checks passed
@ohadzeliger ohadzeliger deleted the supported-version-tests branch March 14, 2025 16:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
testing improvement Change that improves our testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add SupportedVersionTest for illegal versions
2 participants