Working on refactor release test class

Author: Basim-Elayan

src/app/app.module.ts

@@ -6,7 +6,7 @@ import { AppComponent } from "./app.component";
 import { ItemsComponent } from "./item/items.component";
 import { ItemDetailComponent } from "./item/item-detail.component";
 import { LoginComponent } from './login/login.component';
-import { ReleaseService } from "./Services/release.service";
+import { ReleaseService } from "./services/release.service";
 
 // Uncomment and add to NgModule imports if you need to use two-way binding
 // import { NativeScriptFormsModule } from "nativescript-angular/forms";

src/tests/release.test.ts

@@ -3,6 +3,7 @@ import { ReleaseService } from "../app/services/release.service";
 describe('Release notes: ', function () {
 	var releaseService: ReleaseService = new ReleaseService();
 	var data: string;
+	var dataUpperCase: string;
 	var isReleaseFileExist: boolean;
 
 	it("Release notes is exist.", function () {
@@ -12,7 +13,8 @@ describe('Release notes: ', function () {
 
 	it("Read most recent release notes.", function () {
 		if (!!isReleaseFileExist) {
-			data = releaseService.readReleaseFile()
+			data = releaseService.readReleaseFile();
+			dataUpperCase = data.toUpperCase();
 			expect(data).not.equal(null);
 			expect(data).not.equal(undefined);
 			expect(data).not.equal("");
@@ -56,17 +58,17 @@ describe('Release notes: ', function () {
 
 		it("Not Include IOS keyword.", function () {
 			if (!!data) {
-				expect(data.toUpperCase()).not.contain("IOS");
-				expect(data.toUpperCase()).not.contain("APPLE");
+				expect(dataUpperCase).not.contain("IOS");
+				expect(dataUpperCase).not.contain("APPLE");
 			}
 		});
 	});
 
 	describe('(IOS Platform)', function () {
 		it("Not Include android keyword.", function () {
 			if (!!data) {
-				expect(data.toUpperCase()).not.contain("ANDROID");
-				expect(data.toUpperCase()).not.contain("GOOGLE");
+				expect(dataUpperCase).not.contain("ANDROID");
+				expect(dataUpperCase).not.contain("GOOGLE");
 			}
 		});
 	});