Skip to content
This repository was archived by the owner on Aug 1, 2021. It is now read-only.

Commit 33236f8

Browse files
Merge pull request #205 from aeleftheriadis/feature/fix-e2e-tests
Added Greek translation and fixed Frontend tests and configuration
2 parents 8943c9c + 13a3ff4 commit 33236f8

31 files changed

+862
-315
lines changed

src/Frontend/Jp.AdminUI/.vscode/launch.json

Lines changed: 13 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,18 @@
44
// For more information, visit: https://go.microsoft.com/fwlink/?linkid=830387
55
"version": "0.2.0",
66
"configurations": [
7-
7+
{
8+
"type": "node",
9+
"request": "launch",
10+
"name": "Launch Program",
11+
"program": "${workspaceFolder}/node_modules/protractor/bin/protractor",
12+
"args":["${workspaceFolder}/e2e/protractor.conf.js"],
13+
"preLaunchTask": "npm: e2e",
14+
"outFiles": [
15+
"${workspaceFolder}/dist/out-tsc/**/*.js"
16+
]
17+
},
18+
819
{
920
"type": "chrome",
1021
"request": "launch",
@@ -13,4 +24,4 @@
1324
"webRoot": "${workspaceFolder}"
1425
}
1526
]
16-
}
27+
}
Lines changed: 17 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,17 @@
1+
{
2+
// See https://go.microsoft.com/fwlink/?LinkId=733558
3+
// for the documentation about the tasks.json format
4+
"version": "2.0.0",
5+
"tasks": [
6+
{
7+
"type": "npm",
8+
"script": "e2e",
9+
"problemMatcher": []
10+
},
11+
{
12+
"type": "npm",
13+
"script": "start",
14+
"problemMatcher": []
15+
},
16+
]
17+
}

src/Frontend/Jp.AdminUI/e2e/app.e2e-spec.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ describe("jpproject WebApp", function () {
1111

1212
it("should display sign-in page", () => {
1313
page.navigateTo();
14-
expect(page.getParagraphText()).toEqual("SIGN IN TO CONTINUE.");
14+
expect(page.getButtonText()).toEqual("Login");
1515
expect(page.getUrl()).toContain("/login");
1616
});
1717
});

src/Frontend/Jp.AdminUI/e2e/app.po.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ export class JpProjectWebAppPage {
99
return browser.getCurrentUrl();
1010
}
1111

12-
getParagraphText() {
13-
return element(by.css("login-container text-center py-2")).getText();
12+
getButtonText() {
13+
return element(by.css(".login-container .card-body .mt-3")).getText();
1414
}
1515
}

src/Frontend/Jp.AdminUI/protractor.conf.js renamed to src/Frontend/Jp.AdminUI/e2e/protractor.conf.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,9 +4,9 @@
44
const { SpecReporter } = require('jasmine-spec-reporter');
55

66
exports.config = {
7-
allScriptsTimeout: 11000,
7+
allScriptsTimeout: 30000,
88
specs: [
9-
'./e2e/**/*.e2e-spec.ts'
9+
'./*.e2e-spec.ts'
1010
],
1111
capabilities: {
1212
'browserName': 'chrome'

src/Frontend/Jp.AdminUI/e2e/tsconfig.e2e.json

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,8 @@
22
"extends": "../tsconfig.json",
33
"compilerOptions": {
44
"outDir": "../out-tsc/e2e",
5+
"mapRoot": "../out-tsc/e2e",
6+
"sourceMap": true,
57
"baseUrl": "./",
68
"module": "commonjs",
79
"target": "es5",

src/Frontend/Jp.AdminUI/karma.conf.js

Lines changed: 8 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -3,48 +3,40 @@
33

44
module.exports = function (config) {
55
config.set({
6-
basePath: './',
7-
frameworks: ['jasmine', '@angular/cli'],
6+
basePath: '',
7+
frameworks: ['jasmine', '@angular-devkit/build-angular'],
88
plugins: [
99
require('karma-jasmine'),
1010
require('karma-chrome-launcher'),
1111
require('karma-jasmine-html-reporter'),
1212
require('karma-coverage-istanbul-reporter'),
13-
require('@angular/cli/plugins/karma')
13+
require('@angular-devkit/build-angular/plugins/karma')
1414
],
1515
client:{
1616
clearContext: false // leave Jasmine Spec Runner output visible in browser
1717
},
1818
files: [
1919
{ pattern: './src/test.ts', watched: false },
2020
{ pattern: 'src/assets/**/*', watched: false, included: false, served: true },
21+
{ pattern: 'src/assets/i18n/*.json', watched: false, included: false, served: true },
2122
],
2223
proxies: {
2324
'/assets/': '/base/src/assets/'
2425
},
25-
preprocessors: {
26-
'./src/test.ts': ['@angular/cli']
27-
},
28-
mime: {
29-
'text/x-typescript': ['ts','tsx']
26+
angularCli: {
27+
environment: 'dev'
3028
},
3129
coverageIstanbulReporter: {
32-
reports: [ 'html', 'lcovonly' ],
30+
reports: ['html', 'lcovonly', 'text-summary'],
3331
fixWebpackSourcePaths: true
3432
},
35-
angularCli: {
36-
environment: 'dev'
37-
},
3833
reporters: ['progress', 'kjhtml'],
3934
port: 9876,
4035
colors: true,
4136
logLevel: config.LOG_INFO,
4237
autoWatch: true,
4338
browsers: ['Chrome'],
4439
singleRun: false,
45-
captureTimeout: 60000, // it was already there
46-
browserDisconnectTimeout : 60000,
47-
browserDisconnectTolerance : 1,
48-
browserNoActivityTimeout : 60000 //by default 10000
40+
restartOnFileChange: true
4941
});
5042
};

src/Frontend/Jp.AdminUI/package-lock.json

Lines changed: 3 additions & 3 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

src/Frontend/Jp.AdminUI/package.json

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,7 @@
1111
"build": "ng build --prod",
1212
"test": "ng test",
1313
"lint": "ng lint",
14+
"lintfix": "ng lint --fix=true",
1415
"modernizr": "modernizr -c modernizr-config.json -d src/modernizr.js",
1516
"postinstall": "node ./node_modules/protractor/bin/webdriver-manager update",
1617
"e2e": "ng e2e",
@@ -116,6 +117,6 @@
116117
"ts-node": "^8.3.0",
117118
"tslint": "^5.14.0",
118119
"typescript": ">=3.4.0 <3.5.0",
119-
"webdriver-manager": "12.1.5"
120+
"webdriver-manager": "12.1.7"
120121
}
121122
}

src/Frontend/Jp.AdminUI/src/app/core/settings/settings.service.spec.ts

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,12 +3,14 @@
33
import { TestBed, async, inject } from "@angular/core/testing";
44
import { SettingsService } from "./settings.service";
55
import { HttpClientModule } from "@angular/common/http";
6+
import { OAuthModule } from 'angular-oauth2-oidc';
7+
import { VersionService } from '@shared/services/version.service';
68

79
describe("Service: Settings", () => {
810
beforeEach(() => {
911
TestBed.configureTestingModule({
10-
imports: [HttpClientModule],
11-
providers: [SettingsService]
12+
imports: [HttpClientModule, OAuthModule.forRoot()],
13+
providers: [SettingsService, VersionService]
1214
});
1315
});
1416

0 commit comments

Comments
 (0)