Skip to content

Commit 3263b5d

Browse files
wip. check test
1 parent d9e067b commit 3263b5d

File tree

2 files changed

+7
-25
lines changed

2 files changed

+7
-25
lines changed

packages/devextreme-cli/testing/__tests__/nextjs-ts.test.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,6 @@ const { nextjsTs } = require('../env.nextjs');
22
const testAppTemplate = require('../app-template.test.shared.js');
33
testAppTemplate(nextjsTs, {
44
port: 3001,
5-
useDifferentPorts: true,
65
urls: {
76
profile: 'pages/profile',
87
tasks: 'pages/tasks',

packages/devextreme-cli/testing/app-template.test.shared.js

Lines changed: 7 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ const DevServer = require('./dev-server');
77

88
const defaultLayout = 'side-nav-outer-toolbar';
99

10-
module.exports = (env, { port, urls, useDifferentPorts } = { port: '8080', urls: {}, useDifferentPorts: false }) => {
10+
module.exports = (env, { port, urls } = { port: '8080', urls: {} }) => {
1111
const diffSnapshotsDir = path.join('testing/__tests__/__diff_snapshots__', env.engine);
1212
const pageUrls = {
1313
profile: 'profile',
@@ -16,42 +16,30 @@ module.exports = (env, { port, urls, useDifferentPorts } = { port: '8080', urls:
1616
'change-password': 'change-password/123',
1717
...urls,
1818
};
19-
let currentPort = port;
2019

2120
describe(`${env.engine} app-template`, () => {
22-
let browser;
23-
let page;
24-
25-
/* beforeAll(async() => {
26-
browser = await getBrowser();
27-
page = await browser.newPage();
28-
});
29-
30-
afterAll(async() => {
31-
console.log('-----afterAll---CLOSE-->', Boolean(process.env.LAUNCH_BROWSER));
32-
await (Boolean(process.env.LAUNCH_BROWSER) ? browser : page).close();
33-
}); */
34-
3521
Object.keys(themes).forEach((theme) => {
3622
describe(theme, () => {
3723
layouts.forEach((layout) => {
3824
const isDefaultLayout = layout === defaultLayout;
3925

4026
describe(layout, () => {
27+
let browser;
28+
let page;
4129
const getPageURL = (name) => `${appUrl}${(env.engine.indexOf('nextjs') !== 0 ? '#/' : '')}${pageUrls[name]}`;
42-
console.log('----currentPort------>', currentPort);
30+
console.log('----currentPort------>', port);
4331
let devServer;
4432
let appUrl;
4533

4634
beforeAll(async() => {
47-
appUrl = `http://${ip.address()}:${currentPort}/`;
48-
devServer = new DevServer(env, { port: currentPort });
35+
appUrl = `http://${ip.address()}:${port}/`;
36+
devServer = new DevServer(env, { port });
4937

5038
browser = await getBrowser();
5139
page = await browser.newPage();
5240

5341
try {
54-
console.log('----beforeAll--wait-->', currentPort);
42+
console.log('----beforeAll--wait-->', port);
5543
await waitOn({
5644
reverse: true,
5745
resources: [appUrl],
@@ -76,12 +64,7 @@ module.exports = (env, { port, urls, useDifferentPorts } = { port: '8080', urls:
7664
});
7765

7866
afterAll(async() => {
79-
console.log('----afterAll---before stop--->');
8067
await devServer.stop();
81-
console.log('----afterAll---after stop--->');
82-
if(useDifferentPorts) {
83-
// currentPort++;
84-
}
8568
await (Boolean(process.env.LAUNCH_BROWSER) ? browser : page).close();
8669
});
8770

0 commit comments

Comments
 (0)