[typescript-angular] Invalid regular expression error

[kimamula]

Description

The code generated from the following template results in a runtime error, Invalid regular expression: /(?i)^(application/json|[^;/ ]+/[^;/ ]+[+]json)[ ]*(;.*)?$/: Invalid group

swagger-codegen/modules/swagger-codegen/src/main/resources/typescript-angular/api.service.mustache

Line 82 in d2b9107

const jsonMime: RegExp = new RegExp('(?i)^(application/json|[^;/ \t]+/[^;/ \t]+[+]json)[ \t]*(;.*)?$');

Swagger-codegen version

Master branch (2.3.0)

Steps to reproduce

Paste the regular expression in Chrome Developer console, and Chrome raises the syntax error.

[taxpon]

@wing328 I think we need to add some basic tests for angular and run them on CI in order to avoid this kind of bugs. What do you say?

[wing328]

@taxpon definitely. It's better to rely on CI to catch similar issue moving forward.

[wing328]

Btw, npm run build didn't catch this issue so we need to write some integration tests similar to what we've done for TS Fetch: https://github.com/swagger-api/swagger-codegen/tree/master/samples/client/petstore/typescript-fetch

[wing328]

cc @TiFu @taxpon @sebastianhaas @kenisteward @Vrolijkx

[mattrc]

+1

[gimre]

+1 - also worth mentioning that typescript-angular is still using the old and about-to-be-deprecated HttpModule instead of HttpClientModule

[kenisteward]

There is a PR to use the new one by default and optionally use the old one

…

On Wed, Sep 27, 2017, 9:09 AM Gabriel Imre ***@***.***> wrote: +1 - also worth mentioning that typescript-angular is still using the old and about-to-be-deprecated HttpModule instead of HttpClientModule — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#6566 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AMPLtcq97Egsb1Mia38sExaUTNXyhysLks5smkjygaJpZM4Pj3EQ> .

[wing328]

@kimamula the issue has been fixed by #6584. Please pull the latest master to give it a try.