implab/implabjs-di Commit - r2:154d88dba49c

code comments

cin -

r2:154d88dba49c default

parent child

Context file:

r2:154d88dba49c

Collapse all files

src/main/ts/FluentConfiguration.ts +32 -5

             import { DescriptorBuilder } from "./DescriptorBuilder";
-            import { ConfigurableKeys, ContainerServices, ConfigurableServices, RegistrationBuildersMap, RequiredKeys } from "./interfaces";
+            import { ConfigurableKeys, ContainerServices, ConfigurableServices, RegistrationBuildersMap, ExtractRequired } from "./interfaces";
             import { ServiceContainer } from "./interfaces";
             import { argumentNotNull, each, isKey } from "./traits";
             export class FluentConfiguration<S extends object, Y extends ConfigurableKeys<S> = ConfigurableKeys<S>> {
-                _builders: Partial<RegistrationBuildersMap<S>> = {};
+                private _builders: Partial<RegistrationBuildersMap<S>> = {};
+                /** Adds a declaration of the services to the current config.
+                 *
+                 * @template D The map of the services
+                 * @returns self
+                 */
                 declare<D extends Partial<Pick<S, keyof D & keyof S>>>(): FluentConfiguration<S & D, Y | ConfigurableKeys<D>> {
                     return this as FluentConfiguration<S & D, Y | ConfigurableKeys<D>>;
                 }
+                /** Adds compile-time information about the already provided services
+                 *
+                 * @template P The map of the provided services
+                 * @returns self
+                 */
                 provided<P extends Pick<S, keyof P & keyof S>>(): FluentConfiguration<S & P, Exclude<Y, keyof P>> {
                     return this as FluentConfiguration<S & P, Exclude<Y, keyof P>>;
                 }
+                /** Register the service.
+                 *
+                 * @param name The name of the service
+                 * @param builder The service builder
+                 * @returns self
+                 */
                 register<K extends Y>(name: K, builder: RegistrationBuildersMap<S>[K]): FluentConfiguration<S, Exclude<Y, K>>;
+                /** Registers the collection of services
+                 * @param config The collection of services to register.
+                 * @returns self
+                 */
                 register<K extends Y>(config: RegistrationBuildersMap<S, K>): FluentConfiguration<S, Exclude<Y, K>>;
                 register<K extends Y>(nameOrConfig: K | RegistrationBuildersMap<S, K>, builder?: RegistrationBuildersMap<S>[K]) {
                     if (isKey(nameOrConfig)) {
                     return this as FluentConfiguration<S, Exclude<Y, K>>;
                 }
-                done(missing: RequiredKeys<S, Y> extends never ? void : RequiredKeys<S, Y>) {
+                /**
-                    if (missing !== undefined)
+                 * This method is used to enable a compile time check of the configuration.
-                        throw new Error("The configuration isn't complete");
+                 * If there are not configured services in the configuration the compiler
+                 * will trigger the error.
+                 *
+                 * @param missing Empty object literal should always be specified.
+                 * @returns self
+                 */
+                // eslint-disable-next-line @typescript-eslint/no-unused-vars
+                done<M extends ExtractRequired<S,Y>>(missing: M) {
                     return this;
                 }

src/main/ts/interfaces.ts +1 -1

                 store(item: T, cleanup?: (item: T) => void): void;
             }
-            export type RequiredKeys<T, K extends keyof T = keyof T> = keyof { [p in K as (T[p] extends undefined ? never : p)]-?: T[p] };
+            export type ExtractRequired<T, K extends keyof T = keyof T> = { [p in K as (undefined extends T[p] ? never : p)]-?: T[p] };

src/test/ts/t/container.ts +1 -1

                     foo: it => it.factory($ => new Foo()),
                     baz: it => it.value(new Foo())
                 })
-                .done();
+                .done({});
                 declare const container: Container<SharedServices>;

General Comments 0

Write
Preview

You need to be logged in to leave comments. Login now

No TODOs yet

	Site-wide shortcuts
/	Use quick search box
g h	Goto home page
g g	Goto my private gists page
g G	Goto my public gists page
g 0-9	Goto bookmarked items from 0-9
n r	New repository page
n g	New gist page

	Repositories
g s	Goto summary page
g c	Goto changelog page
g f	Goto files page
g F	Goto files page with file search activated
g p	Goto pull requests page
g o	Goto repository settings
g O	Goto repository access permissions settings
t s	Toggle sidebar on some pages