Add Basic Color Test for RGB

[kyledecot]

Issues

Fixes #387

Proposed changes

Updates the substring logic in Color.ts to properly grab the RGB values. Without this change numbers ends up being an array w/ a single item in it with eventually causes the method to return null.

Checklist

• I consent that this change becomes part of alphaTab under it's current or any future open source license
• Changes are implemented
• Existing builds tests pass
• New tests were added

Further details

• This is a breaking change
• This change will require update of the documentation/website

[kyledecot]

I might need some guidance here @Danielku15 – I'm not sure how these C# tests are generated / what I'm doing incorrectly.

[Danielku15]

Hi there.
You can usually just run locally the build-csharp NPM script to generate the C# sources and compile them: https://alphatab.net/docs/contributing#npm-scripts

This way you get the same error locally and can inspect the C# code. You will notice that the test method is having a space which is not supported for C#:

You will notice that all test names within alphaTab are following the kebab-case naming convention. This way you can fix the test. Either you simply adapt the test name, or you dare to touch the C# transpiler to add support for spaces 😉

alphaTab/src.compiler/csharp/CSharpAstTransformer.ts

Line 583 in a9f84fb

name: this._context.toPascalCase((d.arguments[0] as ts.StringLiteral).text),

[kyledecot]

Ah–okay. Thanks for the explanation. This should be fixed in bb1e130

[Danielku15]

Thanks for the contribution. Merged the changes 👍