ParseIntPipe in cats example not working correctly? (v5 beta)

[lo78cn]

I'm submitting a...

[ ] Regression 
[x] Bug report
[ ] Feature request
[ ] Documentation issue or request
[ ] Support request => Please do not submit support request here, instead post your question on Stack Overflow.

Current behavior

{"data":"string"}

Expected behavior

{"data":"number"}

Minimal reproduction of the problem with instructions

Download nest-master
extract zip file
cd sample/01-cats-app

change package.json

    "@nestjs/common": "^5.0.0-beta.3",
    "@nestjs/core": "^5.0.0-beta.3",
    "@nestjs/microservices": "^5.0.0-beta.3",
    "@nestjs/testing": "^5.0.0-beta.3",
    "@nestjs/websockets": "^5.0.0-beta.3",

change cats.controller.ts

@Get(':id')
  findOne(@Param('id', new ParseIntPipe()) id) {
    return typeof id;
  }

npm install
npm start

curl http://localhost:3000/cats/1

What is the motivation / use case for changing the behavior?

It should convert string to number.

Environment

Nest version: 5.0.0-beta.3

 
For Tooling issues:
- Node version: 8.9.1  
- Platform: Mac 

Others:

[kamilmysliwiec]

Thanks for reporting. I'm on it

[kamilmysliwiec]

Fixed in v5.0.0-beta.4. Let me know if you face any issue. 🙂

[lock]

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.