asyncapi / java-template

Java template for the AsyncAPI Generator
Apache License 2.0
10 stars 12 forks source link

fix: correct invalid use of cases in switch #187

Closed dalelane closed 6 months ago

dalelane commented 6 months ago

The use of OR's in switch isn't valid.

For an example, try this in Node.js:

const testvalue = 'option4';

switch (testvalue) {
    case ('option1' || 'option2'):
        console.log('1 or 2');
        break;
    case ('option3' || 'option4' || 'option5'):
        console.log('3, 4, or 5');
        break;
    default:
        console.log('unrecognised ' + testvalue)
}

This means we fail to recognise some type options.

sonarcloud[bot] commented 6 months ago

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
2.8% Duplication on New Code

See analysis details on SonarCloud

dalelane commented 6 months ago

/rtm

asyncapi-bot commented 6 months ago

:tada: This PR is included in version 0.3.0 :tada:

The release is available on:

Your semantic-release bot :package::rocket: