Skip to content

Commit df50641

Browse files
authored
Clean up test comments and describe text, update tinylicious readme (#21831)
1 parent 76d9569 commit df50641

File tree

12 files changed

+17
-20
lines changed

12 files changed

+17
-20
lines changed

examples/apps/contact-collection/tests/contactCollection.test.ts

-1
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,6 @@
55

66
import { globals } from "../jest.config.cjs";
77

8-
// Tests disabled -- requires Tinylicious to be running, which our test environment doesn't do.
98
describe("contactCollection", () => {
109
beforeAll(async () => {
1110
// Wait for the page to load first before running any tests

examples/apps/task-selection/tests/diceRoller.test.ts

-1
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,6 @@
55

66
import { globals } from "../jest.config.cjs";
77

8-
// Tests disabled -- requires Tinylicious to be running, which our test environment doesn't do.
98
describe("diceRoller", () => {
109
beforeAll(async () => {
1110
// Wait for the page to load first before running any tests

examples/apps/tree-comparison/tests/inventoryList.test.ts renamed to examples/apps/tree-comparison/tests/treeComparison.test.ts

+1-2
Original file line numberDiff line numberDiff line change
@@ -5,8 +5,7 @@
55

66
import { globals } from "../jest.config.cjs";
77

8-
// Tests disabled -- requires Tinylicious to be running, which our test environment doesn't do.
9-
describe("inventoryList", () => {
8+
describe("tree-comparison", () => {
109
beforeAll(async () => {
1110
// Wait for the page to load first before running any tests
1211
// so this time isn't attributed to the first test

examples/service-clients/azure-client/external-controller/tests/diceRoller.test.ts

-1
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,6 @@
55

66
import { globals } from "../jest.config.cjs";
77

8-
// Tests disabled -- requires Tinylicious to be running, which our test environment doesn't do.
98
describe("external-controller-diceRoller", () => {
109
beforeAll(async () => {
1110
// Wait for the page to load first before running any tests

examples/version-migration/live-schema-upgrade/tests/diceRoller.test.ts

-1
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,6 @@
55

66
import { globals } from "../jest.config.cjs";
77

8-
// Tests disabled -- requires Tinylicious to be running, which our test environment doesn't do.
98
describe("diceRoller", () => {
109
beforeAll(async () => {
1110
// Wait for the page to load first before running any tests

examples/version-migration/same-container/tests/inventoryList.test.ts renamed to examples/version-migration/same-container/tests/sameContainer.test.ts

+3-2
Original file line numberDiff line numberDiff line change
@@ -7,8 +7,7 @@ import { IMigrator } from "@fluid-example/example-utils";
77
import { globals } from "../jest.config.cjs";
88
import { IContainer } from "@fluidframework/container-definitions/internal";
99

10-
// Tests disabled -- requires Tinylicious to be running, which our test environment doesn't do.
11-
describe("inventoryList", () => {
10+
describe("same-container migration", () => {
1211
beforeAll(async () => {
1312
// Wait for the page to load first before running any tests
1413
// so this time isn't attributed to the first test
@@ -42,6 +41,7 @@ describe("inventoryList", () => {
4241
await expect(containsOne).toEqual(true);
4342
});
4443

44+
// Test skipped, not functioning currently
4545
it.skip("migrates and shows the correct code version after migration", async () => {
4646
// Validate the migration status shows "one" initially
4747
await Promise.all([
@@ -92,6 +92,7 @@ describe("inventoryList", () => {
9292
});
9393
});
9494

95+
// Test skipped, not functioning currently
9596
describe.skip("With summarizer connected", () => {
9697
beforeEach(async () => {
9798
await page.goto(`${globals.PATH}`, { waitUntil: "load" });

examples/version-migration/schema-upgrade/tests/inventoryList.test.ts renamed to examples/version-migration/schema-upgrade/tests/separateContainer.test.ts

+1-2
Original file line numberDiff line numberDiff line change
@@ -7,8 +7,7 @@ import { IMigrator } from "@fluid-example/example-utils";
77
import { globals } from "../jest.config.cjs";
88
import { IContainer } from "@fluidframework/container-definitions/internal";
99

10-
// Tests disabled -- requires Tinylicious to be running, which our test environment doesn't do.
11-
describe("inventoryList", () => {
10+
describe("separate-container migration", () => {
1211
beforeAll(async () => {
1312
// Wait for the page to load first before running any tests
1413
// so this time isn't attributed to the first test

examples/version-migration/tree-shim/tests/inventoryList.test.ts renamed to examples/version-migration/tree-shim/tests/treeShim.test.ts

+1-2
Original file line numberDiff line numberDiff line change
@@ -5,8 +5,7 @@
55

66
import { globals } from "../jest.config.cjs";
77

8-
// Tests disabled -- requires Tinylicious to be running, which our test environment doesn't do.
9-
describe("inventoryList", () => {
8+
describe("tree-shim", () => {
109
beforeAll(async () => {
1110
// Wait for the page to load first before running any tests
1211
// so this time isn't attributed to the first test

examples/view-integration/container-views/tests/diceRoller.test.ts

-1
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,6 @@
55

66
import { globals } from "../jest.config.cjs";
77

8-
// Tests disabled -- requires Tinylicious to be running, which our test environment doesn't do.
98
describe("diceRoller", () => {
109
beforeAll(async () => {
1110
// Wait for the page to load first before running any tests

examples/view-integration/external-views/tests/diceRoller.test.ts

-1
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,6 @@
55

66
import { globals } from "../jest.config.cjs";
77

8-
// Tests disabled -- requires Tinylicious to be running, which our test environment doesn't do.
98
describe("diceRoller", () => {
109
beforeAll(async () => {
1110
// Wait for the page to load first before running any tests

examples/view-integration/view-framework-sampler/tests/diceRoller.test.ts

-1
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,6 @@
55

66
import { globals } from "../jest.config.cjs";
77

8-
// Tests disabled -- requires Tinylicious to be running, which our test environment doesn't do.
98
describe("diceRoller", () => {
109
beforeAll(async () => {
1110
// Wait for the page to load first before running any tests

server/routerlicious/packages/tinylicious/README.md

+11-5
Original file line numberDiff line numberDiff line change
@@ -4,19 +4,25 @@ Tinylicious is a minimal, self-contained, test implementation of the Fluid Frame
44

55
## What is this for?
66

7-
Tinylicious includes most of the basic features needed to **test** data stores and containers. While we use the [Webpack Fluid Loader](../../packages/tools/webpack-fluid-loader)'s in browser service for much of our data store and container development, Tinylicious offers some advantages because it's a standalone process. For instance, testing a Fluid Container from 2+ simultaneously connected clients can be easier using Tinylicious.
7+
Tinylicious includes most of the basic features needed to **test** data stores and containers. While we use the [Local Server](../local-server) as an in-browser service for much of our data store and container development, Tinylicious offers some advantages because it's a standalone process. For instance, testing a Fluid Container from 2+ simultaneously connected clients can be easier using Tinylicious.
88

99
If you're looking for a reference implementation of the Fluid service, don't look here! Go check out [Routerlicious](../routerlicious).
1010

1111
## Getting Started
1212

13-
You can install, build, and start this service by running the following
13+
You can build this service by running the following in the /server/routerlicious directory (NOT in this directory):
1414

1515
```sh
1616
npm i -g pnpm
1717
pnpm i
18-
npm run build
19-
npm run start
18+
pnpm build
19+
```
20+
21+
Afterwards, you can start and stop this service with the following commands in this directory:
22+
23+
```sh
24+
pnpm start
25+
pnpm stop
2026
```
2127

2228
## Configuration
@@ -28,7 +34,7 @@ variable named PORT to the desired number. For example:
2834

2935
```sh
3036
$env:PORT=6502
31-
npm run start
37+
pnpm start
3238
```
3339

3440
### Storage

0 commit comments

Comments
 (0)