Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
26 changes: 0 additions & 26 deletions packages/functional-tests/tests/misc.spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -4,32 +4,6 @@

import { expect, test } from '../lib/fixtures/standard';

test.describe('severity-1', () => {
// runs all mocha tests - see output here: http://127.0.0.1:3030/tests/index.html
test('content-server mocha tests', async ({ target, page }, { project }) => {
test.skip(project.name !== 'local', 'mocha tests are local only');
await page.goto(`${target.contentServerUrl}/tests/index.html`, {
waitUntil: 'load',
});
await page.waitForTimeout(2000); // wait for mocha to load
await page.evaluate(() =>
(globalThis as any).runner.on(
'end',
() => ((globalThis as any).done = true)
)
);
await page.waitForFunction(
() => (globalThis as any).done,
{},
{ timeout: 0 }
);
const failures = await page.evaluate(
() => (globalThis as any).runner.failures
);
expect(failures).toBe(0);
});
});

Comment on lines -7 to -32
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If I'm reading right, this was a test to make sure all content-server intern tests completed successfully, and it's safe to remove?

test.describe('robots.txt', () => {
test('should allow bots to access all pages', async ({ target, page }) => {
await page.goto(`${target.contentServerUrl}/robots.txt`, {
Expand Down
6 changes: 0 additions & 6 deletions packages/fxa-content-server/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -51,12 +51,6 @@ This package uses Selenium to perform functional tests. By default `npm test` wi
yarn test -- --grep="change password, sign in with new password"
```

If you need to run tests for a particular suite, you can like so:

```bash
node tests/intern.js --suites=server --grep='check resources entrained by /signin in all locales'
```

#### Changing the Auth Server

To change the default Auth Server edit `server/config/*.json` on your deployed instance.
Expand Down
5 changes: 1 addition & 4 deletions packages/fxa-content-server/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -16,16 +16,14 @@
"audit": "npm audit --json | audit-filter --nsp-config=.nsprc --audit=-",
"l10n-create-json": "NODE_OPTIONS=--openssl-legacy-provider grunt l10n-create-json",
"l10n-prime": "yarn l10n:prime packages/fxa-content-server",
"lint": "yarn lint-glean && eslint app server tests --cache",
"lint": "yarn lint-glean && eslint app server --cache",
"lint-glean": "npx glean glinter ../fxa-shared/metrics/glean/fxa-ui-pings.yaml ../fxa-shared/metrics/glean/fxa-ui-metrics.yaml",
"start": "pm2 start pm2.config.js && yarn check:url localhost:3030/bundle/app.bundle.js",
"stop": "pm2 stop pm2.config.js",
"restart": "pm2 restart pm2.config.js ",
"delete": "pm2 delete pm2.config.js",
"start-production": "NODE_ENV=production grunt build && yarn build-css && CONFIG_FILES=server/config/local.json,server/config/production.json,server/config/secrets.json grunt serverproc:dist",
"start-remote": "scripts/run_remote_dev.sh",
"test": "node tests/intern.js --unit=true",
"test-server": "node tests/intern.js --suites=server",
"format": "prettier --write --config ../../_dev/.prettierrc '**'"
},
"repository": {
Expand Down Expand Up @@ -153,7 +151,6 @@
"grunt-todo": "0.5.0",
"htmlparser2": "9.1.0",
"install": "0.13.0",
"intern": "^4.10.1",
"jsqr": "1.4.0",
"otplib": "^11.0.1",
"pm2": "^5.4.2",
Expand Down
12 changes: 0 additions & 12 deletions packages/fxa-content-server/tests/.eslintrc

This file was deleted.

Loading