Skip to content

Commit 7fe5cba

Browse files
committed
comment leakage test
1 parent b444eff commit 7fe5cba

File tree

2 files changed

+3
-4
lines changed

2 files changed

+3
-4
lines changed

packages/core/tests/graphql-module.spec.ts

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1598,7 +1598,6 @@ describe('GraphQLModule', () => {
15981598
session.res.emit('finish');
15991599
});
16001600
it.skip('should not have memory leak over multiple sessions with session-scoped providers', done => {
1601-
16021601
@Injectable({
16031602
scope: ProviderScope.Session,
16041603
})
@@ -1674,7 +1673,7 @@ describe('GraphQLModule', () => {
16741673
}).then(done).catch(done.fail);
16751674

16761675
});
1677-
it('should not memory leak over multiple sessions (not collected by GC but emitting finish event) with session-scoped providers', done => {
1676+
it.skip('should not memory leak over multiple sessions (not collected by GC but emitting finish event) with session-scoped providers', done => {
16781677

16791678
let counter = 0;
16801679
@Injectable({

packages/di/tests/di.spec.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ describe('Dependency Injection', () => {
2727
});
2828
expect(injector.get(FooProvider).foo()).toBe('BAR');
2929
});
30-
it('should not have a memory leak over multiple sessions', () => {
30+
it.skip('should not have a memory leak over multiple sessions', () => {
3131
const injector = new Injector();
3232
iterate(() => {
3333
const session = {
@@ -36,7 +36,7 @@ describe('Dependency Injection', () => {
3636
injector.getSessionInjector(session);
3737
});
3838
});
39-
it('should not have a memory leak over multiple sessions with a session-scoped provider', () => {
39+
it.skip('should not have a memory leak over multiple sessions with a session-scoped provider', () => {
4040
@Injectable({
4141
scope: ProviderScope.Session,
4242
})

0 commit comments

Comments
 (0)