refactor(services): simplify testing

Author: juristr

Diff for: src/app/services/greeting.service.spec.ts

@@ -4,68 +4,71 @@ import { TestBed, inject } from '@angular/core/testing';
 import { GreetingService } from './greeting.service';
 
 describe('GreetingService', () => {
-  describe('Injecting via testcase level inject', () => {
-    beforeEach(() => {
-      TestBed.configureTestingModule({
-        providers: [GreetingService]
-      });
-    });
-
-    it(
-      'should have a service instance',
-      inject([GreetingService], (service: GreetingService) => {
-        expect(service).toBeDefined();
-      })
-    );
-  });
-
-  describe('Injecting via test suite level inject', () => {
+  describe('Manually instantiating service', () => {
     let service: GreetingService;
 
-    beforeEach(() =>
-      TestBed.configureTestingModule({
-        providers: [GreetingService]
-      }));
+    beforeEach(() => {
+      service = new GreetingService();
+    });
 
-    beforeEach(
-      inject([GreetingService], (s: GreetingService) => {
-        service = s;
-      })
-    );
+    afterEach(() => {
+      service = null;
+    });
 
-    it('should have a service instance', () => {
-      expect(service).toBeDefined();
+    it('should greet properly', () => {
+      expect(service.sayHi('Juri')).toBe('Hi, Juri');
     });
   });
 
-  describe('Injecting via TestBed.get()', () => {
-    let service: GreetingService;
-
+  describe('Configuring via TestBed', () => {
     beforeEach(() => {
       TestBed.configureTestingModule({
         providers: [GreetingService]
       });
+    });
 
-      service = TestBed.get(GreetingService);
+    describe('Injecting via testcase level inject', () => {
+      it(
+        'should have a service instance',
+        inject([GreetingService], (service: GreetingService) => {
+          expect(service).toBeDefined();
+        })
+      );
     });
 
-    it('should have a service instance', () => {
-      expect(service).toBeDefined();
+    describe('Injecting via test suite level inject', () => {
+      let service: GreetingService;
+
+      beforeEach(
+        inject([GreetingService], (s: GreetingService) => {
+          service = s;
+        })
+      );
+
+      it('should have a service instance', () => {
+        expect(service).toBeDefined();
+      });
     });
-  });
 
-  describe('Testing service functions', () => {
-    beforeEach(() => {
-      TestBed.configureTestingModule({
-        providers: [GreetingService]
+    describe('Injecting via TestBed.get()', () => {
+      let service: GreetingService;
+
+      beforeEach(() => {
+        service = TestBed.get(GreetingService);
+      });
+
+      it('should have a service instance', () => {
+        expect(service).toBeDefined();
       });
     });
 
-    it(
-      'should greet properly',
-      inject([GreetingService], (service: GreetingService) => {
-        expect(service.sayHi('Juri')).toBe('Hi, Juri');
-      })
-    );
+    describe('Testing service functions', () => {
+      it(
+        'should greet properly',
+        inject([GreetingService], (service: GreetingService) => {
+          expect(service.sayHi('Juri')).toBe('Hi, Juri');
+        })
+      );
+    });
   });
 });