Add imports to typescript generator

[luca-nardelli]

Q	A
Bug fix?	yes
New feature?	no
BC breaks?	no
Deprecations?	no
Tests pass?	yes
Fixed tickets
License	MIT
Doc PR

I've noticed that the generated interfaces for typescript are missing the import { ResourceName } from "./resourcefile"; statements, making them less useful when used in a Typescript application (imports would need to be manually added everytime).

This PR fixes this and also adds a test to the typescript generator, testing for both interfaces with references to other interfaces and interfaces without references.

[luca-nardelli]

ping @alanpoulain or @dunglas (should I ping someone else as well?)

[luca-nardelli]

Travis was failing, I forgot to run yarn fix, now it's ok

[luca-nardelli]

I've noticed a thing. At the moment the handlebars template for the typescript interface has the @id field marked as not required, whereas id is required.

{{#each imports}}
import { {{type}} } from "{{file}}";
{{/each}}
{{#if imports.length}}

{{/if}}
export interface {{{name}}} {
  '@id'?: string;
  id: string;
{{#each fields}}
 {{#if readonly}} readonly{{/if}} {{{name}}}{{#if notrequired}}?{{/if}}: {{{type}}};
{{/each}}
}

I'd make both non-required, otherwise you might have issues when using that interface for POST (collection add) operations, where the ID might be unspecified (or would you have to set it as null?)

What do you think?

[dunglas]

@luca-nardelli I agree with you. Can you open a new PR please?

[dunglas]

Thanks @luca-nardelli

[luca-nardelli]

You're welcome! I'll open the other PR asap

[luca-nardelli]

Done! #123