bzp2010 commented on code in PR #12214:
URL: https://github.com/apache/apisix/pull/12214#discussion_r2096780849
##########
t/admin/standalone.spec.ts:
##########
@@ -159,7 +192,82 @@ describe("Admin - Standalone", () => {
it('check route "r2"', () =>
expect(client.get("/r2")).rejects.toThrow(
"Request failed with status code 404"
- ));
+ ));
+
+ it("only set routes_conf_version", async () => {
+ const resp = await client.put(
+ ENDPOINT,
+ YAML.stringify({ routes_conf_version: 15 }),
+ {headers: {"Content-Type": "application/yaml"},
+ });
+ expect(resp.status).toEqual(202);
+
+ const resp_1 = await client.get(ENDPOINT);
+ expect(resp_1.status).toEqual(200);
+ expect(resp_1.data.routes_conf_version).toEqual(15);
+ expect(resp_1.data.ssls_conf_version).toEqual(4);
+ expect(resp_1.data.services_conf_version).toEqual(4);
+ expect(resp_1.data.upstreams_conf_version).toEqual(4);
+ expect(resp_1.data.consumers_conf_version).toEqual(4);
+
+ const resp2 = await client.put(
+ ENDPOINT,
+ YAML.stringify({ routes_conf_version: 17 }),
+ {headers: {"Content-Type": "application/yaml"},
+ });
+ expect(resp2.status).toEqual(202);
+
+ const resp2_1 = await client.get(ENDPOINT);
+ expect(resp2_1.status).toEqual(200);
+ expect(resp2_1.data.routes_conf_version).toEqual(17);
+ expect(resp2_1.data.ssls_conf_version).toEqual(5);
+ expect(resp2_1.data.services_conf_version).toEqual(5);
+ expect(resp2_1.data.upstreams_conf_version).toEqual(5);
+ expect(resp2_1.data.consumers_conf_version).toEqual(5);
+ });
+
+ it("control resource changes using modifiedIndex", async () => {
+ const c1 = structuredClone(routeWithModifiedIndex);
+ c1.routes[0].modifiedIndex = 1;
+
+ const c2 = structuredClone(c1);
+ c2.routes[0].uri = "/r2";
+
+ const c3 = structuredClone(c2);
+ c3.routes[0].modifiedIndex = 2;
+
+ // Update with c1
+ const resp = await client.put(ENDPOINT, c1);
+ expect(resp.status).toEqual(202);
+
+ // Check route /r1 exists
+ const resp_1 = await client.get("/r1");
+ expect(resp_1.status).toEqual(200);
+
+ // Update with c2
+ const resp2 = await client.put(ENDPOINT, c2);
+ expect(resp2.status).toEqual(202);
+
+ // Check route /r1 exists
Review Comment:
```suggestion
// Check route /r1 exists
// But it is not applied because the modifiedIndex is the same as the
old value
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscr...@apisix.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
