Return the original response in @exec-time

[masoud-msk]

This example implies that @execTime decorator doesn't change the method signature:

    @execTime()
    getData(): Promise<DataDto> {
      return this.dataProvider.getData();
    }

But it's not true. This PR is fixing this.

Addressing #171

[masoud-msk]

@vlio20 It looks like you should update the CI job.

[vlio20]

trying...

[vlio20]

@masoud-msk do you want to try and updated actions/upload-artifact: v1 ?

[masoud-msk]

@vlio20 Sorry, I don't have any experience with GitHub Actions!

[vlio20]

why this had to be changed?

[masoud-msk]

Firstly, to exactly reflect the specs.
Secondly, the checked variable was used with await keyword and it doesn't error on a non-Promise but I changed the code to call the .then method on the variable and the runtime throws if it's not a valid Promise. The reason for this change is that I think it's impossible to implement otherwise.

For clarification see this:

const o1 = { then: func => func(42) }
const o2 = { then: 42 }

await o1 // 42
await o1 // { then: 42 }

o1.then(console.log) // 42
o2.then(console.log) // Uncaught TypeError: o2.then is not a function

P.S. Maybe it's better to write test for this utility function.

[vlio20]

would you mind to write this utility function as you suggested?

[vlio20]

@masoud-msk I fixed the CI. I have few questions regarding your PR.