test(sample-05): added unit tests

[Tony133]

PR Checklist

Please check if your PR fulfills the following requirements:

• The commit message follows our guidelines: https://github.com/nestjs/nest/blob/master/CONTRIBUTING.md
• Tests for the changes have been added (for bug fixes / features)
• Docs have been added / updated (for bug fixes / features)

PR Type

What kind of change does this PR introduce?

• Bugfix
• Feature
• Code style update (formatting, local variables)
• Refactoring (no functional changes, no api changes)
• Build related changes
• CI related changes
• Other... Please describe:

What is the current behavior?

Issue Number: #1539

What is the new behavior?

added unit tests in sample 05-sql-typeorm

Does this PR introduce a breaking change?

• Yes
• No

Other information

This PR, added unit tests in sample 05-sql-typeorm

[coveralls]

Pull Request Test Coverage Report for Build d10c5f1e-1a8b-4f21-9679-5b8b056b7086

• 0 of 0 changed or added relevant lines in 0 files are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage remained the same at 94.207%

---

Totals
Change from base Build 19b05668-7122-423d-92a0-0f57cbcafcaa:	0.0%
Covered Lines:	5659
Relevant Lines:	6007

---

💛 - Coveralls

[jmcdo29]

Made several comments. Take a look and address them as you can

[Tony133]

@jmcdo29 I made these changes it could go?

• row 36:
  save: jest.fn().mockResolvedValue(oneUser),
• row 54:

      const oneUser = {
        firstName: 'firstName #1',
        lastName: 'lastName #1',
      };
      expect(
        service.create({
          firstName: 'firstName #1',
          lastName: 'lastName #1',
        }),
      ).resolves.toEqual(oneUser);

[Tony133]

Conflicts resolved in package.json

[Tony133]

Conflicts resolved 👍

[Tony133]

@kamilmysliwiec when you find some time can you check if I still have to make changes in this pull or can you merge?

[kamilmysliwiec]

LGTM