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

Switch multi test #1039

Closed
wants to merge 4 commits into from
Closed

Switch multi test #1039

wants to merge 4 commits into from

Conversation

iArnold
Copy link
Contributor

@iArnold iArnold commented Feb 18, 2015

Added a test that for now fails multiple value selection for switch and while editing that file corrected some errors in the commented test switch-all.

Added a test for multiple values selection "switch-multi" and correction of typo's in commented test "switch-all"
@PeterWAWood
Copy link
Contributor

Thanks for submitting this extra test Arnold and special thanks for spotting and correcting the error in the commented out test.

The test fails under the compiler. This is not surprising as this feature is not yet implemented (and may even not be implemented).

The automatic builds rely on the tests passing so we cannot add a failing test to the test suite. I will manually add your tests to the test suite with the assert commented out so that the test does not fail. It can act as a placeholder for us until the decision has been made whether to implement this feature or not.

@iArnold
Copy link
Contributor Author

iArnold commented Feb 19, 2015

Well the test was supposed to succeed, i.e. the compiled version giving the wrong value, but the test was not executed as compiled code :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants